Error in CGI.pm Version 2.90 endform method

[Hero Zzyzzx]

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

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.

[PodMaster]

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.

[Hero Zzyzzx]

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.

[Hero Zzyzzx]

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.