User points Nodes and Comments - 7.x-1.x

Test status
StatusResult
Last receivedWed, 04/11/2012 - 09:03:06
Last requestedWed, 04/11/2012 - 09:04:00
Last testedWed, 04/11/2012 - 09:04:02
Test count9
Environment status
MySQLpass
Code reviewfail
Overall Resultpass
Branch information
Identifier7.x-1.x
ProjectUser points Nodes and Comments
LinkProject information
SummaryPASSED: [[SimpleTest]]: [MySQL] 218 pass(es); [[Coder]]: [Code review] 7 minor(s), 2 critical(s), and 5 normal(s).
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
218 pass(es), 0 fail(s), and 0 exception(s)
Test namePassFailException
LeafUserpoints Node Visits (UserpointsNodeVisitsTestCase) [Userpoints]13800
LeafUserpoints Nodes and Comments (UserpointsNodeCommentTestCase) [Userpoints]8000

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), 2 critical(s), and 5 normal(s)

Non-pass

Test nameMinorCriticalNormal
Expanduserpoints_nc.module415
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otheruserpoints_nc.module-1N/A
Potential problem: "SELECT FROM {node}" statements should probably be wrapped in db_rewrite_sql() and with the alias for {node} table defined (e.g. {node} n)Otheruserpoints_nc.module370N/A
Use "elseif" in place of "else if"Otheruserpoints_nc.module404N/A
Use an indent of 2 spaces, with no tabsOtheruserpoints_nc.module405N/A
Use "elseif" in place of "else if"Otheruserpoints_nc.module416N/A
Use an indent of 2 spaces, with no tabsOtheruserpoints_nc.module417N/A
Use "elseif" in place of "else if"Otheruserpoints_nc.module514N/A
Use an indent of 2 spaces, with no tabsOtheruserpoints_nc.module515N/A
Use "elseif" in place of "else if"Otheruserpoints_nc.module526N/A
Use an indent of 2 spaces, with no tabsOtheruserpoints_nc.module527N/A
Expanduserpoints_nc_visits/userpoints_nc_visits.module010
MessageGroupFilenameLineFunctionStatus
table names should be enclosed in {curly_brackets}Otheruserpoints_nc_visits.module92N/A

All

Test nameMinorCriticalNormal
Leafuserpoints_nc.install000
Expanduserpoints_nc.module415
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otheruserpoints_nc.module-1N/A
Potential problem: "SELECT FROM {node}" statements should probably be wrapped in db_rewrite_sql() and with the alias for {node} table defined (e.g. {node} n)Otheruserpoints_nc.module370N/A
Use "elseif" in place of "else if"Otheruserpoints_nc.module404N/A
Use an indent of 2 spaces, with no tabsOtheruserpoints_nc.module405N/A
Use "elseif" in place of "else if"Otheruserpoints_nc.module416N/A
Use an indent of 2 spaces, with no tabsOtheruserpoints_nc.module417N/A
Use "elseif" in place of "else if"Otheruserpoints_nc.module514N/A
Use an indent of 2 spaces, with no tabsOtheruserpoints_nc.module515N/A
Use "elseif" in place of "else if"Otheruserpoints_nc.module526N/A
Use an indent of 2 spaces, with no tabsOtheruserpoints_nc.module527N/A
Expanduserpoints_nc.test100
MessageGroupFilenameLineFunctionStatus
in most cases, replace the string function with the drupal_ equivalent string functionsOtheruserpoints_nc.test42N/A
Leafuserpoints_nc_revision/userpoints_nc_revision.module000
Leafuserpoints_nc_visits/userpoints_nc_visits.install000
Expanduserpoints_nc_visits/userpoints_nc_visits.module010
MessageGroupFilenameLineFunctionStatus
table names should be enclosed in {curly_brackets}Otheruserpoints_nc_visits.module92N/A
Expanduserpoints_nc_visits/userpoints_nc_visits.test200
MessageGroupFilenameLineFunctionStatus
There should be no trailing spacesOtheruserpoints_nc_visits.test6N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtheruserpoints_nc_visits.test52N/A