Ogone | Ubercart payment - 6.x-1.x

Test status
StatusResult
Last receivedTue, 04/03/2012 - 09:29:38
Last requestedTue, 04/03/2012 - 09:30:03
Last testedTue, 04/03/2012 - 09:30:16
Test count2
Environment status
MySQLpass
Code reviewfail
Overall Resultpass
Branch information
Identifier6.x-1.x
ProjectOgone | Ubercart payment
LinkProject information
SummaryPASSED: [[Coder]]: [Code review] 14 minor(s), 1 critical(s), and 65 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
14 minor(s), 1 critical(s), and 65 normal(s)

Non-pass

Test nameMinorCriticalNormal
Expanduc_ogone_payment.install001
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otheruc_ogone_payment.install-1N/A
Expanduc_ogone_payment.module14164
MessageGroupFilenameLineFunctionStatus
Remove the empty commented line in your function documentation.Otheruc_ogone_payment.module11N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_ogone_payment.module23N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_ogone_payment.module41N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_ogone_payment.module62N/A
Separate comments from comment syntax by a space.Otheruc_ogone_payment.module87N/A
Separate comments from comment syntax by a space.Otheruc_ogone_payment.module88N/A
Separate comments from comment syntax by a space.Otheruc_ogone_payment.module96N/A
Separate comments from comment syntax by a space.Otheruc_ogone_payment.module97N/A
Separate comments from comment syntax by a space.Otheruc_ogone_payment.module101N/A
Separate comments from comment syntax by a space.Otheruc_ogone_payment.module102N/A
Separate comments from comment syntax by a space.Otheruc_ogone_payment.module104N/A
Separate comments from comment syntax by a space.Otheruc_ogone_payment.module109N/A
put a space between the asterisk and the comment textOtheruc_ogone_payment.module116N/A
Control statements should have one space between the control keyword and opening parenthesisOtheruc_ogone_payment.module122N/A
Control statements should have one space between the control keyword and opening parenthesisOtheruc_ogone_payment.module143N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtheruc_ogone_payment.module155N/A
Use the matching Drupal theme functions, not raw HTML.Otheruc_ogone_payment.module175N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_ogone_payment.module189N/A
Separate comments from comment syntax by a space.Otheruc_ogone_payment.module217N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_ogone_payment.module226N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_ogone_payment.module238N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_ogone_payment.module244N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_ogone_payment.module250N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_ogone_payment.module256N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_ogone_payment.module262N/A
put a space between the asterisk and the comment textOtheruc_ogone_payment.module300N/A
Separate comments from comment syntax by a space.Otheruc_ogone_payment.module300N/A
Separate comments from comment syntax by a space.Otheruc_ogone_payment.module308N/A
Separate comments from comment syntax by a space.Otheruc_ogone_payment.module317N/A
Separate comments from comment syntax by a space.Otheruc_ogone_payment.module323N/A
Separate comments from comment syntax by a space.Otheruc_ogone_payment.module328N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtheruc_ogone_payment.module343N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtheruc_ogone_payment.module344N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtheruc_ogone_payment.module345N/A
Separate comments from comment syntax by a space.Otheruc_ogone_payment.module345N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtheruc_ogone_payment.module350N/A
Separate comments from comment syntax by a space.Otheruc_ogone_payment.module358N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtheruc_ogone_payment.module400N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtheruc_ogone_payment.module404N/A
use lowercase html tags to comply with XHTMLOtheruc_ogone_payment.module404N/A
Use the Form API to build forms to help prevent against CSRF attacks. (Drupal Docs)Otheruc_ogone_payment.module404N/A
put a space between the asterisk and the comment textOtheruc_ogone_payment.module404N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtheruc_ogone_payment.module408N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtheruc_ogone_payment.module409N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtheruc_ogone_payment.module411N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtheruc_ogone_payment.module413N/A
use lowercase html tags to comply with XHTMLOtheruc_ogone_payment.module413N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtheruc_ogone_payment.module418N/A
use lowercase html tags to comply with XHTMLOtheruc_ogone_payment.module418N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtheruc_ogone_payment.module419N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtheruc_ogone_payment.module426N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtheruc_ogone_payment.module434N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtheruc_ogone_payment.module436N/A
Control statements should have one space between the control keyword and opening parenthesisOtheruc_ogone_payment.module453N/A
else statements should begin on a new lineOtheruc_ogone_payment.module454N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_ogone_payment.module467N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_ogone_payment.module468N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_ogone_payment.module470N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_ogone_payment.module471N/A
Arrays should be formatted with a space separating each element and assignment operatorOtheruc_ogone_payment.module472N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_ogone_payment.module472N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_ogone_payment.module473N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_ogone_payment.module475N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_ogone_payment.module476N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_ogone_payment.module477N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_ogone_payment.module478N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_ogone_payment.module479N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_ogone_payment.module480N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_ogone_payment.module482N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_ogone_payment.module483N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_ogone_payment.module484N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_ogone_payment.module485N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_ogone_payment.module486N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtheruc_ogone_payment.module489N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtheruc_ogone_payment.module496N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtheruc_ogone_payment.module497N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtheruc_ogone_payment.module498N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtheruc_ogone_payment.module506N/A
Doxygen uses @todo and @bug to markup things to be done.Otheruc_ogone_payment.module521N/A

All

Test nameMinorCriticalNormal
Expanduc_ogone_payment.install001
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otheruc_ogone_payment.install-1N/A
Expanduc_ogone_payment.module14164
MessageGroupFilenameLineFunctionStatus
Remove the empty commented line in your function documentation.Otheruc_ogone_payment.module11N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_ogone_payment.module23N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_ogone_payment.module41N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_ogone_payment.module62N/A
Separate comments from comment syntax by a space.Otheruc_ogone_payment.module87N/A
Separate comments from comment syntax by a space.Otheruc_ogone_payment.module88N/A
Separate comments from comment syntax by a space.Otheruc_ogone_payment.module96N/A
Separate comments from comment syntax by a space.Otheruc_ogone_payment.module97N/A
Separate comments from comment syntax by a space.Otheruc_ogone_payment.module101N/A
Separate comments from comment syntax by a space.Otheruc_ogone_payment.module102N/A
Separate comments from comment syntax by a space.Otheruc_ogone_payment.module104N/A
Separate comments from comment syntax by a space.Otheruc_ogone_payment.module109N/A
put a space between the asterisk and the comment textOtheruc_ogone_payment.module116N/A
Control statements should have one space between the control keyword and opening parenthesisOtheruc_ogone_payment.module122N/A
Control statements should have one space between the control keyword and opening parenthesisOtheruc_ogone_payment.module143N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtheruc_ogone_payment.module155N/A
Use the matching Drupal theme functions, not raw HTML.Otheruc_ogone_payment.module175N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_ogone_payment.module189N/A
Separate comments from comment syntax by a space.Otheruc_ogone_payment.module217N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_ogone_payment.module226N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_ogone_payment.module238N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_ogone_payment.module244N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_ogone_payment.module250N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_ogone_payment.module256N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_ogone_payment.module262N/A
put a space between the asterisk and the comment textOtheruc_ogone_payment.module300N/A
Separate comments from comment syntax by a space.Otheruc_ogone_payment.module300N/A
Separate comments from comment syntax by a space.Otheruc_ogone_payment.module308N/A
Separate comments from comment syntax by a space.Otheruc_ogone_payment.module317N/A
Separate comments from comment syntax by a space.Otheruc_ogone_payment.module323N/A
Separate comments from comment syntax by a space.Otheruc_ogone_payment.module328N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtheruc_ogone_payment.module343N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtheruc_ogone_payment.module344N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtheruc_ogone_payment.module345N/A
Separate comments from comment syntax by a space.Otheruc_ogone_payment.module345N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtheruc_ogone_payment.module350N/A
Separate comments from comment syntax by a space.Otheruc_ogone_payment.module358N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtheruc_ogone_payment.module400N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtheruc_ogone_payment.module404N/A
use lowercase html tags to comply with XHTMLOtheruc_ogone_payment.module404N/A
Use the Form API to build forms to help prevent against CSRF attacks. (Drupal Docs)Otheruc_ogone_payment.module404N/A
put a space between the asterisk and the comment textOtheruc_ogone_payment.module404N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtheruc_ogone_payment.module408N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtheruc_ogone_payment.module409N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtheruc_ogone_payment.module411N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtheruc_ogone_payment.module413N/A
use lowercase html tags to comply with XHTMLOtheruc_ogone_payment.module413N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtheruc_ogone_payment.module418N/A
use lowercase html tags to comply with XHTMLOtheruc_ogone_payment.module418N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtheruc_ogone_payment.module419N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtheruc_ogone_payment.module426N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtheruc_ogone_payment.module434N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtheruc_ogone_payment.module436N/A
Control statements should have one space between the control keyword and opening parenthesisOtheruc_ogone_payment.module453N/A
else statements should begin on a new lineOtheruc_ogone_payment.module454N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_ogone_payment.module467N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_ogone_payment.module468N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_ogone_payment.module470N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_ogone_payment.module471N/A
Arrays should be formatted with a space separating each element and assignment operatorOtheruc_ogone_payment.module472N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_ogone_payment.module472N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_ogone_payment.module473N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_ogone_payment.module475N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_ogone_payment.module476N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_ogone_payment.module477N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_ogone_payment.module478N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_ogone_payment.module479N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_ogone_payment.module480N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_ogone_payment.module482N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_ogone_payment.module483N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_ogone_payment.module484N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_ogone_payment.module485N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruc_ogone_payment.module486N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtheruc_ogone_payment.module489N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtheruc_ogone_payment.module496N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtheruc_ogone_payment.module497N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtheruc_ogone_payment.module498N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtheruc_ogone_payment.module506N/A
Doxygen uses @todo and @bug to markup things to be done.Otheruc_ogone_payment.module521N/A