Guiders-JS - 7.x-1.2

Test status
StatusResult
Last receivedTue, 01/10/2012 - 12:30:52
Last requestedTue, 01/10/2012 - 12:31:28
Last testedTue, 01/10/2012 - 12:31:34
Test count2
Environment status
MySQLpass
Code reviewfail
Overall Resultpass
Branch information
Identifier7.x-1.2
ProjectGuiders-JS
LinkProject information
SummaryPASSED: [[SimpleTest]]: [MySQL] 0 pass(es); [[Coder]]: [Code review] 47 minor(s), 0 critical(s), and 41 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
47 minor(s), 0 critical(s), and 41 normal(s)

Non-pass

Test nameMinorCriticalNormal
Expandguiders_js.admin.inc19019
MessageGroupFilenameLineFunctionStatus
There should be no trailing spacesOtherguiders_js.admin.inc16N/A
There should be no trailing spacesOtherguiders_js.admin.inc17N/A
There should be no trailing spacesOtherguiders_js.admin.inc18N/A
There should be no trailing spacesOtherguiders_js.admin.inc19N/A
There should be no trailing spacesOtherguiders_js.admin.inc39N/A
There should be no trailing spacesOtherguiders_js.admin.inc41N/A
There should be no trailing spacesOtherguiders_js.admin.inc42N/A
There should be no trailing spacesOtherguiders_js.admin.inc44N/A
There should be no trailing spacesOtherguiders_js.admin.inc45N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.admin.inc55N/A
If you define a @param or @return, you should document it as well.Otherguiders_js.admin.inc71N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.admin.inc84N/A
There should be no trailing spacesOtherguiders_js.admin.inc95N/A
There should be no trailing spacesOtherguiders_js.admin.inc155N/A
There should be no trailing spacesOtherguiders_js.admin.inc156N/A
There should be no trailing spacesOtherguiders_js.admin.inc225N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.admin.inc231N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.admin.inc239N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.admin.inc264N/A
There should be no trailing spacesOtherguiders_js.admin.inc267N/A
There should be no trailing spacesOtherguiders_js.admin.inc277N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.admin.inc277N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.admin.inc316N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.admin.inc332N/A
There should be no trailing spacesOtherguiders_js.admin.inc374N/A
There should be no trailing spacesOtherguiders_js.admin.inc375N/A
There should be no trailing spacesOtherguiders_js.admin.inc423N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.admin.inc429N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.admin.inc437N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.admin.inc461N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.admin.inc550N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.admin.inc615N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.admin.inc633N/A
There should be no trailing spacesOtherguiders_js.admin.inc712N/A
If you define a @param or @return, you should document it as well.Otherguiders_js.admin.inc737N/A
If you define a @param or @return, you should document it as well.Otherguiders_js.admin.inc740N/A
If you define a @param or @return, you should document it as well.Otherguiders_js.admin.inc743N/A
Remove the empty commented line in your function documentation.Otherguiders_js.admin.inc745N/A
Expandguiders_js.install206
MessageGroupFilenameLineFunctionStatus
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.install37N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.install46N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.install78N/A
There should be no trailing spacesOtherguiders_js.install86N/A
There should be no trailing spacesOtherguiders_js.install95N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.install121N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.install163N/A
Control statements should have one space between the control keyword and opening parenthesisOtherguiders_js.install238N/A
Expandguiders_js.module26016
MessageGroupFilenameLineFunctionStatus
There should be no trailing spacesOtherguiders_js.module16N/A
There should be no trailing spacesOtherguiders_js.module21N/A
There should be no trailing spacesOtherguiders_js.module22N/A
There should be no trailing spacesOtherguiders_js.module38N/A
There should be no trailing spacesOtherguiders_js.module49N/A
There should be no trailing spacesOtherguiders_js.module51N/A
There should be no trailing spacesOtherguiders_js.module61N/A
There should be no trailing spacesOtherguiders_js.module66N/A
Control statements should have one space between the control keyword and opening parenthesisOtherguiders_js.module76N/A
There should be no trailing spacesOtherguiders_js.module83N/A
There should be no trailing spacesOtherguiders_js.module97N/A
There should be no trailing spacesOtherguiders_js.module132N/A
There should be no trailing spacesOtherguiders_js.module133N/A
Remove the empty commented line in your function documentation.Otherguiders_js.module143N/A
There should be no trailing spacesOtherguiders_js.module153N/A
There should be no trailing spacesOtherguiders_js.module162N/A
There should be no trailing spacesOtherguiders_js.module163N/A
There should be no trailing spacesOtherguiders_js.module171N/A
There should be no trailing spacesOtherguiders_js.module172N/A
There should be no trailing spacesOtherguiders_js.module175N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.module202N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.module210N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.module238N/A
There should be no trailing spacesOtherguiders_js.module241N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.module245N/A
There should be no trailing spacesOtherguiders_js.module248N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.module252N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.module260N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.module267N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.module274N/A
Control statements should have one space between the control keyword and opening parenthesisOtherguiders_js.module294N/A
There should be no trailing spacesOtherguiders_js.module294N/A
There should be no trailing spacesOtherguiders_js.module297N/A
There should be no trailing spacesOtherguiders_js.module298N/A
There should be no trailing spacesOtherguiders_js.module303N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.module325N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.module329N/A
Function documentation should be less than 80 characters per line.Otherguiders_js.module373N/A
There should be no trailing spacesOtherguiders_js.module391N/A
Function documentation should be less than 80 characters per line.Otherguiders_js.module407N/A
Function documentation should be less than 80 characters per line.Otherguiders_js.module429N/A
There should be no trailing spacesOtherguiders_js.module442N/A

All

Test nameMinorCriticalNormal
Expandguiders_js.admin.inc19019
MessageGroupFilenameLineFunctionStatus
There should be no trailing spacesOtherguiders_js.admin.inc16N/A
There should be no trailing spacesOtherguiders_js.admin.inc17N/A
There should be no trailing spacesOtherguiders_js.admin.inc18N/A
There should be no trailing spacesOtherguiders_js.admin.inc19N/A
There should be no trailing spacesOtherguiders_js.admin.inc39N/A
There should be no trailing spacesOtherguiders_js.admin.inc41N/A
There should be no trailing spacesOtherguiders_js.admin.inc42N/A
There should be no trailing spacesOtherguiders_js.admin.inc44N/A
There should be no trailing spacesOtherguiders_js.admin.inc45N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.admin.inc55N/A
If you define a @param or @return, you should document it as well.Otherguiders_js.admin.inc71N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.admin.inc84N/A
There should be no trailing spacesOtherguiders_js.admin.inc95N/A
There should be no trailing spacesOtherguiders_js.admin.inc155N/A
There should be no trailing spacesOtherguiders_js.admin.inc156N/A
There should be no trailing spacesOtherguiders_js.admin.inc225N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.admin.inc231N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.admin.inc239N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.admin.inc264N/A
There should be no trailing spacesOtherguiders_js.admin.inc267N/A
There should be no trailing spacesOtherguiders_js.admin.inc277N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.admin.inc277N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.admin.inc316N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.admin.inc332N/A
There should be no trailing spacesOtherguiders_js.admin.inc374N/A
There should be no trailing spacesOtherguiders_js.admin.inc375N/A
There should be no trailing spacesOtherguiders_js.admin.inc423N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.admin.inc429N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.admin.inc437N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.admin.inc461N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.admin.inc550N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.admin.inc615N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.admin.inc633N/A
There should be no trailing spacesOtherguiders_js.admin.inc712N/A
If you define a @param or @return, you should document it as well.Otherguiders_js.admin.inc737N/A
If you define a @param or @return, you should document it as well.Otherguiders_js.admin.inc740N/A
If you define a @param or @return, you should document it as well.Otherguiders_js.admin.inc743N/A
Remove the empty commented line in your function documentation.Otherguiders_js.admin.inc745N/A
Expandguiders_js.install206
MessageGroupFilenameLineFunctionStatus
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.install37N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.install46N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.install78N/A
There should be no trailing spacesOtherguiders_js.install86N/A
There should be no trailing spacesOtherguiders_js.install95N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.install121N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.install163N/A
Control statements should have one space between the control keyword and opening parenthesisOtherguiders_js.install238N/A
Expandguiders_js.module26016
MessageGroupFilenameLineFunctionStatus
There should be no trailing spacesOtherguiders_js.module16N/A
There should be no trailing spacesOtherguiders_js.module21N/A
There should be no trailing spacesOtherguiders_js.module22N/A
There should be no trailing spacesOtherguiders_js.module38N/A
There should be no trailing spacesOtherguiders_js.module49N/A
There should be no trailing spacesOtherguiders_js.module51N/A
There should be no trailing spacesOtherguiders_js.module61N/A
There should be no trailing spacesOtherguiders_js.module66N/A
Control statements should have one space between the control keyword and opening parenthesisOtherguiders_js.module76N/A
There should be no trailing spacesOtherguiders_js.module83N/A
There should be no trailing spacesOtherguiders_js.module97N/A
There should be no trailing spacesOtherguiders_js.module132N/A
There should be no trailing spacesOtherguiders_js.module133N/A
Remove the empty commented line in your function documentation.Otherguiders_js.module143N/A
There should be no trailing spacesOtherguiders_js.module153N/A
There should be no trailing spacesOtherguiders_js.module162N/A
There should be no trailing spacesOtherguiders_js.module163N/A
There should be no trailing spacesOtherguiders_js.module171N/A
There should be no trailing spacesOtherguiders_js.module172N/A
There should be no trailing spacesOtherguiders_js.module175N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.module202N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.module210N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.module238N/A
There should be no trailing spacesOtherguiders_js.module241N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.module245N/A
There should be no trailing spacesOtherguiders_js.module248N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.module252N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.module260N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.module267N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.module274N/A
Control statements should have one space between the control keyword and opening parenthesisOtherguiders_js.module294N/A
There should be no trailing spacesOtherguiders_js.module294N/A
There should be no trailing spacesOtherguiders_js.module297N/A
There should be no trailing spacesOtherguiders_js.module298N/A
There should be no trailing spacesOtherguiders_js.module303N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.module325N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherguiders_js.module329N/A
Function documentation should be less than 80 characters per line.Otherguiders_js.module373N/A
There should be no trailing spacesOtherguiders_js.module391N/A
Function documentation should be less than 80 characters per line.Otherguiders_js.module407N/A
Function documentation should be less than 80 characters per line.Otherguiders_js.module429N/A
There should be no trailing spacesOtherguiders_js.module442N/A