Auto create Group - 6.x-1.0

Test status
StatusResult
Last receivedWed, 09/12/2012 - 05:48:15
Last requestedWed, 09/12/2012 - 05:49:03
Last testedWed, 09/12/2012 - 05:49:22
Test count2
Environment status
MySQLpass
Code reviewfail
Overall Resultpass
Branch information
Identifier6.x-1.0
ProjectAuto create Group
LinkProject information
SummaryPASSED: [[Coder]]: [Code review] 0 minor(s), 2 critical(s), and 13 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
0 minor(s), 2 critical(s), and 13 normal(s)

Non-pass

Test nameMinorCriticalNormal
Expandautocreategroup.install002
MessageGroupFilenameLineFunctionStatus
Commits to the Git repository do not require the CVS $Id$ keyword in each file. (Drupal Docs)Otherautocreategroup.install2N/A
Use an indent of 2 spaces, with no tabsOtherautocreategroup.install23N/A
Expandautocreategroup.module0211
MessageGroupFilenameLineFunctionStatus
Commits to the Git repository do not require the CVS $Id$ keyword in each file. (Drupal Docs)Otherautocreategroup.module2N/A
Use "elseif" in place of "else if"Otherautocreategroup.module98N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Otherautocreategroup.module99N/A
Use "elseif" in place of "else if"Otherautocreategroup.module103N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Otherautocreategroup.module104N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Otherautocreategroup.module110N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Otherautocreategroup.module116N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Otherautocreategroup.module266N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Otherautocreategroup.module316N/A
The $message argument to drupal_set_message() should be enclosed within t() so that it is translatable.Otherautocreategroup.module360N/A
Potential problem: drupal_set_message() only accepts filtered text, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized. (Drupal Docs)Otherautocreategroup.module360N/A
The $message argument to drupal_set_message() should be enclosed within t() so that it is translatable.Otherautocreategroup.module370N/A
Potential problem: form_set_error() and form_error() only accept filtered text, be sure all !placeholders for $variables in t() are fully sanitized using check_plain(), filter_xss() or similar. (Drupal Docs)Otherautocreategroup.module380N/A

All

Test nameMinorCriticalNormal
Expandautocreategroup.install002
MessageGroupFilenameLineFunctionStatus
Commits to the Git repository do not require the CVS $Id$ keyword in each file. (Drupal Docs)Otherautocreategroup.install2N/A
Use an indent of 2 spaces, with no tabsOtherautocreategroup.install23N/A
Expandautocreategroup.module0211
MessageGroupFilenameLineFunctionStatus
Commits to the Git repository do not require the CVS $Id$ keyword in each file. (Drupal Docs)Otherautocreategroup.module2N/A
Use "elseif" in place of "else if"Otherautocreategroup.module98N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Otherautocreategroup.module99N/A
Use "elseif" in place of "else if"Otherautocreategroup.module103N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Otherautocreategroup.module104N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Otherautocreategroup.module110N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Otherautocreategroup.module116N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Otherautocreategroup.module266N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Otherautocreategroup.module316N/A
The $message argument to drupal_set_message() should be enclosed within t() so that it is translatable.Otherautocreategroup.module360N/A
Potential problem: drupal_set_message() only accepts filtered text, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized. (Drupal Docs)Otherautocreategroup.module360N/A
The $message argument to drupal_set_message() should be enclosed within t() so that it is translatable.Otherautocreategroup.module370N/A
Potential problem: form_set_error() and form_error() only accept filtered text, be sure all !placeholders for $variables in t() are fully sanitized using check_plain(), filter_xss() or similar. (Drupal Docs)Otherautocreategroup.module380N/A