Philippe M. Chiasson wrote:


Stas Bekman wrote:

Philippe M. Chiasson wrote:
[...]

it's probably always a good idea to discuss first before spending time on things :)



So, what do we do about it? Yank it out, document better alternatives, like
CGI::Util::unescape() and possibly implement in Apache/compat.pm as :
unescape_url() =~ s/\+/ /g;



Implement it in Apache/compat.pm *and* document to use CGI::Util::unescape() in mp2?


Here it is:

+1 (plus doc of course :)

There is one more things that needs to be updated, it's $methods_compat in MethodLookup.pm (which is autogenerated from lib/ModPerl/WrapXS.pm), this and the escape_html method should be there (did we forget any other recently added to compat.pm methods?)

--
__________________________________________________________________
Stas Bekman            JAm_pH ------> Just Another mod_perl Hacker
http://stason.org/     mod_perl Guide ---> http://perl.apache.org
mailto:[EMAIL PROTECTED] http://use.perl.org http://apacheweek.com
http://modperlbook.org http://apache.org   http://ticketmaster.com

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Reply via email to