Re: area tag broken for full URLs

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