Currently in the my_module.coder_ignores.txt files, you have to specify the full path to the module installation.
Thus, on my dev installation it may be sites/dev.mysite.com/modules/specific/mymodule/ but on my QA environment it may be sites/qa.mysite.com/modules/specific/mymodule/ or anything else. Then because i did specify sites/dev.mysite.com/modules/specific/mymodule/ path in coder_ignores file, coder reports me lines i did ignored.
Then i would have allow the hook_coder_ignores to prefix lines with his own path. To be backward-compatible, i would have set it as argument in the returned array.
See attached path for my solution.
Comment | File | Size | Author |
---|---|---|---|
#1 | code_review_ignore_alterable_path.patch | 1.31 KB | superbaloo |
code_review_ignore_alterable_path.patch | 1.33 KB | superbaloo |
Comments
Comment #1
superbaloo CreditAttribution: superbaloo commentedUpdated version of patch without debug messages this time :D
Comment #2
stella CreditAttribution: stella commentedCommitted to both D6 and D7 branches, thanks!
Comment #4
Dane Powell CreditAttribution: Dane Powell commentedFYI, I updated the corresponding docs at http://drupal.org/node/144172
Also, there's a bug with this patch: http://drupal.org/node/1360744
Comment #5
Dane Powell CreditAttribution: Dane Powell commentedSorry, didn't mean to change the status.