Wysiwyg - 6.x-2.x

Test status
StatusResult
Last receivedFri, 09/21/2012 - 16:44:06
Last requestedFri, 09/21/2012 - 16:45:30
Last testedFri, 09/21/2012 - 16:45:41
Test count2
Environment status
MySQLpass
Code reviewfail
Overall Resultpass
Branch information
Identifier6.x-2.x
ProjectWysiwyg
LinkProject information
SummaryPASSED: [[SimpleTest]]: [MySQL] 0 pass(es); [[Coder]]: [Code review] 46 minor(s), 2 critical(s), and 2 normal(s).
Details

MySQL

  1. Setup environment
  2. Detect an invalid patch URL
  3. Detect invalid repository information
  4. Detect invalid patch format
  5. Detect a non-applicable patch
  6. Detect invalid PHP syntax
  7. Detect a Drupal installation failure
  8. Detect a test run failure
  9. Detect a failing test
  10. Complete test suite with all tests passing

Code review

  1. Setup environment
  2. Detect an invalid patch URL
  3. Detect invalid repository information
  4. Detect invalid patch format
  5. Detect a non-applicable patch
  6. Detect invalid PHP syntax
  7. Detect a review run failure
  8. Detect coder flags
    Ensure that your code follows the Drupal standard and passes a coder review.
  9. Complete coder review with all passed
46 minor(s), 2 critical(s), and 2 normal(s)

Non-pass

Test nameMinorCriticalNormal
Expandwysiwyg.admin.inc020
MessageGroupFilenameLineFunctionStatus
Potential problem: FAPI elements '#title' and '#description' only accept filtered text, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized.Otherwysiwyg.admin.inc142N/A
Potential problem: FAPI elements '#title' and '#description' only accept filtered text, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized.Otherwysiwyg.admin.inc151N/A
Expandwysiwyg.install102
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherwysiwyg.install-1N/A
Missing periodOtherwysiwyg.install89N/A
Use update_sql() instead of db_query() in hook_update_N()Otherwysiwyg.install202N/A

All

Test nameMinorCriticalNormal
Expandeditors/ckeditor.inc700
MessageGroupFilenameLineFunctionStatus
Format should be * Implementation of hook_foo().Otherckeditor.inc9N/A
'Implementation' should be at the start of the sentence and begin with a capitialized letterOtherckeditor.inc9N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherckeditor.inc109N/A
@see should always be followed by a filename, a URL, class/interface name (optionally including method), or a function name including ().Otherckeditor.inc131N/A
@see references should be separated by "," followed by a single space and with no trailing punctuationOtherckeditor.inc131N/A
Format should be * Implementation of hook_foo().Otherckeditor.inc385N/A
'Implementation' should be at the start of the sentence and begin with a capitialized letterOtherckeditor.inc385N/A
Expandeditors/fckeditor.inc400
MessageGroupFilenameLineFunctionStatus
Format should be * Implementation of hook_foo().Otherfckeditor.inc9N/A
'Implementation' should be at the start of the sentence and begin with a capitialized letterOtherfckeditor.inc9N/A
Format should be * Implementation of hook_foo().Otherfckeditor.inc212N/A
'Implementation' should be at the start of the sentence and begin with a capitialized letterOtherfckeditor.inc212N/A
Expandeditors/jwysiwyg.inc200
MessageGroupFilenameLineFunctionStatus
Format should be * Implementation of hook_foo().Otherjwysiwyg.inc9N/A
'Implementation' should be at the start of the sentence and begin with a capitialized letterOtherjwysiwyg.inc9N/A
Expandeditors/markitup.inc400
MessageGroupFilenameLineFunctionStatus
Format should be * Implementation of hook_foo().Othermarkitup.inc9N/A
'Implementation' should be at the start of the sentence and begin with a capitialized letterOthermarkitup.inc9N/A
Format should be * Implementation of hook_foo().Othermarkitup.inc169N/A
'Implementation' should be at the start of the sentence and begin with a capitialized letterOthermarkitup.inc169N/A
Expandeditors/nicedit.inc400
MessageGroupFilenameLineFunctionStatus
Format should be * Implementation of hook_foo().Othernicedit.inc9N/A
'Implementation' should be at the start of the sentence and begin with a capitialized letterOthernicedit.inc9N/A
Format should be * Implementation of hook_foo().Othernicedit.inc94N/A
'Implementation' should be at the start of the sentence and begin with a capitialized letterOthernicedit.inc94N/A
Expandeditors/openwysiwyg.inc500
MessageGroupFilenameLineFunctionStatus
Format should be * Implementation of hook_foo().Otheropenwysiwyg.inc9N/A
'Implementation' should be at the start of the sentence and begin with a capitialized letterOtheropenwysiwyg.inc9N/A
There should be no trailing spacesOtheropenwysiwyg.inc129N/A
Format should be * Implementation of hook_foo().Otheropenwysiwyg.inc138N/A
'Implementation' should be at the start of the sentence and begin with a capitialized letterOtheropenwysiwyg.inc138N/A
Expandeditors/tinymce.inc500
MessageGroupFilenameLineFunctionStatus
Format should be * Implementation of hook_foo().Othertinymce.inc9N/A
'Implementation' should be at the start of the sentence and begin with a capitialized letterOthertinymce.inc9N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthertinymce.inc374N/A
Format should be * Implementation of hook_foo().Othertinymce.inc381N/A
'Implementation' should be at the start of the sentence and begin with a capitialized letterOthertinymce.inc381N/A
Expandeditors/whizzywig.inc400
MessageGroupFilenameLineFunctionStatus
Format should be * Implementation of hook_foo().Otherwhizzywig.inc9N/A
'Implementation' should be at the start of the sentence and begin with a capitialized letterOtherwhizzywig.inc9N/A
Format should be * Implementation of hook_foo().Otherwhizzywig.inc121N/A
'Implementation' should be at the start of the sentence and begin with a capitialized letterOtherwhizzywig.inc121N/A
Expandeditors/wymeditor.inc500
MessageGroupFilenameLineFunctionStatus
Format should be * Implementation of hook_foo().Otherwymeditor.inc9N/A
'Implementation' should be at the start of the sentence and begin with a capitialized letterOtherwymeditor.inc9N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherwymeditor.inc164N/A
Format should be * Implementation of hook_foo().Otherwymeditor.inc187N/A
'Implementation' should be at the start of the sentence and begin with a capitialized letterOtherwymeditor.inc187N/A
Expandeditors/yui.inc400
MessageGroupFilenameLineFunctionStatus
Format should be * Implementation of hook_foo().Otheryui.inc9N/A
'Implementation' should be at the start of the sentence and begin with a capitialized letterOtheryui.inc9N/A
Format should be * Implementation of hook_foo().Otheryui.inc273N/A
'Implementation' should be at the start of the sentence and begin with a capitialized letterOtheryui.inc273N/A
Leafplugins/break.inc000
Leafwysiwyg-dialog-page.tpl.php000
Expandwysiwyg.admin.inc020
MessageGroupFilenameLineFunctionStatus
Potential problem: FAPI elements '#title' and '#description' only accept filtered text, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized.Otherwysiwyg.admin.inc142N/A
Potential problem: FAPI elements '#title' and '#description' only accept filtered text, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized.Otherwysiwyg.admin.inc151N/A
Leafwysiwyg.api.php000
Leafwysiwyg.dialog.inc000
Expandwysiwyg.install102
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherwysiwyg.install-1N/A
Missing periodOtherwysiwyg.install89N/A
Use update_sql() instead of db_query() in hook_update_N()Otherwysiwyg.install202N/A
Expandwysiwyg.module100
MessageGroupFilenameLineFunctionStatus
Format should be * Implementation of hook_foo().Otherwysiwyg.module380N/A