Views OAI-PMH - 6.x-2.x

Test status
StatusResult
Last receivedSat, 01/26/2013 - 01:24:10
Last requestedSat, 01/26/2013 - 01:25:01
Last testedSat, 01/26/2013 - 01:25:37
Test count2
Environment status
MySQLpass
Code reviewfail
Overall Resultpass
Branch information
Identifier6.x-2.x
ProjectViews OAI-PMH
LinkProject information
SummaryPASSED: [[Coder]]: [Code review] 156 minor(s), 1 critical(s), and 9 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
156 minor(s), 1 critical(s), and 9 normal(s)

Non-pass

Test nameMinorCriticalNormal
Expandplugins/views_oai_pmh_plugin_row_dc.inc103
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherviews_oai_pmh_plugin_row_dc.inc-1N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherviews_oai_pmh_plugin_row_dc.inc25N/A
Use an indent of 2 spaces, with no tabsOtherviews_oai_pmh_plugin_row_dc.inc44N/A
Use an indent of 2 spaces, with no tabsOtherviews_oai_pmh_plugin_row_dc.inc45N/A
Expandplugins/views_oai_pmh_plugin_row_etdms.inc103
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherviews_oai_pmh_plugin_row_etdms.inc-1N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherviews_oai_pmh_plugin_row_etdms.inc23N/A
Use an indent of 2 spaces, with no tabsOtherviews_oai_pmh_plugin_row_etdms.inc42N/A
Use an indent of 2 spaces, with no tabsOtherviews_oai_pmh_plugin_row_etdms.inc43N/A
Expandtheme/views-oai-pmh-response.tpl.php101
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherviews-oai-pmh-response.tpl.php-1N/A
use lowercase html tags to comply with XHTMLOtherviews-oai-pmh-response.tpl.php2N/A
Expandtheme/views_oai_pmh.theme.inc1610
MessageGroupFilenameLineFunctionStatus
There should be no trailing spacesOtherviews_oai_pmh.theme.inc16N/A
There should be no trailing spacesOtherviews_oai_pmh.theme.inc19N/A
There should be no trailing spacesOtherviews_oai_pmh.theme.inc24N/A
There should be no trailing spacesOtherviews_oai_pmh.theme.inc29N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherviews_oai_pmh.theme.inc35N/A
There should be no trailing spacesOtherviews_oai_pmh.theme.inc49N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherviews_oai_pmh.theme.inc55N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherviews_oai_pmh.theme.inc61N/A
There should be no trailing spacesOtherviews_oai_pmh.theme.inc64N/A
There should be no trailing spacesOtherviews_oai_pmh.theme.inc69N/A
There should be no trailing spacesOtherviews_oai_pmh.theme.inc74N/A
There should be no trailing spacesOtherviews_oai_pmh.theme.inc79N/A
There should be no trailing spacesOtherviews_oai_pmh.theme.inc82N/A
There should be no trailing spacesOtherviews_oai_pmh.theme.inc103N/A
There should be no trailing spacesOtherviews_oai_pmh.theme.inc110N/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)Otherviews_oai_pmh.theme.inc176N/A
There should be no trailing spacesOtherviews_oai_pmh.theme.inc225N/A
Expandviews_oai_pmh.install101
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherviews_oai_pmh.install-1N/A
Missing periodOtherviews_oai_pmh.install12N/A
Expandviews_oai_pmh.views_default.inc001
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherviews_oai_pmh.views_default.inc-1N/A

All

Test nameMinorCriticalNormal
Expandplugins/includes/views_oai_pmh_metadata_type.inc1600
MessageGroupFilenameLineFunctionStatus
There should be no trailing spacesOtherviews_oai_pmh_metadata_type.inc11N/A
There should be no trailing spacesOtherviews_oai_pmh_metadata_type.inc18N/A
There should be no trailing spacesOtherviews_oai_pmh_metadata_type.inc21N/A
There should be no trailing spacesOtherviews_oai_pmh_metadata_type.inc24N/A
There should be no trailing spacesOtherviews_oai_pmh_metadata_type.inc27N/A
There should be no trailing spacesOtherviews_oai_pmh_metadata_type.inc33N/A
There should be no trailing spacesOtherviews_oai_pmh_metadata_type.inc39N/A
There should be no trailing spacesOtherviews_oai_pmh_metadata_type.inc45N/A
There should be no trailing spacesOtherviews_oai_pmh_metadata_type.inc51N/A
There should be no trailing spacesOtherviews_oai_pmh_metadata_type.inc57N/A
There should be no trailing spacesOtherviews_oai_pmh_metadata_type.inc63N/A
There should be no trailing spacesOtherviews_oai_pmh_metadata_type.inc69N/A
There should be no trailing spacesOtherviews_oai_pmh_metadata_type.inc72N/A
There should be no trailing spacesOtherviews_oai_pmh_metadata_type.inc75N/A
There should be no trailing spacesOtherviews_oai_pmh_metadata_type.inc81N/A
There should be no trailing spacesOtherviews_oai_pmh_metadata_type.inc83N/A
Expandplugins/includes/views_oai_pmh_metadata_type_definitions.inc3100
MessageGroupFilenameLineFunctionStatus
There should be no trailing spacesOtherviews_oai_pmh_metadata_type_definitions.inc10N/A
There should be no trailing spacesOtherviews_oai_pmh_metadata_type_definitions.inc12N/A
There should be no trailing spacesOtherviews_oai_pmh_metadata_type_definitions.inc17N/A
There should be no trailing spacesOtherviews_oai_pmh_metadata_type_definitions.inc58N/A
There should be no trailing spacesOtherviews_oai_pmh_metadata_type_definitions.inc78N/A
There should be no trailing spacesOtherviews_oai_pmh_metadata_type_definitions.inc157N/A
There should be no trailing spacesOtherviews_oai_pmh_metadata_type_definitions.inc176N/A
There should be no trailing spacesOtherviews_oai_pmh_metadata_type_definitions.inc181N/A
There should be no trailing spacesOtherviews_oai_pmh_metadata_type_definitions.inc185N/A
There should be no trailing spacesOtherviews_oai_pmh_metadata_type_definitions.inc189N/A
There should be no trailing spacesOtherviews_oai_pmh_metadata_type_definitions.inc202N/A
There should be no trailing spacesOtherviews_oai_pmh_metadata_type_definitions.inc210N/A
There should be no trailing spacesOtherviews_oai_pmh_metadata_type_definitions.inc219N/A
There should be no trailing spacesOtherviews_oai_pmh_metadata_type_definitions.inc223N/A
There should be no trailing spacesOtherviews_oai_pmh_metadata_type_definitions.inc241N/A
There should be no trailing spacesOtherviews_oai_pmh_metadata_type_definitions.inc248N/A
There should be no trailing spacesOtherviews_oai_pmh_metadata_type_definitions.inc255N/A
There should be no trailing spacesOtherviews_oai_pmh_metadata_type_definitions.inc260N/A
There should be no trailing spacesOtherviews_oai_pmh_metadata_type_definitions.inc275N/A
There should be no trailing spacesOtherviews_oai_pmh_metadata_type_definitions.inc294N/A
There should be no trailing spacesOtherviews_oai_pmh_metadata_type_definitions.inc299N/A
There should be no trailing spacesOtherviews_oai_pmh_metadata_type_definitions.inc303N/A
There should be no trailing spacesOtherviews_oai_pmh_metadata_type_definitions.inc307N/A
There should be no trailing spacesOtherviews_oai_pmh_metadata_type_definitions.inc320N/A
There should be no trailing spacesOtherviews_oai_pmh_metadata_type_definitions.inc328N/A
There should be no trailing spacesOtherviews_oai_pmh_metadata_type_definitions.inc337N/A
There should be no trailing spacesOtherviews_oai_pmh_metadata_type_definitions.inc341N/A
There should be no trailing spacesOtherviews_oai_pmh_metadata_type_definitions.inc359N/A
There should be no trailing spacesOtherviews_oai_pmh_metadata_type_definitions.inc366N/A
There should be no trailing spacesOtherviews_oai_pmh_metadata_type_definitions.inc373N/A
There should be no trailing spacesOtherviews_oai_pmh_metadata_type_definitions.inc378N/A
Expandplugins/includes/views_oai_pmh_xml_node.inc1600
MessageGroupFilenameLineFunctionStatus
There should be no trailing spacesOtherviews_oai_pmh_xml_node.inc9N/A
There should be no trailing spacesOtherviews_oai_pmh_xml_node.inc33N/A
There should be no trailing spacesOtherviews_oai_pmh_xml_node.inc36N/A
There should be no trailing spacesOtherviews_oai_pmh_xml_node.inc41N/A
There should be no trailing spacesOtherviews_oai_pmh_xml_node.inc53N/A
There should be no trailing spacesOtherviews_oai_pmh_xml_node.inc68N/A
There should be no trailing spacesOtherviews_oai_pmh_xml_node.inc72N/A
There should be no trailing spacesOtherviews_oai_pmh_xml_node.inc76N/A
There should be no trailing spacesOtherviews_oai_pmh_xml_node.inc84N/A
There should be no trailing spacesOtherviews_oai_pmh_xml_node.inc96N/A
There should be no trailing spacesOtherviews_oai_pmh_xml_node.inc128N/A
There should be no trailing spacesOtherviews_oai_pmh_xml_node.inc132N/A
There should be no trailing spacesOtherviews_oai_pmh_xml_node.inc138N/A
There should be no trailing spacesOtherviews_oai_pmh_xml_node.inc167N/A
There should be no trailing spacesOtherviews_oai_pmh_xml_node.inc232N/A
There should be no trailing spacesOtherviews_oai_pmh_xml_node.inc235N/A
Expandplugins/views_oai_pmh_plugin_display.inc1000
MessageGroupFilenameLineFunctionStatus
There should be no trailing spacesOtherviews_oai_pmh_plugin_display.inc8N/A
There should be no trailing spacesOtherviews_oai_pmh_plugin_display.inc57N/A
There should be no trailing spacesOtherviews_oai_pmh_plugin_display.inc61N/A
There should be no trailing spacesOtherviews_oai_pmh_plugin_display.inc63N/A
There should be no trailing spacesOtherviews_oai_pmh_plugin_display.inc67N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherviews_oai_pmh_plugin_display.inc216N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherviews_oai_pmh_plugin_display.inc231N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherviews_oai_pmh_plugin_display.inc307N/A
Use an indent of 2 spaces, with no tabsOtherviews_oai_pmh_plugin_display.inc388N/A
Use an indent of 2 spaces, with no tabsOtherviews_oai_pmh_plugin_display.inc495N/A
Expandplugins/views_oai_pmh_plugin_row_auto.inc1900
MessageGroupFilenameLineFunctionStatus
There should be no trailing spacesOtherviews_oai_pmh_plugin_row_auto.inc11N/A
There should be no trailing spacesOtherviews_oai_pmh_plugin_row_auto.inc14N/A
There should be no trailing spacesOtherviews_oai_pmh_plugin_row_auto.inc17N/A
There should be no trailing spacesOtherviews_oai_pmh_plugin_row_auto.inc23N/A
There should be no trailing spacesOtherviews_oai_pmh_plugin_row_auto.inc26N/A
There should be no trailing spacesOtherviews_oai_pmh_plugin_row_auto.inc29N/A
There should be no trailing spacesOtherviews_oai_pmh_plugin_row_auto.inc38N/A
There should be no trailing spacesOtherviews_oai_pmh_plugin_row_auto.inc43N/A
There should be no trailing spacesOtherviews_oai_pmh_plugin_row_auto.inc54N/A
There should be no trailing spacesOtherviews_oai_pmh_plugin_row_auto.inc67N/A
There should be no trailing spacesOtherviews_oai_pmh_plugin_row_auto.inc93N/A
There should be no trailing spacesOtherviews_oai_pmh_plugin_row_auto.inc97N/A
There should be no trailing spacesOtherviews_oai_pmh_plugin_row_auto.inc106N/A
There should be no trailing spacesOtherviews_oai_pmh_plugin_row_auto.inc117N/A
There should be no trailing spacesOtherviews_oai_pmh_plugin_row_auto.inc126N/A
There should be no trailing spacesOtherviews_oai_pmh_plugin_row_auto.inc136N/A
There should be no trailing spacesOtherviews_oai_pmh_plugin_row_auto.inc145N/A
There should be no trailing spacesOtherviews_oai_pmh_plugin_row_auto.inc154N/A
There should be no trailing spacesOtherviews_oai_pmh_plugin_row_auto.inc173N/A
Expandplugins/views_oai_pmh_plugin_row_dc.inc103
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherviews_oai_pmh_plugin_row_dc.inc-1N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherviews_oai_pmh_plugin_row_dc.inc25N/A
Use an indent of 2 spaces, with no tabsOtherviews_oai_pmh_plugin_row_dc.inc44N/A
Use an indent of 2 spaces, with no tabsOtherviews_oai_pmh_plugin_row_dc.inc45N/A
Expandplugins/views_oai_pmh_plugin_row_etdms.inc103
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherviews_oai_pmh_plugin_row_etdms.inc-1N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherviews_oai_pmh_plugin_row_etdms.inc23N/A
Use an indent of 2 spaces, with no tabsOtherviews_oai_pmh_plugin_row_etdms.inc42N/A
Use an indent of 2 spaces, with no tabsOtherviews_oai_pmh_plugin_row_etdms.inc43N/A
Expandplugins/views_oai_pmh_plugin_row_misc.inc1900
MessageGroupFilenameLineFunctionStatus
There should be no trailing spacesOtherviews_oai_pmh_plugin_row_misc.inc27N/A
There should be no trailing spacesOtherviews_oai_pmh_plugin_row_misc.inc47N/A
There should be no trailing spacesOtherviews_oai_pmh_plugin_row_misc.inc54N/A
There should be no trailing spacesOtherviews_oai_pmh_plugin_row_misc.inc57N/A
There should be no trailing spacesOtherviews_oai_pmh_plugin_row_misc.inc70N/A
There should be no trailing spacesOtherviews_oai_pmh_plugin_row_misc.inc73N/A
There should be no trailing spacesOtherviews_oai_pmh_plugin_row_misc.inc76N/A
There should be no trailing spacesOtherviews_oai_pmh_plugin_row_misc.inc79N/A
There should be no trailing spacesOtherviews_oai_pmh_plugin_row_misc.inc90N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherviews_oai_pmh_plugin_row_misc.inc94N/A
There should be no trailing spacesOtherviews_oai_pmh_plugin_row_misc.inc120N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherviews_oai_pmh_plugin_row_misc.inc134N/A
There should be no trailing spacesOtherviews_oai_pmh_plugin_row_misc.inc170N/A
There should be no trailing spacesOtherviews_oai_pmh_plugin_row_misc.inc173N/A
There should be no trailing spacesOtherviews_oai_pmh_plugin_row_misc.inc184N/A
There should be no trailing spacesOtherviews_oai_pmh_plugin_row_misc.inc187N/A
There should be no trailing spacesOtherviews_oai_pmh_plugin_row_misc.inc207N/A
There should be no trailing spacesOtherviews_oai_pmh_plugin_row_misc.inc220N/A
There should be no trailing spacesOtherviews_oai_pmh_plugin_row_misc.inc224N/A
Expandplugins/views_oai_pmh_plugin_style.inc200
MessageGroupFilenameLineFunctionStatus
There should be no trailing spacesOtherviews_oai_pmh_plugin_style.inc34N/A
There should be no trailing spacesOtherviews_oai_pmh_plugin_style.inc90N/A
Expandplugins/views_oai_pmh_plugin_style_auto.inc1400
MessageGroupFilenameLineFunctionStatus
There should be no trailing spacesOtherviews_oai_pmh_plugin_style_auto.inc11N/A
There should be no trailing spacesOtherviews_oai_pmh_plugin_style_auto.inc14N/A
There should be no trailing spacesOtherviews_oai_pmh_plugin_style_auto.inc17N/A
There should be no trailing spacesOtherviews_oai_pmh_plugin_style_auto.inc23N/A
There should be no trailing spacesOtherviews_oai_pmh_plugin_style_auto.inc26N/A
There should be no trailing spacesOtherviews_oai_pmh_plugin_style_auto.inc29N/A
There should be no trailing spacesOtherviews_oai_pmh_plugin_style_auto.inc38N/A
There should be no trailing spacesOtherviews_oai_pmh_plugin_style_auto.inc43N/A
There should be no trailing spacesOtherviews_oai_pmh_plugin_style_auto.inc49N/A
There should be no trailing spacesOtherviews_oai_pmh_plugin_style_auto.inc55N/A
There should be no trailing spacesOtherviews_oai_pmh_plugin_style_auto.inc73N/A
There should be no trailing spacesOtherviews_oai_pmh_plugin_style_auto.inc92N/A
There should be no trailing spacesOtherviews_oai_pmh_plugin_style_auto.inc96N/A
There should be no trailing spacesOtherviews_oai_pmh_plugin_style_auto.inc105N/A
Leafplugins/views_oai_pmh_plugin_style_dc.inc000
Leafplugins/views_oai_pmh_plugin_style_etdms.inc000
Expandplugins/views_oai_pmh_plugin_style_misc.inc500
MessageGroupFilenameLineFunctionStatus
There should be no trailing spacesOtherviews_oai_pmh_plugin_style_misc.inc13N/A
There should be no trailing spacesOtherviews_oai_pmh_plugin_style_misc.inc21N/A
There should be no trailing spacesOtherviews_oai_pmh_plugin_style_misc.inc42N/A
There should be no trailing spacesOtherviews_oai_pmh_plugin_style_misc.inc47N/A
There should be no trailing spacesOtherviews_oai_pmh_plugin_style_misc.inc52N/A
Expandtheme/views-oai-pmh-response.tpl.php101
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherviews-oai-pmh-response.tpl.php-1N/A
use lowercase html tags to comply with XHTMLOtherviews-oai-pmh-response.tpl.php2N/A
Expandtheme/views_oai_pmh.theme.inc1610
MessageGroupFilenameLineFunctionStatus
There should be no trailing spacesOtherviews_oai_pmh.theme.inc16N/A
There should be no trailing spacesOtherviews_oai_pmh.theme.inc19N/A
There should be no trailing spacesOtherviews_oai_pmh.theme.inc24N/A
There should be no trailing spacesOtherviews_oai_pmh.theme.inc29N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherviews_oai_pmh.theme.inc35N/A
There should be no trailing spacesOtherviews_oai_pmh.theme.inc49N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherviews_oai_pmh.theme.inc55N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOtherviews_oai_pmh.theme.inc61N/A
There should be no trailing spacesOtherviews_oai_pmh.theme.inc64N/A
There should be no trailing spacesOtherviews_oai_pmh.theme.inc69N/A
There should be no trailing spacesOtherviews_oai_pmh.theme.inc74N/A
There should be no trailing spacesOtherviews_oai_pmh.theme.inc79N/A
There should be no trailing spacesOtherviews_oai_pmh.theme.inc82N/A
There should be no trailing spacesOtherviews_oai_pmh.theme.inc103N/A
There should be no trailing spacesOtherviews_oai_pmh.theme.inc110N/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)Otherviews_oai_pmh.theme.inc176N/A
There should be no trailing spacesOtherviews_oai_pmh.theme.inc225N/A
Expandviews_oai_pmh.install101
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherviews_oai_pmh.install-1N/A
Missing periodOtherviews_oai_pmh.install12N/A
Leafviews_oai_pmh.module000
Expandviews_oai_pmh.views.inc400
MessageGroupFilenameLineFunctionStatus
There should be no trailing spacesOtherviews_oai_pmh.views.inc39N/A
There should be no trailing spacesOtherviews_oai_pmh.views.inc47N/A
There should be no trailing spacesOtherviews_oai_pmh.views.inc69N/A
indent secondary line of comment one space Otherviews_oai_pmh.views.inc126N/A
Expandviews_oai_pmh.views_default.inc001
MessageGroupFilenameLineFunctionStatus
@file block missing (Drupal Docs)Otherviews_oai_pmh.views_default.inc-1N/A