Galleria - 7.x-1.0-beta3

Test status
StatusResult
Last receivedThu, 06/07/2012 - 18:41:19
Last requestedThu, 06/07/2012 - 18:42:03
Last testedThu, 06/07/2012 - 18:42:11
Test count2
Environment status
MySQLpass
Code reviewfail
Overall Resultpass
Branch information
Identifier7.x-1.0-beta3
ProjectGalleria
LinkProject information
SummaryPASSED: [[SimpleTest]]: [MySQL] 0 pass(es); [[Coder]]: [Code review] 8 minor(s), 1 critical(s), and 10 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
8 minor(s), 1 critical(s), and 10 normal(s)

Non-pass

Test nameMinorCriticalNormal
Expandgalleria.module311
MessageGroupFilenameLineFunctionStatus
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthergalleria.module75N/A
There should be no trailing spacesOthergalleria.module402N/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.Othergalleria.module421N/A
There should be no trailing spacesOthergalleria.module509N/A
There should be no trailing spacesOthergalleria.module514N/A
Expandincludes/galleria.admin.inc109
MessageGroupFilenameLineFunctionStatus
do not use mixed case (camelCase), use lower case and _Othergalleria.admin.inc121N/A
do not use mixed case (camelCase), use lower case and _Othergalleria.admin.inc129N/A
do not use mixed case (camelCase), use lower case and _Othergalleria.admin.inc199N/A
do not use mixed case (camelCase), use lower case and _Othergalleria.admin.inc212N/A
do not use mixed case (camelCase), use lower case and _Othergalleria.admin.inc246N/A
do not use mixed case (camelCase), use lower case and _Othergalleria.admin.inc287N/A
do not use mixed case (camelCase), use lower case and _Othergalleria.admin.inc405N/A
do not use mixed case (camelCase), use lower case and _Othergalleria.admin.inc449N/A
do not use mixed case (camelCase), use lower case and _Othergalleria.admin.inc456N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthergalleria.admin.inc717N/A

All

Test nameMinorCriticalNormal
Leafgalleria.install000
Expandgalleria.module311
MessageGroupFilenameLineFunctionStatus
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthergalleria.module75N/A
There should be no trailing spacesOthergalleria.module402N/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.Othergalleria.module421N/A
There should be no trailing spacesOthergalleria.module509N/A
There should be no trailing spacesOthergalleria.module514N/A
Leafgalleria.views.inc000
Expandincludes/galleria.admin.inc109
MessageGroupFilenameLineFunctionStatus
do not use mixed case (camelCase), use lower case and _Othergalleria.admin.inc121N/A
do not use mixed case (camelCase), use lower case and _Othergalleria.admin.inc129N/A
do not use mixed case (camelCase), use lower case and _Othergalleria.admin.inc199N/A
do not use mixed case (camelCase), use lower case and _Othergalleria.admin.inc212N/A
do not use mixed case (camelCase), use lower case and _Othergalleria.admin.inc246N/A
do not use mixed case (camelCase), use lower case and _Othergalleria.admin.inc287N/A
do not use mixed case (camelCase), use lower case and _Othergalleria.admin.inc405N/A
do not use mixed case (camelCase), use lower case and _Othergalleria.admin.inc449N/A
do not use mixed case (camelCase), use lower case and _Othergalleria.admin.inc456N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthergalleria.admin.inc717N/A
Leafincludes/views_plugin_style_galleria.inc000
Expandtheme/galleria-container.tpl.php100
MessageGroupFilenameLineFunctionStatus
indent secondary line of comment one space Othergalleria-container.tpl.php5N/A
Expandtheme/theme.inc300
MessageGroupFilenameLineFunctionStatus
There should be no trailing spacesOthertheme.inc5N/A
Use an indent of 2 spaces, with no tabsOthertheme.inc42N/A
Use an indent of 2 spaces, with no tabsOthertheme.inc45N/A