BlogAPI Alternative - 6.x-1.x

Test status
StatusResult
Last receivedSun, 01/06/2013 - 02:48:02
Last requestedSun, 01/06/2013 - 02:48:03
Last testedSun, 01/06/2013 - 02:48:24
Test count4
Environment status
MySQLpass
Code reviewfail
Overall Resultpass
Branch information
Identifier6.x-1.x
ProjectBlogAPI Alternative
LinkProject information
SummaryPASSED: [[Coder]]: [Code review] 4 minor(s), 2 critical(s), and 4 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
4 minor(s), 2 critical(s), and 4 normal(s)

Non-pass

Test nameMinorCriticalNormal
Expandblogapi.install002
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherblogapi.install-1N/A
The $message argument to drupal_set_message() should be enclosed within t() so that it is translatable.Otherblogapi.install69N/A
Expandblogapi.module422
MessageGroupFilenameLineFunctionStatus
There should be no trailing spacesOtherblogapi.module327N/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)Otherblogapi.module378N/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)Otherblogapi.module381N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherblogapi.module417N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherblogapi.module422N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherblogapi.module444N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtherblogapi.module751N/A
Use "elseif" in place of "else if"Otherblogapi.module905N/A

All

Test nameMinorCriticalNormal
Expandblogapi.install002
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherblogapi.install-1N/A
The $message argument to drupal_set_message() should be enclosed within t() so that it is translatable.Otherblogapi.install69N/A
Expandblogapi.module422
MessageGroupFilenameLineFunctionStatus
There should be no trailing spacesOtherblogapi.module327N/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)Otherblogapi.module378N/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)Otherblogapi.module381N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherblogapi.module417N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherblogapi.module422N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherblogapi.module444N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtherblogapi.module751N/A
Use "elseif" in place of "else if"Otherblogapi.module905N/A