Hello everyone,

It has been a long time since I've used "coder_review" and I had used it a lot for D5->D6 migrations but I'm stuck with D6->D7 for one of my sites. When I enable "Converting 6.x modules to 7.x" to review my module the coder_review module simply does not give ANY report at all. When I hit the report page before selecting that option it gave me results for minor things like spacing issues.

I installed PHP_CodeSniffer using the pear command because the recommended drush command doesn't do anything.

Grammar Parser shows OK on the status report. I have both grammar_parser and grammar_parser_lib, with grammar_parser set to DISABLED as recommended by the module (weird) and a second copy of grammar_parser in the sites/all/libraries folder (weird also, but ok whatever...). grammar_praser_lib is enabled and living in the sites/all/modules directory.

Unchecking "Converting 6.x modules to 7.x" also results in a blank report. If I load the URL of the report with a fresh request it gives me the default report as it did before I tried to customize it. It seems any customizations result in a blank report.

No watchdog errors when I try to run the report either. Any ideas?

Comments

solotandem’s picture

solotandem commented

This has (or should have) nothing to do with your issue, but ...

To run coder review, you do not need to install coder upgrade and its dependencies (grammar parser and lib). As you have it, grammar parser should not be in a modules directory, but only in the libraries directory.

You could remove the grammar parser from modules directory and try again. Don't expect a change in behavior from this, but...

klausi’s picture

klausi
he/him||they/them
Primary language German
Location 🇦🇹 Vienna
commented
Issue summary: View changes
Status: Active » Closed (won't fix)

Coder 7.x is frozen now and will not receive updates. Coder 8.x-2.x can be used to check code for any Drupal version, Coder 8.x-2.x also supports the phpcbf command to automatically fix conding standard errors. Please check if this issue is still relevant and reopen against that version if necessary.