API page: http://api.drupal.org/api/drupal/includes%21form.inc/function/theme_chec...

in element allowed properties you mentioned #value .. #value isn't used, #checked is

CommentFileSizeAuthor
#3 checked-attribute-1488668-D7.patch476 byteschertzog
#2 checked-attribute-1488668.patch496 byteschertzog

Comments

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Title: #checked not #value » #checked attribute not mentioned in theme_checkbox()
Version: 7.x-dev » 8.x-dev
Issue tags: +Novice, +Needs backport to D7

Actually, #value is used -- in form_process_checkbox(). But I agree that #checked should be added to the list.

chertzog’s picture

chertzog
he/him
Primary language English
Location Harrisburg, PA, USA
commented
Status: Active » Needs review
StatusFileSize
new checked-attribute-1488668.patch496 bytes

Patch attached.

chertzog’s picture

chertzog
he/him
Primary language English
Location Harrisburg, PA, USA
commented
Version: 8.x-dev » 7.x-dev
StatusFileSize
new checked-attribute-1488668-D7.patch476 bytes

And for D7.

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Version: 7.x-dev » 8.x-dev
Status: Needs review » Fixed

See http://drupal.org/node/1488414#comment-5748422 on the wisdom of uploading D7 patches, and the need to keep issues at D8 until fixed in D8. Thanks!

Meanwhile, the patch in #2 is fine for D8 and D7. Committed to both. Thanks!

Automatically closed -- issue fixed for 2 weeks with no activity.

Anonymous’s picture

(not verified) commented
Issue summary: View changes

fixing typo