The first stab at a 5.0 update for the links package. Much work to be done, especially in the admin screens, but the core module functions are all working. the function links_l() is (in this patch, at least) removed because all the stuff that uses it has to deal with the new array-based link structure. Thus, feeding elements into l() has the same effect as links_l().

CommentFileSizeAuthor
links_12.patch26.92 KBeaton
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

syscrusher’s picture

syscrusher CreditAttribution: syscrusher commented
Status: Needs work » Reviewed & tested by the community

Eaton, you rule! I have done a quick test of this, though not a comprehensive review yet, with Drupal HEAD. Seems to work as advertised for core functionality. I'm not sure it's perfect yet (as you yourself point out), but it's a damned nice step in the right direction. Many, many thanks for this.

Patch has been committed to CVS tonight.

Syscrusher

syscrusher’s picture

syscrusher CreditAttribution: syscrusher commented

I took a walk through the patch file to see the code changes. There's nothing whatsoever in there that I have a problem with. Great job!

Syscrusher

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented
Status: Reviewed & tested by the community » Fixed

Looks like this was committed. Marking fixed.

Anonymous’s picture

(not verified) CreditAttribution: commented
Status: Fixed » Closed (fixed)
fiLi’s picture

fiLi CreditAttribution: fiLi commented

You forgot to change a few links_l functions in the code. See - http://drupal.org/node/111251
Any fix for that? am I the only one experiencing trouble?