Re: area tag broken for full URLs
Frank Reitzenstein <[email protected]>
| Newsgroups | gmane.comp.web.interchange.users |
|---|---|
| Message-ID | <[email protected]> |
hehe you are right Racke. I would never have thought of doing that, because that was never the context. Whatever will they think of next? On 2/9/2017 4:19 PM, Stefan Hornburg (Racke) wrote: > On 02/09/2017 08:10 AM, Peter wrote: >> On 05/02/17 23:05, Peter wrote: >>> On 05/02/17 23:01, Peter wrote: >>>> If you have a catalog with a VendUrl of www.example.com and you do this: >>>> [area href=http://www.example.net/test form=foo=bar] >>>> >>>> ...you get this as a result: >>>> http://www.example.com/test?foo=bar >>>> >>>> (note how [area] swapped the domain in the URL for the VendUrl domain) >>>> >>>> I note that the parsing of the page name in tag_area() parses out the >>>> domain name but puts it in $r instead. $r is not passed onto vendUrl() >>>> at line 2767 of Interpolate.pm, but I believe it was intended to be the >>>> third argument instead of the current undef. >>>> >>>> Any comments on why it is undef and if changing this to $r will fix the >>>> issue? >>> And indeed a quick test shows that changing the undef to $r does indeed >>> fix the issue. >> Any comments on this? This seems to be an old bug and I want to try to >> make certain that I'm not mucking something important up by fixing it. >> >> >> Peter > Hello Peter, > > I never thought of using [area] for full URLs ... > > Regards > Racke > >> _______________________________________________ >> interchange-users mailing list >> [email protected] >> http://www.icdevgroup.org/mailman/listinfo/interchange-users >> >