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: 66194
Status: rejected
Priority: 0/
Queue: CGI

People
Owner: Nobody in particular
Requestors: rfrancis [...] ichr.uwa.edu.au
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



From rfrancis [...] ichr.uwa.edu.au Sat Feb 26 04: 32:28 2011
MIME-Version: 1.0
X-Spam-Status: No, score=-3.763 tagged_above=-99.9 required=10 tests=[BAYES_00=-1.9, DEAR_SOMETHING=1.973, RCVD_IN_DNSWL_HI=-5, RCVD_NUMERIC_HELO=1.164] autolearn=ham
X-Mailer: Evolution 2.26.3 (2.26.3-1.fc11)
X-Ironport-Anti-Spam-Result: AjUIANxXaE2CX4DX/2dsb2JhbACYNI4QdrMoiGqFYASFDopTgheDEg
X-Spam-Flag: NO
Message-ID: <1298712734.2787.22.camel [...] acs-pc-a0966.ichr.uwa.edu.au>
content-type: text/plain; charset="utf-8"
X-Ironport-Anti-Spam-Filtered: true
X-Virus-Scanned: Debian amavisd-new at bestpractical.com
X-Ironport-Av: E=Sophos;i="4.62,231,1297008000"; d="scan'208";a="69667060"
X-Spam-Score: -3.763
Received: from localhost (localhost [127.0.0.1]) by hipster.bestpractical.com (Postfix) with ESMTP id 8F748241609 for <cpan-bug+CGI.pm [...] hipster.bestpractical.com>; Sat, 26 Feb 2011 04:32:28 -0500 (EST)
Received: from hipster.bestpractical.com ([127.0.0.1]) by localhost (hipster.bestpractical.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 4ePRi+7imdeV for <cpan-bug+CGI.pm [...] hipster.bestpractical.com>; Sat, 26 Feb 2011 04:32:26 -0500 (EST)
Received: from la.mx.develooper.com (x1.develooper.com [207.171.7.70]) by hipster.bestpractical.com (Postfix) with SMTP id F064324120C for <bug-CGI.pm [...] rt.cpan.org>; Sat, 26 Feb 2011 04:32:25 -0500 (EST)
Received: (qmail 21494 invoked by uid 103); 26 Feb 2011 09:32:25 -0000
Received: from x16.dev (10.0.100.26) by x1.dev with QMQP; 26 Feb 2011 09:32:25 -0000
Received: from mail-ext-out2.uwa.edu.au (HELO mail-ext-out2.uwa.edu.au) (130.95.3.211) by 16.mx.develooper.com (qpsmtpd/0.80) with ESMTP; Sat, 26 Feb 2011 01:32:22 -0800
Received: from f5-float.net.uwa.edu.au (HELO mailmarshal.ichr.uwa.edu.au) ([130.95.128.215]) by mail-ext-out2.uwa.edu.au with ESMTP; 26 Feb 2011 17:32:16 +0800
Received: from tweety.ichr.uwa.edu.au (Not Verified[130.95.146.20]) by mailmarshal.ichr.uwa.edu.au with MailMarshal (v6,8,4,9558) id <B4d68c89e0000>; Sat, 26 Feb 2011 17:32:14 +0800
Received: from 130.95.146.168 ([130.95.146.168]) by TWEETY.ichr.uwa.edu.au ([130.95.146.20]) via Exchange Front-End Server owa.ichr.uwa.edu.au ([130.95.146.21]) with Microsoft Exchange Server HTTP-DAV ; Sat, 26 Feb 2011 09:32:14 +0000
Received: from acs-pc-a0966.ichr.uwa.edu.au by owa.ichr.uwa.edu.au; 26 Feb 2011 17:32:14 +0800
Delivered-To: cpan-bug+CGI.pm [...] hipster.bestpractical.com
Subject: labelattributes misbehaves in checkbox_group
Return-Path: <rfrancis [...] ichr.uwa.edu.au>
X-RT-Mail-Extension: cgi.pm
X-Original-To: cpan-bug+CGI.pm [...] hipster.bestpractical.com
X-Spam-Check-BY: 16.mx.develooper.com
Date: Sat, 26 Feb 2011 17:32:14 +0800
X-Spam-Level:
To: bug-CGI.pm [...] rt.cpan.org
Content-Transfer-Encoding: 7bit
From: Richard Francis <rfrancis [...] ichr.uwa.edu.au>
X-RT-Original-Encoding: ascii
Content-Length: 1549
Download (untitled) / with headers
text/plain 1.5k
Dear Sir/Madam, Many thanks for taking the time to look at this bug. I'm using: CGI version 3.52 Perl version 5.10.1 for x86_64 OS Fedora Core 13 2.6.33.6-147.2.4.fc13.x86_64 I assumed that in the context of a checkbox_group labelattributes would behave the same as attributes so as to be able to assign attributes to *specific* labels in a group. However using the code below I get ALL labels being given a hashref to display. #!/usr/bin/perl use strict; use CGI qw/:standard/; my $cgi = new CGI; print $cgi->header; print $cgi->start_html; my %labels = ( 'eenie'=>'first choice', 'meenie'=>'second choice', 'minie'=>'third choice'); my %labelattr = ('eenie'=> { 'class' => 'somecssclass' } ); my %attr = ('eenie'=> { 'class' => 'someothercssclass' } ); print $cgi->checkbox_group(-name => 'my_checkbox', -values => ['eenie','meenie','minie'], -labels => \%labels, -attributes => \%attr, -labelattributes => \%labelattr ); print $cgi->end_html; Regards, Richard Francis -- Mr Richard W Francis, MSc Senior Bioinformatician UWA Centre for Child Health Research Telethon Institute for Child Health Research University of Western Australia 100 Roberts Rd, Subiaco Perth WA 6008 Australia Email: rfrancis@ichr.uwa.edu.au Tel: +61-8-9489 7930 Fax: +61-8-9489 7700 ##################################################################################### This e-mail message has been scanned for Viruses and Content and cleared by MailMarshal #####################################################################################
X-RT-Interface: REST
MIME-Version: 1.0
X-Mailer: MIME-tools 5.504 (Entity 5.504)
RT-Message-ID: <rt-4.0.18-26599-1400760607-864.66194-0-0 [...] rt.cpan.org>
Content-Type: text/plain; charset="utf-8"
Content-Disposition: inline
Content-Transfer-Encoding: binary
Content-Length: 240
Download (untitled) / with headers
text/plain 240b
This issue has been copied to: https://github.com/leejo/CGI.pm/issues/82 please take all future correspondence there. This ticket will remain open but please do not reply here. This ticket will be closed when the github issue is dealt with.
MIME-Version: 1.0
In-Reply-To: <1298712734.2787.22.camel [...] acs-pc-a0966.ichr.uwa.edu.au>
X-Mailer: MIME-tools 5.504 (Entity 5.504)
Content-Disposition: inline
X-RT-Interface: Web
References: <1298712734.2787.22.camel [...] acs-pc-a0966.ichr.uwa.edu.au>
Content-Type: text/plain; charset="utf-8"
Message-ID: <rt-4.0.18-19577-1405252069-160.66194-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
X-RT-Encrypt: 0
X-RT-Sign: 0
Content-Length: 198
Download (untitled) / with headers
text/plain 198b
Rejecting. I'm not accepting any feature requests, patches, or updates to the HTML generating functions of CGI.pm unless they are of a critical nature (i.e. fixing fundamentally broken HTML output).
MIME-Version: 1.0
X-Ironport-Anti-Spam-Result: ApwEAIDiwlOCX4DP/2dsb2JhbADPOnU
X-Spam-Flag: NO
X-Virus-Checked: Checked
X-Virus-Scanned: Debian amavisd-new at bestpractical.com
Content-Type: multipart/mixed; boundary="_002_AFD734325842DE46BDE1293AA8FB5B870122513BHOEXC01ichruwae_"
X-Spam-Score: -4.6
Received: from localhost (localhost [127.0.0.1]) by hipster.bestpractical.com (Postfix) with ESMTP id 0AB5F240621 for <cpan-bug+CGI [...] hipster.bestpractical.com>; Sun, 13 Jul 2014 15:53:44 -0400 (EDT)
Received: from hipster.bestpractical.com ([127.0.0.1]) by localhost (hipster.bestpractical.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id ZzBD2RLy1yGC for <cpan-bug+CGI [...] hipster.bestpractical.com>; Sun, 13 Jul 2014 15:53:41 -0400 (EDT)
Received: from la.mx.develooper.com (x1.develooper.com [207.171.7.70]) by hipster.bestpractical.com (Postfix) with SMTP id 3AE8224031E for <bug-CGI [...] rt.cpan.org>; Sun, 13 Jul 2014 15:53:39 -0400 (EDT)
Received: (qmail 20330 invoked by alias); 13 Jul 2014 19:53:38 -0000
Received: from mail-ext-out1.uwa.edu.au (HELO mail-ext-out1.uwa.edu.au) (130.95.3.210) by la.mx.develooper.com (qpsmtpd/0.28) with ESMTP; Sun, 13 Jul 2014 12:53:32 -0700
Received: from f5-new.net.uwa.edu.au (HELO tweety.ichr.uwa.edu.au) ([130.95.128.207]) by mail-ext-out1.uwa.edu.au with ESMTP; 14 Jul 2014 03:53:23 +0800
Received: from HOEXC01.ichr.uwa.edu.au ([10.0.1.31]) by tweety.ichr.uwa.edu.au with Microsoft SMTPSVC(6.0.3790.4675); Mon, 14 Jul 2014 03:53:23 +0800
Received: from HOEXC01.ichr.uwa.edu.au ([fe80::d5c6:36d8:d089:9e5a]) by HOEXC01.ichr.uwa.edu.au ([fe80::d5c6:36d8:d089:9e5a%12]) with mapi id 14.03.0158.001; Mon, 14 Jul 2014 03:53:22 +0800
Delivered-To: cpan-bug+CGI [...] hipster.bestpractical.com
Subject: RE: [rt.cpan.org #66194] labelattributes misbehaves in checkbox_group
X-Spam-Check-BY: la.mx.develooper.com
Thread-Index: AQHPnpBO3FKNnjHuck+SbgQohaAoeZueaIZZ
Date: Sun, 13 Jul 2014 19:53:21 +0000
X-Spam-Level:
X-Ipas-Result: ApwEAIDiwlOCX4DP/2dsb2JhbADPOnU
To: "bug-CGI [...] rt.cpan.org" <bug-CGI [...] rt.cpan.org>
X-Originalarrivaltime: 13 Jul 2014 19:53:23.0471 (UTC) FILETIME=[1AAB19F0:01CF9ED4]
In-Reply-To: <rt-4.0.18-19577-1405252069-840.66194-6-0 [...] rt.cpan.org>
X-Spam-Status: No, score=-4.6 tagged_above=-99.9 required=10 tests=[BAYES_00=-1.9, FROM_OUR_RT=-2, RCVD_IN_DNSWL_LOW=-0.7] autolearn=ham
X-RT-Interface: API
Content-Language: en-GB
References: <RT-Ticket-66194 [...] rt.cpan.org> <1298712734.2787.22.camel [...] acs-pc-a0966.ichr.uwa.edu.au>,<rt-4.0.18-19577-1405252069-840.66194-6-0 [...] rt.cpan.org>
Message-ID: <AFD734325842DE46BDE1293AA8FB5B870122513B [...] HOEXC01.ichr.uwa.edu.au>
X-Ironport-Anti-Spam-Filtered: true
X-MS-Tnef-Correlator:
X-Ironport-Av: E=Sophos;i="5.01,654,1399996800"; d="png'150?scan'150,208,150";a="79613188"
Return-Path: <Richard.Francis [...] telethonkids.org.au>
X-RT-Mail-Extension: cgi
X-Original-To: cpan-bug+CGI [...] hipster.bestpractical.com
X-MS-Has-Attach: yes
Thread-Topic: [rt.cpan.org #66194] labelattributes misbehaves in checkbox_group
X-Originating-Ip: [10.0.1.37]
Accept-Language: en-GB, en-AU, en-US
From: Richard Francis <Richard.Francis [...] telethonkids.org.au>
RT-Message-ID: <rt-4.0.18-5131-1405281225-788.66194-0-0 [...] rt.cpan.org>
Content-Length: 0
content-type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable
X-RT-Original-Encoding: utf-8
Content-Length: 1195
Download (untitled) / with headers
text/plain 1.1k
Thanks Lee. Does this mean that this will never get fixed? Incidentally this bug does lead to broken HTML output. In the attached screenshot you'll see that in the page source, instead of the label for 'eenie' getting a particular CSS class, all labels get a hashref. This to me is broken. If it is likely to be a while before this is fixed then at least it might be good to note that labelattributes doesn't work in the documentation so that others don't try to use it. Thanks again for your time. Regards, Richard www.telethonkids.org.au 100 Roberts Road, Subiaco, Western Australia, 6008 PO Box 855, West Perth, Western Australia, 6872 Proudly supported by the people of Western Australia through Channel 7's Telethon Show quoted text
________________________________________ From: Lee Johnson via RT [bug-CGI@rt.cpan.org] Sent: 13 July 2014 19:47 To: Richard Francis Subject: [rt.cpan.org #66194] labelattributes misbehaves in checkbox_group <URL: https://rt.cpan.org/Ticket/Display.html?id=66194 > Rejecting. I'm not accepting any feature requests, patches, or updates to the HTML generating functions of CGI.pm unless they are of a critical nature (i.e. fixing fundamentally broken HTML output).
Content-Description: Screen Shot 2014-07-13 at 8.43.48 pm.png
Content-Type: image/png; name="Screen Shot 2014-07-13 at 8.43.48 pm.png"
Content-Disposition: attachment; creation-date="Sun, 13 Jul 2014 19:52:05 GMT"; filename="Screen Shot 2014-07-13 at 8.43.48 pm.png"; modification-date="Sun, 13 Jul 2014 19:52:05 GMT"; size="137686"
Content-Transfer-Encoding: base64
Content-ID: <b627cff1-223f-59e4-900d-d4963a128948>
Content-Length: 137686

Message body is not shown because sender requested not to inline it.

MIME-Version: 1.0
In-Reply-To: <rt-4.0.18-5131-1405281225-788.66194-0-0 [...] rt.cpan.org>
X-Mailer: MIME-tools 5.504 (Entity 5.504)
Content-Disposition: inline
X-RT-Interface: Web
References: <RT-Ticket-66194 [...] rt.cpan.org> <1298712734.2787.22.camel [...] acs-pc-a0966.ichr.uwa.edu.au>, <rt-4.0.18-19577-1405252069-840.66194-6-0 [...] rt.cpan.org> <AFD734325842DE46BDE1293AA8FB5B870122513B [...] HOEXC01.ichr.uwa.edu.au> <rt-4.0.18-5131-1405281225-788.66194-0-0 [...] rt.cpan.org>
Content-Type: text/plain; charset="utf-8"
Message-ID: <rt-4.0.18-30641-1405286180-1968.66194-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
X-RT-Encrypt: 0
X-RT-Sign: 0
Content-Length: 1012
Download (untitled) / with headers
text/plain 1012b
On Sun Jul 13 15:53:45 2014, Richard.Francis@telethonkids.org.au wrote: Show quoted text
> Thanks Lee. > Does this mean that this will never get fixed?
Hi Richard, It's possible that this may be fixed in a later version, but i'm trying to set a precedent by rejecting all issues related to the HTML generation functions to wean users off them. I will be updating the perldoc in the next version to explain the state and future of the HTML generation functions. Show quoted text
> Incidentally this bug does lead to broken HTML output. In the attached > screenshot you'll see that in the page source, instead of the label > for 'eenie' getting a particular CSS class, all labels get a hashref. > This to me is broken.
If you can provide a fully tested patch i'm happy to apply it since the bug report was raised before i took over maintenance. However i won't be looking at this myself as i don't consider it broken in the sense that incorrect/invalid attribute values don't actually break the fundamental rendering of the page. Thanks! Lee.
MIME-Version: 1.0
X-Spam-Status: No, score=-4.601 tagged_above=-99.9 required=10 tests=[BAYES_00=-1.9, FROM_OUR_RT=-2, RCVD_IN_DNSWL_LOW=-0.7, SPF_HELO_PASS=-0.001] autolearn=ham
In-Reply-To: <rt-4.0.18-5131-1405281225-347.66194-5-0 [...] rt.cpan.org>
X-Spam-Flag: NO
X-RT-Interface: API
References: <RT-Ticket-66194 [...] rt.cpan.org> <1298712734.2787.22.camel [...] acs-pc-a0966.ichr.uwa.edu.au>, <rt-4.0.18-19577-1405252069-840.66194-6-0 [...] rt.cpan.org> <AFD734325842DE46BDE1293AA8FB5B870122513B [...] HOEXC01.ichr.uwa.edu.au> <rt-4.0.18-5131-1405281225-347.66194-5-0 [...] rt.cpan.org>
X-Virus-Scanned: Debian amavisd-new at bestpractical.com
Message-ID: <53C2F883.7030409 [...] thecommune.net>
content-type: text/plain; charset="utf-8"; format="flowed"
X-RT-Original-Encoding: utf-8
X-Spam-Score: -4.601
Received: from localhost (localhost [127.0.0.1]) by hipster.bestpractical.com (Postfix) with ESMTP id 600D161E00D for <cpan-bug+CGI [...] hipster.bestpractical.com>; Sun, 13 Jul 2014 17:22:30 -0400 (EDT)
Received: from hipster.bestpractical.com ([127.0.0.1]) by localhost (hipster.bestpractical.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id papuouz8PQsF for <cpan-bug+CGI [...] hipster.bestpractical.com>; Sun, 13 Jul 2014 17:22:27 -0400 (EDT)
Received: from la.mx.develooper.com (x1.develooper.com [207.171.7.70]) by hipster.bestpractical.com (Postfix) with SMTP id 37C0461E00B for <bug-CGI [...] rt.cpan.org>; Sun, 13 Jul 2014 17:22:26 -0400 (EDT)
Received: (qmail 27599 invoked by alias); 13 Jul 2014 21:22:25 -0000
Received: from mxout-07.mxes.net (HELO mxout-07.mxes.net) (216.86.168.182) by la.mx.develooper.com (qpsmtpd/0.28) with ESMTP; Sun, 13 Jul 2014 14:22:21 -0700
Received: from [192.168.1.42] (unknown [71.56.239.12]) (using TLSv1 with cipher DHE-RSA-AES128-SHA (128/128 bits)) (No client certificate requested) by smtp.mxes.net (Postfix) with ESMTPSA id 9ED4F22E1F3 for <bug-CGI [...] rt.cpan.org>; Sun, 13 Jul 2014 17:22:18 -0400 (EDT)
Delivered-To: cpan-bug+CGI [...] hipster.bestpractical.com
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.6.0
Subject: Re: [rt.cpan.org #66194] labelattributes misbehaves in checkbox_group
Return-Path: <saj [...] thecommune.net>
X-Spam-Check-BY: la.mx.develooper.com
X-Original-To: cpan-bug+CGI [...] hipster.bestpractical.com
X-RT-Mail-Extension: cgi
Date: Sun, 13 Jul 2014 15:22:11 -0600
X-Spam-Level:
To: bug-CGI [...] rt.cpan.org
Content-Transfer-Encoding: 7bit
From: Stuart A Johnston <saj [...] thecommune.net>
RT-Message-ID: <rt-4.0.18-30641-1405286551-573.66194-0-0 [...] rt.cpan.org>
Content-Length: 1642
Download (untitled) / with headers
text/plain 1.6k
Although I can see why you'd assume that labelattributes would take the same input as attributes, it is not designed that way and the documentation doesn't say it does. The docs could be clarified or examples added but this definitely is not a bug. On 07/13/2014 01:53 PM, Richard Francis via RT wrote: Show quoted text
> Queue: CGI > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=66194 > > > Thanks Lee. > Does this mean that this will never get fixed? > > Incidentally this bug does lead to broken HTML output. In the attached screenshot you'll see that in the page source, instead of the label for 'eenie' getting a particular CSS class, all labels get a hashref. This to me is broken. > > If it is likely to be a while before this is fixed then at least it might be good to note that labelattributes doesn't work in the documentation so that others don't try to use it. > > Thanks again for your time. > Regards, > Richard > > > www.telethonkids.org.au > > 100 Roberts Road, Subiaco, Western Australia, 6008 PO Box 855, West Perth, Western Australia, 6872 > > Proudly supported by the people of Western Australia through Channel 7's Telethon > > ________________________________________ > From: Lee Johnson via RT [bug-CGI@rt.cpan.org] > Sent: 13 July 2014 19:47 > To: Richard Francis > Subject: [rt.cpan.org #66194] labelattributes misbehaves in checkbox_group > > <URL: https://rt.cpan.org/Ticket/Display.html?id=66194 > > > Rejecting. I'm not accepting any feature requests, patches, or updates to the HTML generating functions of CGI.pm unless they are of a critical nature (i.e. fixing fundamentally broken HTML output). >
MIME-Version: 1.0
X-Ironport-Anti-Spam-Result: AqQEANf/wlOCX4DP/2dsb2JhbABZgmp2WsFDh0MBgSB1hAMBAQEEOg8KATUCAQgRBAEBAQoUEDIdCAIEEwiIOgEMxxsXjByCahQdG4MtgRYFpmWIRoNEbIFE
X-Spam-Flag: NO
X-Virus-Scanned: Debian amavisd-new at bestpractical.com
content-type: text/plain; charset="utf-8"
X-RT-Original-Encoding: utf-8
X-Spam-Score: -4.6
Received: from localhost (localhost [127.0.0.1]) by hipster.bestpractical.com (Postfix) with ESMTP id 3BCE261E00B for <cpan-bug+CGI [...] hipster.bestpractical.com>; Sun, 13 Jul 2014 17:58:18 -0400 (EDT)
Received: from hipster.bestpractical.com ([127.0.0.1]) by localhost (hipster.bestpractical.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id tlhLeBfRLc6P for <cpan-bug+CGI [...] hipster.bestpractical.com>; Sun, 13 Jul 2014 17:58:16 -0400 (EDT)
Received: from la.mx.develooper.com (x1.develooper.com [207.171.7.70]) by hipster.bestpractical.com (Postfix) with SMTP id 1A95124031E for <bug-CGI [...] rt.cpan.org>; Sun, 13 Jul 2014 17:58:15 -0400 (EDT)
Received: (qmail 30364 invoked by alias); 13 Jul 2014 21:58:15 -0000
Received: from mail-ext-out2.uwa.edu.au (HELO mail-ext-out2.uwa.edu.au) (130.95.3.211) by la.mx.develooper.com (qpsmtpd/0.28) with ESMTP; Sun, 13 Jul 2014 14:58:10 -0700
Received: from f5-new.net.uwa.edu.au (HELO tweety.ichr.uwa.edu.au) ([130.95.128.207]) by mail-ext-out2.uwa.edu.au with ESMTP; 14 Jul 2014 05:58:03 +0800
Received: from HOEXC01.ichr.uwa.edu.au ([10.0.1.31]) by tweety.ichr.uwa.edu.au with Microsoft SMTPSVC(6.0.3790.4675); Mon, 14 Jul 2014 05:58:03 +0800
Received: from HOEXC01.ichr.uwa.edu.au ([fe80::d5c6:36d8:d089:9e5a]) by HOEXC01.ichr.uwa.edu.au ([fe80::d5c6:36d8:d089:9e5a%12]) with mapi id 14.03.0158.001; Mon, 14 Jul 2014 05:58:02 +0800
Delivered-To: cpan-bug+CGI [...] hipster.bestpractical.com
Subject: RE: [rt.cpan.org #66194] labelattributes misbehaves in checkbox_group
X-Spam-Check-BY: la.mx.develooper.com
Thread-Index: AQHPnpBO3FKNnjHuck+SbgQohaAoeZueaIZZgAAbfAyAAAXTyw==
Date: Sun, 13 Jul 2014 21:58:02 +0000
X-Spam-Level:
X-Ipas-Result: AqQEANf/wlOCX4DP/2dsb2JhbABZgmp2WsFDh0MBgSB1hAMBAQEEOg8KATUCAQgRBAEBAQoUEDIdCAIEEwiIOgEMxxsXjByCahQdG4MtgRYFpmWIRoNEbIFE
To: "bug-CGI [...] rt.cpan.org" <bug-CGI [...] rt.cpan.org>
Content-Transfer-Encoding: quoted-printable
X-Originalarrivaltime: 13 Jul 2014 21:58:03.0439 (UTC) FILETIME=[8513A3F0:01CF9EE5]
In-Reply-To: <rt-4.0.18-30641-1405286551-1633.66194-6-0 [...] rt.cpan.org>
X-Spam-Status: No, score=-4.6 tagged_above=-99.9 required=10 tests=[BAYES_00=-1.9, FROM_OUR_RT=-2, RCVD_IN_DNSWL_LOW=-0.7] autolearn=ham
X-RT-Interface: API
Content-Language: en-GB
References: <RT-Ticket-66194 [...] rt.cpan.org> <1298712734.2787.22.camel [...] acs-pc-a0966.ichr.uwa.edu.au>, <rt-4.0.18-19577-1405252069-840.66194-6-0 [...] rt.cpan.org> <AFD734325842DE46BDE1293AA8FB5B870122513B [...] HOEXC01.ichr.uwa.edu.au> <rt-4.0.18-5131-1405281225-347.66194-5-0 [...] rt.cpan.org> <53C2F883.7030409 [...] thecommune.net>,<rt-4.0.18-30641-1405286551-1633.66194-6-0 [...] rt.cpan.org>
Message-ID: <AFD734325842DE46BDE1293AA8FB5B8701225177 [...] HOEXC01.ichr.uwa.edu.au>
X-Ironport-Anti-Spam-Filtered: true
X-MS-Tnef-Correlator:
X-Ironport-Av: E=Sophos;i="5.01,655,1399996800"; d="scan'208";a="85411902"
Return-Path: <Richard.Francis [...] telethonkids.org.au>
X-RT-Mail-Extension: cgi
X-Original-To: cpan-bug+CGI [...] hipster.bestpractical.com
X-MS-Has-Attach:
Thread-Topic: [rt.cpan.org #66194] labelattributes misbehaves in checkbox_group
X-Originating-Ip: [10.0.1.37]
Accept-Language: en-GB, en-AU, en-US
From: Richard Francis <Richard.Francis [...] telethonkids.org.au>
RT-Message-ID: <rt-4.0.18-30641-1405288698-1215.66194-0-0 [...] rt.cpan.org>
Content-Length: 2495
Download (untitled) / with headers
text/plain 2.4k
OK. Thanks for your time. Maybe some clarification and an example in the docs would be useful to distinguish between the labelattributes switch adding attributes to particular labels as I assumed (given the behaviour of the attributes switch) or all labels as it is actually designed to do. Other than that I'm happy if you want to close this query. Regards, Richard www.telethonkids.org.au 100 Roberts Road, Subiaco, Western Australia, 6008 PO Box 855, West Perth, Western Australia, 6872 Proudly supported by the people of Western Australia through Channel 7's Telethon Show quoted text
________________________________________ From: saj@thecommune.net via RT [bug-CGI@rt.cpan.org] Sent: 14 July 2014 05:22 To: Richard Francis Subject: Re: [rt.cpan.org #66194] labelattributes misbehaves in checkbox_group <URL: https://rt.cpan.org/Ticket/Display.html?id=66194 > Although I can see why you'd assume that labelattributes would take the same input as attributes, it is not designed that way and the documentation doesn't say it does. The docs could be clarified or examples added but this definitely is not a bug. On 07/13/2014 01:53 PM, Richard Francis via RT wrote:
> Queue: CGI > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=66194 > > > Thanks Lee. > Does this mean that this will never get fixed? > > Incidentally this bug does lead to broken HTML output. In the attached screenshot you'll see that in the page source, instead of the label for 'eenie' getting a particular CSS class, all labels get a hashref. This to me is broken. > > If it is likely to be a while before this is fixed then at least it might be good to note that labelattributes doesn't work in the documentation so that others don't try to use it. > > Thanks again for your time. > Regards, > Richard > > > www.telethonkids.org.au > > 100 Roberts Road, Subiaco, Western Australia, 6008 PO Box 855, West Perth, Western Australia, 6872 > > Proudly supported by the people of Western Australia through Channel 7's Telethon > > ________________________________________ > From: Lee Johnson via RT [bug-CGI@rt.cpan.org] > Sent: 13 July 2014 19:47 > To: Richard Francis > Subject: [rt.cpan.org #66194] labelattributes misbehaves in checkbox_group > > <URL: https://rt.cpan.org/Ticket/Display.html?id=66194 > > > Rejecting. I'm not accepting any feature requests, patches, or updates to the HTML generating functions of CGI.pm unless they are of a critical nature (i.e. fixing fundamentally broken HTML output). >
MIME-Version: 1.0
X-Ironport-Anti-Spam-Result: AqQEABlzw1OCX4DP/2dsb2JhbABZgmp2WsFFh0MBgSt1hAMBAQEDAToZASoLAgEIEQQBAQEKFBAyHQgCBBMIiDIIAQzHCBeMHIJNEQELFB0bgy2BFgWmZYhGg0RsgQs5
X-Spam-Flag: NO
X-Virus-Scanned: Debian amavisd-new at bestpractical.com
content-type: text/plain; charset="utf-8"
X-RT-Original-Encoding: utf-8
X-Spam-Score: -4.6
Received: from localhost (localhost [127.0.0.1]) by hipster.bestpractical.com (Postfix) with ESMTP id 46613240621 for <cpan-bug+CGI [...] hipster.bestpractical.com>; Mon, 14 Jul 2014 02:08:59 -0400 (EDT)
Received: from hipster.bestpractical.com ([127.0.0.1]) by localhost (hipster.bestpractical.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id xaPaCskKYqOP for <cpan-bug+CGI [...] hipster.bestpractical.com>; Mon, 14 Jul 2014 02:08:57 -0400 (EDT)
Received: from la.mx.develooper.com (x1.develooper.com [207.171.7.70]) by hipster.bestpractical.com (Postfix) with SMTP id E44AD24031E for <bug-CGI [...] rt.cpan.org>; Mon, 14 Jul 2014 02:08:56 -0400 (EDT)
Received: (qmail 29811 invoked by alias); 14 Jul 2014 06:08:55 -0000
Received: from mail-ext-out2.uwa.edu.au (HELO mail-ext-out2.uwa.edu.au) (130.95.3.211) by la.mx.develooper.com (qpsmtpd/0.28) with ESMTP; Sun, 13 Jul 2014 23:08:51 -0700
Received: from f5-new.net.uwa.edu.au (HELO tweety.ichr.uwa.edu.au) ([130.95.128.207]) by mail-ext-out2.uwa.edu.au with ESMTP; 14 Jul 2014 14:08:46 +0800
Received: from HOEXC01.ichr.uwa.edu.au ([10.0.1.31]) by tweety.ichr.uwa.edu.au with Microsoft SMTPSVC(6.0.3790.4675); Mon, 14 Jul 2014 14:08:31 +0800
Received: from HOEXC01.ichr.uwa.edu.au ([fe80::d5c6:36d8:d089:9e5a]) by HOEXC01.ichr.uwa.edu.au ([fe80::d5c6:36d8:d089:9e5a%12]) with mapi id 14.03.0158.001; Mon, 14 Jul 2014 14:08:30 +0800
Delivered-To: cpan-bug+CGI [...] hipster.bestpractical.com
Subject: RE: [rt.cpan.org #66194] labelattributes misbehaves in checkbox_group
X-Spam-Check-BY: la.mx.develooper.com
Thread-Index: AQHPnpBO3FKNnjHuck+SbgQohaAoeZueaIZZgAAZww6AAIwyqA==
Date: Mon, 14 Jul 2014 06:08:29 +0000
X-Spam-Level:
X-Ipas-Result: AqQEABlzw1OCX4DP/2dsb2JhbABZgmp2WsFFh0MBgSt1hAMBAQEDAToZASoLAgEIEQQBAQEKFBAyHQgCBBMIiDIIAQzHCBeMHIJNEQELFB0bgy2BFgWmZYhGg0RsgQs5
To: "bug-CGI [...] rt.cpan.org" <bug-CGI [...] rt.cpan.org>
Content-Transfer-Encoding: quoted-printable
X-Originalarrivaltime: 14 Jul 2014 06:08:31.0131 (UTC) FILETIME=[095916B0:01CF9F2A]
In-Reply-To: <rt-4.0.18-30641-1405286181-363.66194-6-0 [...] rt.cpan.org>
X-Spam-Status: No, score=-4.6 tagged_above=-99.9 required=10 tests=[BAYES_00=-1.9, FROM_OUR_RT=-2, RCVD_IN_DNSWL_LOW=-0.7] autolearn=ham
X-RT-Interface: API
Content-Language: en-GB
References: <RT-Ticket-66194 [...] rt.cpan.org> <1298712734.2787.22.camel [...] acs-pc-a0966.ichr.uwa.edu.au>, <rt-4.0.18-19577-1405252069-840.66194-6-0 [...] rt.cpan.org> <AFD734325842DE46BDE1293AA8FB5B870122513B [...] HOEXC01.ichr.uwa.edu.au> <rt-4.0.18-5131-1405281225-788.66194-6-0 [...] rt.cpan.org>,<rt-4.0.18-30641-1405286181-363.66194-6-0 [...] rt.cpan.org>
Message-ID: <AFD734325842DE46BDE1293AA8FB5B87012251EF [...] HOEXC01.ichr.uwa.edu.au>
X-Ironport-Anti-Spam-Filtered: true
X-MS-Tnef-Correlator:
X-Ironport-Av: E=Sophos;i="5.01,656,1399996800"; d="scan'208";a="85491396"
Return-Path: <Richard.Francis [...] telethonkids.org.au>
X-RT-Mail-Extension: cgi
X-Original-To: cpan-bug+CGI [...] hipster.bestpractical.com
X-MS-Has-Attach:
Thread-Topic: [rt.cpan.org #66194] labelattributes misbehaves in checkbox_group
X-Originating-Ip: [10.0.1.37]
Accept-Language: en-GB, en-AU, en-US
From: Richard Francis <Richard.Francis [...] telethonkids.org.au>
RT-Message-ID: <rt-4.0.18-20244-1405318139-1869.66194-0-0 [...] rt.cpan.org>
Content-Length: 1922
Download (untitled) / with headers
text/plain 1.8k
Hi Lee, Just a follow on from your email. If you are still prepared to apply a simple patch then the one suggested here works well for me so that each label can be given a specific attribute. https://rtcpan.develooper.com/Public/Bug/Display.html?id=88451 However this does very slightly alter the behaviour as (very briefly) described in the docs. I'll leave it to you to decide if you wish to apply this. Kind regards, Richard www.telethonkids.org.au 100 Roberts Road, Subiaco, Western Australia, 6008 PO Box 855, West Perth, Western Australia, 6872 Proudly supported by the people of Western Australia through Channel 7's Telethon Show quoted text
________________________________________ From: Lee Johnson via RT [bug-CGI@rt.cpan.org] Sent: 14 July 2014 05:16 To: Richard Francis Subject: [rt.cpan.org #66194] labelattributes misbehaves in checkbox_group <URL: https://rt.cpan.org/Ticket/Display.html?id=66194 > On Sun Jul 13 15:53:45 2014, Richard.Francis@telethonkids.org.au wrote:
> Thanks Lee. > Does this mean that this will never get fixed?
Hi Richard, It's possible that this may be fixed in a later version, but i'm trying to set a precedent by rejecting all issues related to the HTML generation functions to wean users off them. I will be updating the perldoc in the next version to explain the state and future of the HTML generation functions.
> Incidentally this bug does lead to broken HTML output. In the attached > screenshot you'll see that in the page source, instead of the label > for 'eenie' getting a particular CSS class, all labels get a hashref. > This to me is broken.
If you can provide a fully tested patch i'm happy to apply it since the bug report was raised before i took over maintenance. However i won't be looking at this myself as i don't consider it broken in the sense that incorrect/invalid attribute values don't actually break the fundamental rendering of the page. Thanks! Lee.
MIME-Version: 1.0
X-Spam-Status: No, score=-4.601 tagged_above=-99.9 required=10 tests=[BAYES_00=-1.9, FROM_OUR_RT=-2, RCVD_IN_DNSWL_LOW=-0.7, SPF_HELO_PASS=-0.001] autolearn=ham
In-Reply-To: <rt-4.0.18-20244-1405318139-495.66194-5-0 [...] rt.cpan.org>
X-Spam-Flag: NO
X-RT-Interface: API
References: <RT-Ticket-66194 [...] rt.cpan.org> <1298712734.2787.22.camel [...] acs-pc-a0966.ichr.uwa.edu.au>, <rt-4.0.18-19577-1405252069-840.66194-6-0 [...] rt.cpan.org> <AFD734325842DE46BDE1293AA8FB5B870122513B [...] HOEXC01.ichr.uwa.edu.au> <rt-4.0.18-5131-1405281225-788.66194-6-0 [...] rt.cpan.org>, <rt-4.0.18-30641-1405286181-363.66194-6-0 [...] rt.cpan.org> <AFD734325842DE46BDE1293AA8FB5B87012251EF [...] HOEXC01.ichr.uwa.edu.au> <rt-4.0.18-20244-1405318139-495.66194-5-0 [...] rt.cpan.org>
X-Virus-Scanned: Debian amavisd-new at bestpractical.com
Message-ID: <53C40633.9020903 [...] thecommune.net>
content-type: text/plain; charset="utf-8"; format="flowed"
X-RT-Original-Encoding: utf-8
X-Spam-Score: -4.601
Received: from localhost (localhost [127.0.0.1]) by hipster.bestpractical.com (Postfix) with ESMTP id 784712409EA for <cpan-bug+CGI [...] hipster.bestpractical.com>; Mon, 14 Jul 2014 12:33:03 -0400 (EDT)
Received: from hipster.bestpractical.com ([127.0.0.1]) by localhost (hipster.bestpractical.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id J93-u7giaxR3 for <cpan-bug+CGI [...] hipster.bestpractical.com>; Mon, 14 Jul 2014 12:33:02 -0400 (EDT)
Received: from la.mx.develooper.com (x1.develooper.com [207.171.7.70]) by hipster.bestpractical.com (Postfix) with SMTP id E0124240045 for <bug-CGI [...] rt.cpan.org>; Mon, 14 Jul 2014 12:33:01 -0400 (EDT)
Received: (qmail 16096 invoked by alias); 14 Jul 2014 16:33:00 -0000
Received: from mxout-07.mxes.net (HELO mxout-07.mxes.net) (216.86.168.182) by la.mx.develooper.com (qpsmtpd/0.28) with ESMTP; Mon, 14 Jul 2014 09:32:56 -0700
Received: from [192.168.1.42] (unknown [71.56.239.12]) (using TLSv1 with cipher DHE-RSA-AES128-SHA (128/128 bits)) (No client certificate requested) by smtp.mxes.net (Postfix) with ESMTPSA id 42B8C22E1F4 for <bug-CGI [...] rt.cpan.org>; Mon, 14 Jul 2014 12:32:52 -0400 (EDT)
Delivered-To: cpan-bug+CGI [...] hipster.bestpractical.com
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.6.0
Subject: Re: [rt.cpan.org #66194] labelattributes misbehaves in checkbox_group
Return-Path: <saj [...] thecommune.net>
X-Spam-Check-BY: la.mx.develooper.com
X-Original-To: cpan-bug+CGI [...] hipster.bestpractical.com
X-RT-Mail-Extension: cgi
Date: Mon, 14 Jul 2014 10:32:51 -0600
X-Spam-Level:
To: bug-CGI [...] rt.cpan.org
Content-Transfer-Encoding: 7bit
From: Stuart A Johnston <saj [...] thecommune.net>
RT-Message-ID: <rt-4.0.18-31281-1405355584-1161.66194-0-0 [...] rt.cpan.org>
Content-Length: 2447
Download (untitled) / with headers
text/plain 2.3k
I don't think that patch correctly handles backwards compatibility, though I haven't tested. It would also need to be applied to all other functions with labelattributes and include tests. Probably a new option would be better to maintain compatibility rather than trying to overload the existing one. On 07/14/2014 12:09 AM, Richard Francis via RT wrote: Show quoted text
> Queue: CGI > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=66194 > > > Hi Lee, > > Just a follow on from your email. If you are still prepared to apply a simple patch then the one suggested here works well for me so that each label can be given a specific attribute. > > https://rtcpan.develooper.com/Public/Bug/Display.html?id=88451 > > However this does very slightly alter the behaviour as (very briefly) described in the docs. > I'll leave it to you to decide if you wish to apply this. > > Kind regards, > Richard > > > www.telethonkids.org.au > > 100 Roberts Road, Subiaco, Western Australia, 6008 PO Box 855, West Perth, Western Australia, 6872 > > Proudly supported by the people of Western Australia through Channel 7's Telethon > > ________________________________________ > From: Lee Johnson via RT [bug-CGI@rt.cpan.org] > Sent: 14 July 2014 05:16 > To: Richard Francis > Subject: [rt.cpan.org #66194] labelattributes misbehaves in checkbox_group > > <URL: https://rt.cpan.org/Ticket/Display.html?id=66194 > > > On Sun Jul 13 15:53:45 2014, Richard.Francis@telethonkids.org.au wrote:
>> Thanks Lee. >> Does this mean that this will never get fixed?
> > Hi Richard, > > It's possible that this may be fixed in a later version, but i'm trying to set a precedent by rejecting all issues related to the HTML generation functions to wean users off them. > > I will be updating the perldoc in the next version to explain the state and future of the HTML generation functions. >
>> Incidentally this bug does lead to broken HTML output. In the attached >> screenshot you'll see that in the page source, instead of the label >> for 'eenie' getting a particular CSS class, all labels get a hashref. >> This to me is broken.
> > If you can provide a fully tested patch i'm happy to apply it since the bug report was raised before i took over maintenance. However i won't be looking at this myself as i don't consider it broken in the sense that incorrect/invalid attribute values don't actually break the fundamental rendering of the page. > > Thanks! > Lee. >


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.