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.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

superbaloo’s picture

Updated version of patch without debug messages this time :D

stella’s picture

Status: Active » Fixed

Committed to both D6 and D7 branches, thanks!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

Dane Powell’s picture

Status: Closed (fixed) » Needs review

FYI, I updated the corresponding docs at http://drupal.org/node/144172

Also, there's a bug with this patch: http://drupal.org/node/1360744

Dane Powell’s picture

Status: Needs review » Closed (fixed)

Sorry, didn't mean to change the status.