Setup - 7.x-1.0-beta1

Test status
StatusResult
Last receivedMon, 09/17/2012 - 01:30:16
Last requestedMon, 09/17/2012 - 01:31:13
Last testedMon, 09/17/2012 - 01:31:38
Test count2
Environment status
MySQLpass
Code reviewfail
Overall Resultpass
Branch information
Identifier7.x-1.0-beta1
ProjectSetup
LinkProject information
SummaryPASSED: [[Coder]]: [Code review] 3 minor(s), 0 critical(s), and 2 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
3 minor(s), 0 critical(s), and 2 normal(s)

Non-pass

Test nameMinorCriticalNormal
Expandtemplates/setup.tpl.php001
MessageGroupFilenameLineFunctionStatus
The control statement should be on a separate line from the control conditionalOthersetup.tpl.php19N/A
Expandtemplates/system.setup_theme.tpl.php001
MessageGroupFilenameLineFunctionStatus
the final ?> should be omitted from all code filesOthersystem.setup_theme.tpl.php10N/A

All

Test nameMinorCriticalNormal
Leafincludes/admin_menu.inc000
Leafincludes/node.inc000
Leafincludes/setup.inc000
Expandincludes/system.inc100
MessageGroupFilenameLineFunctionStatus
Use an indent of 2 spaces, with no tabsOthersystem.inc111N/A
Leafincludes/variable.inc000
Leafmodules/setup_example/setup_example.install000
Leafmodules/setup_example/setup_example.module000
Leafsetup.api.php000
Expandsetup.module200
MessageGroupFilenameLineFunctionStatus
Format should be * Implementation of hook_foo().Othersetup.module441N/A
Format should be * Implementation of hook_foo().Othersetup.module452N/A
Expandtemplates/setup.tpl.php001
MessageGroupFilenameLineFunctionStatus
The control statement should be on a separate line from the control conditionalOthersetup.tpl.php19N/A
Expandtemplates/system.setup_theme.tpl.php001
MessageGroupFilenameLineFunctionStatus
the final ?> should be omitted from all code filesOthersystem.setup_theme.tpl.php10N/A