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().

links_12.patch26.92 KBeaton
Members fund testing for the Drupal project. Drupal Association Learn more


syscrusher’s picture

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’s picture

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!


webchick’s picture

Status: Reviewed & tested by the community » Fixed

Looks like this was committed. Marking fixed.

Anonymous’s picture

Status: Fixed » Closed (fixed)
fiLi’s picture

You forgot to change a few links_l functions in the code. See -
Any fix for that? am I the only one experiencing trouble?