Commerce CCAvenue Payment Gateway - 7.x-1.x

Test status
StatusResult
Last receivedSun, 03/25/2012 - 15:17:38
Last requestedSun, 03/25/2012 - 15:18:03
Last testedSun, 03/25/2012 - 15:18:27
Test count8
Environment status
MySQLpass
Code reviewfail
Overall Resultpass
Branch information
Identifier7.x-1.x
ProjectCommerce CCAvenue Payment Gateway
LinkProject information
SummaryPASSED: [[Coder]]: [Code review] 8 minor(s), 0 critical(s), and 9 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
8 minor(s), 0 critical(s), and 9 normal(s)

Non-pass

Test nameMinorCriticalNormal
Expandcommerce_ccavenue.module809
MessageGroupFilenameLineFunctionStatus
put a space between the asterisk and the comment textOthercommerce_ccavenue.module80N/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)Othercommerce_ccavenue.module158N/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)Othercommerce_ccavenue.module159N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othercommerce_ccavenue.module159N/A
Function summaries should be one line only. (Drupal Docs)Othercommerce_ccavenue.module189N/A
If you define a @param or @return, you should document it as well.Othercommerce_ccavenue.module190N/A
If you define a @param or @return, you should document it as well.Othercommerce_ccavenue.module192N/A
If you define a @param or @return, you should document it as well.Othercommerce_ccavenue.module194N/A
If you define a @param or @return, you should document it as well.Othercommerce_ccavenue.module196N/A
Use Drupal's format_date(), not PHP's default date().Othercommerce_ccavenue.module210N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthercommerce_ccavenue.module238N/A
Separate comments from comment syntax by a space.Othercommerce_ccavenue.module241N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthercommerce_ccavenue.module248N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthercommerce_ccavenue.module252N/A
Separate comments from comment syntax by a space.Othercommerce_ccavenue.module253N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthercommerce_ccavenue.module260N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthercommerce_ccavenue.module262N/A

All

Test nameMinorCriticalNormal
Expandcommerce_ccavenue.module809
MessageGroupFilenameLineFunctionStatus
put a space between the asterisk and the comment textOthercommerce_ccavenue.module80N/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)Othercommerce_ccavenue.module158N/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)Othercommerce_ccavenue.module159N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othercommerce_ccavenue.module159N/A
Function summaries should be one line only. (Drupal Docs)Othercommerce_ccavenue.module189N/A
If you define a @param or @return, you should document it as well.Othercommerce_ccavenue.module190N/A
If you define a @param or @return, you should document it as well.Othercommerce_ccavenue.module192N/A
If you define a @param or @return, you should document it as well.Othercommerce_ccavenue.module194N/A
If you define a @param or @return, you should document it as well.Othercommerce_ccavenue.module196N/A
Use Drupal's format_date(), not PHP's default date().Othercommerce_ccavenue.module210N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthercommerce_ccavenue.module238N/A
Separate comments from comment syntax by a space.Othercommerce_ccavenue.module241N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthercommerce_ccavenue.module248N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthercommerce_ccavenue.module252N/A
Separate comments from comment syntax by a space.Othercommerce_ccavenue.module253N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthercommerce_ccavenue.module260N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthercommerce_ccavenue.module262N/A