Taskbar - 7.x-1.x

Test status
StatusResult
Last receivedTue, 04/10/2012 - 15:22:38
Last requestedTue, 04/10/2012 - 15:23:03
Last testedTue, 04/10/2012 - 15:23:52
Test count4
Environment status
MySQLpass
Code reviewfail
Overall Resultpass
Branch information
Identifier7.x-1.x
ProjectTaskbar
LinkProject information
SummaryPASSED: [[Coder]]: [Code review] 1 minor(s), 5 critical(s), and 49 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
1 minor(s), 5 critical(s), and 49 normal(s)

Non-pass

Test nameMinorCriticalNormal
Expandincludes/taskbar.theme.inc003
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othertaskbar.theme.inc-1N/A
Separate comments from comment syntax by a space.Othertaskbar.theme.inc2N/A
Separate comments from comment syntax by a space.Othertaskbar.theme.inc4N/A
Expandmodules/taskbar_migrate/taskbar_migrate.module011
MessageGroupFilenameLineFunctionStatus
Doxygen uses @todo and @bug to markup things to be done.Othertaskbar_migrate.module85N/A
In SQL strings, Use db_query() placeholders in place of variables. This is a potential source of SQL injection attacks when the variable can come from user data. (Drupal Docs)Othertaskbar_migrate.module86N/A
Expandmodules/taskbar_tacore/taskbar_tacore.api.php004
MessageGroupFilenameLineFunctionStatus
Function documentation should be less than 80 characters per line.Othertaskbar_tacore.api.php15N/A
Function documentation should be less than 80 characters per line.Othertaskbar_tacore.api.php20N/A
Function documentation should be less than 80 characters per line.Othertaskbar_tacore.api.php21N/A
Remove the empty commented line in your function documentation.Othertaskbar_tacore.api.php25N/A
Expandmodules/taskbar_tacore/taskbar_tacore.module004
MessageGroupFilenameLineFunctionStatus
Separate comments from comment syntax by a space.Othertaskbar_tacore.module50N/A
Separate comments from comment syntax by a space.Othertaskbar_tacore.module52N/A
Separate comments from comment syntax by a space.Othertaskbar_tacore.module83N/A
Separate comments from comment syntax by a space.Othertaskbar_tacore.module85N/A
Expandmodules/taskbar_ui/taskbar_ui-list-form.tpl.php001
MessageGroupFilenameLineFunctionStatus
the final ?> should be omitted from all code filesOthertaskbar_ui-list-form.tpl.php68N/A
Expandmodules/taskbar_ui/taskbar_ui.module049
MessageGroupFilenameLineFunctionStatus
Administrative menu items should have a description.Othertaskbar_ui.module69N/A
Administrative menu items should have a description.Othertaskbar_ui.module76N/A
Administrative menu items should have a description.Othertaskbar_ui.module83N/A
Separate comments from comment syntax by a space.Othertaskbar_ui.module92N/A
Separate comments from comment syntax by a space.Othertaskbar_ui.module94N/A
Potential problem: confirm_form() only accepts filtered text, be sure all !placeholders for $variables in t() are fully sanitized using check_plain(), filter_xss() or similar. (Drupal Docs)Othertaskbar_ui.module465N/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)Othertaskbar_ui.module474N/A
Function documentation should be less than 80 characters per line.Othertaskbar_ui.module482N/A
Function documentation should be less than 80 characters per line.Othertaskbar_ui.module483N/A
Using eval() or drupal_eval() in your module's code could have a security risk if the PHP input provided to the function contains malicious code. (Drupal Docs)Othertaskbar_ui.module528N/A
table names should be enclosed in {curly_brackets}Othertaskbar_ui.module595N/A
Separate comments from comment syntax by a space.Othertaskbar_ui.module640N/A
Separate comments from comment syntax by a space.Othertaskbar_ui.module642N/A
Expandmodules/taskbar_views/taskbar_views.install003
MessageGroupFilenameLineFunctionStatus
Doxygen uses @todo and @bug to markup things to be done.Othertaskbar_views.install20N/A
Doxygen uses @todo and @bug to markup things to be done.Othertaskbar_views.install104N/A
Doxygen uses @todo and @bug to markup things to be done.Othertaskbar_views.install106N/A
Expandmodules/taskbar_views/taskbar_views.module003
MessageGroupFilenameLineFunctionStatus
Function summaries should be one line only. (Drupal Docs)Othertaskbar_views.module103N/A
If you define a @param or @return, you should document it as well.Othertaskbar_views.module104N/A
If you define a @param or @return, you should document it as well.Othertaskbar_views.module105N/A
Expandtaskbar.api.php108
MessageGroupFilenameLineFunctionStatus
Function documentation should be less than 80 characters per line.Othertaskbar.api.php8N/A
Function documentation should be less than 80 characters per line.Othertaskbar.api.php9N/A
There should be no trailing spacesOthertaskbar.api.php11N/A
Function documentation should be less than 80 characters per line.Othertaskbar.api.php17N/A
Function documentation should be less than 80 characters per line.Othertaskbar.api.php21N/A
Function documentation should be less than 80 characters per line.Othertaskbar.api.php53N/A
Function documentation should be less than 80 characters per line.Othertaskbar.api.php58N/A
Function documentation should be less than 80 characters per line.Othertaskbar.api.php59N/A
@param and @return descriptions begin indented on the next line. (Drupal Docs)Othertaskbar.api.php129N/A
Expandtaskbar.module0012
MessageGroupFilenameLineFunctionStatus
Separate comments from comment syntax by a space.Othertaskbar.module142N/A
Separate comments from comment syntax by a space.Othertaskbar.module144N/A
Function summaries should be one line only. (Drupal Docs)Othertaskbar.module173N/A
If you define a @param or @return, you should document it as well.Othertaskbar.module205N/A
If you define a @param or @return, you should document it as well.Othertaskbar.module220N/A
Function summaries should be one line only. (Drupal Docs)Othertaskbar.module287N/A
Function summaries should be one line only. (Drupal Docs)Othertaskbar.module577N/A
Function summaries should be one line only. (Drupal Docs)Othertaskbar.module652N/A
If you define a @param or @return, you should document it as well.Othertaskbar.module653N/A
@param and @return descriptions begin indented on the next line. (Drupal Docs)Othertaskbar.module653N/A
If you define a @param or @return, you should document it as well.Othertaskbar.module654N/A
@param and @return descriptions begin indented on the next line. (Drupal Docs)Othertaskbar.module654N/A
Expandtaskbar.taskbar.inc001
MessageGroupFilenameLineFunctionStatus
Use sentence case, not title case, for end-user strings. (Wikipedia)Othertaskbar.taskbar.inc66N/A

All

Test nameMinorCriticalNormal
Leafexamples/taskbar_activity_example/taskbar_activity_example.install000
Leafexamples/taskbar_activity_example/taskbar_activity_example.module000
Expandincludes/taskbar.theme.inc003
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othertaskbar.theme.inc-1N/A
Separate comments from comment syntax by a space.Othertaskbar.theme.inc2N/A
Separate comments from comment syntax by a space.Othertaskbar.theme.inc4N/A
Leafmodules/taskbar_activity/taskbar_activity.admin.inc000
Leafmodules/taskbar_activity/taskbar_activity.api.php000
Leafmodules/taskbar_activity/taskbar_activity.install000
Leafmodules/taskbar_activity/taskbar_activity.module000
Expandmodules/taskbar_migrate/taskbar_migrate.module011
MessageGroupFilenameLineFunctionStatus
Doxygen uses @todo and @bug to markup things to be done.Othertaskbar_migrate.module85N/A
In SQL strings, Use db_query() placeholders in place of variables. This is a potential source of SQL injection attacks when the variable can come from user data. (Drupal Docs)Othertaskbar_migrate.module86N/A
Expandmodules/taskbar_tacore/taskbar_tacore.api.php004
MessageGroupFilenameLineFunctionStatus
Function documentation should be less than 80 characters per line.Othertaskbar_tacore.api.php15N/A
Function documentation should be less than 80 characters per line.Othertaskbar_tacore.api.php20N/A
Function documentation should be less than 80 characters per line.Othertaskbar_tacore.api.php21N/A
Remove the empty commented line in your function documentation.Othertaskbar_tacore.api.php25N/A
Leafmodules/taskbar_tacore/taskbar_tacore.install000
Expandmodules/taskbar_tacore/taskbar_tacore.module004
MessageGroupFilenameLineFunctionStatus
Separate comments from comment syntax by a space.Othertaskbar_tacore.module50N/A
Separate comments from comment syntax by a space.Othertaskbar_tacore.module52N/A
Separate comments from comment syntax by a space.Othertaskbar_tacore.module83N/A
Separate comments from comment syntax by a space.Othertaskbar_tacore.module85N/A
Expandmodules/taskbar_ui/taskbar_ui-list-form.tpl.php001
MessageGroupFilenameLineFunctionStatus
the final ?> should be omitted from all code filesOthertaskbar_ui-list-form.tpl.php68N/A
Expandmodules/taskbar_ui/taskbar_ui.module049
MessageGroupFilenameLineFunctionStatus
Administrative menu items should have a description.Othertaskbar_ui.module69N/A
Administrative menu items should have a description.Othertaskbar_ui.module76N/A
Administrative menu items should have a description.Othertaskbar_ui.module83N/A
Separate comments from comment syntax by a space.Othertaskbar_ui.module92N/A
Separate comments from comment syntax by a space.Othertaskbar_ui.module94N/A
Potential problem: confirm_form() only accepts filtered text, be sure all !placeholders for $variables in t() are fully sanitized using check_plain(), filter_xss() or similar. (Drupal Docs)Othertaskbar_ui.module465N/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)Othertaskbar_ui.module474N/A
Function documentation should be less than 80 characters per line.Othertaskbar_ui.module482N/A
Function documentation should be less than 80 characters per line.Othertaskbar_ui.module483N/A
Using eval() or drupal_eval() in your module's code could have a security risk if the PHP input provided to the function contains malicious code. (Drupal Docs)Othertaskbar_ui.module528N/A
table names should be enclosed in {curly_brackets}Othertaskbar_ui.module595N/A
Separate comments from comment syntax by a space.Othertaskbar_ui.module640N/A
Separate comments from comment syntax by a space.Othertaskbar_ui.module642N/A
Leafmodules/taskbar_views/taskbar_views.admin.inc000
Expandmodules/taskbar_views/taskbar_views.install003
MessageGroupFilenameLineFunctionStatus
Doxygen uses @todo and @bug to markup things to be done.Othertaskbar_views.install20N/A
Doxygen uses @todo and @bug to markup things to be done.Othertaskbar_views.install104N/A
Doxygen uses @todo and @bug to markup things to be done.Othertaskbar_views.install106N/A
Expandmodules/taskbar_views/taskbar_views.module003
MessageGroupFilenameLineFunctionStatus
Function summaries should be one line only. (Drupal Docs)Othertaskbar_views.module103N/A
If you define a @param or @return, you should document it as well.Othertaskbar_views.module104N/A
If you define a @param or @return, you should document it as well.Othertaskbar_views.module105N/A
Leafproviders/taskbar_provider.block.inc000
Leafproviders/taskbar_provider.custom.inc000
Leafproviders/taskbar_provider.link.inc000
Expandtaskbar.api.php108
MessageGroupFilenameLineFunctionStatus
Function documentation should be less than 80 characters per line.Othertaskbar.api.php8N/A
Function documentation should be less than 80 characters per line.Othertaskbar.api.php9N/A
There should be no trailing spacesOthertaskbar.api.php11N/A
Function documentation should be less than 80 characters per line.Othertaskbar.api.php17N/A
Function documentation should be less than 80 characters per line.Othertaskbar.api.php21N/A
Function documentation should be less than 80 characters per line.Othertaskbar.api.php53N/A
Function documentation should be less than 80 characters per line.Othertaskbar.api.php58N/A
Function documentation should be less than 80 characters per line.Othertaskbar.api.php59N/A
@param and @return descriptions begin indented on the next line. (Drupal Docs)Othertaskbar.api.php129N/A
Leaftaskbar.install000
Expandtaskbar.module0012
MessageGroupFilenameLineFunctionStatus
Separate comments from comment syntax by a space.Othertaskbar.module142N/A
Separate comments from comment syntax by a space.Othertaskbar.module144N/A
Function summaries should be one line only. (Drupal Docs)Othertaskbar.module173N/A
If you define a @param or @return, you should document it as well.Othertaskbar.module205N/A
If you define a @param or @return, you should document it as well.Othertaskbar.module220N/A
Function summaries should be one line only. (Drupal Docs)Othertaskbar.module287N/A
Function summaries should be one line only. (Drupal Docs)Othertaskbar.module577N/A
Function summaries should be one line only. (Drupal Docs)Othertaskbar.module652N/A
If you define a @param or @return, you should document it as well.Othertaskbar.module653N/A
@param and @return descriptions begin indented on the next line. (Drupal Docs)Othertaskbar.module653N/A
If you define a @param or @return, you should document it as well.Othertaskbar.module654N/A
@param and @return descriptions begin indented on the next line. (Drupal Docs)Othertaskbar.module654N/A
Expandtaskbar.taskbar.inc001
MessageGroupFilenameLineFunctionStatus
Use sentence case, not title case, for end-user strings. (Wikipedia)Othertaskbar.taskbar.inc66N/A