[Catalyst] Patch for Catalyst::Plugin::Unicode::Encoding

Jonathan Rockway jon at jrock.us
Wed Mar 19 05:27:47 GMT 2008


* On Tue, Mar 18 2008, Bill Moseley wrote:

> Without testing is_utf8 and if the parameters are already decoded I 
> rightly get:
>
>     [error] Caught exception in engine "Cannot decode string with wide
>     characters at
>     /usr/local/share/perl/5.8.8/Catalyst/Plugin/Unicode/Encoding.pm
>     line 74."

What this means is "we don't know what the fuck the encoding is in the
incoming data, so just try random shit until something appears to
work".  I would prefer a more thought-out approach.

We should not need to check the flag.  The incoming data should be
encoded.  Then we should decode it.  Then we should not try to decode it
again.

Regards,
Jonathan Rockway

-- 
print just => another => perl => hacker => if $,=$"



More information about the Catalyst mailing list