WhatCounts API - 6.x-1.x

Test status
StatusResult
Last receivedFri, 09/07/2012 - 15:51:45
Last requestedFri, 09/07/2012 - 15:52:03
Last testedFri, 09/07/2012 - 15:52:19
Test count2
Environment status
MySQLpass
Code reviewfail
Overall Resultpass
Branch information
Identifier6.x-1.x
ProjectWhatCounts API
LinkProject information
SummaryPASSED: [[Coder]]: [Code review] 32 minor(s), 4 critical(s), and 3 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
32 minor(s), 4 critical(s), and 3 normal(s)

Non-pass

Test nameMinorCriticalNormal
Expandwhatcounts.install001
MessageGroupFilenameLineFunctionStatus
Commits to the Git repository do not require the CVS $Id$ keyword in each file. (Drupal Docs)Otherwhatcounts.install2N/A
Expandwhatcounts.module3242
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherwhatcounts.module-1N/A
Commits to the Git repository do not require the CVS $Id$ keyword in each file. (Drupal Docs)Otherwhatcounts.module2N/A
put a space between the asterisk and the comment textOtherwhatcounts.module10N/A
put a space between the asterisk and the comment textOtherwhatcounts.module26N/A
put a space between the asterisk and the comment textOtherwhatcounts.module35N/A
put a space between the asterisk and the comment textOtherwhatcounts.module64N/A
There should be no trailing spacesOtherwhatcounts.module76N/A
put a space between the asterisk and the comment textOtherwhatcounts.module122N/A
indent secondary line of comment one space Otherwhatcounts.module163N/A
There should be no trailing spacesOtherwhatcounts.module521N/A
indent secondary line of comment one space Otherwhatcounts.module524N/A
put a space between the asterisk and the comment textOtherwhatcounts.module533N/A
put a space between the asterisk and the comment textOtherwhatcounts.module559N/A
put a space between the asterisk and the comment textOtherwhatcounts.module574N/A
There should be no trailing spacesOtherwhatcounts.module581N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherwhatcounts.module695N/A
Use an indent of 2 spaces, with no tabsOtherwhatcounts.module780N/A
Use an indent of 2 spaces, with no tabsOtherwhatcounts.module781N/A
There should be no trailing spacesOtherwhatcounts.module805N/A
put a space between the asterisk and the comment textOtherwhatcounts.module811N/A
There should be no trailing spacesOtherwhatcounts.module818N/A
There should be no trailing spacesOtherwhatcounts.module895N/A
There should be no trailing spacesOtherwhatcounts.module904N/A
There should be no trailing spacesOtherwhatcounts.module910N/A
indent secondary line of comment one space Otherwhatcounts.module916N/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.Otherwhatcounts.module951N/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.Otherwhatcounts.module960N/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.Otherwhatcounts.module969N/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.Otherwhatcounts.module978N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherwhatcounts.module1028N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherwhatcounts.module1047N/A
There should be no trailing spacesOtherwhatcounts.module1050N/A
There should be no trailing spacesOtherwhatcounts.module1138N/A
There should be no trailing spacesOtherwhatcounts.module1141N/A
There should be no trailing spacesOtherwhatcounts.module1142N/A
There should be no trailing spacesOtherwhatcounts.module1144N/A
There should be no trailing spacesOtherwhatcounts.module1146N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherwhatcounts.module1151N/A

All

Test nameMinorCriticalNormal
Expandwhatcounts.install001
MessageGroupFilenameLineFunctionStatus
Commits to the Git repository do not require the CVS $Id$ keyword in each file. (Drupal Docs)Otherwhatcounts.install2N/A
Expandwhatcounts.module3242
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherwhatcounts.module-1N/A
Commits to the Git repository do not require the CVS $Id$ keyword in each file. (Drupal Docs)Otherwhatcounts.module2N/A
put a space between the asterisk and the comment textOtherwhatcounts.module10N/A
put a space between the asterisk and the comment textOtherwhatcounts.module26N/A
put a space between the asterisk and the comment textOtherwhatcounts.module35N/A
put a space between the asterisk and the comment textOtherwhatcounts.module64N/A
There should be no trailing spacesOtherwhatcounts.module76N/A
put a space between the asterisk and the comment textOtherwhatcounts.module122N/A
indent secondary line of comment one space Otherwhatcounts.module163N/A
There should be no trailing spacesOtherwhatcounts.module521N/A
indent secondary line of comment one space Otherwhatcounts.module524N/A
put a space between the asterisk and the comment textOtherwhatcounts.module533N/A
put a space between the asterisk and the comment textOtherwhatcounts.module559N/A
put a space between the asterisk and the comment textOtherwhatcounts.module574N/A
There should be no trailing spacesOtherwhatcounts.module581N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherwhatcounts.module695N/A
Use an indent of 2 spaces, with no tabsOtherwhatcounts.module780N/A
Use an indent of 2 spaces, with no tabsOtherwhatcounts.module781N/A
There should be no trailing spacesOtherwhatcounts.module805N/A
put a space between the asterisk and the comment textOtherwhatcounts.module811N/A
There should be no trailing spacesOtherwhatcounts.module818N/A
There should be no trailing spacesOtherwhatcounts.module895N/A
There should be no trailing spacesOtherwhatcounts.module904N/A
There should be no trailing spacesOtherwhatcounts.module910N/A
indent secondary line of comment one space Otherwhatcounts.module916N/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.Otherwhatcounts.module951N/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.Otherwhatcounts.module960N/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.Otherwhatcounts.module969N/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.Otherwhatcounts.module978N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherwhatcounts.module1028N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherwhatcounts.module1047N/A
There should be no trailing spacesOtherwhatcounts.module1050N/A
There should be no trailing spacesOtherwhatcounts.module1138N/A
There should be no trailing spacesOtherwhatcounts.module1141N/A
There should be no trailing spacesOtherwhatcounts.module1142N/A
There should be no trailing spacesOtherwhatcounts.module1144N/A
There should be no trailing spacesOtherwhatcounts.module1146N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherwhatcounts.module1151N/A