UC WePay - 6.x-1.1

Test status
StatusResult
Last receivedSat, 03/31/2012 - 00:11:38
Last requestedSat, 03/31/2012 - 00:12:03
Last testedSat, 03/31/2012 - 00:12:18
Test count2
Environment status
MySQLpass
Code reviewfail
Overall Resultpass
Branch information
Identifier6.x-1.1
ProjectUC WePay
LinkProject information
SummaryPASSED: [[Coder]]: [Code review] 7 minor(s), 1 critical(s), and 17 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
7 minor(s), 1 critical(s), and 17 normal(s)

Non-pass

Test nameMinorCriticalNormal
Expanduc_wepay.module4012
MessageGroupFilenameLineFunctionStatus
There should be no trailing spacesOtheruc_wepay.module6N/A
There should be no trailing spacesOtheruc_wepay.module8N/A
Remove the empty commented line in your function documentation.Otheruc_wepay.module8N/A
Function documentation should be less than 80 characters per line.Otheruc_wepay.module24N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_wepay.module31N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_wepay.module41N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_wepay.module50N/A
Function documentation should be less than 80 characters per line.Otheruc_wepay.module83N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_wepay.module92N/A
Function documentation should be less than 80 characters per line.Otheruc_wepay.module165N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_wepay.module218N/A
There should be no trailing spacesOtheruc_wepay.module222N/A
Function documentation should be less than 80 characters per line.Otheruc_wepay.module229N/A
Function summaries should be one line only. (Drupal Docs)Otheruc_wepay.module232N/A
There should be no trailing spacesOtheruc_wepay.module233N/A
Use uppercase for PHP constants, e.g. NULL, TRUE, FALSEOtheruc_wepay.module240N/A
Expanduc_wepay.pages.inc315
MessageGroupFilenameLineFunctionStatus
There should be no trailing spacesOtheruc_wepay.pages.inc6N/A
There should be no trailing spacesOtheruc_wepay.pages.inc8N/A
Remove the empty commented line in your function documentation.Otheruc_wepay.pages.inc8N/A
Function summaries should be one line only. (Drupal Docs)Otheruc_wepay.pages.inc12N/A
Potential problem: use the Form API to prevent against CSRF attacks. If you need to use $_POST variables, ensure they are fully sanitized if displayed by using check_plain(), filter_xss() or similar. (Drupal Docs)Otheruc_wepay.pages.inc33N/A
Use the matching Drupal theme functions, not raw HTML.Otheruc_wepay.pages.inc86N/A
The $message argument to drupal_set_message() should be enclosed within t() so that it is translatable.Otheruc_wepay.pages.inc126N/A
Potential problem: drupal_set_message() only accepts filtered text, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized. (Drupal Docs)Otheruc_wepay.pages.inc126N/A
The $message argument to drupal_set_message() should be enclosed within t() so that it is translatable.Otheruc_wepay.pages.inc129N/A

All

Test nameMinorCriticalNormal
Leafuc_wepay.install000
Expanduc_wepay.module4012
MessageGroupFilenameLineFunctionStatus
There should be no trailing spacesOtheruc_wepay.module6N/A
There should be no trailing spacesOtheruc_wepay.module8N/A
Remove the empty commented line in your function documentation.Otheruc_wepay.module8N/A
Function documentation should be less than 80 characters per line.Otheruc_wepay.module24N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_wepay.module31N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_wepay.module41N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_wepay.module50N/A
Function documentation should be less than 80 characters per line.Otheruc_wepay.module83N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_wepay.module92N/A
Function documentation should be less than 80 characters per line.Otheruc_wepay.module165N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_wepay.module218N/A
There should be no trailing spacesOtheruc_wepay.module222N/A
Function documentation should be less than 80 characters per line.Otheruc_wepay.module229N/A
Function summaries should be one line only. (Drupal Docs)Otheruc_wepay.module232N/A
There should be no trailing spacesOtheruc_wepay.module233N/A
Use uppercase for PHP constants, e.g. NULL, TRUE, FALSEOtheruc_wepay.module240N/A
Expanduc_wepay.pages.inc315
MessageGroupFilenameLineFunctionStatus
There should be no trailing spacesOtheruc_wepay.pages.inc6N/A
There should be no trailing spacesOtheruc_wepay.pages.inc8N/A
Remove the empty commented line in your function documentation.Otheruc_wepay.pages.inc8N/A
Function summaries should be one line only. (Drupal Docs)Otheruc_wepay.pages.inc12N/A
Potential problem: use the Form API to prevent against CSRF attacks. If you need to use $_POST variables, ensure they are fully sanitized if displayed by using check_plain(), filter_xss() or similar. (Drupal Docs)Otheruc_wepay.pages.inc33N/A
Use the matching Drupal theme functions, not raw HTML.Otheruc_wepay.pages.inc86N/A
The $message argument to drupal_set_message() should be enclosed within t() so that it is translatable.Otheruc_wepay.pages.inc126N/A
Potential problem: drupal_set_message() only accepts filtered text, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized. (Drupal Docs)Otheruc_wepay.pages.inc126N/A
The $message argument to drupal_set_message() should be enclosed within t() so that it is translatable.Otheruc_wepay.pages.inc129N/A