CCK Signup - 7.x-1.x

Test status
StatusResult
Last receivedWed, 03/07/2012 - 03:29:03
Last requestedWed, 03/07/2012 - 05:56:09
Last testedWed, 03/07/2012 - 05:56:19
Test count11
Environment status
MySQLfail
Code reviewfail
Overall Resultfail
Branch information
Identifier7.x-1.x
ProjectCCK Signup
LinkProject information
SummaryFAILED: [[SimpleTest]]: [MySQL] Setup environment: failed to drop checkout database; [[Coder]]: [Code review] 24 minor(s), 2 critical(s), and 23 normal(s).
Details

MySQL

  1. Setup environment
    The testbot client is probably malfunctioning.
  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
Review log
[20:29:01] Log initialized (/var/lib/drupaltestbot/sites/default/files/review.test_132009.pifr_simpletest.1331090941.log)
[20:29:01] Review timestamp: (Tue, 03/06/2012 - 20:29)
[20:29:01] Review host: (drupaltestbot664-mysql)
[20:29:01] Environment variables and review information initialized.
[20:29:01] core_url = [git://git.drupal.org/project/drupal.git];  Test repository = [git://git.drupal.org/project/cck_signup.git]
[20:29:01] Project directory is [cck_signup]
[20:29:01] Review started.
[20:29:01] Invoking operation [setup]...
[20:29:01] Command [chmod -R u+w sites/default/files/checkout 2>&1] succeeded.
[20:29:01] Command [rm -rf sites/default/files/checkout 2>&1] succeeded.
[20:29:01] Command [rm -rf sites/default/files/review 2>&1] succeeded.
[20:29:01] Database backend [mysql] loaded.
[20:29:02] Encountered error on [setup], details:
array (
  '@reason' => 'failed to drop checkout database',
)
[20:29:02] Review complete. test_id=132009 result code=1 details=Array
(
    [@reason] => failed to drop checkout database
)

[20:29:02] Static variables reset.

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

Non-pass

Test nameMinorCriticalNormal
Expandcck_signup.admin.inc401
MessageGroupFilenameLineFunctionStatus
Use an indent of 2 spaces, with no tabsOthercck_signup.admin.inc38N/A
Use an indent of 2 spaces, with no tabsOthercck_signup.admin.inc39N/A
Format should be * Implementation of hook_foo().Othercck_signup.admin.inc100N/A
'Implementation' should be at the start of the sentence and begin with a capitialized letterOthercck_signup.admin.inc100N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othercck_signup.admin.inc109N/A
Expandcck_signup.module211
MessageGroupFilenameLineFunctionStatus
Use an indent of 2 spaces, with no tabsOthercck_signup.module33N/A
Use an indent of 2 spaces, with no tabsOthercck_signup.module34N/A
Potential problem: form_set_error() and form_error() only accept filtered text, be sure all !placeholders for $variables in t() are fully sanitized using check_plain(), filter_xss() or similar. (Drupal Docs)Othercck_signup.module140N/A
Use the matching Drupal theme functions, not raw HTML.Othercck_signup.module140N/A
Expandcck_signup.test103
MessageGroupFilenameLineFunctionStatus
Use sentence case, not title case, for end-user strings. (Wikipedia)Othercck_signup.test178N/A
There should be no trailing spacesOthercck_signup.test196N/A
Doxygen uses @todo and @bug to markup things to be done.Othercck_signup.test384N/A
Doxygen uses @todo and @bug to markup things to be done.Othercck_signup.test386N/A
Expandcck_signup_basic/cck_signup_basic.module001
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othercck_signup_basic.module-1N/A
Expandcck_signup_basic/cck_signup_basic.views_default.inc002
MessageGroupFilenameLineFunctionStatus
Use sentence case, not title case, for end-user strings. (Wikipedia)Othercck_signup_basic.views_default.inc18N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othercck_signup_basic.views_default.inc25N/A
Expandcck_signup_group/cck_signup_group.admin.inc002
MessageGroupFilenameLineFunctionStatus
Use sentence case, not title case, for end-user strings. (Wikipedia)Othercck_signup_group.admin.inc20N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othercck_signup_group.admin.inc50N/A
Expandcck_signup_group/cck_signup_group.module101
MessageGroupFilenameLineFunctionStatus
Use an indent of 2 spaces, with no tabsOthercck_signup_group.module47N/A
Function summaries should be one line only. (Drupal Docs)Othercck_signup_group.module110N/A
Expandcck_signup_group_confirmation/cck_signup_group_confirmation.module813
MessageGroupFilenameLineFunctionStatus
Use an indent of 2 spaces, with no tabsOthercck_signup_group_confirmation.module24N/A
Use an indent of 2 spaces, with no tabsOthercck_signup_group_confirmation.module25N/A
Doxygen uses @todo and @bug to markup things to be done.Othercck_signup_group_confirmation.module65N/A
Doxygen uses @todo and @bug to markup things to be done.Othercck_signup_group_confirmation.module67N/A
Use an indent of 2 spaces, with no tabsOthercck_signup_group_confirmation.module73N/A
Use an indent of 2 spaces, with no tabsOthercck_signup_group_confirmation.module74N/A
Use an indent of 2 spaces, with no tabsOthercck_signup_group_confirmation.module75N/A
Use an indent of 2 spaces, with no tabsOthercck_signup_group_confirmation.module93N/A
Use an indent of 2 spaces, with no tabsOthercck_signup_group_confirmation.module94N/A
Use an indent of 2 spaces, with no tabsOthercck_signup_group_confirmation.module257N/A
Doxygen uses @todo and @bug to markup things to be done.Othercck_signup_group_confirmation.module309N/A
Potential problem: "SELECT FROM {node}" statements should probably be wrapped in db_rewrite_sql() and with the alias for {node} table defined (e.g. {node} n)Othercck_signup_group_confirmation.module310N/A
Expandnotifications_cck_signup/notifications_cck_signup.admin.inc001
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othernotifications_cck_signup.admin.inc-1N/A
Expandnotifications_cck_signup/notifications_cck_signup.module408
MessageGroupFilenameLineFunctionStatus
Use an indent of 2 spaces, with no tabsOthernotifications_cck_signup.module21N/A
Use an indent of 2 spaces, with no tabsOthernotifications_cck_signup.module22N/A
Doxygen uses @todo and @bug to markup things to be done.Othernotifications_cck_signup.module74N/A
Doxygen uses @todo and @bug to markup things to be done.Othernotifications_cck_signup.module75N/A
Doxygen uses @todo and @bug to markup things to be done.Othernotifications_cck_signup.module176N/A
Use an indent of 2 spaces, with no tabsOthernotifications_cck_signup.module329N/A
Function summaries should be one line only. (Drupal Docs)Othernotifications_cck_signup.module338N/A
Doxygen uses @todo and @bug to markup things to be done.Othernotifications_cck_signup.module369N/A
Doxygen uses @todo and @bug to markup things to be done.Othernotifications_cck_signup.module370N/A
Use an indent of 2 spaces, with no tabsOthernotifications_cck_signup.module377N/A
Doxygen uses @todo and @bug to markup things to be done.Othernotifications_cck_signup.module402N/A
Doxygen uses @todo and @bug to markup things to be done.Othernotifications_cck_signup.module423N/A

All

Test nameMinorCriticalNormal
Expandcck_signup.admin.inc401
MessageGroupFilenameLineFunctionStatus
Use an indent of 2 spaces, with no tabsOthercck_signup.admin.inc38N/A
Use an indent of 2 spaces, with no tabsOthercck_signup.admin.inc39N/A
Format should be * Implementation of hook_foo().Othercck_signup.admin.inc100N/A
'Implementation' should be at the start of the sentence and begin with a capitialized letterOthercck_signup.admin.inc100N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othercck_signup.admin.inc109N/A
Expandcck_signup.module211
MessageGroupFilenameLineFunctionStatus
Use an indent of 2 spaces, with no tabsOthercck_signup.module33N/A
Use an indent of 2 spaces, with no tabsOthercck_signup.module34N/A
Potential problem: form_set_error() and form_error() only accept filtered text, be sure all !placeholders for $variables in t() are fully sanitized using check_plain(), filter_xss() or similar. (Drupal Docs)Othercck_signup.module140N/A
Use the matching Drupal theme functions, not raw HTML.Othercck_signup.module140N/A
Expandcck_signup.test103
MessageGroupFilenameLineFunctionStatus
Use sentence case, not title case, for end-user strings. (Wikipedia)Othercck_signup.test178N/A
There should be no trailing spacesOthercck_signup.test196N/A
Doxygen uses @todo and @bug to markup things to be done.Othercck_signup.test384N/A
Doxygen uses @todo and @bug to markup things to be done.Othercck_signup.test386N/A
Leafcck_signup_basic/cck_signup_basic.features.field.inc000
Leafcck_signup_basic/cck_signup_basic.features.inc000
Leafcck_signup_basic/cck_signup_basic.features.user_permission.inc000
Expandcck_signup_basic/cck_signup_basic.module001
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othercck_signup_basic.module-1N/A
Leafcck_signup_basic/cck_signup_basic.strongarm.inc000
Expandcck_signup_basic/cck_signup_basic.views_default.inc002
MessageGroupFilenameLineFunctionStatus
Use sentence case, not title case, for end-user strings. (Wikipedia)Othercck_signup_basic.views_default.inc18N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othercck_signup_basic.views_default.inc25N/A
Expandcck_signup_group/cck_signup_group.admin.inc002
MessageGroupFilenameLineFunctionStatus
Use sentence case, not title case, for end-user strings. (Wikipedia)Othercck_signup_group.admin.inc20N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othercck_signup_group.admin.inc50N/A
Expandcck_signup_group/cck_signup_group.module101
MessageGroupFilenameLineFunctionStatus
Use an indent of 2 spaces, with no tabsOthercck_signup_group.module47N/A
Function summaries should be one line only. (Drupal Docs)Othercck_signup_group.module110N/A
Expandcck_signup_group_confirmation/cck_signup_group_confirmation.admin.inc200
MessageGroupFilenameLineFunctionStatus
Format should be * Implementation of hook_foo().Othercck_signup_group_confirmation.admin.inc9N/A
'Implementation' should be at the start of the sentence and begin with a capitialized letterOthercck_signup_group_confirmation.admin.inc9N/A
Expandcck_signup_group_confirmation/cck_signup_group_confirmation.module813
MessageGroupFilenameLineFunctionStatus
Use an indent of 2 spaces, with no tabsOthercck_signup_group_confirmation.module24N/A
Use an indent of 2 spaces, with no tabsOthercck_signup_group_confirmation.module25N/A
Doxygen uses @todo and @bug to markup things to be done.Othercck_signup_group_confirmation.module65N/A
Doxygen uses @todo and @bug to markup things to be done.Othercck_signup_group_confirmation.module67N/A
Use an indent of 2 spaces, with no tabsOthercck_signup_group_confirmation.module73N/A
Use an indent of 2 spaces, with no tabsOthercck_signup_group_confirmation.module74N/A
Use an indent of 2 spaces, with no tabsOthercck_signup_group_confirmation.module75N/A
Use an indent of 2 spaces, with no tabsOthercck_signup_group_confirmation.module93N/A
Use an indent of 2 spaces, with no tabsOthercck_signup_group_confirmation.module94N/A
Use an indent of 2 spaces, with no tabsOthercck_signup_group_confirmation.module257N/A
Doxygen uses @todo and @bug to markup things to be done.Othercck_signup_group_confirmation.module309N/A
Potential problem: "SELECT FROM {node}" statements should probably be wrapped in db_rewrite_sql() and with the alias for {node} table defined (e.g. {node} n)Othercck_signup_group_confirmation.module310N/A
Expandcck_signup_restrictions/cck_signup_restrictions.admin.inc200
MessageGroupFilenameLineFunctionStatus
Format should be * Implementation of hook_foo().Othercck_signup_restrictions.admin.inc9N/A
'Implementation' should be at the start of the sentence and begin with a capitialized letterOthercck_signup_restrictions.admin.inc9N/A
Leafcck_signup_restrictions/cck_signup_restrictions.module000
Expandnotifications_cck_signup/notifications_cck_signup.admin.inc001
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othernotifications_cck_signup.admin.inc-1N/A
Expandnotifications_cck_signup/notifications_cck_signup.module408
MessageGroupFilenameLineFunctionStatus
Use an indent of 2 spaces, with no tabsOthernotifications_cck_signup.module21N/A
Use an indent of 2 spaces, with no tabsOthernotifications_cck_signup.module22N/A
Doxygen uses @todo and @bug to markup things to be done.Othernotifications_cck_signup.module74N/A
Doxygen uses @todo and @bug to markup things to be done.Othernotifications_cck_signup.module75N/A
Doxygen uses @todo and @bug to markup things to be done.Othernotifications_cck_signup.module176N/A
Use an indent of 2 spaces, with no tabsOthernotifications_cck_signup.module329N/A
Function summaries should be one line only. (Drupal Docs)Othernotifications_cck_signup.module338N/A
Doxygen uses @todo and @bug to markup things to be done.Othernotifications_cck_signup.module369N/A
Doxygen uses @todo and @bug to markup things to be done.Othernotifications_cck_signup.module370N/A
Use an indent of 2 spaces, with no tabsOthernotifications_cck_signup.module377N/A
Doxygen uses @todo and @bug to markup things to be done.Othernotifications_cck_signup.module402N/A
Doxygen uses @todo and @bug to markup things to be done.Othernotifications_cck_signup.module423N/A