Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the CGI CPAN distribution.

Report information
The Basics
Id: 35153
Status: rejected
Priority: 0/
Queue: CGI
People
Owner: MARKSTOS [...] cpan.org
Requestors: EDAVIS [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: Normal
Broken in: (no value)
Fixed in: (no value)

History Show all quoted text
  Thu Apr 17 11:26:30 2008 EDAVIS [...] cpan.org - Ticket created
This code fragment: print popup_menu(-name => 'n', -values => [ qw(a b c) ], 'x'); generates the HTML fragment <select name="n" x> <option value="a">a</option> <option value="b">b</option> <option value="c">c</option> </select> which is ill-formed. popup_menu() should warn about the extra argument rather than generating bad HTML.
  Wed Jul 22 19:05:26 2009 MARKSTOS [...] cpan.org - Taken
  Wed Jul 22 19:08:18 2009 MARKSTOS [...] cpan.org - Correspondence added
On Thu Apr 17 11:26:30 2008, EDAVIS wrote: Show quoted text
> This code fragment: > > print popup_menu(-name => 'n', -values => [ qw(a b c) ], 'x'); > > generates the HTML fragment > > <select name="n" x> > <option value="a">a</option> > <option value="b">b</option> > <option value="c">c</option> > </select> > > which is ill-formed. popup_menu() should warn about the extra argument > rather than generating bad HTML.
I believe this part of the documented behavior of the CGI.pm HTML shortcuts-- they turn extra arguments into additional attribute name/value pairs. This is documented here: http://search.cpan.org/~lds/CGI.pm-3.43/CGI.pm#PROVIDING_ARGUMENTS_TO_HTML_SHORTCUTS
  Wed Jul 22 19:08:18 2009 The RT System itself - Status changed from 'new' to 'open'
  Wed Jul 22 19:08:19 2009 MARKSTOS [...] cpan.org - Status changed from 'open' to 'rejected'
  Fri May 23 14:28:17 2014 The RT System itself - Queue changed from CGI.pm to CGI