P6 Base - 7.x-1.x

Test status
StatusResult
Last receivedThu, 09/20/2012 - 00:40:05
Last requestedThu, 09/20/2012 - 00:41:03
Last testedThu, 09/20/2012 - 00:41:35
Test count10
Environment status
MySQLpass
Code reviewfail
Overall Resultpass
Branch information
Identifier7.x-1.x
ProjectP6 Base
LinkProject information
SummaryPASSED: [[Coder]]: [Code review] 6 minor(s), 0 critical(s), and 9 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
6 minor(s), 0 critical(s), and 9 normal(s)

Non-pass

Test nameMinorCriticalNormal
Expandcore/p6_base.drush.inc101
MessageGroupFilenameLineFunctionStatus
in most cases, replace the string function with the drupal_ equivalent string functionsOtherp6_base.drush.inc49N/A
Control statements should have one space between the control keyword and opening parenthesisOtherp6_base.drush.inc112N/A
Expandcore/template.php203
MessageGroupFilenameLineFunctionStatus
Use an indent of 2 spaces, with no tabsOthertemplate.php93N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Othertemplate.php406N/A
@see should always be followed by a filename, a URL, class/interface name (optionally including method), or a function name including ().Othertemplate.php409N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Othertemplate.php423N/A
use a space between the closing parenthesis and the open bracketOthertemplate.php509N/A
Expandcore/templates/block/block--no-wrapper.tpl.php001
MessageGroupFilenameLineFunctionStatus
the final ?> should be omitted from all code filesOtherblock--no-wrapper.tpl.php51N/A
Expandcore/templates/search/search-results.tpl.php001
MessageGroupFilenameLineFunctionStatus
the final ?> should be omitted from all code filesOthersearch-results.tpl.php32N/A
Expandcore/templates/system/region.tpl.php001
MessageGroupFilenameLineFunctionStatus
the final ?> should be omitted from all code filesOtherregion.tpl.php32N/A
Expandcore/templates/user/user-picture.tpl.php001
MessageGroupFilenameLineFunctionStatus
the final ?> should be omitted from all code filesOtheruser-picture.tpl.php16N/A
Expandcore/theme-settings.php301
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othertheme-settings.php-1N/A
@see should always be followed by a filename, a URL, class/interface name (optionally including method), or a function name including ().Othertheme-settings.php10N/A
@see references should be separated by "," followed by a single space and with no trailing punctuationOthertheme-settings.php10N/A
@see should be separated from the references by one space onlyOthertheme-settings.php10N/A

All

Test nameMinorCriticalNormal
Expandcore/p6_base.drush.inc101
MessageGroupFilenameLineFunctionStatus
in most cases, replace the string function with the drupal_ equivalent string functionsOtherp6_base.drush.inc49N/A
Control statements should have one space between the control keyword and opening parenthesisOtherp6_base.drush.inc112N/A
Expandcore/template.php203
MessageGroupFilenameLineFunctionStatus
Use an indent of 2 spaces, with no tabsOthertemplate.php93N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Othertemplate.php406N/A
@see should always be followed by a filename, a URL, class/interface name (optionally including method), or a function name including ().Othertemplate.php409N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Othertemplate.php423N/A
use a space between the closing parenthesis and the open bracketOthertemplate.php509N/A
Expandcore/templates/block/block--no-wrapper.tpl.php001
MessageGroupFilenameLineFunctionStatus
the final ?> should be omitted from all code filesOtherblock--no-wrapper.tpl.php51N/A
Leafcore/templates/block/block.tpl.php000
Leafcore/templates/comment/comment-wrapper.tpl.php000
Leafcore/templates/comment/comment.tpl.php000
Leafcore/templates/node/node.tpl.php000
Leafcore/templates/search/search-block-form.tpl.php000
Leafcore/templates/search/search-result.tpl.php000
Expandcore/templates/search/search-results.tpl.php001
MessageGroupFilenameLineFunctionStatus
the final ?> should be omitted from all code filesOthersearch-results.tpl.php32N/A
Leafcore/templates/system/html.tpl.php000
Leafcore/templates/system/page.tpl.php000
Expandcore/templates/system/region.tpl.php001
MessageGroupFilenameLineFunctionStatus
the final ?> should be omitted from all code filesOtherregion.tpl.php32N/A
Expandcore/templates/user/user-picture.tpl.php001
MessageGroupFilenameLineFunctionStatus
the final ?> should be omitted from all code filesOtheruser-picture.tpl.php16N/A
Leafcore/templates/views/views-more.tpl.php000
Expandcore/theme-settings.php301
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othertheme-settings.php-1N/A
@see should always be followed by a filename, a URL, class/interface name (optionally including method), or a function name including ().Othertheme-settings.php10N/A
@see references should be separated by "," followed by a single space and with no trailing punctuationOthertheme-settings.php10N/A
@see should be separated from the references by one space onlyOthertheme-settings.php10N/A
LeafP6_SUBTHEME/template.php000