Multi-Language Link and Redirect (MultiLink) - 6.x-2.7

Test status
StatusResult
Last receivedMon, 01/30/2012 - 08:51:11
Last requestedMon, 01/30/2012 - 08:51:43
Last testedMon, 01/30/2012 - 08:51:51
Test count2
Environment status
MySQLpass
Code reviewfail
Overall Resultpass
Branch information
Identifier6.x-2.7
ProjectMulti-Language Link and Redirect (MultiLink)
LinkProject information
SummaryPASSED: [[SimpleTest]]: [MySQL] 0 pass(es); [[Coder]]: [Code review] 4 minor(s), 2 critical(s), and 88 normal(s).
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
4 minor(s), 2 critical(s), and 88 normal(s)

Non-pass

Test nameMinorCriticalNormal
Expandmultilink.install004
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othermultilink.install-1N/A
Function summaries should be one line only. (Drupal Docs)Othermultilink.install3N/A
Function summaries should be one line only. (Drupal Docs)Othermultilink.install11N/A
Doxygen uses @todo and @bug to markup things to be done.Othermultilink.install30N/A
Expandmultilink.module3156
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othermultilink.module-1N/A
Commits to the Git repository do not require the CVS $Id$ keyword in each file. (Drupal Docs)Othermultilink.module2N/A
Use uppercase for PHP constants, e.g. NULL, TRUE, FALSEOthermultilink.module12N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthermultilink.module30N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthermultilink.module33N/A
Use uppercase for PHP constants, e.g. NULL, TRUE, FALSEOthermultilink.module42N/A
Use uppercase for PHP constants, e.g. NULL, TRUE, FALSEOthermultilink.module43N/A
Function documentation should be less than 80 characters per line.Othermultilink.module61N/A
use a space between the closing parenthesis and the open bracketOthermultilink.module73N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthermultilink.module94N/A
Separate comments from comment syntax by a space.Othermultilink.module95N/A
Separate comments from comment syntax by a space.Othermultilink.module123N/A
Function documentation should be less than 80 characters per line.Othermultilink.module129N/A
Function documentation should be less than 80 characters per line.Othermultilink.module130N/A
Control statements should have one space between the control keyword and opening parenthesisOthermultilink.module175N/A
Separate comments from comment syntax by a space.Othermultilink.module176N/A
Function documentation should be less than 80 characters per line.Othermultilink.module207N/A
missing space after commaOthermultilink.module231N/A
Control statements should have one space between the control keyword and opening parenthesisOthermultilink.module243N/A
Control statements should have one space between the control keyword and opening parenthesisOthermultilink.module265N/A
Control statements should have one space between the control keyword and opening parenthesisOthermultilink.module273N/A
Separate comments from comment syntax by a space.Othermultilink.module280N/A
Separate comments from comment syntax by a space.Othermultilink.module317N/A
Separate comments from comment syntax by a space.Othermultilink.module319N/A
Control statements should have one space between the control keyword and opening parenthesisOthermultilink.module334N/A
Separate comments from comment syntax by a space.Othermultilink.module340N/A
Separate comments from comment syntax by a space.Othermultilink.module342N/A
Separate comments from comment syntax by a space.Othermultilink.module351N/A
Separate comments from comment syntax by a space.Othermultilink.module354N/A
Function documentation should be less than 80 characters per line.Othermultilink.module359N/A
Control statements should have one space between the control keyword and opening parenthesisOthermultilink.module371N/A
Use uppercase for PHP constants, e.g. NULL, TRUE, FALSEOthermultilink.module393N/A
Control statements should have one space between the control keyword and opening parenthesisOthermultilink.module406N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthermultilink.module504N/A
Separate comments from comment syntax by a space.Othermultilink.module515N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthermultilink.module523N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthermultilink.module539N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthermultilink.module547N/A
Control statements should have one space between the control keyword and opening parenthesisOthermultilink.module573N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthermultilink.module575N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthermultilink.module585N/A
indent secondary line of comment one space Othermultilink.module603N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthermultilink.module614N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthermultilink.module622N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthermultilink.module646N/A
Function documentation should be less than 80 characters per line.Othermultilink.module663N/A
Control statements should have one space between the control keyword and opening parenthesisOthermultilink.module682N/A
Function summaries should be one line only. (Drupal Docs)Othermultilink.module690N/A
Control statements should have one space between the control keyword and opening parenthesisOthermultilink.module695N/A
Function summaries should be one line only. (Drupal Docs)Othermultilink.module701N/A
Function documentation should be less than 80 characters per line.Othermultilink.module703N/A
Control statements should have one space between the control keyword and opening parenthesisOthermultilink.module707N/A
Separate comments from comment syntax by a space.Othermultilink.module720N/A
Function documentation should be less than 80 characters per line.Othermultilink.module725N/A
Separate comments from comment syntax by a space.Othermultilink.module731N/A
Separate comments from comment syntax by a space.Othermultilink.module739N/A
Separate comments from comment syntax by a space.Othermultilink.module766N/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.Othermultilink.module777N/A
indent secondary line of comment one space Othermultilink.module788N/A
put a space between the asterisk and the comment textOthermultilink.module788N/A
Expandmultilink_redirect/multilink_redirect.install004
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othermultilink_redirect.install-1N/A
Function summaries should be one line only. (Drupal Docs)Othermultilink_redirect.install9N/A
Use update_sql() instead of db_query() in hook_update_N()Othermultilink_redirect.install20N/A
Use update_sql() instead of db_query() in hook_update_N()Othermultilink_redirect.install26N/A
Expandmultilink_redirect/multilink_redirect.module018
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othermultilink_redirect.module-1N/A
Commits to the Git repository do not require the CVS $Id$ keyword in each file. (Drupal Docs)Othermultilink_redirect.module2N/A
Separate comments from comment syntax by a space.Othermultilink_redirect.module21N/A
Separate comments from comment syntax by a space.Othermultilink_redirect.module29N/A
Separate comments from comment syntax by a space.Othermultilink_redirect.module64N/A
Use the matching Drupal theme functions, not raw HTML.Othermultilink_redirect.module100N/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)Othermultilink_redirect.module104N/A
Separate comments from comment syntax by a space.Othermultilink_redirect.module105N/A
Use uppercase for PHP constants, e.g. NULL, TRUE, FALSEOthermultilink_redirect.module109N/A
Expandmultilink_redirect_cached/multilink_redirect_cached.install001
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othermultilink_redirect_cached.install-1N/A
Expandmultilink_redirect_cached/multilink_redirect_cached.module004
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othermultilink_redirect_cached.module-1N/A
Commits to the Git repository do not require the CVS $Id$ keyword in each file. (Drupal Docs)Othermultilink_redirect_cached.module2N/A
Control statements should have one space between the control keyword and opening parenthesisOthermultilink_redirect_cached.module19N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othermultilink_redirect_cached.module35N/A
Expandmultilink_securepages/multilink_securepages.install003
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othermultilink_securepages.install-1N/A
Function summaries should be one line only. (Drupal Docs)Othermultilink_securepages.install4N/A
The $message argument to drupal_set_message() should be enclosed within t() so that it is translatable.Othermultilink_securepages.install10N/A
Expandmultilink_securepages/multilink_securepages.module108
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othermultilink_securepages.module-1N/A
Commits to the Git repository do not require the CVS $Id$ keyword in each file. (Drupal Docs)Othermultilink_securepages.module2N/A
Function documentation should be less than 80 characters per line.Othermultilink_securepages.module12N/A
Separate comments from comment syntax by a space.Othermultilink_securepages.module22N/A
indent secondary line of comment one space Othermultilink_securepages.module41N/A
Separate comments from comment syntax by a space.Othermultilink_securepages.module52N/A
Separate comments from comment syntax by a space.Othermultilink_securepages.module54N/A
Separate comments from comment syntax by a space.Othermultilink_securepages.module73N/A
curly braces { should end a line, not start oneOthermultilink_securepages.module96N/A

All

Test nameMinorCriticalNormal
Expandmultilink.install004
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othermultilink.install-1N/A
Function summaries should be one line only. (Drupal Docs)Othermultilink.install3N/A
Function summaries should be one line only. (Drupal Docs)Othermultilink.install11N/A
Doxygen uses @todo and @bug to markup things to be done.Othermultilink.install30N/A
Expandmultilink.module3156
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othermultilink.module-1N/A
Commits to the Git repository do not require the CVS $Id$ keyword in each file. (Drupal Docs)Othermultilink.module2N/A
Use uppercase for PHP constants, e.g. NULL, TRUE, FALSEOthermultilink.module12N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthermultilink.module30N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthermultilink.module33N/A
Use uppercase for PHP constants, e.g. NULL, TRUE, FALSEOthermultilink.module42N/A
Use uppercase for PHP constants, e.g. NULL, TRUE, FALSEOthermultilink.module43N/A
Function documentation should be less than 80 characters per line.Othermultilink.module61N/A
use a space between the closing parenthesis and the open bracketOthermultilink.module73N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthermultilink.module94N/A
Separate comments from comment syntax by a space.Othermultilink.module95N/A
Separate comments from comment syntax by a space.Othermultilink.module123N/A
Function documentation should be less than 80 characters per line.Othermultilink.module129N/A
Function documentation should be less than 80 characters per line.Othermultilink.module130N/A
Control statements should have one space between the control keyword and opening parenthesisOthermultilink.module175N/A
Separate comments from comment syntax by a space.Othermultilink.module176N/A
Function documentation should be less than 80 characters per line.Othermultilink.module207N/A
missing space after commaOthermultilink.module231N/A
Control statements should have one space between the control keyword and opening parenthesisOthermultilink.module243N/A
Control statements should have one space between the control keyword and opening parenthesisOthermultilink.module265N/A
Control statements should have one space between the control keyword and opening parenthesisOthermultilink.module273N/A
Separate comments from comment syntax by a space.Othermultilink.module280N/A
Separate comments from comment syntax by a space.Othermultilink.module317N/A
Separate comments from comment syntax by a space.Othermultilink.module319N/A
Control statements should have one space between the control keyword and opening parenthesisOthermultilink.module334N/A
Separate comments from comment syntax by a space.Othermultilink.module340N/A
Separate comments from comment syntax by a space.Othermultilink.module342N/A
Separate comments from comment syntax by a space.Othermultilink.module351N/A
Separate comments from comment syntax by a space.Othermultilink.module354N/A
Function documentation should be less than 80 characters per line.Othermultilink.module359N/A
Control statements should have one space between the control keyword and opening parenthesisOthermultilink.module371N/A
Use uppercase for PHP constants, e.g. NULL, TRUE, FALSEOthermultilink.module393N/A
Control statements should have one space between the control keyword and opening parenthesisOthermultilink.module406N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthermultilink.module504N/A
Separate comments from comment syntax by a space.Othermultilink.module515N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthermultilink.module523N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthermultilink.module539N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthermultilink.module547N/A
Control statements should have one space between the control keyword and opening parenthesisOthermultilink.module573N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthermultilink.module575N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthermultilink.module585N/A
indent secondary line of comment one space Othermultilink.module603N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthermultilink.module614N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthermultilink.module622N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOthermultilink.module646N/A
Function documentation should be less than 80 characters per line.Othermultilink.module663N/A
Control statements should have one space between the control keyword and opening parenthesisOthermultilink.module682N/A
Function summaries should be one line only. (Drupal Docs)Othermultilink.module690N/A
Control statements should have one space between the control keyword and opening parenthesisOthermultilink.module695N/A
Function summaries should be one line only. (Drupal Docs)Othermultilink.module701N/A
Function documentation should be less than 80 characters per line.Othermultilink.module703N/A
Control statements should have one space between the control keyword and opening parenthesisOthermultilink.module707N/A
Separate comments from comment syntax by a space.Othermultilink.module720N/A
Function documentation should be less than 80 characters per line.Othermultilink.module725N/A
Separate comments from comment syntax by a space.Othermultilink.module731N/A
Separate comments from comment syntax by a space.Othermultilink.module739N/A
Separate comments from comment syntax by a space.Othermultilink.module766N/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.Othermultilink.module777N/A
indent secondary line of comment one space Othermultilink.module788N/A
put a space between the asterisk and the comment textOthermultilink.module788N/A
Expandmultilink_redirect/multilink_redirect.install004
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othermultilink_redirect.install-1N/A
Function summaries should be one line only. (Drupal Docs)Othermultilink_redirect.install9N/A
Use update_sql() instead of db_query() in hook_update_N()Othermultilink_redirect.install20N/A
Use update_sql() instead of db_query() in hook_update_N()Othermultilink_redirect.install26N/A
Expandmultilink_redirect/multilink_redirect.module018
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othermultilink_redirect.module-1N/A
Commits to the Git repository do not require the CVS $Id$ keyword in each file. (Drupal Docs)Othermultilink_redirect.module2N/A
Separate comments from comment syntax by a space.Othermultilink_redirect.module21N/A
Separate comments from comment syntax by a space.Othermultilink_redirect.module29N/A
Separate comments from comment syntax by a space.Othermultilink_redirect.module64N/A
Use the matching Drupal theme functions, not raw HTML.Othermultilink_redirect.module100N/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)Othermultilink_redirect.module104N/A
Separate comments from comment syntax by a space.Othermultilink_redirect.module105N/A
Use uppercase for PHP constants, e.g. NULL, TRUE, FALSEOthermultilink_redirect.module109N/A
Expandmultilink_redirect_cached/multilink_redirect_cached.install001
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othermultilink_redirect_cached.install-1N/A
Expandmultilink_redirect_cached/multilink_redirect_cached.module004
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othermultilink_redirect_cached.module-1N/A
Commits to the Git repository do not require the CVS $Id$ keyword in each file. (Drupal Docs)Othermultilink_redirect_cached.module2N/A
Control statements should have one space between the control keyword and opening parenthesisOthermultilink_redirect_cached.module19N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othermultilink_redirect_cached.module35N/A
Expandmultilink_securepages/multilink_securepages.install003
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othermultilink_securepages.install-1N/A
Function summaries should be one line only. (Drupal Docs)Othermultilink_securepages.install4N/A
The $message argument to drupal_set_message() should be enclosed within t() so that it is translatable.Othermultilink_securepages.install10N/A
Expandmultilink_securepages/multilink_securepages.module108
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othermultilink_securepages.module-1N/A
Commits to the Git repository do not require the CVS $Id$ keyword in each file. (Drupal Docs)Othermultilink_securepages.module2N/A
Function documentation should be less than 80 characters per line.Othermultilink_securepages.module12N/A
Separate comments from comment syntax by a space.Othermultilink_securepages.module22N/A
indent secondary line of comment one space Othermultilink_securepages.module41N/A
Separate comments from comment syntax by a space.Othermultilink_securepages.module52N/A
Separate comments from comment syntax by a space.Othermultilink_securepages.module54N/A
Separate comments from comment syntax by a space.Othermultilink_securepages.module73N/A
curly braces { should end a line, not start oneOthermultilink_securepages.module96N/A