Common Functionality - master

Test status
StatusResult
Last receivedMon, 05/07/2012 - 17:51:38
Last requestedMon, 05/07/2012 - 17:52:03
Last testedMon, 05/07/2012 - 17:52:30
Test count38
Environment status
MySQLpass
Code reviewfail
Overall Resultpass
Branch information
Identifiermaster
ProjectCommon Functionality
LinkProject information
SummaryPASSED: [[Coder]]: [Code review] 9 minor(s), 5 critical(s), and 28 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
9 minor(s), 5 critical(s), and 28 normal(s)

Non-pass

Test nameMinorCriticalNormal
Expandcf.module001
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othercf.module-1N/A
Expandmodules/cf_1x_compatibility/cf_1x_compatibility.module001
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othercf_1x_compatibility.module-1N/A
Expandmodules/cf_db_options/cf_db_options.module001
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othercf_db_options.module-1N/A
Expandmodules/cf_error/cf_error.module051
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othercf_error.module-1N/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)Othercf_error.module123N/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)Othercf_error.module131N/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)Othercf_error.module137N/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)Othercf_error.module144N/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)Othercf_error.module150N/A
Expandmodules/cf_http/cf_http.module507
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othercf_http.module-1N/A
Control statements should have one space between the control keyword and opening parenthesisOthercf_http.module78N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthercf_http.module123N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthercf_http.module206N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthercf_http.module215N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthercf_http.module219N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthercf_http.module220N/A
Control statements should have one space between the control keyword and opening parenthesisOthercf_http.module222N/A
missing space after commaOthercf_http.module274N/A
use a space between the closing parenthesis and the open bracketOthercf_http.module306N/A
use a space between the closing parenthesis and the open bracketOthercf_http.module478N/A
use a space between the closing parenthesis and the open bracketOthercf_http.module630N/A
Expandmodules/cf_menu/cf_menu.install001
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othercf_menu.install-1N/A
Expandmodules/cf_menu/cf_menu.module101
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othercf_menu.module-1N/A
@see should always be followed by a filename, a URL, class/interface name (optionally including method), or a function name including ().Othercf_menu.module6N/A
Expandmodules/cf_node/cf_node.module101
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othercf_node.module-1N/A
put a space between the asterisk and the comment textOthercf_node.module85N/A
Expandmodules/cf_settings/cf_settings.module001
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othercf_settings.module-1N/A
Expandmodules/cf_theme/cf_theme.module2010
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othercf_theme.module-1N/A
Use uppercase for PHP constants, e.g. NULL, TRUE, FALSEOthercf_theme.module71N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthercf_theme.module93N/A
Use "elseif" in place of "else if"Othercf_theme.module197N/A
global variables should start with a single underscore followed by the module and another underscoreOthercf_theme.module238N/A
use a space between the closing parenthesis and the open bracketOthercf_theme.module244N/A
Use an indent of 2 spaces, with no tabsOthercf_theme.module245N/A
Use "elseif" in place of "else if"Othercf_theme.module255N/A
Use "elseif" in place of "else if"Othercf_theme.module262N/A
use a space between the closing parenthesis and the open bracketOthercf_theme.module323N/A
use a space between the closing parenthesis and the open bracketOthercf_theme.module326N/A
use a space between the closing parenthesis and the open bracketOthercf_theme.module515N/A
Expandmodules/cf_user/cf_user.module002
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othercf_user.module-1N/A
Use "elseif" in place of "else if"Othercf_user.module31N/A
Expandmodules/cf_views/cf_views.module001
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othercf_views.module-1N/A

All

Test nameMinorCriticalNormal
Expandcf.module001
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othercf.module-1N/A
Expandmodules/cf_1x_compatibility/cf_1x_compatibility.module001
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othercf_1x_compatibility.module-1N/A
Expandmodules/cf_db_options/cf_db_options.module001
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othercf_db_options.module-1N/A
Expandmodules/cf_error/cf_error.module051
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othercf_error.module-1N/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)Othercf_error.module123N/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)Othercf_error.module131N/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)Othercf_error.module137N/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)Othercf_error.module144N/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)Othercf_error.module150N/A
Expandmodules/cf_http/cf_http.module507
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othercf_http.module-1N/A
Control statements should have one space between the control keyword and opening parenthesisOthercf_http.module78N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthercf_http.module123N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthercf_http.module206N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthercf_http.module215N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthercf_http.module219N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthercf_http.module220N/A
Control statements should have one space between the control keyword and opening parenthesisOthercf_http.module222N/A
missing space after commaOthercf_http.module274N/A
use a space between the closing parenthesis and the open bracketOthercf_http.module306N/A
use a space between the closing parenthesis and the open bracketOthercf_http.module478N/A
use a space between the closing parenthesis and the open bracketOthercf_http.module630N/A
Expandmodules/cf_menu/cf_menu.install001
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othercf_menu.install-1N/A
Expandmodules/cf_menu/cf_menu.module101
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othercf_menu.module-1N/A
@see should always be followed by a filename, a URL, class/interface name (optionally including method), or a function name including ().Othercf_menu.module6N/A
Leafmodules/cf_menu/includes/menu.inc000
Expandmodules/cf_node/cf_node.module101
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othercf_node.module-1N/A
put a space between the asterisk and the comment textOthercf_node.module85N/A
Expandmodules/cf_settings/cf_settings.module001
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othercf_settings.module-1N/A
Expandmodules/cf_theme/cf_theme.module2010
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othercf_theme.module-1N/A
Use uppercase for PHP constants, e.g. NULL, TRUE, FALSEOthercf_theme.module71N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthercf_theme.module93N/A
Use "elseif" in place of "else if"Othercf_theme.module197N/A
global variables should start with a single underscore followed by the module and another underscoreOthercf_theme.module238N/A
use a space between the closing parenthesis and the open bracketOthercf_theme.module244N/A
Use an indent of 2 spaces, with no tabsOthercf_theme.module245N/A
Use "elseif" in place of "else if"Othercf_theme.module255N/A
Use "elseif" in place of "else if"Othercf_theme.module262N/A
use a space between the closing parenthesis and the open bracketOthercf_theme.module323N/A
use a space between the closing parenthesis and the open bracketOthercf_theme.module326N/A
use a space between the closing parenthesis and the open bracketOthercf_theme.module515N/A
Expandmodules/cf_user/cf_user.module002
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othercf_user.module-1N/A
Use "elseif" in place of "else if"Othercf_user.module31N/A
Expandmodules/cf_views/cf_views.module001
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othercf_views.module-1N/A