Ubercart Custom Sized Products - 6.x-2.x

Test status
StatusResult
Last receivedMon, 01/30/2012 - 13:04:37
Last requestedMon, 01/30/2012 - 13:05:03
Last testedMon, 01/30/2012 - 13:05:55
Test count2
Environment status
MySQLpass
Code reviewfail
Overall Resultpass
Branch information
Identifier6.x-2.x
ProjectUbercart Custom Sized Products
LinkProject information
SummaryPASSED: [[Coder]]: [Code review] 16 minor(s), 2 critical(s), and 2 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
16 minor(s), 2 critical(s), and 2 normal(s)

Non-pass

Test nameMinorCriticalNormal
Expanduc_custom_sized_products.install1021
MessageGroupFilenameLineFunctionStatus
There should be no trailing spacesOtheruc_custom_sized_products.install39N/A
There should be no trailing spacesOtheruc_custom_sized_products.install42N/A
There should be no trailing spacesOtheruc_custom_sized_products.install45N/A
Control statements should have one space between the control keyword and opening parenthesisOtheruc_custom_sized_products.install46N/A
There should be no trailing spacesOtheruc_custom_sized_products.install50N/A
There should be no trailing spacesOtheruc_custom_sized_products.install55N/A
There should be no trailing spacesOtheruc_custom_sized_products.install60N/A
In SQL strings, Use db_query() placeholders in place of variables. This is a potential source of SQL injection attacks when the variable can come from user data. (Drupal Docs)Otheruc_custom_sized_products.install63N/A
There should be no trailing spacesOtheruc_custom_sized_products.install64N/A
There should be no trailing spacesOtheruc_custom_sized_products.install67N/A
In SQL strings, Use db_query() placeholders in place of variables. This is a potential source of SQL injection attacks when the variable can come from user data. (Drupal Docs)Otheruc_custom_sized_products.install67N/A
There should be no trailing spacesOtheruc_custom_sized_products.install152N/A
There should be no trailing spacesOtheruc_custom_sized_products.install156N/A
Expanduc_custom_sized_products.module601
MessageGroupFilenameLineFunctionStatus
There should be no trailing spacesOtheruc_custom_sized_products.module325N/A
There should be no trailing spacesOtheruc_custom_sized_products.module335N/A
There should be no trailing spacesOtheruc_custom_sized_products.module340N/A
There should be no trailing spacesOtheruc_custom_sized_products.module361N/A
There should be no trailing spacesOtheruc_custom_sized_products.module367N/A
Remove the empty commented line in your function documentation.Otheruc_custom_sized_products.module632N/A
There should be no trailing spacesOtheruc_custom_sized_products.module828N/A

All

Test nameMinorCriticalNormal
Expanduc_custom_sized_products.install1021
MessageGroupFilenameLineFunctionStatus
There should be no trailing spacesOtheruc_custom_sized_products.install39N/A
There should be no trailing spacesOtheruc_custom_sized_products.install42N/A
There should be no trailing spacesOtheruc_custom_sized_products.install45N/A
Control statements should have one space between the control keyword and opening parenthesisOtheruc_custom_sized_products.install46N/A
There should be no trailing spacesOtheruc_custom_sized_products.install50N/A
There should be no trailing spacesOtheruc_custom_sized_products.install55N/A
There should be no trailing spacesOtheruc_custom_sized_products.install60N/A
In SQL strings, Use db_query() placeholders in place of variables. This is a potential source of SQL injection attacks when the variable can come from user data. (Drupal Docs)Otheruc_custom_sized_products.install63N/A
There should be no trailing spacesOtheruc_custom_sized_products.install64N/A
There should be no trailing spacesOtheruc_custom_sized_products.install67N/A
In SQL strings, Use db_query() placeholders in place of variables. This is a potential source of SQL injection attacks when the variable can come from user data. (Drupal Docs)Otheruc_custom_sized_products.install67N/A
There should be no trailing spacesOtheruc_custom_sized_products.install152N/A
There should be no trailing spacesOtheruc_custom_sized_products.install156N/A
Expanduc_custom_sized_products.module601
MessageGroupFilenameLineFunctionStatus
There should be no trailing spacesOtheruc_custom_sized_products.module325N/A
There should be no trailing spacesOtheruc_custom_sized_products.module335N/A
There should be no trailing spacesOtheruc_custom_sized_products.module340N/A
There should be no trailing spacesOtheruc_custom_sized_products.module361N/A
There should be no trailing spacesOtheruc_custom_sized_products.module367N/A
Remove the empty commented line in your function documentation.Otheruc_custom_sized_products.module632N/A
There should be no trailing spacesOtheruc_custom_sized_products.module828N/A