SocialFlow - 7.x-1.x

Test status
StatusResult
Last receivedSun, 01/01/2012 - 01:20:13
Last requestedSun, 01/01/2012 - 01:21:21
Last testedSun, 01/01/2012 - 01:21:29
Test count2
Environment status
MySQLpass
Code reviewfail
Overall Resultpass
Branch information
Identifier7.x-1.x
ProjectSocialFlow
LinkProject information
SummaryPASSED: [[SimpleTest]]: [MySQL] 0 pass(es); [[Coder]]: [Code review] 1 minor(s), 2 critical(s), and 56 normal(s).
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), 2 critical(s), and 56 normal(s)

Non-pass

Test nameMinorCriticalNormal
Expandsocialflow.forms.inc0021
MessageGroupFilenameLineFunctionStatus
Use the matching Drupal theme functions, not raw HTML.Othersocialflow.forms.inc50N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othersocialflow.forms.inc109N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othersocialflow.forms.inc164N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othersocialflow.forms.inc171N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othersocialflow.forms.inc179N/A
Use Drupal's format_date(), not PHP's default date().Othersocialflow.forms.inc266N/A
Doxygen uses @todo and @bug to markup things to be done.Othersocialflow.forms.inc267N/A
Use the matching Drupal theme functions, not raw HTML.Othersocialflow.forms.inc447N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othersocialflow.forms.inc452N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othersocialflow.forms.inc462N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othersocialflow.forms.inc497N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othersocialflow.forms.inc503N/A
Use Drupal's format_date(), not PHP's default date().Othersocialflow.forms.inc513N/A
Use Drupal's format_date(), not PHP's default date().Othersocialflow.forms.inc560N/A
Use Drupal's format_date(), not PHP's default date().Othersocialflow.forms.inc565N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othersocialflow.forms.inc703N/A
Use the matching Drupal theme functions, not raw HTML.Othersocialflow.forms.inc724N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othersocialflow.forms.inc729N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othersocialflow.forms.inc773N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othersocialflow.forms.inc791N/A
Use Drupal's format_date(), not PHP's default date().Othersocialflow.forms.inc802N/A
Expandsocialflow.inc001
MessageGroupFilenameLineFunctionStatus
Function summaries should be one line only. (Drupal Docs)Othersocialflow.inc10N/A
Expandsocialflow.module1234
MessageGroupFilenameLineFunctionStatus
Administrative menu items should have a description.Othersocialflow.module23N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othersocialflow.module44N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othersocialflow.module55N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othersocialflow.module65N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othersocialflow.module75N/A
Administrative menu items should have a description.Othersocialflow.module85N/A
Administrative menu items should have a description.Othersocialflow.module94N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othersocialflow.module104N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othersocialflow.module131N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Othersocialflow.module227N/A
Use the matching Drupal theme functions, not raw HTML.Othersocialflow.module233N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Othersocialflow.module234N/A
Use the matching Drupal theme functions, not raw HTML.Othersocialflow.module265N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Othersocialflow.module288N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Othersocialflow.module290N/A
missing space after commaOthersocialflow.module295N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Othersocialflow.module317N/A
Use the matching Drupal theme functions, not raw HTML.Othersocialflow.module332N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Othersocialflow.module386N/A
Use the matching Drupal theme functions, not raw HTML.Othersocialflow.module469N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othersocialflow.module474N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othersocialflow.module519N/A
Use Drupal's format_date(), not PHP's default date().Othersocialflow.module528N/A
Use Drupal's format_date(), not PHP's default date().Othersocialflow.module531N/A
Use Drupal's format_date(), not PHP's default date().Othersocialflow.module606N/A
Use Drupal's format_date(), not PHP's default date().Othersocialflow.module611N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othersocialflow.module687N/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)Othersocialflow.module693N/A
Use an indent of 2 spaces, with no tabsOthersocialflow.module780N/A
Use Drupal's format_date(), not PHP's default date().Othersocialflow.module790N/A
Use Drupal's format_date(), not PHP's default date().Othersocialflow.module872N/A
Use Drupal's format_date(), not PHP's default date().Othersocialflow.module877N/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)Othersocialflow.module1059N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Othersocialflow.module1061N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Othersocialflow.module1062N/A
Doxygen uses @todo and @bug to markup things to be done.Othersocialflow.module1079N/A
Use the matching Drupal theme functions, not raw HTML.Othersocialflow.module1085N/A

All

Test nameMinorCriticalNormal
Expandsocialflow.forms.inc0021
MessageGroupFilenameLineFunctionStatus
Use the matching Drupal theme functions, not raw HTML.Othersocialflow.forms.inc50N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othersocialflow.forms.inc109N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othersocialflow.forms.inc164N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othersocialflow.forms.inc171N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othersocialflow.forms.inc179N/A
Use Drupal's format_date(), not PHP's default date().Othersocialflow.forms.inc266N/A
Doxygen uses @todo and @bug to markup things to be done.Othersocialflow.forms.inc267N/A
Use the matching Drupal theme functions, not raw HTML.Othersocialflow.forms.inc447N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othersocialflow.forms.inc452N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othersocialflow.forms.inc462N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othersocialflow.forms.inc497N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othersocialflow.forms.inc503N/A
Use Drupal's format_date(), not PHP's default date().Othersocialflow.forms.inc513N/A
Use Drupal's format_date(), not PHP's default date().Othersocialflow.forms.inc560N/A
Use Drupal's format_date(), not PHP's default date().Othersocialflow.forms.inc565N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othersocialflow.forms.inc703N/A
Use the matching Drupal theme functions, not raw HTML.Othersocialflow.forms.inc724N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othersocialflow.forms.inc729N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othersocialflow.forms.inc773N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othersocialflow.forms.inc791N/A
Use Drupal's format_date(), not PHP's default date().Othersocialflow.forms.inc802N/A
Expandsocialflow.inc001
MessageGroupFilenameLineFunctionStatus
Function summaries should be one line only. (Drupal Docs)Othersocialflow.inc10N/A
Leafsocialflow.install000
Expandsocialflow.module1234
MessageGroupFilenameLineFunctionStatus
Administrative menu items should have a description.Othersocialflow.module23N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othersocialflow.module44N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othersocialflow.module55N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othersocialflow.module65N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othersocialflow.module75N/A
Administrative menu items should have a description.Othersocialflow.module85N/A
Administrative menu items should have a description.Othersocialflow.module94N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othersocialflow.module104N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othersocialflow.module131N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Othersocialflow.module227N/A
Use the matching Drupal theme functions, not raw HTML.Othersocialflow.module233N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Othersocialflow.module234N/A
Use the matching Drupal theme functions, not raw HTML.Othersocialflow.module265N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Othersocialflow.module288N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Othersocialflow.module290N/A
missing space after commaOthersocialflow.module295N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Othersocialflow.module317N/A
Use the matching Drupal theme functions, not raw HTML.Othersocialflow.module332N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Othersocialflow.module386N/A
Use the matching Drupal theme functions, not raw HTML.Othersocialflow.module469N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othersocialflow.module474N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othersocialflow.module519N/A
Use Drupal's format_date(), not PHP's default date().Othersocialflow.module528N/A
Use Drupal's format_date(), not PHP's default date().Othersocialflow.module531N/A
Use Drupal's format_date(), not PHP's default date().Othersocialflow.module606N/A
Use Drupal's format_date(), not PHP's default date().Othersocialflow.module611N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othersocialflow.module687N/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)Othersocialflow.module693N/A
Use an indent of 2 spaces, with no tabsOthersocialflow.module780N/A
Use Drupal's format_date(), not PHP's default date().Othersocialflow.module790N/A
Use Drupal's format_date(), not PHP's default date().Othersocialflow.module872N/A
Use Drupal's format_date(), not PHP's default date().Othersocialflow.module877N/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)Othersocialflow.module1059N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Othersocialflow.module1061N/A
The $text argument to l() should be enclosed within t() so that it is translatable.Othersocialflow.module1062N/A
Doxygen uses @todo and @bug to markup things to be done.Othersocialflow.module1079N/A
Use the matching Drupal theme functions, not raw HTML.Othersocialflow.module1085N/A