Privatemsg - 6.x-2.x

Test status
StatusResult
Last receivedFri, 09/28/2012 - 21:22:25
Last requestedFri, 09/28/2012 - 21:25:34
Last testedFri, 09/28/2012 - 21:25:48
Test count98
Environment status
MySQLfail
Code reviewfail
Overall Resultfail
Branch information
Identifier6.x-2.x
ProjectPrivatemsg
LinkProject information
SummaryFAILED: [[SimpleTest]]: [MySQL] Repository checkout: failed to checkout from [git://git.drupal.org/project/privatemsg.git]; [[Coder]]: [Code review] 28 minor(s), 16 critical(s), and 40 normal(s).
Details

MySQL

  1. Setup environment
  2. Detect an invalid patch URL
  3. Detect invalid repository information
    Ensure the version control system is functioning properly.
  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
Review log
[14:25:28] Log initialized (/var/lib/drupaltestbot/sites/default/files/review.test_26760.pifr_simpletest.1348867528.log)
[14:25:28] Review timestamp: (Fri, 09/28/2012 - 14:25)
[14:25:28] Review host: (drupaltestbot813)
[14:25:28] Operations:Array
(
    [0] => setup
    [1] => fetch
    [2] => checkout
    [3] => check
    [4] => apply
    [5] => syntax
    [6] => install
    [7] => review
)

[14:25:28] Result Codes: Array
(
    [fail] => 9
    [pass] => 10
)

[14:25:28] Environment variables and review information initialized.
[14:25:28] core_url = [git://git.drupal.org/project/drupal.git];  Test repository = [git://git.drupal.org/project/privatemsg.git]
[14:25:28] Project directory is [privatemsg]
[14:25:28] Review started.
[14:25:28] Test: Array
(
    [test_id] => 26760
    [review] => Array
        (
            [plugin] => pifr_simpletest
            [argument] => Array
                (
                    [test.title] => 
                    [test.directory.review] => Array
                        (
                            [0] => 
                        )

                    [test.directory.apply] => 
                    [test.directory.dependency] => 
                    [test.files] => Array
                        (
                        )

                    [test.extensions] => Array
                        (
                            [0] => php
                            [1] => inc
                            [2] => install
                            [3] => module
                            [4] => test
                        )

                    [drupal.core.version] => 6
                    [drupal.core.url] => git://git.drupal.org/project/drupal.git
                    [drupal.modules] => Array
                        (
                            [0] => pm_block_user
                            [1] => privatemsg
                            [2] => pm_email_notify
                            [3] => privatemsg_attachments
                            [4] => privatemsg_filter
                            [5] => privatemsg_forward
                            [6] => privatemsg_limits
                            [7] => privatemsg_realname
                            [8] => privatemsg_roles
                            [9] => privatemsg_rules
                        )

                    [drupal.user] => admin
                    [drupal.pass] => 
                    [drupal.variables] => Array
                        (
                        )

                    [simpletest.tests] => Array
                        (
                        )

                    [simpletest.db] => 
                )

        )

    [vcs] => Array
        (
            [main] => Array
                (
                    [repository] => Array
                        (
                            [type] => git
                            [url] => git://git.drupal.org/project/drupal.git
                        )

                    [vcs_identifier] => 6.x
                )

            [dependencies] => Array
                (
                    [1] => Array
                        (
                            [repository] => Array
                                (
                                    [type] => git
                                    [url] => git://git.drupal.org/project/privatemsg.git
                                )

                            [vcs_identifier] => 6.x-1.x
                        )

                    [2] => Array
                        (
                            [repository] => Array
                                (
                                    [type] => git
                                    [url] => git://git.drupal.org/project/privatemsg.git
                                )

                            [vcs_identifier] => 6.x-2.x
                        )

                    [3] => Array
                        (
                            [repository] => Array
                                (
                                    [type] => git
                                    [url] => git://git.drupal.org/project/simpletest.git
                                )

                            [vcs_identifier] => 6.x-2.x
                        )

                )

        )

    [files] => Array
        (
        )

)

[14:25:28] Operations: Array
(
    [0] => setup
    [1] => fetch
    [2] => checkout
    [3] => check
    [4] => apply
    [5] => syntax
    [6] => install
    [7] => review
)

[14:25:28] Invoking operation [setup]...
[14:25:28] Command [chmod -R u+rw sites/default/files/checkout 2>&1] succeeded.
[14:25:28] Command [rm -rf sites/default/files/checkout 2>&1] succeeded.
[14:25:28] Command [rm -rf sites/default/files/review 2>&1] succeeded.
[14:25:28] Database backend [mysql] loaded.
[14:25:30] Invoking operation [fetch]...
[14:25:30] Invoking operation [checkout]...
[14:25:30] Version control system backend [git] loaded.
[14:25:31] Command [git clone 'git://git.drupal.org/project/drupal.git' 'checkout' --reference /var/cache/git/reference 2>&1] succeeded.
[14:25:32] Command [git --work-tree='checkout' --git-dir='checkout/.git' checkout '6.x' 2>&1] succeeded.
[14:25:32] Command [git --work-tree='checkout' --git-dir='checkout/.git' status 2>&1] succeeded.
[14:25:32] Command [git --work-tree='checkout' --git-dir='checkout/.git' log --oneline -1 2>&1] succeeded.
[14:25:32] Main branch [6.x] checkout [complete].
[14:25:33] Command [git clone 'git://git.drupal.org/project/privatemsg.git' '/var/lib/drupaltestbot/sites/default/files/checkout/sites/default/modules/privatemsg' --reference /var/cache/git/reference 2>&1] succeeded.
[14:25:33] Command [git --work-tree='/var/lib/drupaltestbot/sites/default/files/checkout/sites/default/modules/privatemsg' --git-dir='/var/lib/drupaltestbot/sites/default/files/checkout/sites/default/modules/privatemsg/.git' checkout '6.x-1.x' 2>&1] succeeded.
[14:25:33] Command [git --work-tree='/var/lib/drupaltestbot/sites/default/files/checkout/sites/default/modules/privatemsg' --git-dir='/var/lib/drupaltestbot/sites/default/files/checkout/sites/default/modules/privatemsg/.git' status 2>&1] succeeded.
[14:25:33] Command [git --work-tree='/var/lib/drupaltestbot/sites/default/files/checkout/sites/default/modules/privatemsg' --git-dir='/var/lib/drupaltestbot/sites/default/files/checkout/sites/default/modules/privatemsg/.git' log --oneline -1 2>&1] succeeded.
[14:25:33] Dependency branch [6.x-1.x] checkout [complete].
[14:25:33] Command [git clone 'git://git.drupal.org/project/privatemsg.git' '/var/lib/drupaltestbot/sites/default/files/checkout/sites/default/modules/privatemsg' --reference /var/cache/git/reference 2>&1] failed
  Duration 0 seconds
  Directory [/var/lib/drupaltestbot/sites/default/files/checkout/sites/default/modules]
  Status [128]
 Output: [fatal: destination path '/var/lib/drupaltestbot/sites/default/files/checkout/sites/default/modules/privatemsg' already exists and is not an empty directory.].
[14:25:33] Dependency branch [6.x-2.x] checkout [failed].
[14:25:33] Encountered error on [checkout], details:
array (
  '@reason' => 'failed to checkout from [git://git.drupal.org/project/privatemsg.git]',
)
[14:25:33] Review complete. test_id=26760 result code=3 details=Array
(
    [@reason] => failed to checkout from [git://git.drupal.org/project/privatemsg.git]
)

[14:25:33] Static variables reset.

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
28 minor(s), 16 critical(s), and 40 normal(s)

Non-pass

Test nameMinorCriticalNormal
Expandpm_block_user/pm_block_user.pages.inc030
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)Otherpm_block_user.pages.inc138N/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)Otherpm_block_user.pages.inc144N/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)Otherpm_block_user.pages.inc176N/A
Expandpm_email_notify/pm_email_notify.admin.inc001
MessageGroupFilenameLineFunctionStatus
missing space after commaOtherpm_email_notify.admin.inc50N/A
Expandpm_email_notify/pm_email_notify.module302
MessageGroupFilenameLineFunctionStatus
else statements should begin on a new lineOtherpm_email_notify.module79N/A
else statements should begin on a new lineOtherpm_email_notify.module150N/A
Use an indent of 2 spaces, with no tabsOtherpm_email_notify.module167N/A
Use an indent of 2 spaces, with no tabsOtherpm_email_notify.module168N/A
indent secondary line of comment one space Otherpm_email_notify.module211N/A
Expandprivatemsg-recipients.tpl.php001
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherprivatemsg-recipients.tpl.php-1N/A
Expandprivatemsg-view.tpl.php001
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherprivatemsg-view.tpl.php-1N/A
Expandprivatemsg.api.php101
MessageGroupFilenameLineFunctionStatus
Control statements should have one space between the control keyword and opening parenthesisOtherprivatemsg.api.php416N/A
@see should always be followed by a filename, a URL, class/interface name (optionally including method), or a function name including ().Otherprivatemsg.api.php724N/A
Expandprivatemsg.author-pane.inc001
MessageGroupFilenameLineFunctionStatus
The $text argument to l() should be enclosed within t() so that it is translatable.Otherprivatemsg.author-pane.inc25N/A
Expandprivatemsg.devel_generate.inc003
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherprivatemsg.devel_generate.inc-1N/A
else statements should begin on a new lineOtherprivatemsg.devel_generate.inc285N/A
missing space after commaOtherprivatemsg.devel_generate.inc292N/A
Expandprivatemsg.install0113
MessageGroupFilenameLineFunctionStatus
Arrays should be formatted with a space separating each element and assignment operatorOtherprivatemsg.install262N/A
Use update_sql() instead of db_query() in hook_update_N()Otherprivatemsg.install343N/A
Use update_sql() instead of db_query() in hook_update_N()Otherprivatemsg.install347N/A
Use update_sql() instead of db_query() in hook_update_N()Otherprivatemsg.install363N/A
Use update_sql() instead of db_query() in hook_update_N()Otherprivatemsg.install370N/A
Use update_sql() instead of db_query() in hook_update_N()Otherprivatemsg.install374N/A
Use update_sql() instead of db_query() in hook_update_N()Otherprivatemsg.install382N/A
Use update_sql() instead of db_query() in hook_update_N()Otherprivatemsg.install386N/A
Use update_sql() instead of db_query() in hook_update_N()Otherprivatemsg.install400N/A
Use update_sql() instead of db_query() in hook_update_N()Otherprivatemsg.install403N/A
Use update_sql() instead of db_query() in hook_update_N()Otherprivatemsg.install409N/A
Use update_sql() instead of db_query() in hook_update_N()Otherprivatemsg.install638N/A
Use update_sql() instead of db_query() in hook_update_N()Otherprivatemsg.install641N/A
table names should be enclosed in {curly_brackets}Otherprivatemsg.install765N/A
Expandprivatemsg.module2328
MessageGroupFilenameLineFunctionStatus
in most cases, replace the string function with the drupal_ equivalent string functionsOtherprivatemsg.module65N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherprivatemsg.module66N/A
The $string argument to t() should not begin or end with a space. (Drupal Docs)Otherprivatemsg.module118N/A
Use an indent of 2 spaces, with no tabsOtherprivatemsg.module357N/A
Use an indent of 2 spaces, with no tabsOtherprivatemsg.module358N/A
Use an indent of 2 spaces, with no tabsOtherprivatemsg.module359N/A
Use an indent of 2 spaces, with no tabsOtherprivatemsg.module360N/A
Use an indent of 2 spaces, with no tabsOtherprivatemsg.module363N/A
Use an indent of 2 spaces, with no tabsOtherprivatemsg.module364N/A
Use an indent of 2 spaces, with no tabsOtherprivatemsg.module365N/A
Use an indent of 2 spaces, with no tabsOtherprivatemsg.module366N/A
Use an indent of 2 spaces, with no tabsOtherprivatemsg.module367N/A
Use an indent of 2 spaces, with no tabsOtherprivatemsg.module368N/A
Use an indent of 2 spaces, with no tabsOtherprivatemsg.module369N/A
Use an indent of 2 spaces, with no tabsOtherprivatemsg.module370N/A
do not use mixed case (camelCase), use lower case and _Otherprivatemsg.module450N/A
do not use mixed case (camelCase), use lower case and _Otherprivatemsg.module552N/A
use a space between the closing parenthesis and the open bracketOtherprivatemsg.module761N/A
Use an indent of 2 spaces, with no tabsOtherprivatemsg.module942N/A
Use an indent of 2 spaces, with no tabsOtherprivatemsg.module945N/A
Use an indent of 2 spaces, with no tabsOtherprivatemsg.module946N/A
Use an indent of 2 spaces, with no tabsOtherprivatemsg.module947N/A
Use an indent of 2 spaces, with no tabsOtherprivatemsg.module948N/A
Use an indent of 2 spaces, with no tabsOtherprivatemsg.module949N/A
Use an indent of 2 spaces, with no tabsOtherprivatemsg.module950N/A
Use an indent of 2 spaces, with no tabsOtherprivatemsg.module982N/A
Menu item titles and descriptions should NOT be enclosed within t(). (Drupal Docs)Otherprivatemsg.module1516N/A
Control statements should have one space between the control keyword and opening parenthesisOtherprivatemsg.module2198N/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.module2276N/A
In SQL strings, Use db_query() placeholders in place of variables. This is a potential source of SQL injection attacks when the variable can come from user data. (Drupal Docs)Otherprivatemsg.module2309N/A
Control statements should have one space between the control keyword and opening parenthesisOtherprivatemsg.module2310N/A
@see references should be separated by "," followed by a single space and with no trailing punctuationOtherprivatemsg.module2992N/A
missing space after commaOtherprivatemsg.module3058N/A
Expandprivatemsg.pages.inc031
MessageGroupFilenameLineFunctionStatus
else statements should begin on a new lineOtherprivatemsg.pages.inc199N/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.Otherprivatemsg.pages.inc534N/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.inc555N/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.inc575N/A
Expandprivatemsg.test002
MessageGroupFilenameLineFunctionStatus
missing space after commaOtherprivatemsg.test1321N/A
missing space after commaOtherprivatemsg.test1330N/A
Expandprivatemsg_attachments/privatemsg_attachments.module031
MessageGroupFilenameLineFunctionStatus
Use an indent of 2 spaces, with no tabsOtherprivatemsg_attachments.module102N/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.Otherprivatemsg_attachments.module123N/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.Otherprivatemsg_attachments.module144N/A
Potential problem: when FAPI element '#type' is set to 'markup' (default), '#value' only accepts filtered text, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized.Otherprivatemsg_attachments.module157N/A
Expandprivatemsg_filter/privatemsg_filter.admin.inc010
MessageGroupFilenameLineFunctionStatus
Potential problem: when FAPI element '#type' is set to 'markup' (default), '#value' only accepts filtered text, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized.Otherprivatemsg_filter.admin.inc209N/A
Expandprivatemsg_filter/privatemsg_filter.install011
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.install97N/A
Use update_sql() instead of db_query() in hook_update_N()Otherprivatemsg_filter.install197N/A
Expandprivatemsg_filter/privatemsg_filter.module003
MessageGroupFilenameLineFunctionStatus
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtherprivatemsg_filter.module89N/A
do not use mixed case (camelCase), use lower case and _Otherprivatemsg_filter.module878N/A
do not use mixed case (camelCase), use lower case and _Otherprivatemsg_filter.module905N/A
Expandprivatemsg_forward/privatemsg_forward.module010
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_forward.module113N/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
Expandpm_block_user/pm_block_user.admin.inc100
MessageGroupFilenameLineFunctionStatus
Potential problem: use the Form API to prevent against CSRF attacks. If you need to use $_POST variables, ensure they are fully sanitized if displayed by using check_plain(), filter_xss() or similar. (Drupal Docs)Otherpm_block_user.admin.inc235N/A
Leafpm_block_user/pm_block_user.install000
Leafpm_block_user/pm_block_user.module000
Expandpm_block_user/pm_block_user.pages.inc030
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)Otherpm_block_user.pages.inc138N/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)Otherpm_block_user.pages.inc144N/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)Otherpm_block_user.pages.inc176N/A
Leafpm_block_user/pm_block_user.test000
Expandpm_email_notify/pm_email_notify.admin.inc001
MessageGroupFilenameLineFunctionStatus
missing space after commaOtherpm_email_notify.admin.inc50N/A
Leafpm_email_notify/pm_email_notify.api.php000
Leafpm_email_notify/pm_email_notify.install000
Expandpm_email_notify/pm_email_notify.module302
MessageGroupFilenameLineFunctionStatus
else statements should begin on a new lineOtherpm_email_notify.module79N/A
else statements should begin on a new lineOtherpm_email_notify.module150N/A
Use an indent of 2 spaces, with no tabsOtherpm_email_notify.module167N/A
Use an indent of 2 spaces, with no tabsOtherpm_email_notify.module168N/A
indent secondary line of comment one space Otherpm_email_notify.module211N/A
Leafpm_email_notify/pm_email_notify.test000
Expandprivatemsg-recipients.tpl.php001
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherprivatemsg-recipients.tpl.php-1N/A
Expandprivatemsg-view.tpl.php001
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherprivatemsg-view.tpl.php-1N/A
Leafprivatemsg.admin.inc000
Expandprivatemsg.api.php101
MessageGroupFilenameLineFunctionStatus
Control statements should have one space between the control keyword and opening parenthesisOtherprivatemsg.api.php416N/A
@see should always be followed by a filename, a URL, class/interface name (optionally including method), or a function name including ().Otherprivatemsg.api.php724N/A
Expandprivatemsg.author-pane.inc001
MessageGroupFilenameLineFunctionStatus
The $text argument to l() should be enclosed within t() so that it is translatable.Otherprivatemsg.author-pane.inc25N/A
Expandprivatemsg.devel_generate.inc003
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherprivatemsg.devel_generate.inc-1N/A
else statements should begin on a new lineOtherprivatemsg.devel_generate.inc285N/A
missing space after commaOtherprivatemsg.devel_generate.inc292N/A
Expandprivatemsg.install0113
MessageGroupFilenameLineFunctionStatus
Arrays should be formatted with a space separating each element and assignment operatorOtherprivatemsg.install262N/A
Use update_sql() instead of db_query() in hook_update_N()Otherprivatemsg.install343N/A
Use update_sql() instead of db_query() in hook_update_N()Otherprivatemsg.install347N/A
Use update_sql() instead of db_query() in hook_update_N()Otherprivatemsg.install363N/A
Use update_sql() instead of db_query() in hook_update_N()Otherprivatemsg.install370N/A
Use update_sql() instead of db_query() in hook_update_N()Otherprivatemsg.install374N/A
Use update_sql() instead of db_query() in hook_update_N()Otherprivatemsg.install382N/A
Use update_sql() instead of db_query() in hook_update_N()Otherprivatemsg.install386N/A
Use update_sql() instead of db_query() in hook_update_N()Otherprivatemsg.install400N/A
Use update_sql() instead of db_query() in hook_update_N()Otherprivatemsg.install403N/A
Use update_sql() instead of db_query() in hook_update_N()Otherprivatemsg.install409N/A
Use update_sql() instead of db_query() in hook_update_N()Otherprivatemsg.install638N/A
Use update_sql() instead of db_query() in hook_update_N()Otherprivatemsg.install641N/A
table names should be enclosed in {curly_brackets}Otherprivatemsg.install765N/A
Expandprivatemsg.module2328
MessageGroupFilenameLineFunctionStatus
in most cases, replace the string function with the drupal_ equivalent string functionsOtherprivatemsg.module65N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherprivatemsg.module66N/A
The $string argument to t() should not begin or end with a space. (Drupal Docs)Otherprivatemsg.module118N/A
Use an indent of 2 spaces, with no tabsOtherprivatemsg.module357N/A
Use an indent of 2 spaces, with no tabsOtherprivatemsg.module358N/A
Use an indent of 2 spaces, with no tabsOtherprivatemsg.module359N/A
Use an indent of 2 spaces, with no tabsOtherprivatemsg.module360N/A
Use an indent of 2 spaces, with no tabsOtherprivatemsg.module363N/A
Use an indent of 2 spaces, with no tabsOtherprivatemsg.module364N/A
Use an indent of 2 spaces, with no tabsOtherprivatemsg.module365N/A
Use an indent of 2 spaces, with no tabsOtherprivatemsg.module366N/A
Use an indent of 2 spaces, with no tabsOtherprivatemsg.module367N/A
Use an indent of 2 spaces, with no tabsOtherprivatemsg.module368N/A
Use an indent of 2 spaces, with no tabsOtherprivatemsg.module369N/A
Use an indent of 2 spaces, with no tabsOtherprivatemsg.module370N/A
do not use mixed case (camelCase), use lower case and _Otherprivatemsg.module450N/A
do not use mixed case (camelCase), use lower case and _Otherprivatemsg.module552N/A
use a space between the closing parenthesis and the open bracketOtherprivatemsg.module761N/A
Use an indent of 2 spaces, with no tabsOtherprivatemsg.module942N/A
Use an indent of 2 spaces, with no tabsOtherprivatemsg.module945N/A
Use an indent of 2 spaces, with no tabsOtherprivatemsg.module946N/A
Use an indent of 2 spaces, with no tabsOtherprivatemsg.module947N/A
Use an indent of 2 spaces, with no tabsOtherprivatemsg.module948N/A
Use an indent of 2 spaces, with no tabsOtherprivatemsg.module949N/A
Use an indent of 2 spaces, with no tabsOtherprivatemsg.module950N/A
Use an indent of 2 spaces, with no tabsOtherprivatemsg.module982N/A
Menu item titles and descriptions should NOT be enclosed within t(). (Drupal Docs)Otherprivatemsg.module1516N/A
Control statements should have one space between the control keyword and opening parenthesisOtherprivatemsg.module2198N/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.module2276N/A
In SQL strings, Use db_query() placeholders in place of variables. This is a potential source of SQL injection attacks when the variable can come from user data. (Drupal Docs)Otherprivatemsg.module2309N/A
Control statements should have one space between the control keyword and opening parenthesisOtherprivatemsg.module2310N/A
@see references should be separated by "," followed by a single space and with no trailing punctuationOtherprivatemsg.module2992N/A
missing space after commaOtherprivatemsg.module3058N/A
Expandprivatemsg.pages.inc031
MessageGroupFilenameLineFunctionStatus
else statements should begin on a new lineOtherprivatemsg.pages.inc199N/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.Otherprivatemsg.pages.inc534N/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.inc555N/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.inc575N/A
Expandprivatemsg.test002
MessageGroupFilenameLineFunctionStatus
missing space after commaOtherprivatemsg.test1321N/A
missing space after commaOtherprivatemsg.test1330N/A
Leafprivatemsg.theme.inc000
Leafprivatemsg_attachments/privatemsg_attachments.install000
Expandprivatemsg_attachments/privatemsg_attachments.module031
MessageGroupFilenameLineFunctionStatus
Use an indent of 2 spaces, with no tabsOtherprivatemsg_attachments.module102N/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.Otherprivatemsg_attachments.module123N/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.Otherprivatemsg_attachments.module144N/A
Potential problem: when FAPI element '#type' is set to 'markup' (default), '#value' only accepts filtered text, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized.Otherprivatemsg_attachments.module157N/A
Leafprivatemsg_attachments/privatemsg_attachments.test000
Expandprivatemsg_filter/privatemsg_filter.admin.inc010
MessageGroupFilenameLineFunctionStatus
Potential problem: when FAPI element '#type' is set to 'markup' (default), '#value' only accepts filtered text, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized.Otherprivatemsg_filter.admin.inc209N/A
Expandprivatemsg_filter/privatemsg_filter.install011
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.install97N/A
Use update_sql() instead of db_query() in hook_update_N()Otherprivatemsg_filter.install197N/A
Expandprivatemsg_filter/privatemsg_filter.module003
MessageGroupFilenameLineFunctionStatus
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtherprivatemsg_filter.module89N/A
do not use mixed case (camelCase), use lower case and _Otherprivatemsg_filter.module878N/A
do not use mixed case (camelCase), use lower case and _Otherprivatemsg_filter.module905N/A
Leafprivatemsg_filter/privatemsg_filter.test000
Expandprivatemsg_forward/privatemsg_forward.module010
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_forward.module113N/A
Leafprivatemsg_forward/privatemsg_forward.test000
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
Leafprivatemsg_service/privatemsg_service.api.php000
Leafprivatemsg_service/privatemsg_service.inc000
Leafprivatemsg_service/privatemsg_service.module000
Leafprivatemsg_service/privatemsg_service.test000
Leafviews/privatemsg.views.inc000
Leafviews/views_handler_field_privatemsg_link.inc000