BlogAPI Alternative - 7.x-1.x

Test status
StatusResult
Last receivedMon, 12/24/2012 - 12:44:56
Last requestedMon, 12/24/2012 - 12:45:50
Last testedMon, 12/24/2012 - 12:46:27
Test count2
Environment status
MySQLfail
Code reviewfail
Overall Resultfail
Branch information
Identifier7.x-1.x
ProjectBlogAPI Alternative
LinkProject information
SummaryFAILED: [[Coder]]: [Code review] 19 minor(s), 2 critical(s), and 31 normal(s); [[SimpleTest]]: [MySQL] 4 pass(es), 2 fail(s), and 2 exception(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
    Review the test failures below then if necessary run them locally until the problem has been fixed.
  10. Complete test suite with all tests passing
4 pass(es), 2 fail(s), and 2 exception(s)

Non-pass

Test namePassFailException
ExpandBlog API functionality (BlogAPITestCase) [Blog API]422
MessageGroupFilenameLineFunctionStatus
The test did not complete due to a fatal error.Completion checkblogapi.test19BlogAPITestCase->testBlogAPI()
key() expects parameter 1 to be array, string givenWarningxmlrpc.inc557_xmlrpc()
Invalid argument supplied for foreach()Warningxmlrpc.inc380xmlrpc_request()
Cannot use string offset as an arrayPHP Fatal errorxmlrpc.inc563Unknown

All

Test namePassFailException
ExpandBlog API functionality (BlogAPITestCase) [Blog API]422
MessageGroupFilenameLineFunctionStatus
The test did not complete due to a fatal error.Completion checkblogapi.test19BlogAPITestCase->testBlogAPI()
key() expects parameter 1 to be array, string givenWarningxmlrpc.inc557_xmlrpc()
Invalid argument supplied for foreach()Warningxmlrpc.inc380xmlrpc_request()
Cannot use string offset as an arrayPHP Fatal errorxmlrpc.inc563Unknown

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

Non-pass

Test nameMinorCriticalNormal
Expandblogapi.module18230
MessageGroupFilenameLineFunctionStatus
There should be no trailing spacesOtherblogapi.module6N/A
Use uppercase for PHP constants, e.g. NULL, TRUE, FALSEOtherblogapi.module12N/A
There should be no trailing spacesOtherblogapi.module15N/A
missing space after commaOtherblogapi.module18N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtherblogapi.module19N/A
missing space after commaOtherblogapi.module19N/A
There should be no trailing spacesOtherblogapi.module24N/A
use a space between the closing parenthesis and the open bracketOtherblogapi.module25N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtherblogapi.module26N/A
missing space after commaOtherblogapi.module26N/A
Use uppercase for PHP constants, e.g. NULL, TRUE, FALSEOtherblogapi.module26N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtherblogapi.module255N/A
There should be no trailing spacesOtherblogapi.module304N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtherblogapi.module321N/A
There should be no trailing spacesOtherblogapi.module360N/A
There should be no trailing spacesOtherblogapi.module370N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtherblogapi.module380N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtherblogapi.module423N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtherblogapi.module439N/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.module452N/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.module460N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtherblogapi.module472N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtherblogapi.module492N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherblogapi.module504N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherblogapi.module509N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherblogapi.module531N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtherblogapi.module571N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtherblogapi.module596N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtherblogapi.module604N/A
There should be no trailing spacesOtherblogapi.module609N/A
There should be no trailing spacesOtherblogapi.module613N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtherblogapi.module634N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtherblogapi.module658N/A
There should be no trailing spacesOtherblogapi.module659N/A
There should be no trailing spacesOtherblogapi.module778N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtherblogapi.module780N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtherblogapi.module789N/A
There should be no trailing spacesOtherblogapi.module790N/A
missing space after commaOtherblogapi.module792N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtherblogapi.module793N/A
There should be no trailing spacesOtherblogapi.module794N/A
Control statements should have one space between the control keyword and opening parenthesisOtherblogapi.module795N/A
There should be no trailing spacesOtherblogapi.module798N/A
curly braces { should end a line, not start oneOtherblogapi.module799N/A
There should be no trailing spacesOtherblogapi.module802N/A
curly braces { should end a line, not start oneOtherblogapi.module805N/A
curly braces { should end a line, not start oneOtherblogapi.module811N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtherblogapi.module1039N/A
There should be no trailing spacesOtherblogapi.module1040N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtherblogapi.module1079N/A
Expandblogapi.test101
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherblogapi.test-1N/A
There should be no trailing spacesOtherblogapi.test128N/A

All

Test nameMinorCriticalNormal
Leafblogapi.install000
Expandblogapi.module18230
MessageGroupFilenameLineFunctionStatus
There should be no trailing spacesOtherblogapi.module6N/A
Use uppercase for PHP constants, e.g. NULL, TRUE, FALSEOtherblogapi.module12N/A
There should be no trailing spacesOtherblogapi.module15N/A
missing space after commaOtherblogapi.module18N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtherblogapi.module19N/A
missing space after commaOtherblogapi.module19N/A
There should be no trailing spacesOtherblogapi.module24N/A
use a space between the closing parenthesis and the open bracketOtherblogapi.module25N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtherblogapi.module26N/A
missing space after commaOtherblogapi.module26N/A
Use uppercase for PHP constants, e.g. NULL, TRUE, FALSEOtherblogapi.module26N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtherblogapi.module255N/A
There should be no trailing spacesOtherblogapi.module304N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtherblogapi.module321N/A
There should be no trailing spacesOtherblogapi.module360N/A
There should be no trailing spacesOtherblogapi.module370N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtherblogapi.module380N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtherblogapi.module423N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtherblogapi.module439N/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.module452N/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.module460N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtherblogapi.module472N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtherblogapi.module492N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherblogapi.module504N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherblogapi.module509N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherblogapi.module531N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtherblogapi.module571N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtherblogapi.module596N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtherblogapi.module604N/A
There should be no trailing spacesOtherblogapi.module609N/A
There should be no trailing spacesOtherblogapi.module613N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtherblogapi.module634N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtherblogapi.module658N/A
There should be no trailing spacesOtherblogapi.module659N/A
There should be no trailing spacesOtherblogapi.module778N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtherblogapi.module780N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtherblogapi.module789N/A
There should be no trailing spacesOtherblogapi.module790N/A
missing space after commaOtherblogapi.module792N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtherblogapi.module793N/A
There should be no trailing spacesOtherblogapi.module794N/A
Control statements should have one space between the control keyword and opening parenthesisOtherblogapi.module795N/A
There should be no trailing spacesOtherblogapi.module798N/A
curly braces { should end a line, not start oneOtherblogapi.module799N/A
There should be no trailing spacesOtherblogapi.module802N/A
curly braces { should end a line, not start oneOtherblogapi.module805N/A
curly braces { should end a line, not start oneOtherblogapi.module811N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtherblogapi.module1039N/A
There should be no trailing spacesOtherblogapi.module1040N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtherblogapi.module1079N/A
Expandblogapi.test101
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherblogapi.test-1N/A
There should be no trailing spacesOtherblogapi.test128N/A