Commerce Registry - 7.x-1.x

Test status
StatusResult
Last receivedThu, 05/31/2012 - 20:56:47
Last requestedThu, 05/31/2012 - 21:13:34
Last testedThu, 05/31/2012 - 21:13:46
Test count8
Environment status
MySQLpass
Code reviewfail
Overall Resultpass
Branch information
Identifier7.x-1.x
ProjectCommerce Registry
LinkProject information
SummaryPASSED: [[Coder]]: [Code review] 15 minor(s), 4 critical(s), and 6 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
15 minor(s), 4 critical(s), and 6 normal(s)

Non-pass

Test nameMinorCriticalNormal
Expandcommerce_registry.module311
MessageGroupFilenameLineFunctionStatus
Menu item titles and descriptions should NOT be enclosed within t(). (Drupal Docs)Othercommerce_registry.module26N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthercommerce_registry.module484N/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)Othercommerce_registry.module551N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthercommerce_registry.module566N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthercommerce_registry.module571N/A
Expandincludes/commerce_registry.entity.inc501
MessageGroupFilenameLineFunctionStatus
in most cases, replace the string function with the drupal_ equivalent string functionsOthercommerce_registry.entity.inc215N/A
Use an indent of 2 spaces, with no tabsOthercommerce_registry.entity.inc468N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthercommerce_registry.entity.inc637N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthercommerce_registry.entity.inc641N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthercommerce_registry.entity.inc660N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthercommerce_registry.entity.inc662N/A
Expandincludes/commerce_registry.forms.inc732
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)Othercommerce_registry.forms.inc52N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthercommerce_registry.forms.inc149N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthercommerce_registry.forms.inc153N/A
Control statements should have one space between the control keyword and opening parenthesisOthercommerce_registry.forms.inc168N/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)Othercommerce_registry.forms.inc180N/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)Othercommerce_registry.forms.inc224N/A
Use an indent of 2 spaces, with no tabsOthercommerce_registry.forms.inc497N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthercommerce_registry.forms.inc501N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthercommerce_registry.forms.inc517N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthercommerce_registry.forms.inc519N/A
Use "elseif" in place of "else if"Othercommerce_registry.forms.inc533N/A
There should be no trailing spacesOthercommerce_registry.forms.inc597N/A
Expandincludes/commerce_registry.theme.inc002
MessageGroupFilenameLineFunctionStatus
The $string argument to t() should not begin or end with a space. (Drupal Docs)Othercommerce_registry.theme.inc146N/A
The $string argument to t() should not begin or end with a space. (Drupal Docs)Othercommerce_registry.theme.inc161N/A

All

Test nameMinorCriticalNormal
Leafcommerce_registry.install000
Expandcommerce_registry.module311
MessageGroupFilenameLineFunctionStatus
Menu item titles and descriptions should NOT be enclosed within t(). (Drupal Docs)Othercommerce_registry.module26N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthercommerce_registry.module484N/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)Othercommerce_registry.module551N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthercommerce_registry.module566N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthercommerce_registry.module571N/A
Leafcommerce_registry.rules.inc000
Leafincludes/commerce_registry.admin.inc000
Expandincludes/commerce_registry.entity.inc501
MessageGroupFilenameLineFunctionStatus
in most cases, replace the string function with the drupal_ equivalent string functionsOthercommerce_registry.entity.inc215N/A
Use an indent of 2 spaces, with no tabsOthercommerce_registry.entity.inc468N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthercommerce_registry.entity.inc637N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthercommerce_registry.entity.inc641N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthercommerce_registry.entity.inc660N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthercommerce_registry.entity.inc662N/A
Expandincludes/commerce_registry.forms.inc732
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)Othercommerce_registry.forms.inc52N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthercommerce_registry.forms.inc149N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthercommerce_registry.forms.inc153N/A
Control statements should have one space between the control keyword and opening parenthesisOthercommerce_registry.forms.inc168N/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)Othercommerce_registry.forms.inc180N/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)Othercommerce_registry.forms.inc224N/A
Use an indent of 2 spaces, with no tabsOthercommerce_registry.forms.inc497N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthercommerce_registry.forms.inc501N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthercommerce_registry.forms.inc517N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthercommerce_registry.forms.inc519N/A
Use "elseif" in place of "else if"Othercommerce_registry.forms.inc533N/A
There should be no trailing spacesOthercommerce_registry.forms.inc597N/A
Leafincludes/commerce_registry.pages.inc000
Expandincludes/commerce_registry.theme.inc002
MessageGroupFilenameLineFunctionStatus
The $string argument to t() should not begin or end with a space. (Drupal Docs)Othercommerce_registry.theme.inc146N/A
The $string argument to t() should not begin or end with a space. (Drupal Docs)Othercommerce_registry.theme.inc161N/A