Countries - 7.x-2.0

Test status
StatusResult
Last receivedWed, 09/26/2012 - 04:31:30
Last requestedWed, 09/26/2012 - 04:32:15
Last testedWed, 09/26/2012 - 04:34:22
Test count4
Environment status
MySQLpass
Code reviewfail
Overall Resultpass
Branch information
Identifier7.x-2.0
ProjectCountries
LinkProject information
SummaryPASSED: [[Coder]]: [Code review] 5 minor(s), 7 critical(s), and 1 normal(s); [[SimpleTest]]: [MySQL] 747 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
747 pass(es), 0 fail(s), and 0 exception(s)
Test namePassFailException
LeafCountry administration UI (CountriesCRUDUIUnitTest) [Countries]23800
LeafCountry core (CountriesFunctionsUnitTest) [Countries]16700
LeafCountry CRUD functions (CountriesCRUDUnitTest) [Countries]1800
LeafCountry Element (CountriesElementUnitTest) [Countries]4700
LeafCountry imports from CSV file (CountriesBaseImportUnitTest) [Countries]22600
LeafCountry overrides (CountriesBaseInstallUnitTest) [Countries]5100

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
5 minor(s), 7 critical(s), and 1 normal(s)

Non-pass

Test nameMinorCriticalNormal
Expandcountries.admin.inc041
MessageGroupFilenameLineFunctionStatus
Potential problem: form_set_error() and form_error() only accept filtered text, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized.Othercountries.admin.inc214N/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)Othercountries.admin.inc393N/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)Othercountries.admin.inc419N/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.Othercountries.admin.inc428N/A
use stdClass caseCapitalization, it's the one exception to the mixed case style standardOthercountries.admin.inc461N/A
Expandcountries.install010
MessageGroupFilenameLineFunctionStatus
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)Othercountries.install126N/A
Expandcountries.module010
MessageGroupFilenameLineFunctionStatus
Potential problem: form_set_error() and form_error() only accept filtered text, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized.Othercountries.module314N/A
Expandmodules/countries_configuration/countries_configuration.admin.inc010
MessageGroupFilenameLineFunctionStatus
Potential problem: drupal_set_title() only accepts filtered text, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized.Othercountries_configuration.admin.inc33N/A

All

Test nameMinorCriticalNormal
Expandcountries.admin.inc041
MessageGroupFilenameLineFunctionStatus
Potential problem: form_set_error() and form_error() only accept filtered text, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized.Othercountries.admin.inc214N/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)Othercountries.admin.inc393N/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)Othercountries.admin.inc419N/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.Othercountries.admin.inc428N/A
use stdClass caseCapitalization, it's the one exception to the mixed case style standardOthercountries.admin.inc461N/A
Leafcountries.elements.inc000
Leafcountries.feeds.inc000
Leafcountries.fields.inc000
Expandcountries.install010
MessageGroupFilenameLineFunctionStatus
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)Othercountries.install126N/A
Leafcountries.migrate.inc000
Expandcountries.module010
MessageGroupFilenameLineFunctionStatus
Potential problem: form_set_error() and form_error() only accept filtered text, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized.Othercountries.module314N/A
Leafcountries.tokens.inc000
Expandmodules/countries_configuration/countries_configuration.admin.inc010
MessageGroupFilenameLineFunctionStatus
Potential problem: drupal_set_title() only accepts filtered text, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized.Othercountries_configuration.admin.inc33N/A
Leafmodules/countries_configuration/countries_configuration.api.php000
Leafmodules/countries_configuration/countries_configuration.install000
Leafmodules/countries_configuration/countries_configuration.module000
Leafmodules/countries_example/countries_example.install000
Leafmodules/countries_example/countries_example.module000
Leafmodules/countries_i18n/countries_i18n.module000
Expandtests/countries.test400
MessageGroupFilenameLineFunctionStatus
@see should always be followed by a filename, a URL, class/interface name (optionally including method), or a function name including ().Othercountries.test9N/A
@see should always be followed by a filename, a URL, class/interface name (optionally including method), or a function name including ().Othercountries.test11N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthercountries.test198N/A
in most cases, replace the string function with the drupal_ equivalent string functionsOthercountries.test199N/A
Leafviews/countries.views.inc000
Leafviews/views_handler_field_countries_continent.inc000
Leafviews/views_handler_filter_countries_continents_list.inc000
Expandviews/views_handler_filter_countries_list.inc100
MessageGroupFilenameLineFunctionStatus
in most cases, replace the string function with the drupal_ equivalent string functionsOtherviews_handler_filter_countries_list.inc88N/A