When using cvs head of both module and code (untested elsewhere) i get broken links (actually i just get a "<" on entry)

This patch fixes that

CommentFileSizeAuthor
drupal.subscriptions.nolinks.patch2.44 KBhalkeye

Comments

dziemecki’s picture

dziemecki commented

I've never seen this bug. How do you ge to it?

halkeye’s picture

halkeye commented

i'm not entire sure...
I installed the module last night
I don't remember the links showing up at all last night
This morning did a cvs update to the entire drupal tree
and then started to get "<" instead of any link

if i helps, i can revert the patch and show you on my test install.

halkeye’s picture

halkeye commented

looks like its http://cvs.drupal.org/viewcvs/drupal/drupal/includes/theme.inc?r1=1.295&...

halkeye’s picture

halkeye commented

this cvs patch

sorry

dziemecki’s picture

dziemecki commented
Assigned: Unassigned » dziemecki

Ah. So hook_link has changed in cvs. I've been coding in 4.7, only. Okay. Guess I need to start a new dev environment.

halkeye’s picture

halkeye commented

wouldn't be all that hard to fix the theme.inc patch to make it backwards compatible.

i think i'm going to submit a patch that way.

dziemecki’s picture

dziemecki commented
Status: Active » Postponed

Good idea. Given the non-static state of CVS, I'm going to hold up reacting to it until code freese in September.

arthurf’s picture

arthurf commented
Status: Postponed » Closed (fixed)

I've fixed this in relation to 50