Textimage - 7.x-2.0-alpha2

Test status
StatusResult
Last receivedMon, 08/20/2012 - 14:31:28
Last requestedMon, 08/20/2012 - 14:59:45
Last testedMon, 08/20/2012 - 14:59:53
Test count2
Environment status
MySQLpass
Code reviewfail
Overall Resultpass
Branch information
Identifier7.x-2.0-alpha2
ProjectTextimage
LinkProject information
SummaryPASSED: [[SimpleTest]]: [MySQL] 0 pass(es); [[Coder]]: [Code review] 3 minor(s), 5 critical(s), and 24 normal(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
  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
3 minor(s), 5 critical(s), and 24 normal(s)

Non-pass

Test nameMinorCriticalNormal
Expandtextimage.admin.inc022
MessageGroupFilenameLineFunctionStatus
Commits to the Git repository do not require the CVS $Id$ keyword in each file. (Drupal Docs)Othertextimage.admin.inc2N/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.Othertextimage.admin.inc16N/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.Othertextimage.admin.inc33N/A
missing space after commaOthertextimage.admin.inc553N/A
Expandtextimage.install104
MessageGroupFilenameLineFunctionStatus
use a space between the closing parenthesis and the open bracketOthertextimage.install10N/A
use a space between the closing parenthesis and the open bracketOthertextimage.install15N/A
use a space between the closing parenthesis and the open bracketOthertextimage.install24N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthertextimage.install119N/A
indent secondary line of comment one space Othertextimage.install271N/A
Expandtextimage.module0114
MessageGroupFilenameLineFunctionStatus
use a space between the closing parenthesis and the open bracketOthertextimage.module19N/A
use a space between the closing parenthesis and the open bracketOthertextimage.module52N/A
Control statements should have one space between the control keyword and opening parenthesisOthertextimage.module55N/A
use a space between the closing parenthesis and the open bracketOthertextimage.module55N/A
use a space between the closing parenthesis and the open bracketOthertextimage.module100N/A
Use uppercase for PHP constants, e.g. NULL, TRUE, FALSEOthertextimage.module339N/A
Use uppercase for PHP constants, e.g. NULL, TRUE, FALSEOthertextimage.module341N/A
Control statements should have one space between the control keyword and opening parenthesisOthertextimage.module374N/A
use a space between the closing parenthesis and the open bracketOthertextimage.module402N/A
Use preg_replace_callback() instead of the 'e' modifier to preg_replace()Othertextimage.module458N/A
use a space between the closing parenthesis and the open bracketOthertextimage.module619N/A
missing space after commaOthertextimage.module636N/A
use a space between the closing parenthesis and the open bracketOthertextimage.module640N/A
Use uppercase for PHP constants, e.g. NULL, TRUE, FALSEOthertextimage.module649N/A
Arrays should be formatted with a space separating each element and assignment operatorOthertextimage.module758N/A
Expandtextimage.utils.inc204
MessageGroupFilenameLineFunctionStatus
use a space between the closing parenthesis and the open bracketOthertextimage.utils.inc13N/A
use a space between the closing parenthesis and the open bracketOthertextimage.utils.inc40N/A
use a space between the closing parenthesis and the open bracketOthertextimage.utils.inc45N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthertextimage.utils.inc224N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthertextimage.utils.inc231N/A
use a space between the closing parenthesis and the open bracketOthertextimage.utils.inc275N/A
Expandtextimage_admin.inc020
MessageGroupFilenameLineFunctionStatus
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.Othertextimage_admin.inc14N/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.Othertextimage_admin.inc31N/A

All

Test nameMinorCriticalNormal
Leafincludes/ctools.inc000
Expandtextimage.admin.inc022
MessageGroupFilenameLineFunctionStatus
Commits to the Git repository do not require the CVS $Id$ keyword in each file. (Drupal Docs)Othertextimage.admin.inc2N/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.Othertextimage.admin.inc16N/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.Othertextimage.admin.inc33N/A
missing space after commaOthertextimage.admin.inc553N/A
Expandtextimage.install104
MessageGroupFilenameLineFunctionStatus
use a space between the closing parenthesis and the open bracketOthertextimage.install10N/A
use a space between the closing parenthesis and the open bracketOthertextimage.install15N/A
use a space between the closing parenthesis and the open bracketOthertextimage.install24N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthertextimage.install119N/A
indent secondary line of comment one space Othertextimage.install271N/A
Expandtextimage.module0114
MessageGroupFilenameLineFunctionStatus
use a space between the closing parenthesis and the open bracketOthertextimage.module19N/A
use a space between the closing parenthesis and the open bracketOthertextimage.module52N/A
Control statements should have one space between the control keyword and opening parenthesisOthertextimage.module55N/A
use a space between the closing parenthesis and the open bracketOthertextimage.module55N/A
use a space between the closing parenthesis and the open bracketOthertextimage.module100N/A
Use uppercase for PHP constants, e.g. NULL, TRUE, FALSEOthertextimage.module339N/A
Use uppercase for PHP constants, e.g. NULL, TRUE, FALSEOthertextimage.module341N/A
Control statements should have one space between the control keyword and opening parenthesisOthertextimage.module374N/A
use a space between the closing parenthesis and the open bracketOthertextimage.module402N/A
Use preg_replace_callback() instead of the 'e' modifier to preg_replace()Othertextimage.module458N/A
use a space between the closing parenthesis and the open bracketOthertextimage.module619N/A
missing space after commaOthertextimage.module636N/A
use a space between the closing parenthesis and the open bracketOthertextimage.module640N/A
Use uppercase for PHP constants, e.g. NULL, TRUE, FALSEOthertextimage.module649N/A
Arrays should be formatted with a space separating each element and assignment operatorOthertextimage.module758N/A
Expandtextimage.utils.inc204
MessageGroupFilenameLineFunctionStatus
use a space between the closing parenthesis and the open bracketOthertextimage.utils.inc13N/A
use a space between the closing parenthesis and the open bracketOthertextimage.utils.inc40N/A
use a space between the closing parenthesis and the open bracketOthertextimage.utils.inc45N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthertextimage.utils.inc224N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthertextimage.utils.inc231N/A
use a space between the closing parenthesis and the open bracketOthertextimage.utils.inc275N/A
Expandtextimage_admin.inc020
MessageGroupFilenameLineFunctionStatus
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.Othertextimage_admin.inc14N/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.Othertextimage_admin.inc31N/A