The recommended way to change "checked" status for checkbox is using .prop() instead of .attr(). See jQuery .prop() function. Otherwise the checkbox ui status may not be updated event the checked attribute is correctly set on the html element.

CommentFileSizeAuthor
#3 2827710-3-attr-to-prop.patch2.04 KBjoelpittet
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

qichanghai created an issue. See original summary.

joelpittet’s picture

joelpittet
Primary language English
Location Vancouver
CreditAttribution: joelpittet at The University of British Columbia commented

@qichanghai, there is no .prop() for jQuery 1.4 which is core drupal. I've fixed this in VBO by putting in a quick polyfill

https://www.drupal.org/node/2608360

Maybe the same approach could work here too?

joelpittet’s picture

joelpittet
Primary language English
Location Vancouver
CreditAttribution: joelpittet at The University of British Columbia commented
Status: Active » Needs review
FileSize
2827710-3-attr-to-prop.patch2.04 KB

Here's the patch

  • hansfn committed b799af0 on 7.x-1.x authored by joelpittet 
    Issue #2827710 by joelpittet: Should set "checked" using .prop() instead...
hansfn’s picture

hansfn CreditAttribution: hansfn as a volunteer commented
Status: Needs review » Fixed

Thx for the patch - committed.

Status: Fixed » Closed (fixed)

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