Stas Bekman wrote:
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 im
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
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
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;
Imp
Stas Bekman wrote:
Philippe M. Chiasson wrote:
[...]
>
also continuing yesterdays story, this function doesn't know to deal
with utf8 strings, right? in which case we might just as well drop it
too? is there a CPAN implementation for that functionality?
I am 99.9% sure there is something on CPAN
Philippe M. Chiasson wrote:
As usual, I usually try and complete the documentation once the code
gets a go ;-)
of course :)
also continuing yesterdays story, this function doesn't know to deal
with utf8 strings, right? in which case we might just as well drop it
too? is there a CPAN implementati
Stas Bekman wrote:
Philippe M. Chiasson wrote:
as per todo/release:
Apache->unescape_url{_info}:
- not yet implemented. should be moved to Apache::Util (or may be
- APR::URI?)
I'd think it should live under ModPerl:: since it's not an Apache API
function. But I'm not sure if it'll serve
Philippe M. Chiasson wrote:
as per todo/release:
Apache->unescape_url{_info}:
- not yet implemented. should be moved to Apache::Util (or may be
- APR::URI?)
I'd think it should live under ModPerl:: since it's not an Apache API
function. But I'm not sure if it'll serve the best the user to
as per todo/release:
Apache->unescape_url{_info}:
- not yet implemented. should be moved to Apache::Util (or may be
- APR::URI?)
unescape_url has already been implemented, so this small patch ports the mp1
code to Apache::URI::unescape_url_info
--
--
