|
Wrongful suggestion to use db_query_range on LIMITed DELETE FROM |
needs review |
normal |
bug reports |
7.x-1.x-dev |
Review/Rules |
|
|
Check for 'access arguments' => user_access(...) |
active |
normal |
tasks |
6.x-2.x-dev |
Code |
|
|
Line indentation style should be optional or of minor priority (tabs instead of spaces) |
by design |
normal |
feature requests |
6.x-2.x-dev |
User interface |
|
|
Camel case is checked also for class functions |
active |
normal |
bug reports |
6.x-1.x-dev |
Code |
|
|
Add simpletest coding standards to Coder Module |
active |
normal |
tasks |
7.x-1.x-dev |
SimpleTest |
|
|
Add spacing and string checks on comments. |
active |
normal |
feature requests |
7.x-1.x-dev |
Review/Rules |
|
|
string concatenation: wrong rules for spaces |
duplicate |
normal |
bug reports |
6.x-1.0 |
Code |
|
|
does coder check spacing of "x=y"? |
duplicate |
normal |
bug reports |
6.x-2.x-dev |
Review/Rules |
|
|
getInfo() cannot be called statically |
closed |
normal |
bug reports |
7.x-1.x-dev |
SimpleTest |
|
|
Fatal error: Call to undefined function drupal_get_page() |
closed |
normal |
bug reports |
7.x-1.x-dev |
Code |
|
|
Check to make sure hook_install()/hook_uninstall() are implemented? |
won't fix |
normal |
feature requests |
6.x-1.0 |
Code |
|
|
Restore settings menu |
closed |
normal |
bug reports |
7.x-1.x-dev |
Code |
|
|
Failed to include coder_review_test_case.tinc |
closed |
normal |
bug reports |
6.x-2.x-dev |
SimpleTest |
|
|
"Drupal Docs" links broken (again) |
closed |
normal |
bug reports |
6.x-2.x-dev |
Code |
|
|
Coder should not check camelCase for class methods |
closed |
normal |
bug reports |
6.x-2.x-dev |
Code |
douggreen |
|
Coding standards of string concatenation have changed |
closed |
normal |
feature requests |
6.x-1.x-dev |
Code |
|
|
empty lines added in patches are lost |
closed |
normal |
bug reports |
6.x-1.x-dev |
Code |
|
|
port to D7 |
closed |
normal |
tasks |
6.x-2.x-dev |
Code |
|
|
String concatenations |
duplicate |
normal |
support requests |
5.x-2.8 |
Review/Rules |
|
|
Bad Demo Link on Coder Project Page |
closed |
minor |
tasks |
5.x-2.x-dev |
Miscellaneous |
|
|
Permissions mostly capitalized |
active |
normal |
feature requests |
7.x-1.x-dev |
Review/Rules |
|
|
Metrics? |
closed |
minor |
feature requests |
6.x-2.x-dev |
Code |
|
|
Add check for incorrect use of 'access callback' |
needs review |
critical |
feature requests |
6.x-1.x-dev |
Review/Rules |
|
|
Corrected code review links for individual modules |
closed |
normal |
bug reports |
7.x-1.x-dev |
Code |
chiddicks |
|
wrong message when reviewing a patch |
closed |
normal |
bug reports |
6.x-2.x-dev |
User interface |
|
|
PHP notice errors: undefined indexes and offsets |
closed |
normal |
bug reports |
6.x-2.x-dev |
Code |
|
|
Blame gratitious use of heredoc syntax |
active |
normal |
feature requests |
7.x-1.x-dev |
Code |
|
|
Include file exclusions don't work |
needs review |
normal |
bug reports |
6.x-1.x-dev |
Code |
|
|
Wrong line displayed in watchdog() errors. |
active |
normal |
bug reports |
6.x-2.x-dev |
Review/Rules |
|
|
coder should not complain about closing php tag within template files |
needs review |
normal |
bug reports |
6.x-2.x-dev |
Code |
|
|
Form property #DANGEROUS_SKIP_CHECK removed in D6 |
closed |
normal |
tasks |
6.x-1.x-dev |
Code |
|
|
fatal error in tests, WSOD on simpletest page |
won't fix |
critical |
bug reports |
6.x-1.x-dev |
Code |
|
|
Coder incorrectly detects validate and submit functions |
active |
normal |
bug reports |
6.x-1.x-dev |
Code |
|
|
Coder issues critical errors for valid drupal_mail calls |
active |
normal |
bug reports |
6.x-1.x-dev |
Code |
|
|
Catch .info file syntax error |
active |
normal |
feature requests |
6.x-1.0 |
Code |
|
|
GHOP #136: Improve coder_format to re-format Drupal 6 core without wrong formattings |
needs review |
normal |
tasks |
6.x-1.x-dev |
Coder Format |
sun |
|
provide javascript review |
active |
normal |
feature requests |
6.x-1.x-dev |
Code |
stella |
|
Doc-Comment checker, some questions |
closed |
normal |
support requests |
6.x-2.x-dev |
Miscellaneous |
|
|
check for pgsql compatibility - double quotes in SQL queries |
needs work |
normal |
feature requests |
6.x-2.x-dev |
Code |
|
|
Performance optimization support |
won't fix |
normal |
feature requests |
5.x-2.2 |
Code |
douggreen |
|
Supply a front color with the back color |
needs review |
minor |
bug reports |
6.x-1.0 |
User interface |
|
|
No warning that hook_link_alter() parameters have changed in 6.x. |
active |
minor |
bug reports |
6.x-2.x-dev |
Review/Rules |
|
|
wrong path name for all.tests |
closed |
normal |
bug reports |
6.x-1.x-dev |
Coder Format |
|
|
Locale check doesnt consider Unicode strings: period |
needs review |
normal |
bug reports |
6.x-1.0 |
Code |
|
|
Locale check doesnt consider Unicode strings: first letter case |
needs review |
normal |
bug reports |
6.x-1.0 |
Code |
|
|
Empty translation is pointed as an error |
needs review |
normal |
bug reports |
6.x-1.0 |
Code |
|
|
t() called in a global scope |
active |
normal |
feature requests |
6.x-2.x-dev |
Review/Rules |
|
|
Zerod lines are set as -1 |
needs review |
normal |
bug reports |
6.x-1.x-dev |
Code |
|
|
How to check array content from *.install? |
active |
normal |
support requests |
6.x-2.x-dev |
Review/Rules |
|
|
false positive in patch on final ?> |
active |
normal |
bug reports |
6.x-2.x-dev |
Code |
|
