Re: area tag encoding dash in sku to %2d

Quoting Peter ([email protected]):
> On 10/06/17 23:32, Mike Heins wrote:
> >> GlobalSub <<<EOR
> > 
> > I think you have one too many < in the here doc. Should work fine, though.
> 
> Oops, you're right.  Score one for the typo monster.
> 
> > We could consider defining that string in Vend::Util like is done
> > for HTML::Entities, but the above is the functional equivalent and not
> > difficult to do.
> 
> I suppose you could also do:
> 
> GlobalSub <<<EOR
> sub override_hacking {
>     package Vend::Util;
> 	*hexify = \&HTML::Entities::encode_entities;
> }
> EOR
> 
> ...or something like that.  In fact, is there any reason why we should
> keep using hexify when we are using encode_entities everywhere else?

Because CGI urlencoding is not the same. It might be nice to have a pragma
or directive to route hexify, as you have done, but to change the behavior
would introduce an element of risk to applications I would not be willing
to take on.

-- 
Mike Heins
End Point -- Expert Internet Consulting    http://www.endpoint.com/
phone +1.765.253.4194  <[email protected]>

Opportunity is missed by most people because it is dressed in
overalls and looks like work. -- Thomas Edison