Role Change Notify - 7.x-1.x

Test status
StatusResult
Last receivedThu, 01/31/2013 - 04:16:39
Last requestedThu, 01/31/2013 - 04:16:41
Last testedThu, 01/31/2013 - 04:17:17
Test count20
Environment status
MySQLfail
Code reviewfail
Overall Resultfail
Branch information
Identifier7.x-1.x
ProjectRole Change Notify
LinkProject information
SummaryPASSED: [[Coder]]: [Code review] 0 minor(s), 2 critical(s), and 13 normal(s); [[SimpleTest]]: [MySQL] 64 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
64 pass(es), 0 fail(s), and 0 exception(s)
Test namePassFailException
LeafRole Change Notify (RoleChangeNotifyTestCase) [Role Change Notify]6400

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
Expandrole_change_notify.install002
MessageGroupFilenameLineFunctionStatus
Control statements should have one space between the control keyword and opening parenthesisOtherrole_change_notify.install41N/A
Use the matching Drupal theme functions, not raw HTML.Otherrole_change_notify.install41N/A
Expandrole_change_notify.module029
MessageGroupFilenameLineFunctionStatus
Remove the empty commented line in your function documentation.Otherrole_change_notify.module6N/A
Use the matching Drupal theme functions, not raw HTML.Otherrole_change_notify.module20N/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)Otherrole_change_notify.module25N/A
Use the matching Drupal theme functions, not raw HTML.Otherrole_change_notify.module31N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherrole_change_notify.module44N/A
Function documentation should be less than 80 characters per line.Otherrole_change_notify.module80N/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.Otherrole_change_notify.module101N/A
Core uses "e-mail" in end-user text and "mail" elsewhere (database, function names, etc.)Otherrole_change_notify.module116N/A
Core uses "e-mail" in end-user text and "mail" elsewhere (database, function names, etc.)Otherrole_change_notify.module123N/A
Function summaries should be one line only. (Drupal Docs)Otherrole_change_notify.module163N/A
Function summaries should be one line only. (Drupal Docs)Otherrole_change_notify.module187N/A
Expandrole_change_notify.test001
MessageGroupFilenameLineFunctionStatus
Remove the empty commented line in your function documentation.Otherrole_change_notify.test9N/A
Expandrole_change_notify.tokens.inc001
MessageGroupFilenameLineFunctionStatus
Remove the empty commented line in your function documentation.Otherrole_change_notify.tokens.inc6N/A

All

Test nameMinorCriticalNormal
Expandrole_change_notify.install002
MessageGroupFilenameLineFunctionStatus
Control statements should have one space between the control keyword and opening parenthesisOtherrole_change_notify.install41N/A
Use the matching Drupal theme functions, not raw HTML.Otherrole_change_notify.install41N/A
Expandrole_change_notify.module029
MessageGroupFilenameLineFunctionStatus
Remove the empty commented line in your function documentation.Otherrole_change_notify.module6N/A
Use the matching Drupal theme functions, not raw HTML.Otherrole_change_notify.module20N/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)Otherrole_change_notify.module25N/A
Use the matching Drupal theme functions, not raw HTML.Otherrole_change_notify.module31N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherrole_change_notify.module44N/A
Function documentation should be less than 80 characters per line.Otherrole_change_notify.module80N/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.Otherrole_change_notify.module101N/A
Core uses "e-mail" in end-user text and "mail" elsewhere (database, function names, etc.)Otherrole_change_notify.module116N/A
Core uses "e-mail" in end-user text and "mail" elsewhere (database, function names, etc.)Otherrole_change_notify.module123N/A
Function summaries should be one line only. (Drupal Docs)Otherrole_change_notify.module163N/A
Function summaries should be one line only. (Drupal Docs)Otherrole_change_notify.module187N/A
Expandrole_change_notify.test001
MessageGroupFilenameLineFunctionStatus
Remove the empty commented line in your function documentation.Otherrole_change_notify.test9N/A
Expandrole_change_notify.tokens.inc001
MessageGroupFilenameLineFunctionStatus
Remove the empty commented line in your function documentation.Otherrole_change_notify.tokens.inc6N/A