Privatemsg - 7.x-1.x

Test status
StatusResult
Last receivedSat, 01/19/2013 - 08:37:11
Last requestedSat, 01/19/2013 - 08:38:01
Last testedSat, 01/19/2013 - 08:40:10
Test count182
Environment status
MySQLpass
Code reviewfail
Overall Resultpass
Branch information
Identifier7.x-1.x
ProjectPrivatemsg
LinkProject information
SummaryPASSED: [[Coder]]: [Code review] 6 minor(s), 7 critical(s), and 19 normal(s); [[SimpleTest]]: [MySQL] 3,125 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
3,125 pass(es), 0 fail(s), and 0 exception(s)
Test namePassFailException
LeafPrivatemsg API functionality. (PrivatemsgAPITestCase) [Privatemsg]11000
LeafPrivatemsg fields. (PrivatemsgFieldsTestCase) [Privatemsg]10700
LeafPrivatemsg Filter functionality. (PrivatemsgFilterTestCase) [Privatemsg]11200
LeafPrivatemsg filter widget (PrivatemsgFilterWidgetTestCase) [Privatemsg]4900
LeafPrivatemsg functionality. (PrivatemsgTestCase) [Privatemsg]74800
LeafPrivatemsg Limits (PrivatemsgLimitsTestCase) [Privatemsg]53100
LeafPrivatemsg links (PrivatemsgLinksTestCase) [Privatemsg]14700
LeafPrivatemsg Roles functionality (PrivatemsgRolesTestCase) [Privatemsg]84800
LeafPrivatemsg Tags functionality. (PrivatemsgTagsTestCase) [Privatemsg]20700
LeafPrivatemsg tokens (PrivatemsgTokenTestCase) [Privatemsg]9000
LeafUser blocking functionality. (PrivatemsgBlockUserCase) [Privatemsg]17600

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

Non-pass

Test nameMinorCriticalNormal
Expandpm_email_notify/pm_email_notify.module001
MessageGroupFilenameLineFunctionStatus
missing space after commaOtherpm_email_notify.module146N/A
Expandprivatemsg-recipients.tpl.php202
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherprivatemsg-recipients.tpl.php-1N/A
There should be no trailing spacesOtherprivatemsg-recipients.tpl.php1N/A
There should be no trailing spacesOtherprivatemsg-recipients.tpl.php2N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtherprivatemsg-recipients.tpl.php3N/A
Expandprivatemsg-view.tpl.php002
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherprivatemsg-view.tpl.php-1N/A
The control statement should be on a separate line from the control conditionalOtherprivatemsg-view.tpl.php9N/A
Expandprivatemsg.author-pane.inc001
MessageGroupFilenameLineFunctionStatus
The $text argument to l() should be enclosed within t() so that it is translatable.Otherprivatemsg.author-pane.inc23N/A
Expandprivatemsg.module019
MessageGroupFilenameLineFunctionStatus
The $string argument to t() should not begin or end with a space. (Drupal Docs)Otherprivatemsg.module146N/A
do not use mixed case (camelCase), use lower case and _Otherprivatemsg.module415N/A
do not use mixed case (camelCase), use lower case and _Otherprivatemsg.module444N/A
do not use mixed case (camelCase), use lower case and _Otherprivatemsg.module445N/A
do not use mixed case (camelCase), use lower case and _Otherprivatemsg.module446N/A
do not use mixed case (camelCase), use lower case and _Otherprivatemsg.module516N/A
Control statements should have one space between the control keyword and opening parenthesisOtherprivatemsg.module1950N/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)Otherprivatemsg.module2200N/A
use a space between the closing parenthesis and the open bracketOtherprivatemsg.module2346N/A
use a space between the closing parenthesis and the open bracketOtherprivatemsg.module2364N/A
Expandprivatemsg.pages.inc040
MessageGroupFilenameLineFunctionStatus
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.Otherprivatemsg.pages.inc591N/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)Otherprivatemsg.pages.inc627N/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)Otherprivatemsg.pages.inc649N/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)Otherprivatemsg.pages.inc652N/A
Expandprivatemsg_filter/privatemsg_filter.install010
MessageGroupFilenameLineFunctionStatus
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)Otherprivatemsg_filter.install104N/A
Expandprivatemsg_filter/privatemsg_filter.module003
MessageGroupFilenameLineFunctionStatus
missing space after commaOtherprivatemsg_filter.module709N/A
do not use mixed case (camelCase), use lower case and _Otherprivatemsg_filter.module895N/A
do not use mixed case (camelCase), use lower case and _Otherprivatemsg_filter.module916N/A
Expandprivatemsg_limits/privatemsg_limits.admin.inc010
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.Otherprivatemsg_limits.admin.inc136N/A
Expandprivatemsg_realname/privatemsg_realname.module001
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherprivatemsg_realname.module-1N/A

All

Test nameMinorCriticalNormal
Leafpm_block_user/pm_block_user.admin.inc000
Leafpm_block_user/pm_block_user.install000
Expandpm_block_user/pm_block_user.module100
MessageGroupFilenameLineFunctionStatus
There should be no trailing spacesOtherpm_block_user.module63N/A
Leafpm_block_user/pm_block_user.pages.inc000
Leafpm_block_user/pm_block_user.test000
Leafpm_email_notify/pm_email_notify.install000
Expandpm_email_notify/pm_email_notify.module001
MessageGroupFilenameLineFunctionStatus
missing space after commaOtherpm_email_notify.module146N/A
Expandprivatemsg-recipients.tpl.php202
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherprivatemsg-recipients.tpl.php-1N/A
There should be no trailing spacesOtherprivatemsg-recipients.tpl.php1N/A
There should be no trailing spacesOtherprivatemsg-recipients.tpl.php2N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtherprivatemsg-recipients.tpl.php3N/A
Expandprivatemsg-view.tpl.php002
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherprivatemsg-view.tpl.php-1N/A
The control statement should be on a separate line from the control conditionalOtherprivatemsg-view.tpl.php9N/A
Leafprivatemsg.admin.inc000
Leafprivatemsg.api.php000
Expandprivatemsg.author-pane.inc001
MessageGroupFilenameLineFunctionStatus
The $text argument to l() should be enclosed within t() so that it is translatable.Otherprivatemsg.author-pane.inc23N/A
Leafprivatemsg.install000
Expandprivatemsg.module019
MessageGroupFilenameLineFunctionStatus
The $string argument to t() should not begin or end with a space. (Drupal Docs)Otherprivatemsg.module146N/A
do not use mixed case (camelCase), use lower case and _Otherprivatemsg.module415N/A
do not use mixed case (camelCase), use lower case and _Otherprivatemsg.module444N/A
do not use mixed case (camelCase), use lower case and _Otherprivatemsg.module445N/A
do not use mixed case (camelCase), use lower case and _Otherprivatemsg.module446N/A
do not use mixed case (camelCase), use lower case and _Otherprivatemsg.module516N/A
Control statements should have one space between the control keyword and opening parenthesisOtherprivatemsg.module1950N/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)Otherprivatemsg.module2200N/A
use a space between the closing parenthesis and the open bracketOtherprivatemsg.module2346N/A
use a space between the closing parenthesis and the open bracketOtherprivatemsg.module2364N/A
Expandprivatemsg.pages.inc040
MessageGroupFilenameLineFunctionStatus
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.Otherprivatemsg.pages.inc591N/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)Otherprivatemsg.pages.inc627N/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)Otherprivatemsg.pages.inc649N/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)Otherprivatemsg.pages.inc652N/A
Expandprivatemsg.test200
MessageGroupFilenameLineFunctionStatus
in most cases, replace the string function with the drupal_ equivalent string functionsOtherprivatemsg.test901N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherprivatemsg.test960N/A
Leafprivatemsg.theme.inc000
Leafprivatemsg_filter/privatemsg_filter.admin.inc000
Expandprivatemsg_filter/privatemsg_filter.install010
MessageGroupFilenameLineFunctionStatus
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)Otherprivatemsg_filter.install104N/A
Expandprivatemsg_filter/privatemsg_filter.module003
MessageGroupFilenameLineFunctionStatus
missing space after commaOtherprivatemsg_filter.module709N/A
do not use mixed case (camelCase), use lower case and _Otherprivatemsg_filter.module895N/A
do not use mixed case (camelCase), use lower case and _Otherprivatemsg_filter.module916N/A
Leafprivatemsg_filter/privatemsg_filter.pages.inc000
Expandprivatemsg_filter/privatemsg_filter.test100
MessageGroupFilenameLineFunctionStatus
in most cases, replace the string function with the drupal_ equivalent string functionsOtherprivatemsg_filter.test249N/A
Expandprivatemsg_limits/privatemsg_limits.admin.inc010
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.Otherprivatemsg_limits.admin.inc136N/A
Leafprivatemsg_limits/privatemsg_limits.module000
Leafprivatemsg_limits/privatemsg_limits.test000
Expandprivatemsg_realname/privatemsg_realname.module001
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherprivatemsg_realname.module-1N/A
Leafprivatemsg_roles/privatemsg_roles.module000
Leafprivatemsg_roles/privatemsg_roles.test000
Leafprivatemsg_rules/privatemsg_rules.module000
Leafprivatemsg_rules/privatemsg_rules.rules.inc000
Leafviews/privatemsg.views.inc000
Leafviews/views_handler_field_privatemsg_link.inc000