Folks,

Just did a redhat 8.0 reinstall on my development laptop, which includes me pulling new stuff of CPAN. Anyhow, I discovered an error in CGI's endform() method in version 2.90. I've notified the author, but I thought the rest of you might like to know:

1624 #### Method: endform
1625 # End a form
1626 'endform' => <<'END_OF_FUNC',
1627 sub endform {
1628     my($self,@p) = self_or_default(@_);    
1629     if ( $NOSTICKY ) {
1630     return wantarray ? ("</form>") : "\n</form>";
1631     } else {
1632     return wantarray ? (("<div>",$self->get_fields,"</div>","</fo
+rm>") : 
1633                         "<div>".$self->get_fields ."</div>\n</for
+m>";
1634     }
1635 }
1636 END_OF_FUNC

[download]

Line 1632 has an extra "(" in it. Take it out and you'll be good to go.

-Any sufficiently advanced technology is
indistinguishable from doubletalk.

Replies are listed 'Best First'.
Re: Error in CGI.pm Version 2.90 endform method
by PodMaster (Abbot) on Feb 18, 2003 at 22:46 UTC
    um, sorry to burst your bubble, but this has already been fixed in CGI.pm-2.91 a week ago. It helps to keep up

    Version 2.91

    1. Attribute generation now correctly respects the value of autoEscape().
    2. Fixed endofrm() syntax error introduced by Ben Edgington's patch.

    MJD says you can't just make shit up and expect the computer to know what you mean, retardo!
    I run a Win32 PPM repository for perl 5.6x+5.8x. I take requests.
    ** The Third rule of perl club is a statement of fact: pod is sexy.

[reply]

      I just downloaded it via the CPAN shell from a mirror last night, the mirror itself must've been out of date. <sarcasm>Thanks for your kind words.</sarcasm>

      -Any sufficiently advanced technology is
      indistinguishable from doubletalk.

[reply]
Re: Error in CGI.pm Version 2.90 endform method
by Hero Zzyzzx (Curate) on Feb 18, 2003 at 16:59 UTC

    Surprisingly enough, endform() isn't in the included tests. I thought maybe I had done something stupid like installing a module when the test failed, but I didn't. Just a plain old syntax error, not caught because the method wasn't tested.

    -Any sufficiently advanced technology is
    indistinguishable from doubletalk.

[reply]

Back to Perl News