Image Zoomer - 7.x-1.0

Test status
StatusResult
Last receivedThu, 01/31/2013 - 12:33:10
Last requestedThu, 01/31/2013 - 12:34:02
Last testedThu, 01/31/2013 - 12:34:45
Test count2
Environment status
MySQLpass
Code reviewfail
Overall Resultpass
Branch information
Identifier7.x-1.0
ProjectImage Zoomer
LinkProject information
SummaryPASSED: [[Coder]]: [Code review] 17 minor(s), 2 critical(s), and 10 normal(s); [[SimpleTest]]: [MySQL] 0 pass(es).
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
17 minor(s), 2 critical(s), and 10 normal(s)

Non-pass

Test nameMinorCriticalNormal
Expandimagezoomer.module17210
MessageGroupFilenameLineFunctionStatus
There should be no trailing spacesOtherimagezoomer.module24N/A
There should be no trailing spacesOtherimagezoomer.module25N/A
There should be no trailing spacesOtherimagezoomer.module33N/A
There should be no trailing spacesOtherimagezoomer.module34N/A
There should be no trailing spacesOtherimagezoomer.module50N/A
There should be no trailing spacesOtherimagezoomer.module58N/A
There should be no trailing spacesOtherimagezoomer.module65N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherimagezoomer.module81N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherimagezoomer.module104N/A
There should be no trailing spacesOtherimagezoomer.module169N/A
There should be no trailing spacesOtherimagezoomer.module185N/A
There should be no trailing spacesOtherimagezoomer.module241N/A
There should be no trailing spacesOtherimagezoomer.module247N/A
There should be no trailing spacesOtherimagezoomer.module304N/A
There should be no trailing spacesOtherimagezoomer.module313N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Otherimagezoomer.module378N/A
Use uppercase for PHP constants, e.g. NULL, TRUE, FALSEOtherimagezoomer.module378N/A
Potential problem: drupal_set_message() only accepts filtered text, be sure all !placeholders for $variables in t() are fully sanitized using check_plain(), filter_xss() or similar. (Drupal Docs)Otherimagezoomer.module379N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Otherimagezoomer.module384N/A
Use uppercase for PHP constants, e.g. NULL, TRUE, FALSEOtherimagezoomer.module384N/A
Potential problem: drupal_set_message() only accepts filtered text, be sure all !placeholders for $variables in t() are fully sanitized using check_plain(), filter_xss() or similar. (Drupal Docs)Otherimagezoomer.module385N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Otherimagezoomer.module401N/A
The $string argument to t() should not begin or end with a space. (Drupal Docs)Otherimagezoomer.module401N/A
Use uppercase for PHP constants, e.g. NULL, TRUE, FALSEOtherimagezoomer.module401N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Otherimagezoomer.module468N/A
The $string argument to t() should not begin or end with a space. (Drupal Docs)Otherimagezoomer.module468N/A
Use uppercase for PHP constants, e.g. NULL, TRUE, FALSEOtherimagezoomer.module468N/A
There should be no trailing spacesOtherimagezoomer.module502N/A
There should be no trailing spacesOtherimagezoomer.module561N/A

All

Test nameMinorCriticalNormal
Expandimagezoomer.module17210
MessageGroupFilenameLineFunctionStatus
There should be no trailing spacesOtherimagezoomer.module24N/A
There should be no trailing spacesOtherimagezoomer.module25N/A
There should be no trailing spacesOtherimagezoomer.module33N/A
There should be no trailing spacesOtherimagezoomer.module34N/A
There should be no trailing spacesOtherimagezoomer.module50N/A
There should be no trailing spacesOtherimagezoomer.module58N/A
There should be no trailing spacesOtherimagezoomer.module65N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherimagezoomer.module81N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherimagezoomer.module104N/A
There should be no trailing spacesOtherimagezoomer.module169N/A
There should be no trailing spacesOtherimagezoomer.module185N/A
There should be no trailing spacesOtherimagezoomer.module241N/A
There should be no trailing spacesOtherimagezoomer.module247N/A
There should be no trailing spacesOtherimagezoomer.module304N/A
There should be no trailing spacesOtherimagezoomer.module313N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Otherimagezoomer.module378N/A
Use uppercase for PHP constants, e.g. NULL, TRUE, FALSEOtherimagezoomer.module378N/A
Potential problem: drupal_set_message() only accepts filtered text, be sure all !placeholders for $variables in t() are fully sanitized using check_plain(), filter_xss() or similar. (Drupal Docs)Otherimagezoomer.module379N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Otherimagezoomer.module384N/A
Use uppercase for PHP constants, e.g. NULL, TRUE, FALSEOtherimagezoomer.module384N/A
Potential problem: drupal_set_message() only accepts filtered text, be sure all !placeholders for $variables in t() are fully sanitized using check_plain(), filter_xss() or similar. (Drupal Docs)Otherimagezoomer.module385N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Otherimagezoomer.module401N/A
The $string argument to t() should not begin or end with a space. (Drupal Docs)Otherimagezoomer.module401N/A
Use uppercase for PHP constants, e.g. NULL, TRUE, FALSEOtherimagezoomer.module401N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Otherimagezoomer.module468N/A
The $string argument to t() should not begin or end with a space. (Drupal Docs)Otherimagezoomer.module468N/A
Use uppercase for PHP constants, e.g. NULL, TRUE, FALSEOtherimagezoomer.module468N/A
There should be no trailing spacesOtherimagezoomer.module502N/A
There should be no trailing spacesOtherimagezoomer.module561N/A