Ubercart Systempay Credit Card Payment - 6.x-1.0

Test status
StatusResult
Last receivedTue, 09/04/2012 - 14:41:27
Last requestedTue, 09/04/2012 - 14:42:02
Last testedTue, 09/04/2012 - 14:43:36
Test count2
Environment status
MySQLpass
Code reviewfail
Overall Resultpass
Branch information
Identifier6.x-1.0
ProjectUbercart Systempay Credit Card Payment
LinkProject information
SummaryPASSED: [[Coder]]: [Code review] 12 minor(s), 0 critical(s), and 5 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
12 minor(s), 0 critical(s), and 5 normal(s)

Non-pass

Test nameMinorCriticalNormal
Expanduc_systempay.install002
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otheruc_systempay.install-1N/A
Commits to the Git repository do not require the CVS $Id$ keyword in each file. (Drupal Docs)Otheruc_systempay.install2N/A
Expanduc_systempay.module1203
MessageGroupFilenameLineFunctionStatus
Commits to the Git repository do not require the CVS $Id$ keyword in each file. (Drupal Docs)Otheruc_systempay.module2N/A
@file description should be on the following line (Drupal Docs)Otheruc_systempay.module5N/A
put a space between the asterisk and the comment textOtheruc_systempay.module15N/A
Use an indent of 2 spaces, with no tabsOtheruc_systempay.module73N/A
Use an indent of 2 spaces, with no tabsOtheruc_systempay.module74N/A
Use an indent of 2 spaces, with no tabsOtheruc_systempay.module75N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtheruc_systempay.module134N/A
put a space between the asterisk and the comment textOtheruc_systempay.module143N/A
put a space between the asterisk and the comment textOtheruc_systempay.module170N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtheruc_systempay.module241N/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_systempay.module309N/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_systempay.module310N/A
There should be no trailing spacesOtheruc_systempay.module373N/A
Use an indent of 2 spaces, with no tabsOtheruc_systempay.module419N/A
Use an indent of 2 spaces, with no tabsOtheruc_systempay.module420N/A

All

Test nameMinorCriticalNormal
Expanduc_systempay.install002
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otheruc_systempay.install-1N/A
Commits to the Git repository do not require the CVS $Id$ keyword in each file. (Drupal Docs)Otheruc_systempay.install2N/A
Expanduc_systempay.module1203
MessageGroupFilenameLineFunctionStatus
Commits to the Git repository do not require the CVS $Id$ keyword in each file. (Drupal Docs)Otheruc_systempay.module2N/A
@file description should be on the following line (Drupal Docs)Otheruc_systempay.module5N/A
put a space between the asterisk and the comment textOtheruc_systempay.module15N/A
Use an indent of 2 spaces, with no tabsOtheruc_systempay.module73N/A
Use an indent of 2 spaces, with no tabsOtheruc_systempay.module74N/A
Use an indent of 2 spaces, with no tabsOtheruc_systempay.module75N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtheruc_systempay.module134N/A
put a space between the asterisk and the comment textOtheruc_systempay.module143N/A
put a space between the asterisk and the comment textOtheruc_systempay.module170N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtheruc_systempay.module241N/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_systempay.module309N/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_systempay.module310N/A
There should be no trailing spacesOtheruc_systempay.module373N/A
Use an indent of 2 spaces, with no tabsOtheruc_systempay.module419N/A
Use an indent of 2 spaces, with no tabsOtheruc_systempay.module420N/A