CCK Multiple Field Remove - 7.x-1.1

Test status
StatusResult
Last receivedTue, 10/02/2012 - 09:56:06
Last requestedTue, 10/02/2012 - 09:57:03
Last testedTue, 10/02/2012 - 09:57:30
Test count2
Environment status
MySQLpass
Code reviewfail
Overall Resultpass
Branch information
Identifier7.x-1.1
ProjectCCK Multiple Field Remove
LinkProject information
SummaryPASSED: [[Coder]]: [Code review] 4 minor(s), 1 critical(s), and 92 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), 1 critical(s), and 92 normal(s)

Non-pass

Test nameMinorCriticalNormal
Expandcck_multiple_field_remove.install001
MessageGroupFilenameLineFunctionStatus
Use uppercase for PHP constants, e.g. NULL, TRUE, FALSEOthercck_multiple_field_remove.install14N/A
Expandcck_multiple_field_remove.module4191
MessageGroupFilenameLineFunctionStatus
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module52N/A
There should be no trailing spacesOthercck_multiple_field_remove.module52N/A
missing space after commaOthercck_multiple_field_remove.module56N/A
missing space after commaOthercck_multiple_field_remove.module57N/A
Control statements should have one space between the control keyword and opening parenthesisOthercck_multiple_field_remove.module58N/A
use a space between the closing parenthesis and the open bracketOthercck_multiple_field_remove.module58N/A
Control statements should have one space between the control keyword and opening parenthesisOthercck_multiple_field_remove.module59N/A
use a space between the closing parenthesis and the open bracketOthercck_multiple_field_remove.module59N/A
missing space after commaOthercck_multiple_field_remove.module59N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module60N/A
Potential problem: FAPI elements '#title' and '#description' only accept filtered text, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized.Othercck_multiple_field_remove.module62N/A
missing space after commaOthercck_multiple_field_remove.module63N/A
There should be no trailing spacesOthercck_multiple_field_remove.module67N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module81N/A
Control statements should have one space between the control keyword and opening parenthesisOthercck_multiple_field_remove.module81N/A
use a space between the closing parenthesis and the open bracketOthercck_multiple_field_remove.module81N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module82N/A
Control statements should have one space between the control keyword and opening parenthesisOthercck_multiple_field_remove.module82N/A
use a space between the closing parenthesis and the open bracketOthercck_multiple_field_remove.module82N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module83N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module84N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module85N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module86N/A
missing space after commaOthercck_multiple_field_remove.module86N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module87N/A
The $message argument to drupal_set_message() should be enclosed within t() so that it is translatable.Othercck_multiple_field_remove.module88N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module88N/A
There should be no trailing spacesOthercck_multiple_field_remove.module88N/A
curly braces { should end a line, not start oneOthercck_multiple_field_remove.module97N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module98N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module98N/A
missing space after commaOthercck_multiple_field_remove.module98N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module99N/A
Use uppercase for PHP constants, e.g. NULL, TRUE, FALSEOthercck_multiple_field_remove.module99N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module100N/A
Control statements should have one space between the control keyword and opening parenthesisOthercck_multiple_field_remove.module100N/A
use a space between the closing parenthesis and the open bracketOthercck_multiple_field_remove.module100N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module101N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module102N/A
Use uppercase for PHP constants, e.g. NULL, TRUE, FALSEOthercck_multiple_field_remove.module102N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module103N/A
else statements should begin on a new lineOthercck_multiple_field_remove.module103N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module104N/A
Control statements should have one space between the control keyword and opening parenthesisOthercck_multiple_field_remove.module104N/A
use a space between the closing parenthesis and the open bracketOthercck_multiple_field_remove.module104N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module105N/A
Control statements should have one space between the control keyword and opening parenthesisOthercck_multiple_field_remove.module105N/A
use a space between the closing parenthesis and the open bracketOthercck_multiple_field_remove.module105N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module106N/A
Use uppercase for PHP constants, e.g. NULL, TRUE, FALSEOthercck_multiple_field_remove.module106N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module107N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module108N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module109N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module110N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module111N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module112N/A
Control statements should have one space between the control keyword and opening parenthesisOthercck_multiple_field_remove.module112N/A
use a space between the closing parenthesis and the open bracketOthercck_multiple_field_remove.module112N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module113N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module114N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module115N/A
Control statements should have one space between the control keyword and opening parenthesisOthercck_multiple_field_remove.module115N/A
missing space after commaOthercck_multiple_field_remove.module115N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module116N/A
curly braces { should end a line, not start oneOthercck_multiple_field_remove.module116N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module117N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthercck_multiple_field_remove.module117N/A
missing space after commaOthercck_multiple_field_remove.module117N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module118N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module119N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module120N/A
Control statements should have one space between the control keyword and opening parenthesisOthercck_multiple_field_remove.module120N/A
use a space between the closing parenthesis and the open bracketOthercck_multiple_field_remove.module120N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module121N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module122N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module123N/A
Control statements should have one space between the control keyword and opening parenthesisOthercck_multiple_field_remove.module123N/A
use a space between the closing parenthesis and the open bracketOthercck_multiple_field_remove.module123N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module124N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module125N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module126N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module127N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module128N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthercck_multiple_field_remove.module128N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module129N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module130N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module131N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module132N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthercck_multiple_field_remove.module132N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module133N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module134N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module135N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module136N/A
use a space between the closing parenthesis and the open bracketOthercck_multiple_field_remove.module144N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module145N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module146N/A

All

Test nameMinorCriticalNormal
Expandcck_multiple_field_remove.install001
MessageGroupFilenameLineFunctionStatus
Use uppercase for PHP constants, e.g. NULL, TRUE, FALSEOthercck_multiple_field_remove.install14N/A
Expandcck_multiple_field_remove.module4191
MessageGroupFilenameLineFunctionStatus
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module52N/A
There should be no trailing spacesOthercck_multiple_field_remove.module52N/A
missing space after commaOthercck_multiple_field_remove.module56N/A
missing space after commaOthercck_multiple_field_remove.module57N/A
Control statements should have one space between the control keyword and opening parenthesisOthercck_multiple_field_remove.module58N/A
use a space between the closing parenthesis and the open bracketOthercck_multiple_field_remove.module58N/A
Control statements should have one space between the control keyword and opening parenthesisOthercck_multiple_field_remove.module59N/A
use a space between the closing parenthesis and the open bracketOthercck_multiple_field_remove.module59N/A
missing space after commaOthercck_multiple_field_remove.module59N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module60N/A
Potential problem: FAPI elements '#title' and '#description' only accept filtered text, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized.Othercck_multiple_field_remove.module62N/A
missing space after commaOthercck_multiple_field_remove.module63N/A
There should be no trailing spacesOthercck_multiple_field_remove.module67N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module81N/A
Control statements should have one space between the control keyword and opening parenthesisOthercck_multiple_field_remove.module81N/A
use a space between the closing parenthesis and the open bracketOthercck_multiple_field_remove.module81N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module82N/A
Control statements should have one space between the control keyword and opening parenthesisOthercck_multiple_field_remove.module82N/A
use a space between the closing parenthesis and the open bracketOthercck_multiple_field_remove.module82N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module83N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module84N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module85N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module86N/A
missing space after commaOthercck_multiple_field_remove.module86N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module87N/A
The $message argument to drupal_set_message() should be enclosed within t() so that it is translatable.Othercck_multiple_field_remove.module88N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module88N/A
There should be no trailing spacesOthercck_multiple_field_remove.module88N/A
curly braces { should end a line, not start oneOthercck_multiple_field_remove.module97N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module98N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module98N/A
missing space after commaOthercck_multiple_field_remove.module98N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module99N/A
Use uppercase for PHP constants, e.g. NULL, TRUE, FALSEOthercck_multiple_field_remove.module99N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module100N/A
Control statements should have one space between the control keyword and opening parenthesisOthercck_multiple_field_remove.module100N/A
use a space between the closing parenthesis and the open bracketOthercck_multiple_field_remove.module100N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module101N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module102N/A
Use uppercase for PHP constants, e.g. NULL, TRUE, FALSEOthercck_multiple_field_remove.module102N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module103N/A
else statements should begin on a new lineOthercck_multiple_field_remove.module103N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module104N/A
Control statements should have one space between the control keyword and opening parenthesisOthercck_multiple_field_remove.module104N/A
use a space between the closing parenthesis and the open bracketOthercck_multiple_field_remove.module104N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module105N/A
Control statements should have one space between the control keyword and opening parenthesisOthercck_multiple_field_remove.module105N/A
use a space between the closing parenthesis and the open bracketOthercck_multiple_field_remove.module105N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module106N/A
Use uppercase for PHP constants, e.g. NULL, TRUE, FALSEOthercck_multiple_field_remove.module106N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module107N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module108N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module109N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module110N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module111N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module112N/A
Control statements should have one space between the control keyword and opening parenthesisOthercck_multiple_field_remove.module112N/A
use a space between the closing parenthesis and the open bracketOthercck_multiple_field_remove.module112N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module113N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module114N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module115N/A
Control statements should have one space between the control keyword and opening parenthesisOthercck_multiple_field_remove.module115N/A
missing space after commaOthercck_multiple_field_remove.module115N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module116N/A
curly braces { should end a line, not start oneOthercck_multiple_field_remove.module116N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module117N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthercck_multiple_field_remove.module117N/A
missing space after commaOthercck_multiple_field_remove.module117N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module118N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module119N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module120N/A
Control statements should have one space between the control keyword and opening parenthesisOthercck_multiple_field_remove.module120N/A
use a space between the closing parenthesis and the open bracketOthercck_multiple_field_remove.module120N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module121N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module122N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module123N/A
Control statements should have one space between the control keyword and opening parenthesisOthercck_multiple_field_remove.module123N/A
use a space between the closing parenthesis and the open bracketOthercck_multiple_field_remove.module123N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module124N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module125N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module126N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module127N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module128N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthercck_multiple_field_remove.module128N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module129N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module130N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module131N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module132N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthercck_multiple_field_remove.module132N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module133N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module134N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module135N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module136N/A
use a space between the closing parenthesis and the open bracketOthercck_multiple_field_remove.module144N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module145N/A
Use an indent of 2 spaces, with no tabsOthercck_multiple_field_remove.module146N/A