mojeID integration - 6.x-1.x

Test status
StatusResult
Last receivedWed, 01/11/2012 - 23:15:36
Last requestedWed, 01/11/2012 - 23:16:03
Last testedWed, 01/11/2012 - 23:16:40
Test count2
Environment status
MySQLpass
Code reviewfail
Overall Resultpass
Branch information
Identifier6.x-1.x
ProjectmojeID integration
LinkProject information
SummaryPASSED: [[Coder]]: [Code review] 14 minor(s), 3 critical(s), and 106 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
14 minor(s), 3 critical(s), and 106 normal(s)

Non-pass

Test nameMinorCriticalNormal
Expandmojeid.admin.inc002
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othermojeid.admin.inc-1N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othermojeid.admin.inc11N/A
Expandmojeid.inc6259
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othermojeid.inc-1N/A
Separate comments from comment syntax by a space.Othermojeid.inc4N/A
Separate comments from comment syntax by a space.Othermojeid.inc6N/A
use lowercase html tags to comply with XHTMLOthermojeid.inc18N/A
use lowercase html tags to comply with XHTMLOthermojeid.inc19N/A
use lowercase html tags to comply with XHTMLOthermojeid.inc20N/A
use lowercase html tags to comply with XHTMLOthermojeid.inc21N/A
Separate comments from comment syntax by a space.Othermojeid.inc29N/A
Separate comments from comment syntax by a space.Othermojeid.inc31N/A
Potential problem: when FAPI element '#type' is set to 'markup' (default), '#value' only accepts filtered text, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized.Othermojeid.inc158N/A
The $message argument to drupal_set_message() should be enclosed within t() so that it is translatable.Othermojeid.inc193N/A
Potential problem: drupal_set_message() only accepts filtered text, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized. (Drupal Docs)Othermojeid.inc193N/A
Separate comments from comment syntax by a space.Othermojeid.inc224N/A
Separate comments from comment syntax by a space.Othermojeid.inc226N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthermojeid.inc230N/A
If you define a @param or @return, you should document it as well.Othermojeid.inc236N/A
Function summaries should be one line only. (Drupal Docs)Othermojeid.inc244N/A
If you define a @param or @return, you should document it as well.Othermojeid.inc250N/A
Function documentation should be less than 80 characters per line.Othermojeid.inc255N/A
If you define a @param or @return, you should document it as well.Othermojeid.inc285N/A
If you define a @param or @return, you should document it as well.Othermojeid.inc287N/A
Separate comments from comment syntax by a space.Othermojeid.inc295N/A
Separate comments from comment syntax by a space.Othermojeid.inc297N/A
Use an indent of 2 spaces, with no tabsOthermojeid.inc378N/A
Separate comments from comment syntax by a space.Othermojeid.inc383N/A
Separate comments from comment syntax by a space.Othermojeid.inc395N/A
Separate comments from comment syntax by a space.Othermojeid.inc404N/A
Separate comments from comment syntax by a space.Othermojeid.inc413N/A
Separate comments from comment syntax by a space.Othermojeid.inc430N/A
Separate comments from comment syntax by a space.Othermojeid.inc439N/A
Separate comments from comment syntax by a space.Othermojeid.inc448N/A
Separate comments from comment syntax by a space.Othermojeid.inc460N/A
Separate comments from comment syntax by a space.Othermojeid.inc469N/A
Separate comments from comment syntax by a space.Othermojeid.inc478N/A
Separate comments from comment syntax by a space.Othermojeid.inc487N/A
Separate comments from comment syntax by a space.Othermojeid.inc496N/A
Separate comments from comment syntax by a space.Othermojeid.inc505N/A
Separate comments from comment syntax by a space.Othermojeid.inc514N/A
Separate comments from comment syntax by a space.Othermojeid.inc526N/A
Separate comments from comment syntax by a space.Othermojeid.inc535N/A
Separate comments from comment syntax by a space.Othermojeid.inc544N/A
Separate comments from comment syntax by a space.Othermojeid.inc553N/A
Separate comments from comment syntax by a space.Othermojeid.inc562N/A
Separate comments from comment syntax by a space.Othermojeid.inc571N/A
Separate comments from comment syntax by a space.Othermojeid.inc580N/A
Separate comments from comment syntax by a space.Othermojeid.inc592N/A
Separate comments from comment syntax by a space.Othermojeid.inc601N/A
Separate comments from comment syntax by a space.Othermojeid.inc613N/A
Separate comments from comment syntax by a space.Othermojeid.inc622N/A
Separate comments from comment syntax by a space.Othermojeid.inc631N/A
Separate comments from comment syntax by a space.Othermojeid.inc640N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othermojeid.inc643N/A
Separate comments from comment syntax by a space.Othermojeid.inc649N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othermojeid.inc652N/A
Separate comments from comment syntax by a space.Othermojeid.inc657N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othermojeid.inc660N/A
Separate comments from comment syntax by a space.Othermojeid.inc669N/A
Separate comments from comment syntax by a space.Othermojeid.inc678N/A
Separate comments from comment syntax by a space.Othermojeid.inc690N/A
Separate comments from comment syntax by a space.Othermojeid.inc699N/A
Separate comments from comment syntax by a space.Othermojeid.inc708N/A
Separate comments from comment syntax by a space.Othermojeid.inc717N/A
Separate comments from comment syntax by a space.Othermojeid.inc726N/A
Separate comments from comment syntax by a space.Othermojeid.inc738N/A
Separate comments from comment syntax by a space.Othermojeid.inc747N/A
Separate comments from comment syntax by a space.Othermojeid.inc756N/A
Separate comments from comment syntax by a space.Othermojeid.inc768N/A
Expandmojeid.install101
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othermojeid.install-1N/A
There should be no trailing spacesOthermojeid.install23N/A
Expandmojeid.module207
MessageGroupFilenameLineFunctionStatus
Separate comments from comment syntax by a space.Othermojeid.module17N/A
Separate comments from comment syntax by a space.Othermojeid.module19N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othermojeid.module51N/A
Use the matching Drupal theme functions, not raw HTML.Othermojeid.module67N/A
Use the matching Drupal theme functions, not raw HTML.Othermojeid.module72N/A
Use the matching Drupal theme functions, not raw HTML.Othermojeid.module75N/A
Missing periodOthermojeid.module96N/A
Separate comments from comment syntax by a space.Othermojeid.module109N/A
put a space between the asterisk and the comment textOthermojeid.module153N/A
Expandmojeid_uc_addresses/mojeid_uc_addresses.install001
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othermojeid_uc_addresses.install-1N/A
Expandmojeid_uc_addresses/mojeid_uc_addresses.module5136
MessageGroupFilenameLineFunctionStatus
Separate comments from comment syntax by a space.Othermojeid_uc_addresses.module20N/A
Separate comments from comment syntax by a space.Othermojeid_uc_addresses.module22N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othermojeid_uc_addresses.module31N/A
Doxygen uses @todo and @bug to markup things to be done.Othermojeid_uc_addresses.module54N/A
Missing periodOthermojeid_uc_addresses.module75N/A
If you define a @param or @return, you should document it as well.Othermojeid_uc_addresses.module95N/A
@param and @return descriptions begin indented on the next line. (Drupal Docs)Othermojeid_uc_addresses.module95N/A
If you define a @param or @return, you should document it as well.Othermojeid_uc_addresses.module96N/A
@param and @return descriptions begin indented on the next line. (Drupal Docs)Othermojeid_uc_addresses.module96N/A
If you define a @param or @return, you should document it as well.Othermojeid_uc_addresses.module98N/A
put a space between the asterisk and the comment textOthermojeid_uc_addresses.module102N/A
Doxygen uses @todo and @bug to markup things to be done.Othermojeid_uc_addresses.module114N/A
If you define a @param or @return, you should document it as well.Othermojeid_uc_addresses.module212N/A
If you define a @param or @return, you should document it as well.Othermojeid_uc_addresses.module215N/A
table names should be enclosed in {curly_brackets}Othermojeid_uc_addresses.module225N/A
If you define a @param or @return, you should document it as well.Othermojeid_uc_addresses.module234N/A
@param and @return descriptions begin indented on the next line. (Drupal Docs)Othermojeid_uc_addresses.module234N/A
If you define a @param or @return, you should document it as well.Othermojeid_uc_addresses.module236N/A
If you define a @param or @return, you should document it as well.Othermojeid_uc_addresses.module245N/A
@param and @return descriptions begin indented on the next line. (Drupal Docs)Othermojeid_uc_addresses.module245N/A
If you define a @param or @return, you should document it as well.Othermojeid_uc_addresses.module247N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthermojeid_uc_addresses.module280N/A
Doxygen uses @todo and @bug to markup things to be done.Othermojeid_uc_addresses.module287N/A
If you define a @param or @return, you should document it as well.Othermojeid_uc_addresses.module295N/A
If you define a @param or @return, you should document it as well.Othermojeid_uc_addresses.module296N/A
If you define a @param or @return, you should document it as well.Othermojeid_uc_addresses.module328N/A
If you define a @param or @return, you should document it as well.Othermojeid_uc_addresses.module345N/A
If you define a @param or @return, you should document it as well.Othermojeid_uc_addresses.module346N/A
If you define a @param or @return, you should document it as well.Othermojeid_uc_addresses.module363N/A
If you define a @param or @return, you should document it as well.Othermojeid_uc_addresses.module379N/A
If you define a @param or @return, you should document it as well.Othermojeid_uc_addresses.module381N/A
Function documentation should be less than 80 characters per line.Othermojeid_uc_addresses.module392N/A
If you define a @param or @return, you should document it as well.Othermojeid_uc_addresses.module397N/A
If you define a @param or @return, you should document it as well.Othermojeid_uc_addresses.module407N/A
If you define a @param or @return, you should document it as well.Othermojeid_uc_addresses.module408N/A
Use an indent of 2 spaces, with no tabsOthermojeid_uc_addresses.module450N/A
If you define a @param or @return, you should document it as well.Othermojeid_uc_addresses.module465N/A
If you define a @param or @return, you should document it as well.Othermojeid_uc_addresses.module529N/A
If you define a @param or @return, you should document it as well.Othermojeid_uc_addresses.module530N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthermojeid_uc_addresses.module533N/A
Function summaries should be one line only. (Drupal Docs)Othermojeid_uc_addresses.module540N/A
If you define a @param or @return, you should document it as well.Othermojeid_uc_addresses.module544N/A

All

Test nameMinorCriticalNormal
Expandmojeid.admin.inc002
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othermojeid.admin.inc-1N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othermojeid.admin.inc11N/A
Expandmojeid.inc6259
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othermojeid.inc-1N/A
Separate comments from comment syntax by a space.Othermojeid.inc4N/A
Separate comments from comment syntax by a space.Othermojeid.inc6N/A
use lowercase html tags to comply with XHTMLOthermojeid.inc18N/A
use lowercase html tags to comply with XHTMLOthermojeid.inc19N/A
use lowercase html tags to comply with XHTMLOthermojeid.inc20N/A
use lowercase html tags to comply with XHTMLOthermojeid.inc21N/A
Separate comments from comment syntax by a space.Othermojeid.inc29N/A
Separate comments from comment syntax by a space.Othermojeid.inc31N/A
Potential problem: when FAPI element '#type' is set to 'markup' (default), '#value' only accepts filtered text, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized.Othermojeid.inc158N/A
The $message argument to drupal_set_message() should be enclosed within t() so that it is translatable.Othermojeid.inc193N/A
Potential problem: drupal_set_message() only accepts filtered text, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized. (Drupal Docs)Othermojeid.inc193N/A
Separate comments from comment syntax by a space.Othermojeid.inc224N/A
Separate comments from comment syntax by a space.Othermojeid.inc226N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthermojeid.inc230N/A
If you define a @param or @return, you should document it as well.Othermojeid.inc236N/A
Function summaries should be one line only. (Drupal Docs)Othermojeid.inc244N/A
If you define a @param or @return, you should document it as well.Othermojeid.inc250N/A
Function documentation should be less than 80 characters per line.Othermojeid.inc255N/A
If you define a @param or @return, you should document it as well.Othermojeid.inc285N/A
If you define a @param or @return, you should document it as well.Othermojeid.inc287N/A
Separate comments from comment syntax by a space.Othermojeid.inc295N/A
Separate comments from comment syntax by a space.Othermojeid.inc297N/A
Use an indent of 2 spaces, with no tabsOthermojeid.inc378N/A
Separate comments from comment syntax by a space.Othermojeid.inc383N/A
Separate comments from comment syntax by a space.Othermojeid.inc395N/A
Separate comments from comment syntax by a space.Othermojeid.inc404N/A
Separate comments from comment syntax by a space.Othermojeid.inc413N/A
Separate comments from comment syntax by a space.Othermojeid.inc430N/A
Separate comments from comment syntax by a space.Othermojeid.inc439N/A
Separate comments from comment syntax by a space.Othermojeid.inc448N/A
Separate comments from comment syntax by a space.Othermojeid.inc460N/A
Separate comments from comment syntax by a space.Othermojeid.inc469N/A
Separate comments from comment syntax by a space.Othermojeid.inc478N/A
Separate comments from comment syntax by a space.Othermojeid.inc487N/A
Separate comments from comment syntax by a space.Othermojeid.inc496N/A
Separate comments from comment syntax by a space.Othermojeid.inc505N/A
Separate comments from comment syntax by a space.Othermojeid.inc514N/A
Separate comments from comment syntax by a space.Othermojeid.inc526N/A
Separate comments from comment syntax by a space.Othermojeid.inc535N/A
Separate comments from comment syntax by a space.Othermojeid.inc544N/A
Separate comments from comment syntax by a space.Othermojeid.inc553N/A
Separate comments from comment syntax by a space.Othermojeid.inc562N/A
Separate comments from comment syntax by a space.Othermojeid.inc571N/A
Separate comments from comment syntax by a space.Othermojeid.inc580N/A
Separate comments from comment syntax by a space.Othermojeid.inc592N/A
Separate comments from comment syntax by a space.Othermojeid.inc601N/A
Separate comments from comment syntax by a space.Othermojeid.inc613N/A
Separate comments from comment syntax by a space.Othermojeid.inc622N/A
Separate comments from comment syntax by a space.Othermojeid.inc631N/A
Separate comments from comment syntax by a space.Othermojeid.inc640N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othermojeid.inc643N/A
Separate comments from comment syntax by a space.Othermojeid.inc649N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othermojeid.inc652N/A
Separate comments from comment syntax by a space.Othermojeid.inc657N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othermojeid.inc660N/A
Separate comments from comment syntax by a space.Othermojeid.inc669N/A
Separate comments from comment syntax by a space.Othermojeid.inc678N/A
Separate comments from comment syntax by a space.Othermojeid.inc690N/A
Separate comments from comment syntax by a space.Othermojeid.inc699N/A
Separate comments from comment syntax by a space.Othermojeid.inc708N/A
Separate comments from comment syntax by a space.Othermojeid.inc717N/A
Separate comments from comment syntax by a space.Othermojeid.inc726N/A
Separate comments from comment syntax by a space.Othermojeid.inc738N/A
Separate comments from comment syntax by a space.Othermojeid.inc747N/A
Separate comments from comment syntax by a space.Othermojeid.inc756N/A
Separate comments from comment syntax by a space.Othermojeid.inc768N/A
Expandmojeid.install101
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othermojeid.install-1N/A
There should be no trailing spacesOthermojeid.install23N/A
Expandmojeid.module207
MessageGroupFilenameLineFunctionStatus
Separate comments from comment syntax by a space.Othermojeid.module17N/A
Separate comments from comment syntax by a space.Othermojeid.module19N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othermojeid.module51N/A
Use the matching Drupal theme functions, not raw HTML.Othermojeid.module67N/A
Use the matching Drupal theme functions, not raw HTML.Othermojeid.module72N/A
Use the matching Drupal theme functions, not raw HTML.Othermojeid.module75N/A
Missing periodOthermojeid.module96N/A
Separate comments from comment syntax by a space.Othermojeid.module109N/A
put a space between the asterisk and the comment textOthermojeid.module153N/A
Expandmojeid_uc_addresses/mojeid_uc_addresses.install001
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Othermojeid_uc_addresses.install-1N/A
Expandmojeid_uc_addresses/mojeid_uc_addresses.module5136
MessageGroupFilenameLineFunctionStatus
Separate comments from comment syntax by a space.Othermojeid_uc_addresses.module20N/A
Separate comments from comment syntax by a space.Othermojeid_uc_addresses.module22N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)Othermojeid_uc_addresses.module31N/A
Doxygen uses @todo and @bug to markup things to be done.Othermojeid_uc_addresses.module54N/A
Missing periodOthermojeid_uc_addresses.module75N/A
If you define a @param or @return, you should document it as well.Othermojeid_uc_addresses.module95N/A
@param and @return descriptions begin indented on the next line. (Drupal Docs)Othermojeid_uc_addresses.module95N/A
If you define a @param or @return, you should document it as well.Othermojeid_uc_addresses.module96N/A
@param and @return descriptions begin indented on the next line. (Drupal Docs)Othermojeid_uc_addresses.module96N/A
If you define a @param or @return, you should document it as well.Othermojeid_uc_addresses.module98N/A
put a space between the asterisk and the comment textOthermojeid_uc_addresses.module102N/A
Doxygen uses @todo and @bug to markup things to be done.Othermojeid_uc_addresses.module114N/A
If you define a @param or @return, you should document it as well.Othermojeid_uc_addresses.module212N/A
If you define a @param or @return, you should document it as well.Othermojeid_uc_addresses.module215N/A
table names should be enclosed in {curly_brackets}Othermojeid_uc_addresses.module225N/A
If you define a @param or @return, you should document it as well.Othermojeid_uc_addresses.module234N/A
@param and @return descriptions begin indented on the next line. (Drupal Docs)Othermojeid_uc_addresses.module234N/A
If you define a @param or @return, you should document it as well.Othermojeid_uc_addresses.module236N/A
If you define a @param or @return, you should document it as well.Othermojeid_uc_addresses.module245N/A
@param and @return descriptions begin indented on the next line. (Drupal Docs)Othermojeid_uc_addresses.module245N/A
If you define a @param or @return, you should document it as well.Othermojeid_uc_addresses.module247N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthermojeid_uc_addresses.module280N/A
Doxygen uses @todo and @bug to markup things to be done.Othermojeid_uc_addresses.module287N/A
If you define a @param or @return, you should document it as well.Othermojeid_uc_addresses.module295N/A
If you define a @param or @return, you should document it as well.Othermojeid_uc_addresses.module296N/A
If you define a @param or @return, you should document it as well.Othermojeid_uc_addresses.module328N/A
If you define a @param or @return, you should document it as well.Othermojeid_uc_addresses.module345N/A
If you define a @param or @return, you should document it as well.Othermojeid_uc_addresses.module346N/A
If you define a @param or @return, you should document it as well.Othermojeid_uc_addresses.module363N/A
If you define a @param or @return, you should document it as well.Othermojeid_uc_addresses.module379N/A
If you define a @param or @return, you should document it as well.Othermojeid_uc_addresses.module381N/A
Function documentation should be less than 80 characters per line.Othermojeid_uc_addresses.module392N/A
If you define a @param or @return, you should document it as well.Othermojeid_uc_addresses.module397N/A
If you define a @param or @return, you should document it as well.Othermojeid_uc_addresses.module407N/A
If you define a @param or @return, you should document it as well.Othermojeid_uc_addresses.module408N/A
Use an indent of 2 spaces, with no tabsOthermojeid_uc_addresses.module450N/A
If you define a @param or @return, you should document it as well.Othermojeid_uc_addresses.module465N/A
If you define a @param or @return, you should document it as well.Othermojeid_uc_addresses.module529N/A
If you define a @param or @return, you should document it as well.Othermojeid_uc_addresses.module530N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthermojeid_uc_addresses.module533N/A
Function summaries should be one line only. (Drupal Docs)Othermojeid_uc_addresses.module540N/A
If you define a @param or @return, you should document it as well.Othermojeid_uc_addresses.module544N/A