lot of times in d6_compat.inc url() arguments contains arrays, but it need simple variables:

http://api.drupal.org/api/function/url/5

Comments

michelle’s picture

michelle
she/her
Primary language English
Location Wisconsin, USA
commented

Is this causing problems somewhere? I haven't noticed any URLs not working. Unless this is affecting the functionality of the module, I'm going to won't fix it as I'm not touching that code unless something is actually broken.

Michelle

andrew_kabai’s picture

andrew_kabai commented

hi,

yes, it makes wrong urls. for example in a topic list a link for the new replys will be "node/nodenum?array" but the corret link is looks like this: node/nodenum#new

an other example: when the new post is not in the first page, then the link still will be "node/nodenum?array" and not "node/nodenum?page=pagenum#new"

Andrew

michelle’s picture

michelle
she/her
Primary language English
Location Wisconsin, USA
commented

Well, damn, how did I manage to miss that? I've clicked those links tons of times and never noticed they were wrong. LOL! You're right, of course. Ok, I'll dig in there and get that fixed. Thanks :)

Michelle

andrew_kabai’s picture

andrew_kabai commented

no problem, this is why we use bug report ;)

by the way, thanks for the great job!

Andrew

melodiesmaster’s picture

melodiesmaster commented
Version: 5.x-1.0-alpha3 » 6.x-1.x-dev

yeah, I just found this bug now while I'm testing. Hope there will be a solution soon.

michelle’s picture

michelle
she/her
Primary language English
Location Wisconsin, USA
commented
Version: 6.x-1.x-dev » 5.x-1.x-dev

Fixing the version.

Michelle

michelle’s picture

michelle
she/her
Primary language English
Location Wisconsin, USA
commented
Status: Active » Fixed

http://drupal.org/cvs?commit=104287

Fixed, thanks. :) Still can't believe I missed that one... LOL

Michelle

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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