MySQL
- Setup environment
- Detect an invalid patch URL
- Detect invalid repository information
- Detect invalid patch format
- Detect a non-applicable patch
- Detect invalid PHP syntax
- Detect a Drupal installation failure
- Detect a test run failure
Ensure that you can install SimpleTest locally and invoke tests using the command line script.
- Detect a failing test
- Complete test suite with all tests passing
Code review
- Setup environment
- Detect an invalid patch URL
- Detect invalid repository information
- Detect invalid patch format
- Detect a non-applicable patch
- Detect invalid PHP syntax
- Detect a review run failure
- Detect coder flags
Ensure that your code follows the Drupal standard and passes a coder review.
- Complete coder review with all passed
4 minor(s), 2 critical(s), and 2 normal(s)
Non-pass
| Test name | Minor | Critical | Normal |
providers/slideshare.inc | 0 | 1 | 1 |
| Message | Group | Filename | Line | Function | Status |
| 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) | Other | slideshare.inc | 104 | N/A |  |
| The $text argument to l() should be enclosed within t() so that it is translatable. | Other | slideshare.inc | 105 | N/A |  |
|
slideshare.install | 4 | 1 | 1 |
| Message | Group | Filename | Line | Function | Status |
| indent secondary line of comment one space | Other | slideshare.install | 9 | N/A |  |
Format should be * Implementation of hook_foo(). | Other | slideshare.install | 9 | N/A |  |
| indent secondary line of comment one space | Other | slideshare.install | 10 | N/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) | Other | slideshare.install | 13 | N/A |  |
| The $text argument to l() should be enclosed within st() so that it is translatable from within the install. | Other | slideshare.install | 14 | N/A |  |
| There should be no trailing spaces | Other | slideshare.install | 29 | N/A |  |
|
All
| Test name | Minor | Critical | Normal |
providers/slideshare.inc | 0 | 1 | 1 |
| Message | Group | Filename | Line | Function | Status |
| 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) | Other | slideshare.inc | 104 | N/A |  |
| The $text argument to l() should be enclosed within t() so that it is translatable. | Other | slideshare.inc | 105 | N/A |  |
|
slideshare.install | 4 | 1 | 1 |
| Message | Group | Filename | Line | Function | Status |
| indent secondary line of comment one space | Other | slideshare.install | 9 | N/A |  |
Format should be * Implementation of hook_foo(). | Other | slideshare.install | 9 | N/A |  |
| indent secondary line of comment one space | Other | slideshare.install | 10 | N/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) | Other | slideshare.install | 13 | N/A |  |
| The $text argument to l() should be enclosed within st() so that it is translatable from within the install. | Other | slideshare.install | 14 | N/A |  |
| There should be no trailing spaces | Other | slideshare.install | 29 | N/A |  |
|
slideshare.module | 0 | 0 | 0 |
slideshare.theme.inc | 0 | 0 | 0 |
tests/slideshare.test | 0 | 0 | 0 |