Gavin (halkeye) wrote in lj_dev,
Gavin
halkeye
lj_dev

poll bug?

in ljpoll.pl on like 97 we have: $qopts{'type'} = $opts->{'type'};
then we have the parts the vertify the scale and text parts

but on like 154, we have $qopts{'type'} = lc($opts->{'type'}) || "text";
I'm assuming the first bit really just sets $qopts{'type'} so the vertifying will work.
But, the second instance makes the type lowercase to check if it is a valid type. thus allowing things like type=Scale in, but not working..
At least thats what my limited perl knowledge has figured out...

Just trying to figure out why the type=scale will work, but type=Scale wont, while type=Radio does as well as type=radio

Yea, from what i can see, the defaults are set before type= is lowercased, so type=Text shouldn't really work either. Its not really a big deal, just thought i'd let people know.
Subscribe
  • Post a new comment

    Error

    Anonymous comments are disabled in this journal

    default userpic

    Your reply will be screened

    Your IP address will be recorded 

  • 6 comments