EHS Ad Bridge - 6.x-1.3

Test status
StatusResult
Last receivedMon, 12/05/2011 - 23:29:37
Last requestedMon, 12/05/2011 - 23:30:02
Last testedMon, 12/05/2011 - 23:30:15
Test count2
Environment status
MySQLpass
Code reviewfail
Overall Resultpass
Branch information
Identifier6.x-1.3
ProjectEHS Ad Bridge
LinkProject information
SummaryPASSED: [[Coder]]: [Code review] 0 minor(s), 0 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
0 minor(s), 0 critical(s), and 10 normal(s)

Non-pass

Test nameMinorCriticalNormal
Expandehs_adbridge.admin.inc004
MessageGroupFilenameLineFunctionStatus
Use the matching Drupal theme functions, not raw HTML.Otherehs_adbridge.admin.inc15N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherehs_adbridge.admin.inc23N/A
The $message argument to drupal_set_message() should be enclosed within t() so that it is translatable.Otherehs_adbridge.admin.inc111N/A
Use the matching Drupal theme functions, not raw HTML.Otherehs_adbridge.admin.inc150N/A
Expandehs_adbridge.module006
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherehs_adbridge.module-1N/A
Remove the empty commented line in your function documentation.Otherehs_adbridge.module13N/A
Remove the empty commented line in your function documentation.Otherehs_adbridge.module35N/A
Administrative menu items should have a description.Otherehs_adbridge.module78N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherehs_adbridge.module88N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Otherehs_adbridge.module150N/A

All

Test nameMinorCriticalNormal
Expandehs_adbridge.admin.inc004
MessageGroupFilenameLineFunctionStatus
Use the matching Drupal theme functions, not raw HTML.Otherehs_adbridge.admin.inc15N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherehs_adbridge.admin.inc23N/A
The $message argument to drupal_set_message() should be enclosed within t() so that it is translatable.Otherehs_adbridge.admin.inc111N/A
Use the matching Drupal theme functions, not raw HTML.Otherehs_adbridge.admin.inc150N/A
Leafehs_adbridge.install000
Expandehs_adbridge.module006
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherehs_adbridge.module-1N/A
Remove the empty commented line in your function documentation.Otherehs_adbridge.module13N/A
Remove the empty commented line in your function documentation.Otherehs_adbridge.module35N/A
Administrative menu items should have a description.Otherehs_adbridge.module78N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherehs_adbridge.module88N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Otherehs_adbridge.module150N/A