Facebook OAuth - 7.x-1.x

Test status
StatusResult
Last receivedMon, 03/26/2012 - 21:53:37
Last requestedMon, 03/26/2012 - 21:54:06
Last testedMon, 03/26/2012 - 21:54:13
Test count2
Environment status
MySQLpass
Code reviewfail
Overall Resultpass
Branch information
Identifier7.x-1.x
ProjectFacebook OAuth (FBOAuth)
LinkProject information
SummaryPASSED: [[SimpleTest]]: [MySQL] 0 pass(es); [[Coder]]: [Code review] 10 minor(s), 2 critical(s), and 22 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
10 minor(s), 2 critical(s), and 22 normal(s)

Non-pass

Test nameMinorCriticalNormal
Expandfboauth.module205
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherfboauth.module-1N/A
Menu item titles and descriptions should NOT be enclosed within t(). (Drupal Docs)Otherfboauth.module16N/A
Menu item titles and descriptions should NOT be enclosed within t(). (Drupal Docs)Otherfboauth.module25N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherfboauth.module354N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherfboauth.module355N/A
Use the matching Drupal theme functions, not raw HTML.Otherfboauth.module404N/A
Use the matching Drupal theme functions, not raw HTML.Otherfboauth.module414N/A
Expandincludes/fboauth.fboauth.inc0011
MessageGroupFilenameLineFunctionStatus
Doxygen uses @todo and @bug to markup things to be done.Otherfboauth.fboauth.inc20N/A
Use the matching Drupal theme functions, not raw HTML.Otherfboauth.fboauth.inc29N/A
Use the matching Drupal theme functions, not raw HTML.Otherfboauth.fboauth.inc34N/A
Core uses "e-mail" in end-user text and "mail" elsewhere (database, function names, etc.)Otherfboauth.fboauth.inc83N/A
Use the matching Drupal theme functions, not raw HTML.Otherfboauth.fboauth.inc111N/A
The $message argument to drupal_set_message() should be enclosed within t() so that it is translatable.Otherfboauth.fboauth.inc117N/A
Core uses "e-mail" in end-user text and "mail" elsewhere (database, function names, etc.)Otherfboauth.fboauth.inc169N/A
Core uses "e-mail" in end-user text and "mail" elsewhere (database, function names, etc.)Otherfboauth.fboauth.inc170N/A
Doxygen uses @todo and @bug to markup things to be done.Otherfboauth.fboauth.inc173N/A
Control statements should have one space between the control keyword and opening parenthesisOtherfboauth.fboauth.inc199N/A
use a space between the closing parenthesis and the open bracketOtherfboauth.fboauth.inc199N/A
Expandincludes/fboauth.field.inc221
MessageGroupFilenameLineFunctionStatus
Use the matching Drupal theme functions, not raw HTML.Otherfboauth.field.inc15N/A
Potential problem: FAPI elements '#title' and '#description' only accept filtered text, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized.Otherfboauth.field.inc39N/A
Potential problem: FAPI elements '#title' and '#description' only accept filtered text, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized.Otherfboauth.field.inc47N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherfboauth.field.inc167N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherfboauth.field.inc170N/A
Expandincludes/fboauth.pages.inc205
MessageGroupFilenameLineFunctionStatus
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherfboauth.pages.inc16N/A
Use the matching Drupal theme functions, not raw HTML.Otherfboauth.pages.inc19N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherfboauth.pages.inc24N/A
Use the matching Drupal theme functions, not raw HTML.Otherfboauth.pages.inc27N/A
Use the matching Drupal theme functions, not raw HTML.Otherfboauth.pages.inc69N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherfboauth.pages.inc121N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherfboauth.pages.inc124N/A

All

Test nameMinorCriticalNormal
Expandfboauth.api.php200
MessageGroupFilenameLineFunctionStatus
@see references should be separated by "," followed by a single space and with no trailing punctuationOtherfboauth.api.php28N/A
@see references should be separated by "," followed by a single space and with no trailing punctuationOtherfboauth.api.php63N/A
Leaffboauth.install000
Expandfboauth.module205
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherfboauth.module-1N/A
Menu item titles and descriptions should NOT be enclosed within t(). (Drupal Docs)Otherfboauth.module16N/A
Menu item titles and descriptions should NOT be enclosed within t(). (Drupal Docs)Otherfboauth.module25N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherfboauth.module354N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherfboauth.module355N/A
Use the matching Drupal theme functions, not raw HTML.Otherfboauth.module404N/A
Use the matching Drupal theme functions, not raw HTML.Otherfboauth.module414N/A
Expandincludes/fboauth.fboauth.inc0011
MessageGroupFilenameLineFunctionStatus
Doxygen uses @todo and @bug to markup things to be done.Otherfboauth.fboauth.inc20N/A
Use the matching Drupal theme functions, not raw HTML.Otherfboauth.fboauth.inc29N/A
Use the matching Drupal theme functions, not raw HTML.Otherfboauth.fboauth.inc34N/A
Core uses "e-mail" in end-user text and "mail" elsewhere (database, function names, etc.)Otherfboauth.fboauth.inc83N/A
Use the matching Drupal theme functions, not raw HTML.Otherfboauth.fboauth.inc111N/A
The $message argument to drupal_set_message() should be enclosed within t() so that it is translatable.Otherfboauth.fboauth.inc117N/A
Core uses "e-mail" in end-user text and "mail" elsewhere (database, function names, etc.)Otherfboauth.fboauth.inc169N/A
Core uses "e-mail" in end-user text and "mail" elsewhere (database, function names, etc.)Otherfboauth.fboauth.inc170N/A
Doxygen uses @todo and @bug to markup things to be done.Otherfboauth.fboauth.inc173N/A
Control statements should have one space between the control keyword and opening parenthesisOtherfboauth.fboauth.inc199N/A
use a space between the closing parenthesis and the open bracketOtherfboauth.fboauth.inc199N/A
Expandincludes/fboauth.field.inc221
MessageGroupFilenameLineFunctionStatus
Use the matching Drupal theme functions, not raw HTML.Otherfboauth.field.inc15N/A
Potential problem: FAPI elements '#title' and '#description' only accept filtered text, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized.Otherfboauth.field.inc39N/A
Potential problem: FAPI elements '#title' and '#description' only accept filtered text, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized.Otherfboauth.field.inc47N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherfboauth.field.inc167N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherfboauth.field.inc170N/A
Expandincludes/fboauth.pages.inc205
MessageGroupFilenameLineFunctionStatus
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherfboauth.pages.inc16N/A
Use the matching Drupal theme functions, not raw HTML.Otherfboauth.pages.inc19N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherfboauth.pages.inc24N/A
Use the matching Drupal theme functions, not raw HTML.Otherfboauth.pages.inc27N/A
Use the matching Drupal theme functions, not raw HTML.Otherfboauth.pages.inc69N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherfboauth.pages.inc121N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherfboauth.pages.inc124N/A
Expandincludes/fboauth.profile.inc200
MessageGroupFilenameLineFunctionStatus
in most cases, replace the string function with the drupal_ equivalent string functionsOtherfboauth.profile.inc108N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherfboauth.profile.inc111N/A