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
0 minor(s), 2 critical(s), and 1 normal(s)
Non-pass
| Test name | Minor | Critical | Normal |
route_planner.install | 0 | 1 | 0 |
| 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 | route_planner.install | 11 | N/A |  |
|
route_planner.module | 0 | 1 | 1 |
| Message | Group | Filename | Line | Function | Status |
| Potential problem: hook_block() only accepts filtered text as the block title, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized. (Drupal Docs) | Other | route_planner.module | 54 | N/A |  |
| Function documentation should be less than 80 characters per line. | Other | route_planner.module | 73 | N/A |  |
|
All
| Test name | Minor | Critical | Normal |
route_planner.admin.inc | 0 | 0 | 0 |
route_planner.install | 0 | 1 | 0 |
| 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 | route_planner.install | 11 | N/A |  |
|
route_planner.module | 0 | 1 | 1 |
| Message | Group | Filename | Line | Function | Status |
| Potential problem: hook_block() only accepts filtered text as the block title, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized. (Drupal Docs) | Other | route_planner.module | 54 | N/A |  |
| Function documentation should be less than 80 characters per line. | Other | route_planner.module | 73 | N/A |  |
|