Just Another Social Module (JASM) - 7.x-1.x

Test status
StatusResult
Last receivedMon, 03/19/2012 - 15:37:36
Last requestedMon, 03/19/2012 - 15:38:04
Last testedMon, 03/19/2012 - 15:38:28
Test count4
Environment status
MySQLpass
Code reviewfail
Overall Resultpass
Branch information
Identifier7.x-1.x
ProjectJust Another Social Module (JASM)
LinkProject information
SummaryPASSED: [[Coder]]: [Code review] 38 minor(s), 1 critical(s), and 59 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
38 minor(s), 1 critical(s), and 59 normal(s)

Non-pass

Test nameMinorCriticalNormal
Expandjasm.install001
MessageGroupFilenameLineFunctionStatus
Commits to the Git repository do not require the CVS $Id$ keyword in each file. (Drupal Docs)Otherjasm.install2N/A
Expandjasm.module38158
MessageGroupFilenameLineFunctionStatus
Commits to the Git repository do not require the CVS $Id$ keyword in each file. (Drupal Docs)Otherjasm.module2N/A
Function documentation should be less than 80 characters per line.Otherjasm.module6N/A
indent secondary line of comment one space Otherjasm.module10N/A
indent secondary line of comment one space Otherjasm.module11N/A
indent secondary line of comment one space Otherjasm.module12N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherjasm.module16N/A
indent secondary line of comment one space Otherjasm.module23N/A
Format should be * Implementation of hook_foo().Otherjasm.module23N/A
indent secondary line of comment one space Otherjasm.module24N/A
"Module" should rarely be capitalized as part of a module's proper name.Otherjasm.module27N/A
indent secondary line of comment one space Otherjasm.module38N/A
indent secondary line of comment one space Otherjasm.module39N/A
indent secondary line of comment one space Otherjasm.module40N/A
Separate comments from comment syntax by a space.Otherjasm.module77N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherjasm.module128N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherjasm.module130N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherjasm.module173N/A
There should be no trailing spacesOtherjasm.module330N/A
There should be no trailing spacesOtherjasm.module331N/A
Doxygen uses @todo and @bug to markup things to be done.Otherjasm.module343N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherjasm.module480N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherjasm.module517N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherjasm.module624N/A
Doxygen uses @todo and @bug to markup things to be done.Otherjasm.module671N/A
There should be no trailing spacesOtherjasm.module699N/A
There should be no trailing spacesOtherjasm.module715N/A
There should be no trailing spacesOtherjasm.module723N/A
do not use mixed case (camelCase), use lower case and _Otherjasm.module726N/A
do not use mixed case (camelCase), use lower case and _Otherjasm.module729N/A
There should be no trailing spacesOtherjasm.module752N/A
There should be no trailing spacesOtherjasm.module804N/A
There should be no trailing spacesOtherjasm.module891N/A
There should be no trailing spacesOtherjasm.module892N/A
Doxygen uses @todo and @bug to markup things to be done.Otherjasm.module904N/A
"Module" should rarely be capitalized as part of a module's proper name.Otherjasm.module1167N/A
"Module" should rarely be capitalized as part of a module's proper name.Otherjasm.module1171N/A
Use the matching Drupal theme functions, not raw HTML.Otherjasm.module1176N/A
Use the matching Drupal theme functions, not raw HTML.Otherjasm.module1176N/A
Doxygen uses @todo and @bug to markup things to be done.Otherjasm.module1180N/A
Doxygen uses @todo and @bug to markup things to be done.Otherjasm.module1182N/A
Doxygen uses @todo and @bug to markup things to be done.Otherjasm.module1228N/A
There should be no trailing spacesOtherjasm.module1234N/A
Function summaries should be one line only. (Drupal Docs)Otherjasm.module1248N/A
If you define a @param or @return, you should document it as well.Otherjasm.module1249N/A
Separate comments from comment syntax by a space.Otherjasm.module1302N/A
Function summaries should be one line only. (Drupal Docs)Otherjasm.module1316N/A
Separate comments from comment syntax by a space.Otherjasm.module1325N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherjasm.module1332N/A
Separate comments from comment syntax by a space.Otherjasm.module1334N/A
Separate comments from comment syntax by a space.Otherjasm.module1335N/A
Separate comments from comment syntax by a space.Otherjasm.module1336N/A
Function summaries should be one line only. (Drupal Docs)Otherjasm.module1403N/A
Function documentation should be less than 80 characters per line.Otherjasm.module1405N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtherjasm.module1435N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtherjasm.module1453N/A
Separate comments from comment syntax by a space.Otherjasm.module1456N/A
Doxygen uses @todo and @bug to markup things to be done.Otherjasm.module1475N/A
There should be no trailing spacesOtherjasm.module1480N/A
There should be no trailing spacesOtherjasm.module1487N/A
There should be no trailing spacesOtherjasm.module1494N/A
There should be no trailing spacesOtherjasm.module1501N/A
There should be no trailing spacesOtherjasm.module1508N/A
There should be no trailing spacesOtherjasm.module1515N/A
put a space between the asterisk and the comment textOtherjasm.module1535N/A
Doxygen uses @todo and @bug to markup things to be done.Otherjasm.module1538N/A
put a space between the asterisk and the comment textOtherjasm.module1553N/A
Doxygen uses @todo and @bug to markup things to be done.Otherjasm.module1556N/A
put a space between the asterisk and the comment textOtherjasm.module1571N/A
Doxygen uses @todo and @bug to markup things to be done.Otherjasm.module1574N/A
Use the matching Drupal theme functions, not raw HTML.Otherjasm.module1589N/A
Doxygen uses @todo and @bug to markup things to be done.Otherjasm.module1602N/A
Separate comments from comment syntax by a space.Otherjasm.module1608N/A
Separate comments from comment syntax by a space.Otherjasm.module1612N/A
Function documentation should be less than 80 characters per line.Otherjasm.module1620N/A
Separate comments from comment syntax by a space.Otherjasm.module1641N/A
Function summaries should be one line only. (Drupal Docs)Otherjasm.module1648N/A
put a space between the asterisk and the comment textOtherjasm.module1650N/A
Use the matching Drupal theme functions, not raw HTML.Otherjasm.module1657N/A
Doxygen uses @todo and @bug to markup things to be done.Otherjasm.module1662N/A
Use the Form API to build forms to help prevent against CSRF attacks. (Drupal Docs)Otherjasm.module1663N/A
Use the matching Drupal theme functions, not raw HTML.Otherjasm.module1666N/A
Separate comments from comment syntax by a space.Otherjasm.module1667N/A
There should be no trailing spacesOtherjasm.module1672N/A
Function summaries should be one line only. (Drupal Docs)Otherjasm.module1720N/A
put a space between the asterisk and the comment textOtherjasm.module1721N/A
Function documentation should be less than 80 characters per line.Otherjasm.module1741N/A
There should be no trailing spacesOtherjasm.module1750N/A
There should be no trailing spacesOtherjasm.module1753N/A
There should be no trailing spacesOtherjasm.module1760N/A
Function summaries should be one line only. (Drupal Docs)Otherjasm.module1766N/A
put a space between the asterisk and the comment textOtherjasm.module1767N/A
Doxygen uses @todo and @bug to markup things to be done.Otherjasm.module1887N/A
Function summaries should be one line only. (Drupal Docs)Otherjasm.module1939N/A
put a space between the asterisk and the comment textOtherjasm.module1941N/A
Function summaries should be one line only. (Drupal Docs)Otherjasm.module1953N/A
put a space between the asterisk and the comment textOtherjasm.module1955N/A
Doxygen uses @todo and @bug to markup things to be done.Otherjasm.module1993N/A

All

Test nameMinorCriticalNormal
Expandjasm.install001
MessageGroupFilenameLineFunctionStatus
Commits to the Git repository do not require the CVS $Id$ keyword in each file. (Drupal Docs)Otherjasm.install2N/A
Expandjasm.module38158
MessageGroupFilenameLineFunctionStatus
Commits to the Git repository do not require the CVS $Id$ keyword in each file. (Drupal Docs)Otherjasm.module2N/A
Function documentation should be less than 80 characters per line.Otherjasm.module6N/A
indent secondary line of comment one space Otherjasm.module10N/A
indent secondary line of comment one space Otherjasm.module11N/A
indent secondary line of comment one space Otherjasm.module12N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherjasm.module16N/A
indent secondary line of comment one space Otherjasm.module23N/A
Format should be * Implementation of hook_foo().Otherjasm.module23N/A
indent secondary line of comment one space Otherjasm.module24N/A
"Module" should rarely be capitalized as part of a module's proper name.Otherjasm.module27N/A
indent secondary line of comment one space Otherjasm.module38N/A
indent secondary line of comment one space Otherjasm.module39N/A
indent secondary line of comment one space Otherjasm.module40N/A
Separate comments from comment syntax by a space.Otherjasm.module77N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherjasm.module128N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherjasm.module130N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherjasm.module173N/A
There should be no trailing spacesOtherjasm.module330N/A
There should be no trailing spacesOtherjasm.module331N/A
Doxygen uses @todo and @bug to markup things to be done.Otherjasm.module343N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherjasm.module480N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherjasm.module517N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Otherjasm.module624N/A
Doxygen uses @todo and @bug to markup things to be done.Otherjasm.module671N/A
There should be no trailing spacesOtherjasm.module699N/A
There should be no trailing spacesOtherjasm.module715N/A
There should be no trailing spacesOtherjasm.module723N/A
do not use mixed case (camelCase), use lower case and _Otherjasm.module726N/A
do not use mixed case (camelCase), use lower case and _Otherjasm.module729N/A
There should be no trailing spacesOtherjasm.module752N/A
There should be no trailing spacesOtherjasm.module804N/A
There should be no trailing spacesOtherjasm.module891N/A
There should be no trailing spacesOtherjasm.module892N/A
Doxygen uses @todo and @bug to markup things to be done.Otherjasm.module904N/A
"Module" should rarely be capitalized as part of a module's proper name.Otherjasm.module1167N/A
"Module" should rarely be capitalized as part of a module's proper name.Otherjasm.module1171N/A
Use the matching Drupal theme functions, not raw HTML.Otherjasm.module1176N/A
Use the matching Drupal theme functions, not raw HTML.Otherjasm.module1176N/A
Doxygen uses @todo and @bug to markup things to be done.Otherjasm.module1180N/A
Doxygen uses @todo and @bug to markup things to be done.Otherjasm.module1182N/A
Doxygen uses @todo and @bug to markup things to be done.Otherjasm.module1228N/A
There should be no trailing spacesOtherjasm.module1234N/A
Function summaries should be one line only. (Drupal Docs)Otherjasm.module1248N/A
If you define a @param or @return, you should document it as well.Otherjasm.module1249N/A
Separate comments from comment syntax by a space.Otherjasm.module1302N/A
Function summaries should be one line only. (Drupal Docs)Otherjasm.module1316N/A
Separate comments from comment syntax by a space.Otherjasm.module1325N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherjasm.module1332N/A
Separate comments from comment syntax by a space.Otherjasm.module1334N/A
Separate comments from comment syntax by a space.Otherjasm.module1335N/A
Separate comments from comment syntax by a space.Otherjasm.module1336N/A
Function summaries should be one line only. (Drupal Docs)Otherjasm.module1403N/A
Function documentation should be less than 80 characters per line.Otherjasm.module1405N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtherjasm.module1435N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote termsOtherjasm.module1453N/A
Separate comments from comment syntax by a space.Otherjasm.module1456N/A
Doxygen uses @todo and @bug to markup things to be done.Otherjasm.module1475N/A
There should be no trailing spacesOtherjasm.module1480N/A
There should be no trailing spacesOtherjasm.module1487N/A
There should be no trailing spacesOtherjasm.module1494N/A
There should be no trailing spacesOtherjasm.module1501N/A
There should be no trailing spacesOtherjasm.module1508N/A
There should be no trailing spacesOtherjasm.module1515N/A
put a space between the asterisk and the comment textOtherjasm.module1535N/A
Doxygen uses @todo and @bug to markup things to be done.Otherjasm.module1538N/A
put a space between the asterisk and the comment textOtherjasm.module1553N/A
Doxygen uses @todo and @bug to markup things to be done.Otherjasm.module1556N/A
put a space between the asterisk and the comment textOtherjasm.module1571N/A
Doxygen uses @todo and @bug to markup things to be done.Otherjasm.module1574N/A
Use the matching Drupal theme functions, not raw HTML.Otherjasm.module1589N/A
Doxygen uses @todo and @bug to markup things to be done.Otherjasm.module1602N/A
Separate comments from comment syntax by a space.Otherjasm.module1608N/A
Separate comments from comment syntax by a space.Otherjasm.module1612N/A
Function documentation should be less than 80 characters per line.Otherjasm.module1620N/A
Separate comments from comment syntax by a space.Otherjasm.module1641N/A
Function summaries should be one line only. (Drupal Docs)Otherjasm.module1648N/A
put a space between the asterisk and the comment textOtherjasm.module1650N/A
Use the matching Drupal theme functions, not raw HTML.Otherjasm.module1657N/A
Doxygen uses @todo and @bug to markup things to be done.Otherjasm.module1662N/A
Use the Form API to build forms to help prevent against CSRF attacks. (Drupal Docs)Otherjasm.module1663N/A
Use the matching Drupal theme functions, not raw HTML.Otherjasm.module1666N/A
Separate comments from comment syntax by a space.Otherjasm.module1667N/A
There should be no trailing spacesOtherjasm.module1672N/A
Function summaries should be one line only. (Drupal Docs)Otherjasm.module1720N/A
put a space between the asterisk and the comment textOtherjasm.module1721N/A
Function documentation should be less than 80 characters per line.Otherjasm.module1741N/A
There should be no trailing spacesOtherjasm.module1750N/A
There should be no trailing spacesOtherjasm.module1753N/A
There should be no trailing spacesOtherjasm.module1760N/A
Function summaries should be one line only. (Drupal Docs)Otherjasm.module1766N/A
put a space between the asterisk and the comment textOtherjasm.module1767N/A
Doxygen uses @todo and @bug to markup things to be done.Otherjasm.module1887N/A
Function summaries should be one line only. (Drupal Docs)Otherjasm.module1939N/A
put a space between the asterisk and the comment textOtherjasm.module1941N/A
Function summaries should be one line only. (Drupal Docs)Otherjasm.module1953N/A
put a space between the asterisk and the comment textOtherjasm.module1955N/A
Doxygen uses @todo and @bug to markup things to be done.Otherjasm.module1993N/A