Nice module. I ran Coder against it and found many coding standards violations (too many to submit a patch).

Comments

mgifford’s picture

mgifford
he/him
Primary language English
CreditAttribution: mgifford commented
Version: 5.x-1.x-dev » 7.x-1.x-dev
Category: feature » task

Note, this is still a problem.

Coder found 1 projects, 8 files, 3 critical warnings, 155 normal warnings, 0 warnings were flagged to be ignored

Also switching this to a task from a feature request. It's just less optional for D7 than it was for D5 when this issue was first posted.

mh86’s picture

mh86 CreditAttribution: mh86 commented

Thanks for updating the issue.

I'm planning a code cleanup before the 1.0 release.

mgifford’s picture

mgifford
he/him
Primary language English
CreditAttribution: mgifford commented

Excellent. Thanks! It's a lot of work to clean it all up.

mh86’s picture

mh86 CreditAttribution: mh86 commented
Status: Active » Fixed

All warnings according to the coder module have been fixed.

Status: Fixed » Closed (fixed)

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