Sms payments Mobio - 6.x-1.x

Test status
StatusResult
Last receivedSun, 01/15/2012 - 11:18:36
Last requestedSun, 01/15/2012 - 11:19:03
Last testedSun, 01/15/2012 - 11:19:25
Test count2
Environment status
MySQLpass
Code reviewfail
Overall Resultpass
Branch information
Identifier6.x-1.x
ProjectSms payments Mobio
LinkProject information
SummaryPASSED: [[Coder]]: [Code review] 17 minor(s), 2 critical(s), and 8 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 8 normal(s)

Non-pass

Test nameMinorCriticalNormal
Expandsms_mobio.module1528
MessageGroupFilenameLineFunctionStatus
Function summaries should be one line only. (Drupal Docs)Othersms_mobio.module9N/A
There should be no trailing spacesOthersms_mobio.module10N/A
If you define a @param or @return, you should document it as well.Othersms_mobio.module11N/A
If you define a @param or @return, you should document it as well.Othersms_mobio.module12N/A
There should be no trailing spacesOthersms_mobio.module20N/A
There should be no trailing spacesOthersms_mobio.module29N/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)Othersms_mobio.module31N/A
There should be no trailing spacesOthersms_mobio.module42N/A
Function summaries should be one line only. (Drupal Docs)Othersms_mobio.module76N/A
There should be no trailing spacesOthersms_mobio.module78N/A
Remove the empty commented line in your function documentation.Othersms_mobio.module78N/A
Use Drupal's format_date(), not PHP's default date().Othersms_mobio.module90N/A
There should be no trailing spacesOthersms_mobio.module95N/A
There should be no trailing spacesOthersms_mobio.module101N/A
There should be no trailing spacesOthersms_mobio.module103N/A
There should be no trailing spacesOthersms_mobio.module108N/A
There should be no trailing spacesOthersms_mobio.module110N/A
There should be no trailing spacesOthersms_mobio.module112N/A
There should be no trailing spacesOthersms_mobio.module114N/A
Use Drupal's format_date(), not PHP's default date().Othersms_mobio.module115N/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)Othersms_mobio.module120N/A
Remove the empty commented line in your function documentation.Othersms_mobio.module136N/A
There should be no trailing spacesOthersms_mobio.module235N/A
indent secondary line of comment one space Othersms_mobio.module240N/A
indent secondary line of comment one space Othersms_mobio.module241N/A

All

Test nameMinorCriticalNormal
Expandsms_mobio.install200
MessageGroupFilenameLineFunctionStatus
There should be no trailing spacesOthersms_mobio.install1N/A
There should be no trailing spacesOthersms_mobio.install11N/A
Expandsms_mobio.module1528
MessageGroupFilenameLineFunctionStatus
Function summaries should be one line only. (Drupal Docs)Othersms_mobio.module9N/A
There should be no trailing spacesOthersms_mobio.module10N/A
If you define a @param or @return, you should document it as well.Othersms_mobio.module11N/A
If you define a @param or @return, you should document it as well.Othersms_mobio.module12N/A
There should be no trailing spacesOthersms_mobio.module20N/A
There should be no trailing spacesOthersms_mobio.module29N/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)Othersms_mobio.module31N/A
There should be no trailing spacesOthersms_mobio.module42N/A
Function summaries should be one line only. (Drupal Docs)Othersms_mobio.module76N/A
There should be no trailing spacesOthersms_mobio.module78N/A
Remove the empty commented line in your function documentation.Othersms_mobio.module78N/A
Use Drupal's format_date(), not PHP's default date().Othersms_mobio.module90N/A
There should be no trailing spacesOthersms_mobio.module95N/A
There should be no trailing spacesOthersms_mobio.module101N/A
There should be no trailing spacesOthersms_mobio.module103N/A
There should be no trailing spacesOthersms_mobio.module108N/A
There should be no trailing spacesOthersms_mobio.module110N/A
There should be no trailing spacesOthersms_mobio.module112N/A
There should be no trailing spacesOthersms_mobio.module114N/A
Use Drupal's format_date(), not PHP's default date().Othersms_mobio.module115N/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)Othersms_mobio.module120N/A
Remove the empty commented line in your function documentation.Othersms_mobio.module136N/A
There should be no trailing spacesOthersms_mobio.module235N/A
indent secondary line of comment one space Othersms_mobio.module240N/A
indent secondary line of comment one space Othersms_mobio.module241N/A