User Delete - 6.x-1.x

Test status
StatusResult
Last receivedFri, 03/02/2012 - 14:48:37
Last requestedFri, 03/02/2012 - 14:49:02
Last testedFri, 03/02/2012 - 14:49:17
Test count5
Environment status
MySQLpass
Code reviewfail
Overall Resultpass
Branch information
Identifier6.x-1.x
ProjectUser Delete
LinkProject information
SummaryPASSED: [[Coder]]: [Code review] 2 minor(s), 2 critical(s), and 14 normal(s); [[SimpleTest]]: [MySQL] 69 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
69 pass(es), 0 fail(s), and 0 exception(s)
Test namePassFailException
LeafUser delete (UserDeleteTest) [User delete]6900

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
2 minor(s), 2 critical(s), and 14 normal(s)

Non-pass

Test nameMinorCriticalNormal
Expanduser_delete.admin.inc001
MessageGroupFilenameLineFunctionStatus
Separate comments from comment syntax by a space.Otheruser_delete.admin.inc18N/A
Expanduser_delete.module1211
MessageGroupFilenameLineFunctionStatus
Doxygen uses @todo and @bug to markup things to be done.Otheruser_delete.module15N/A
Separate comments from comment syntax by a space.Otheruser_delete.module70N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruser_delete.module74N/A
There should be no trailing spacesOtheruser_delete.module99N/A
Doxygen uses @todo and @bug to markup things to be done.Otheruser_delete.module174N/A
Potential problem: "SELECT FROM {node}" statements should probably be wrapped in db_rewrite_sql() and with the alias for {node} table defined (e.g. {node} n)Otheruser_delete.module191N/A
Function summaries should be one line only. (Drupal Docs)Otheruser_delete.module331N/A
Separate comments from comment syntax by a space.Otheruser_delete.module352N/A
The $message argument to watchdog() should NOT be enclosed within t(), so that it can be properly translated at display time.Otheruser_delete.module353N/A
Function summaries should be one line only. (Drupal Docs)Otheruser_delete.module357N/A
Separate comments from comment syntax by a space.Otheruser_delete.module366N/A
Separate comments from comment syntax by a space.Otheruser_delete.module380N/A
Use PHP's master function, not an alias. (List of PHP aliases)Otheruser_delete.module391N/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)Otheruser_delete.module397N/A
Expanduser_delete.test102
MessageGroupFilenameLineFunctionStatus
Use uppercase for PHP constants, e.g. NULL, TRUE, FALSEOtheruser_delete.test70N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruser_delete.test105N/A
There should be no trailing spacesOtheruser_delete.test111N/A

All

Test nameMinorCriticalNormal
Expanduser_delete.admin.inc001
MessageGroupFilenameLineFunctionStatus
Separate comments from comment syntax by a space.Otheruser_delete.admin.inc18N/A
Leafuser_delete.install000
Expanduser_delete.module1211
MessageGroupFilenameLineFunctionStatus
Doxygen uses @todo and @bug to markup things to be done.Otheruser_delete.module15N/A
Separate comments from comment syntax by a space.Otheruser_delete.module70N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruser_delete.module74N/A
There should be no trailing spacesOtheruser_delete.module99N/A
Doxygen uses @todo and @bug to markup things to be done.Otheruser_delete.module174N/A
Potential problem: "SELECT FROM {node}" statements should probably be wrapped in db_rewrite_sql() and with the alias for {node} table defined (e.g. {node} n)Otheruser_delete.module191N/A
Function summaries should be one line only. (Drupal Docs)Otheruser_delete.module331N/A
Separate comments from comment syntax by a space.Otheruser_delete.module352N/A
The $message argument to watchdog() should NOT be enclosed within t(), so that it can be properly translated at display time.Otheruser_delete.module353N/A
Function summaries should be one line only. (Drupal Docs)Otheruser_delete.module357N/A
Separate comments from comment syntax by a space.Otheruser_delete.module366N/A
Separate comments from comment syntax by a space.Otheruser_delete.module380N/A
Use PHP's master function, not an alias. (List of PHP aliases)Otheruser_delete.module391N/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)Otheruser_delete.module397N/A
Expanduser_delete.test102
MessageGroupFilenameLineFunctionStatus
Use uppercase for PHP constants, e.g. NULL, TRUE, FALSEOtheruser_delete.test70N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otheruser_delete.test105N/A
There should be no trailing spacesOtheruser_delete.test111N/A