simpleSAMLphp Authentication - 6.x-2.x

Test status
StatusResult
Last receivedFri, 06/08/2012 - 14:50:11
Last requestedFri, 06/08/2012 - 14:51:03
Last testedFri, 06/08/2012 - 14:51:18
Test count4
Environment status
MySQLpass
Code reviewfail
Overall Resultpass
Branch information
Identifier6.x-2.x
ProjectsimpleSAMLphp Authentication
LinkProject information
SummaryPASSED: [[Coder]]: [Code review] 0 minor(s), 3 critical(s), and 24 normal(s); [[SimpleTest]]: [MySQL] 0 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

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), 3 critical(s), and 24 normal(s)

Non-pass

Test nameMinorCriticalNormal
Expandsimplesamlphp_auth.install004
MessageGroupFilenameLineFunctionStatus
The $message argument to drupal_set_message() should be enclosed within t() so that it is translatable.Othersimplesamlphp_auth.install36N/A
The $message argument to drupal_set_message() should be enclosed within t() so that it is translatable.Othersimplesamlphp_auth.install39N/A
The $message argument to drupal_set_message() should be enclosed within t() so that it is translatable.Othersimplesamlphp_auth.install77N/A
The $message argument to drupal_set_message() should be enclosed within t() so that it is translatable.Othersimplesamlphp_auth.install80N/A
Expandsimplesamlphp_auth.module0320
MessageGroupFilenameLineFunctionStatus
Menu item titles and descriptions should NOT be enclosed within t(). (Drupal Docs)Othersimplesamlphp_auth.module26N/A
Menu item titles and descriptions should NOT be enclosed within t(). (Drupal Docs)Othersimplesamlphp_auth.module27N/A
Menu item titles and descriptions should NOT be enclosed within t(). (Drupal Docs)Othersimplesamlphp_auth.module34N/A
Menu item titles and descriptions should NOT be enclosed within t(). (Drupal Docs)Othersimplesamlphp_auth.module35N/A
do not use mixed case (camelCase), use lower case and _Othersimplesamlphp_auth.module93N/A
do not use mixed case (camelCase), use lower case and _Othersimplesamlphp_auth.module96N/A
The $message argument to watchdog() should NOT be enclosed within t(), so that it can be properly translated at display time.Othersimplesamlphp_auth.module97N/A
The $message argument to watchdog() should NOT be enclosed within t(), so that it can be properly translated at display time.Othersimplesamlphp_auth.module103N/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.Othersimplesamlphp_auth.module245N/A
do not use mixed case (camelCase), use lower case and _Othersimplesamlphp_auth.module297N/A
do not use mixed case (camelCase), use lower case and _Othersimplesamlphp_auth.module298N/A
do not use mixed case (camelCase), use lower case and _Othersimplesamlphp_auth.module301N/A
do not use mixed case (camelCase), use lower case and _Othersimplesamlphp_auth.module306N/A
do not use mixed case (camelCase), use lower case and _Othersimplesamlphp_auth.module309N/A
do not use mixed case (camelCase), use lower case and _Othersimplesamlphp_auth.module311N/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)Othersimplesamlphp_auth.module351N/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)Othersimplesamlphp_auth.module364N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Othersimplesamlphp_auth.module516N/A
do not use mixed case (camelCase), use lower case and _Othersimplesamlphp_auth.module578N/A
do not use mixed case (camelCase), use lower case and _Othersimplesamlphp_auth.module579N/A
do not use mixed case (camelCase), use lower case and _Othersimplesamlphp_auth.module580N/A
do not use mixed case (camelCase), use lower case and _Othersimplesamlphp_auth.module604N/A
The $message argument to watchdog() should NOT be enclosed within t(), so that it can be properly translated at display time.Othersimplesamlphp_auth.module843N/A

All

Test nameMinorCriticalNormal
Expandsimplesamlphp_auth.install004
MessageGroupFilenameLineFunctionStatus
The $message argument to drupal_set_message() should be enclosed within t() so that it is translatable.Othersimplesamlphp_auth.install36N/A
The $message argument to drupal_set_message() should be enclosed within t() so that it is translatable.Othersimplesamlphp_auth.install39N/A
The $message argument to drupal_set_message() should be enclosed within t() so that it is translatable.Othersimplesamlphp_auth.install77N/A
The $message argument to drupal_set_message() should be enclosed within t() so that it is translatable.Othersimplesamlphp_auth.install80N/A
Expandsimplesamlphp_auth.module0320
MessageGroupFilenameLineFunctionStatus
Menu item titles and descriptions should NOT be enclosed within t(). (Drupal Docs)Othersimplesamlphp_auth.module26N/A
Menu item titles and descriptions should NOT be enclosed within t(). (Drupal Docs)Othersimplesamlphp_auth.module27N/A
Menu item titles and descriptions should NOT be enclosed within t(). (Drupal Docs)Othersimplesamlphp_auth.module34N/A
Menu item titles and descriptions should NOT be enclosed within t(). (Drupal Docs)Othersimplesamlphp_auth.module35N/A
do not use mixed case (camelCase), use lower case and _Othersimplesamlphp_auth.module93N/A
do not use mixed case (camelCase), use lower case and _Othersimplesamlphp_auth.module96N/A
The $message argument to watchdog() should NOT be enclosed within t(), so that it can be properly translated at display time.Othersimplesamlphp_auth.module97N/A
The $message argument to watchdog() should NOT be enclosed within t(), so that it can be properly translated at display time.Othersimplesamlphp_auth.module103N/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.Othersimplesamlphp_auth.module245N/A
do not use mixed case (camelCase), use lower case and _Othersimplesamlphp_auth.module297N/A
do not use mixed case (camelCase), use lower case and _Othersimplesamlphp_auth.module298N/A
do not use mixed case (camelCase), use lower case and _Othersimplesamlphp_auth.module301N/A
do not use mixed case (camelCase), use lower case and _Othersimplesamlphp_auth.module306N/A
do not use mixed case (camelCase), use lower case and _Othersimplesamlphp_auth.module309N/A
do not use mixed case (camelCase), use lower case and _Othersimplesamlphp_auth.module311N/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)Othersimplesamlphp_auth.module351N/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)Othersimplesamlphp_auth.module364N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Othersimplesamlphp_auth.module516N/A
do not use mixed case (camelCase), use lower case and _Othersimplesamlphp_auth.module578N/A
do not use mixed case (camelCase), use lower case and _Othersimplesamlphp_auth.module579N/A
do not use mixed case (camelCase), use lower case and _Othersimplesamlphp_auth.module580N/A
do not use mixed case (camelCase), use lower case and _Othersimplesamlphp_auth.module604N/A
The $message argument to watchdog() should NOT be enclosed within t(), so that it can be properly translated at display time.Othersimplesamlphp_auth.module843N/A