Back-port disabled checkboxes fix from 7.x [#314286]

Comment	File	Size	Author
#1	314286.patch	696 bytes	asimmonds

Comment #1

asimmonds CreditAttribution: asimmonds commented 20 December 2008 at 09:21

Status:

Active

» Needs review

File	Size
314286.patch	696 bytes

Did this backport a couple months ago, but forgot to upload the patch. I've had this patch running on my local 6.x-dev instance for that time, haven't noticed any problems so far...

Log in or register to post comments

Comment #2

Damien Tournoud CreditAttribution: Damien Tournoud commented 20 December 2008 at 13:50

Status:

Needs review

» Reviewed & tested by the community

Makes total sense and makes form_type_checkbox_value() of D6 on par with D7.

Log in or register to post comments

Comment #3

Gábor Hojtsy

he/him

Hungarian

Hungary

CreditAttribution: Gábor Hojtsy commented 6 January 2009 at 16:34

How is the rest of the original issue is not applying here? I am not getting what was the issue there, the description was very bad to say the least.

Log in or register to post comments

Comment #4

Gábor Hojtsy

he/him

Hungarian

Hungary

CreditAttribution: Gábor Hojtsy commented 22 February 2009 at 10:13

Drupal.org suffers from this bug, see #378898: Resubscribe users to Maintainer News. Can someone please enlighten me on my question, so we can get this in and deployed on drupal.org sooner then later?

Log in or register to post comments

Comment #5

Damien Tournoud CreditAttribution: Damien Tournoud commented 22 February 2009 at 13:03

Gabor: the only issue here is: disabled checkboxes, even where they are checked, are not sent by the browser (by spec). We need to special case the disabled case so as to provide the correct value to form handlers.