Icon Links - 7.x-1.0

Test status
StatusResult
Last receivedTue, 07/10/2012 - 14:23:15
Last requestedTue, 07/10/2012 - 14:24:02
Last testedTue, 07/10/2012 - 14:24:28
Test count2
Environment status
MySQLpass
Code reviewfail
Overall Resultpass
Branch information
Identifier7.x-1.0
ProjectIcon Links
LinkProject information
SummaryPASSED: [[Coder]]: [Code review] 11 minor(s), 1 critical(s), and 38 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
11 minor(s), 1 critical(s), and 38 normal(s)

Non-pass

Test nameMinorCriticalNormal
Expandicon_links.install205
MessageGroupFilenameLineFunctionStatus
Use an indent of 2 spaces, with no tabsOthericon_links.install13N/A
Use an indent of 2 spaces, with no tabsOthericon_links.install36N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthericon_links.install41N/A
Control statements should have one space between the control keyword and opening parenthesisOthericon_links.install43N/A
use a space between the closing parenthesis and the open bracketOthericon_links.install43N/A
Control statements should have one space between the control keyword and opening parenthesisOthericon_links.install44N/A
use a space between the closing parenthesis and the open bracketOthericon_links.install44N/A
Expandicon_links.module9133
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othericon_links.module-1N/A
There should be no trailing spacesOthericon_links.module5N/A
There should be no trailing spacesOthericon_links.module9N/A
Control statements should have one space between the control keyword and opening parenthesisOthericon_links.module28N/A
use a space between the closing parenthesis and the open bracketOthericon_links.module28N/A
Control statements should have one space between the control keyword and opening parenthesisOthericon_links.module29N/A
use a space between the closing parenthesis and the open bracketOthericon_links.module29N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthericon_links.module32N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthericon_links.module37N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthericon_links.module42N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthericon_links.module55N/A
Control statements should have one space between the control keyword and opening parenthesisOthericon_links.module57N/A
use a space between the closing parenthesis and the open bracketOthericon_links.module57N/A
Control statements should have one space between the control keyword and opening parenthesisOthericon_links.module58N/A
use a space between the closing parenthesis and the open bracketOthericon_links.module58N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthericon_links.module63N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthericon_links.module65N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthericon_links.module67N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthericon_links.module72N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthericon_links.module74N/A
Use an indent of 2 spaces, with no tabsOthericon_links.module78N/A
There should be no trailing spacesOthericon_links.module90N/A
Control statements should have one space between the control keyword and opening parenthesisOthericon_links.module95N/A
use a space between the closing parenthesis and the open bracketOthericon_links.module95N/A
There should be no trailing spacesOthericon_links.module96N/A
Control statements should have one space between the control keyword and opening parenthesisOthericon_links.module97N/A
use a space between the closing parenthesis and the open bracketOthericon_links.module97N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthericon_links.module97N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthericon_links.module98N/A
Control statements should have one space between the control keyword and opening parenthesisOthericon_links.module99N/A
use a space between the closing parenthesis and the open bracketOthericon_links.module99N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthericon_links.module102N/A
missing space after commaOthericon_links.module106N/A
Functions should be called with no spaces between the function name and opening parenthesesOthericon_links.module108N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthericon_links.module110N/A
else statements should begin on a new lineOthericon_links.module111N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthericon_links.module112N/A
Potential problem: drupal_set_message() only accepts filtered text, be sure all !placeholders for $variables in t() are fully sanitized using check_plain(), filter_xss() or similar. (Drupal Docs)Othericon_links.module112N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthericon_links.module121N/A
There should be no trailing spacesOthericon_links.module129N/A
There should be no trailing spacesOthericon_links.module136N/A
There should be no trailing spacesOthericon_links.module144N/A
There should be no trailing spacesOthericon_links.module163N/A

All

Test nameMinorCriticalNormal
Expandicon_links.install205
MessageGroupFilenameLineFunctionStatus
Use an indent of 2 spaces, with no tabsOthericon_links.install13N/A
Use an indent of 2 spaces, with no tabsOthericon_links.install36N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthericon_links.install41N/A
Control statements should have one space between the control keyword and opening parenthesisOthericon_links.install43N/A
use a space between the closing parenthesis and the open bracketOthericon_links.install43N/A
Control statements should have one space between the control keyword and opening parenthesisOthericon_links.install44N/A
use a space between the closing parenthesis and the open bracketOthericon_links.install44N/A
Expandicon_links.module9133
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othericon_links.module-1N/A
There should be no trailing spacesOthericon_links.module5N/A
There should be no trailing spacesOthericon_links.module9N/A
Control statements should have one space between the control keyword and opening parenthesisOthericon_links.module28N/A
use a space between the closing parenthesis and the open bracketOthericon_links.module28N/A
Control statements should have one space between the control keyword and opening parenthesisOthericon_links.module29N/A
use a space between the closing parenthesis and the open bracketOthericon_links.module29N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthericon_links.module32N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthericon_links.module37N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthericon_links.module42N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthericon_links.module55N/A
Control statements should have one space between the control keyword and opening parenthesisOthericon_links.module57N/A
use a space between the closing parenthesis and the open bracketOthericon_links.module57N/A
Control statements should have one space between the control keyword and opening parenthesisOthericon_links.module58N/A
use a space between the closing parenthesis and the open bracketOthericon_links.module58N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthericon_links.module63N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthericon_links.module65N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthericon_links.module67N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthericon_links.module72N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthericon_links.module74N/A
Use an indent of 2 spaces, with no tabsOthericon_links.module78N/A
There should be no trailing spacesOthericon_links.module90N/A
Control statements should have one space between the control keyword and opening parenthesisOthericon_links.module95N/A
use a space between the closing parenthesis and the open bracketOthericon_links.module95N/A
There should be no trailing spacesOthericon_links.module96N/A
Control statements should have one space between the control keyword and opening parenthesisOthericon_links.module97N/A
use a space between the closing parenthesis and the open bracketOthericon_links.module97N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthericon_links.module97N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthericon_links.module98N/A
Control statements should have one space between the control keyword and opening parenthesisOthericon_links.module99N/A
use a space between the closing parenthesis and the open bracketOthericon_links.module99N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthericon_links.module102N/A
missing space after commaOthericon_links.module106N/A
Functions should be called with no spaces between the function name and opening parenthesesOthericon_links.module108N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthericon_links.module110N/A
else statements should begin on a new lineOthericon_links.module111N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthericon_links.module112N/A
Potential problem: drupal_set_message() only accepts filtered text, be sure all !placeholders for $variables in t() are fully sanitized using check_plain(), filter_xss() or similar. (Drupal Docs)Othericon_links.module112N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthericon_links.module121N/A
There should be no trailing spacesOthericon_links.module129N/A
There should be no trailing spacesOthericon_links.module136N/A
There should be no trailing spacesOthericon_links.module144N/A
There should be no trailing spacesOthericon_links.module163N/A