Beefy Boxes and Bandwidth Generously Provided by pair Networks
We don't bite newbies here... much
 
PerlMonks  

Re^5: Building data structures from CGI params

by Anonymous Monk
on Aug 08, 2012 at 03:26 UTC ( #986129=note: print w/ replies, xml ) Need Help??


in reply to Re^4: Building data structures from CGI params
in thread Building data structures from CGI params

I'm not sure what you mean.  { row => ["row", "row", "row", "your boat"]  }   looks like the "right" answer to me... What were you expecting?

That line is generated by the workaround

This is Vars output

{ "row[]" => "row\0row\0row\0your boat" },

And this is what your module does with vars output

{ row => ["row\0row\0row\0your boat"] },

This is the bug, it forgot to split on \0, because Vars joins on \0

or better still it should simply accept a CGI.pm compatible object (one with param method, whether its CGI/CGI::Simple/CGI::Lite.....) and use the Vars workaround on it

Do you get it now?


Comment on Re^5: Building data structures from CGI params
Select or Download Code
Replies are listed 'Best First'.
Re^6: Building data structures from CGI params
by fullermd (Priest) on Aug 08, 2012 at 17:36 UTC

    That line is generated by the workaround

    [...]

    This is the bug, it forgot to split on \0, because Vars joins on \0

    Oh, I see what you mean. Hm. I hate that NUL stuff...

    I guess the best thing would be to add a config'able (probably on by default) splitting on them to the processing. I like that better than expecting a ->param, since that breaks it away from the contract of purely hash->hash.

    I'll see if I can't get that into a new release in the next few weeks. Annoyingly busy... :|

      I like that better than expecting a ->param, since that breaks it away from the contract of purely hash->hash.

      Adding a branch breaks no contracts

      sub build_cgi_struct { my ($iv, $errs, $conf) = @_; use Scalar::Util qw(blessed); my $blessed = blessed $iv; my $splitnull = 1; if( $blessed and UNIVERSAL::can( $iv, 'param') ){ $splitnull = 0; if( $iv->isa('CGI') ){ $iv = $iv->{"param"} ; # cheat :) } else { $iv = { map { $_ => [ $iv->param($_) ] } $iv->param }; } } ... ... if $splitnull;

      I'll see if I can't get that into a new release in the next few weeks.

      Well, OK, it's still just a few weeks. FSVO "few" :)

      I've uploaded a new version that does nullsplits by default (as well as one or two other minor tweaks).

Log In?
Username:
Password:

What's my password?
Create A New User
Node Status?
node history
Node Type: note [id://986129]
help
Chatterbox?
and the web crawler heard nothing...

How do I use this? | Other CB clients
Other Users?
Others chanting in the Monastery: (16)
As of 2015-07-07 17:09 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?

    The top three priorities of my open tasks are (in descending order of likelihood to be worked on) ...









    Results (92 votes), past polls