Many sites now use various means to update their version of jQuery to a later version than 1.4, the version in Drupal 7.

jQuery deprecated (version 1.6) and removed (version 1.9) the use of .attr() for use to set "checked" and "disabled" properties. It also points out that attempting to retrieve values by using .attr() will result in confusion/problems, as in this example: $elem.attr("checked"); will return true if $elem has a "checked" property at all, not if it is true.

jQuery now uses .prop() instead to set and retrieve the value of a "checked" or "disabled" properties of a node.

.removeAttr() may also cause problems/unexpected behavior, as it completely removes the property rather than setting it to false, consequently it will no longer be selectable by selectors that typically would work. 
Example: 
Using <code>jQuery('input[name="test"]').removeAttr('checked');

on <input name="test" type="checkbox" checked="true" /> will result in <input name="test" type="checkbox" />. This is typically not the intended result, as it will cause CSS and jQuery selectors that you would expect to be able to find that item to fail.
Example: jQuery('input[checked]') will now fail to return this node.

See: jQuery 1.9 Upgrade Guide: attr vs. prop

It looks like the jQuery Update module's newest dev branch (7.x-3.x-dev) is including an option via a patch for using jQuery Migrate, which fixes compatibility with legacy jQuery code in newer version of jQuery. However, since most users will not move to that branch until it is out of dev, either a dependency on certain versions of jQuery and possibly certain settings in jQuery update need to be addressed, or the code in question should be made to work with all versions of jQuery.

file: media/js/media.admin.js

  1. Line 50: $('.media-display-thumbnails', $(this).parents('form')).find(':checkbox').attr('checked', true).change();
  2. Line 55: $('.media-display-thumbnails', $(this).parents('form')).find(':checkbox').attr('checked', false).change();
  3. Line 72: checkbox.attr('checked', false).change();
  4. Line 74: checkbox.attr('checked', true).change();

Comments

DrCord’s picture

Title: media: jQuery .attr() deprecated/removed for use with "checked" and "disabled" properties » jQuery .attr() deprecated/removed for use with "checked" and "disabled" properties
dave reid’s picture

Version: 7.x-1.4 » 7.x-2.x-dev
Category: Bug report » Feature request

So how do we support this considering jQuery.prop() was added in 1.6, but Drupal 7 ships with 1.4 by default?

DrCord’s picture

There seem to be a few ways of going about this:

  1. Detect the version of jQuery in use and use code that works with that, this seems like it would add a lot of code doing the same thing, but for different versions of jQuery - YUCK!
  2. Force a full dependency on the jQuery update module and and a >X version of jQuery and update all code to use the newer version
  3. If using the jQuery update module, require the 3.x branch, as it includes jQuery migrate - not sure if feasible or possible to have a module somewhat dependent...
  4. Include the jQuery migrate plugin if the detected version of jQuery is not the expected one from original core and the migrate plugin isn't already included

My first inclination is that #4 sounds the most feasible.

There could be additional options or combinations of these that are better, I am just trying to help the discussion get started and once we have a viable solution, potentially help roll patches.

DrCord’s picture

I reported a similar bug on the imagecache_actions module queue and a user submitted a great looking patch that switched the code to the form #states API, allowing drupal to handle it natively, which should work great...whatever version of jQuery you have gets used then...

imagecache_actions solution/patch

chris matthews’s picture

Status: Active » Closed (outdated)

Closing this issue as outdated. However, if you think this issue is still important, please let us know and we will gladly re-open it for review.
sincerely,
- the Drupal Media Team

chris matthews’s picture

Closing this issue as outdated. However, if you think this issue is still important, please let us know and we will gladly re-open it for review.
sincerely,
- the Drupal Media Team