Printer, email and PDF versions - 7.x-1.x

Test status
StatusResult
Last receivedWed, 04/11/2012 - 22:53:37
Last requestedWed, 04/11/2012 - 22:54:16
Last testedWed, 04/11/2012 - 22:54:34
Test count6
Environment status
MySQLpass
Code reviewfail
Overall Resultpass
Branch information
Identifier7.x-1.x
ProjectPrinter, email and PDF versions
LinkProject information
SummaryPASSED: [[Coder]]: [Code review] 0 minor(s), 5 critical(s), and 0 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
0 minor(s), 5 critical(s), and 0 normal(s)

Non-pass

Test nameMinorCriticalNormal
Expandprint.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)Otherprint.install15N/A
Expandprint.module020
MessageGroupFilenameLineFunctionStatus
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)Otherprint.module290N/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)Otherprint.module551N/A
Expandprint_mail/print_mail.module010
MessageGroupFilenameLineFunctionStatus
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)Otherprint_mail.module123N/A
Expandprint_pdf/print_pdf.module010
MessageGroupFilenameLineFunctionStatus
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)Otherprint_pdf.module146N/A

All

Test nameMinorCriticalNormal
Leafprint.admin.inc000
Expandprint.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)Otherprint.install15N/A
Expandprint.module020
MessageGroupFilenameLineFunctionStatus
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)Otherprint.module290N/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)Otherprint.module551N/A
Leafprint.pages.inc000
Leafprint.tpl.php000
Leafprint_mail/print_mail.admin.inc000
Leafprint_mail/print_mail.inc000
Leafprint_mail/print_mail.install000
Expandprint_mail/print_mail.module010
MessageGroupFilenameLineFunctionStatus
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)Otherprint_mail.module123N/A
Leafprint_mail/views_handler_sort_print_mail_sentcount.inc000
Leafprint_pdf/print_pdf.admin.inc000
Leafprint_pdf/print_pdf.class.inc000
Leafprint_pdf/print_pdf.drush.inc000
Leafprint_pdf/print_pdf.install000
Expandprint_pdf/print_pdf.module010
MessageGroupFilenameLineFunctionStatus
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)Otherprint_pdf.module146N/A
Leafprint_pdf/print_pdf.pages.inc000
Leaftests/print_basic.test000