Pretty much as I said - if you run Coder against Link module to check for Drupal standards, it fails.

Comments

jcfiala’s picture

jcfiala commented
Status: Active » Fixed

And fixed - Link module seems to be to standards.

Status: Fixed » Closed (fixed)

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

mgifford’s picture

mgifford
he/him
Primary language English
commented
Version: 6.x-2.5 » 7.x-1.x-dev
Status: Closed (fixed) » Active

It needs to be re-assessed for D7. My review brought up a number of issues that would be good to clear up. Thanks!

jcfiala’s picture

jcfiala commented
Status: Active » Postponed

I'll worry about passing the coder test once the module actually works and is close to being released.

mgifford’s picture

mgifford
he/him
Primary language English
commented

Yup. Good order of priorities. Coder can be useful for picking out annoying things you may have missed that might actually be stopping your code from working too. Anyways, thanks again for your work.

mgifford’s picture

mgifford
he/him
Primary language English
commented
Status: Postponed » Active

Version 7.x-1.1 is the latest stable release & I just tried it in 7.x-2.x-dev too.

dqd’s picture

dqd
Location London | N.Y.C | Paris | Hamburg | Berlin
commented
Issue summary: View changes
Status: Active » Closed (outdated)

I'll close this issue due to inactivity for 12 years and upcoming EOL of Drupal 7 in January 2025. The project will be further maintained for Drupal 7 and will provide support for security issues and website breaking bug fixes but we should try to minimize the issue queue as much as possible. Thanks for the reports and all the efforts in here.