Contact Forms - 7.x-1.6

Test status
StatusResult
Last receivedFri, 01/18/2013 - 14:20:30
Last requestedFri, 01/18/2013 - 14:21:54
Last testedFri, 01/18/2013 - 14:22:05
Test count2
Environment status
MySQLpass
Code reviewfail
Overall Resultpass
Branch information
Identifier7.x-1.6
ProjectContact Forms
LinkProject information
SummaryPASSED: [[SimpleTest]]: [MySQL] 0 pass(es); [[Coder]]: [Code review] 19 minor(s), 3 critical(s), and 31 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
19 minor(s), 3 critical(s), and 31 normal(s)

Non-pass

Test nameMinorCriticalNormal
Expandcontact_forms.install016
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othercontact_forms.install-1N/A
Commits to the Git repository do not require the CVS $Id$ keyword in each file. (Drupal Docs)Othercontact_forms.install2N/A
Functions should be called with no spaces between the function name and opening parenthesesOthercontact_forms.install15N/A
Use st() instead of t() in hook_install(), hook_uninstall() and hook_update_N()Othercontact_forms.install16N/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)Othercontact_forms.install16N/A
The $text argument to l() should be enclosed within st() so that it is translatable from within the install.Othercontact_forms.install17N/A
Functions should be called with no spaces between the function name and opening parenthesesOthercontact_forms.install41N/A
Expandcontact_forms.module10110
MessageGroupFilenameLineFunctionStatus
Commits to the Git repository do not require the CVS $Id$ keyword in each file. (Drupal Docs)Othercontact_forms.module2N/A
put a space between the asterisk and the comment textOthercontact_forms.module19N/A
Missing parenthesis after function nameOthercontact_forms.module26N/A
Missing parenthesis after function nameOthercontact_forms.module64N/A
use a space between the closing parenthesis and the open bracketOthercontact_forms.module66N/A
Use uppercase for PHP constants, e.g. NULL, TRUE, FALSEOthercontact_forms.module77N/A
Use uppercase for PHP constants, e.g. NULL, TRUE, FALSEOthercontact_forms.module87N/A
Use an indent of 2 spaces, with no tabsOthercontact_forms.module96N/A
Missing periodOthercontact_forms.module103N/A
missing space after commaOthercontact_forms.module119N/A
Potential problem: drupal_set_title() only accepts filtered text, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized.Othercontact_forms.module136N/A
Use an indent of 2 spaces, with no tabsOthercontact_forms.module149N/A
missing space after commaOthercontact_forms.module151N/A
Control statements should have one space between the control keyword and opening parenthesisOthercontact_forms.module175N/A
use a space between the closing parenthesis and the open bracketOthercontact_forms.module175N/A
Missing parenthesis after function nameOthercontact_forms.module214N/A
Use an indent of 2 spaces, with no tabsOthercontact_forms.module216N/A
l() already contains a check_plain() call by defaultOthercontact_forms.module261N/A
Missing parenthesis after function nameOthercontact_forms.module284N/A
Use an indent of 2 spaces, with no tabsOthercontact_forms.module290N/A
Use an indent of 2 spaces, with no tabsOthercontact_forms.module297N/A
Expandcontact_forms_int/contact_forms_int.install9010
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othercontact_forms_int.install-1N/A
indent secondary line of comment one space Othercontact_forms_int.install5N/A
Format should be * Implementation of hook_foo().Othercontact_forms_int.install5N/A
indent secondary line of comment one space Othercontact_forms_int.install6N/A
indent secondary line of comment one space Othercontact_forms_int.install7N/A
indent secondary line of comment one space Othercontact_forms_int.install8N/A
indent secondary line of comment one space Othercontact_forms_int.install9N/A
indent secondary line of comment one space Othercontact_forms_int.install10N/A
use a space between the closing parenthesis and the open bracketOthercontact_forms_int.install35N/A
Control statements should have one space between the control keyword and opening parenthesisOthercontact_forms_int.install51N/A
use a space between the closing parenthesis and the open bracketOthercontact_forms_int.install51N/A
Control statements should have one space between the control keyword and opening parenthesisOthercontact_forms_int.install55N/A
use a space between the closing parenthesis and the open bracketOthercontact_forms_int.install55N/A
Missing periodOthercontact_forms_int.install67N/A
There should be no trailing spacesOthercontact_forms_int.install72N/A
Control statements should have one space between the control keyword and opening parenthesisOthercontact_forms_int.install77N/A
use a space between the closing parenthesis and the open bracketOthercontact_forms_int.install77N/A
Control statements should have one space between the control keyword and opening parenthesisOthercontact_forms_int.install85N/A
use a space between the closing parenthesis and the open bracketOthercontact_forms_int.install85N/A
Expandcontact_forms_int/contact_forms_int.module015
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othercontact_forms_int.module-1N/A
Control statements should have one space between the control keyword and opening parenthesisOthercontact_forms_int.module28N/A
use a space between the closing parenthesis and the open bracketOthercontact_forms_int.module28N/A
Potential problem: drupal_set_title() only accepts filtered text, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized. (Drupal Docs)Othercontact_forms_int.module29N/A
The $string argument to t() should not begin or end with a space. (Drupal Docs)Othercontact_forms_int.module53N/A
The $string argument to t() should not begin or end with a space. (Drupal Docs)Othercontact_forms_int.module63N/A

All

Test nameMinorCriticalNormal
Expandcontact_forms.install016
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othercontact_forms.install-1N/A
Commits to the Git repository do not require the CVS $Id$ keyword in each file. (Drupal Docs)Othercontact_forms.install2N/A
Functions should be called with no spaces between the function name and opening parenthesesOthercontact_forms.install15N/A
Use st() instead of t() in hook_install(), hook_uninstall() and hook_update_N()Othercontact_forms.install16N/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)Othercontact_forms.install16N/A
The $text argument to l() should be enclosed within st() so that it is translatable from within the install.Othercontact_forms.install17N/A
Functions should be called with no spaces between the function name and opening parenthesesOthercontact_forms.install41N/A
Expandcontact_forms.module10110
MessageGroupFilenameLineFunctionStatus
Commits to the Git repository do not require the CVS $Id$ keyword in each file. (Drupal Docs)Othercontact_forms.module2N/A
put a space between the asterisk and the comment textOthercontact_forms.module19N/A
Missing parenthesis after function nameOthercontact_forms.module26N/A
Missing parenthesis after function nameOthercontact_forms.module64N/A
use a space between the closing parenthesis and the open bracketOthercontact_forms.module66N/A
Use uppercase for PHP constants, e.g. NULL, TRUE, FALSEOthercontact_forms.module77N/A
Use uppercase for PHP constants, e.g. NULL, TRUE, FALSEOthercontact_forms.module87N/A
Use an indent of 2 spaces, with no tabsOthercontact_forms.module96N/A
Missing periodOthercontact_forms.module103N/A
missing space after commaOthercontact_forms.module119N/A
Potential problem: drupal_set_title() only accepts filtered text, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized.Othercontact_forms.module136N/A
Use an indent of 2 spaces, with no tabsOthercontact_forms.module149N/A
missing space after commaOthercontact_forms.module151N/A
Control statements should have one space between the control keyword and opening parenthesisOthercontact_forms.module175N/A
use a space between the closing parenthesis and the open bracketOthercontact_forms.module175N/A
Missing parenthesis after function nameOthercontact_forms.module214N/A
Use an indent of 2 spaces, with no tabsOthercontact_forms.module216N/A
l() already contains a check_plain() call by defaultOthercontact_forms.module261N/A
Missing parenthesis after function nameOthercontact_forms.module284N/A
Use an indent of 2 spaces, with no tabsOthercontact_forms.module290N/A
Use an indent of 2 spaces, with no tabsOthercontact_forms.module297N/A
Expandcontact_forms_int/contact_forms_int.install9010
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othercontact_forms_int.install-1N/A
indent secondary line of comment one space Othercontact_forms_int.install5N/A
Format should be * Implementation of hook_foo().Othercontact_forms_int.install5N/A
indent secondary line of comment one space Othercontact_forms_int.install6N/A
indent secondary line of comment one space Othercontact_forms_int.install7N/A
indent secondary line of comment one space Othercontact_forms_int.install8N/A
indent secondary line of comment one space Othercontact_forms_int.install9N/A
indent secondary line of comment one space Othercontact_forms_int.install10N/A
use a space between the closing parenthesis and the open bracketOthercontact_forms_int.install35N/A
Control statements should have one space between the control keyword and opening parenthesisOthercontact_forms_int.install51N/A
use a space between the closing parenthesis and the open bracketOthercontact_forms_int.install51N/A
Control statements should have one space between the control keyword and opening parenthesisOthercontact_forms_int.install55N/A
use a space between the closing parenthesis and the open bracketOthercontact_forms_int.install55N/A
Missing periodOthercontact_forms_int.install67N/A
There should be no trailing spacesOthercontact_forms_int.install72N/A
Control statements should have one space between the control keyword and opening parenthesisOthercontact_forms_int.install77N/A
use a space between the closing parenthesis and the open bracketOthercontact_forms_int.install77N/A
Control statements should have one space between the control keyword and opening parenthesisOthercontact_forms_int.install85N/A
use a space between the closing parenthesis and the open bracketOthercontact_forms_int.install85N/A
Expandcontact_forms_int/contact_forms_int.module015
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othercontact_forms_int.module-1N/A
Control statements should have one space between the control keyword and opening parenthesisOthercontact_forms_int.module28N/A
use a space between the closing parenthesis and the open bracketOthercontact_forms_int.module28N/A
Potential problem: drupal_set_title() only accepts filtered text, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized. (Drupal Docs)Othercontact_forms_int.module29N/A
The $string argument to t() should not begin or end with a space. (Drupal Docs)Othercontact_forms_int.module53N/A
The $string argument to t() should not begin or end with a space. (Drupal Docs)Othercontact_forms_int.module63N/A