Remove @ from selectors

jcmarco - April 17, 2009 - 08:49
Project:Community Tags
Version:6.x-1.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:active
Description

Related with issue #358082: jQuery 1.3 in Drupal 6.x

To help Community Tags work with jQuery 1.3.2 in jQuery Update can we please remove the @ in the selectors. It was depreciated as of 1.2 and removed in 1.3. Change highlighted below.

jQuery 1.2.x:
$("a[@href*='admin/build/testing']").doSomething();

jQuery 1.2.6 and 1.3.x:
$("a[href*='admin/build/testing']").doSomething();

More detail at http://docs.jquery.com/Selectors/attributeHas#attribute

AttachmentSize
community_tags_js.patch533 bytes

#1

neurovation.kiwi - August 19, 2009 - 14:31
Status:needs review» reviewed & tested by the community

reviewed - ok and needed if you use jQuery Update

By the way:
the "@" syntax is already deprecated in jQuery 1.2.x

#2

entrigan - August 27, 2009 - 06:44

thanks for catching this, I will commit tomorrow.

#3

entrigan - August 28, 2009 - 21:59
Status:reviewed & tested by the community» fixed

committed

#4

System Message - September 11, 2009 - 22:00
Status:fixed» closed

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

#5

valderama - November 2, 2009 - 17:14
Status:closed» active

this patch should also be applied for community tags version 6.x-1.0-beta2

best,
walter

 
 

Drupal is a registered trademark of Dries Buytaert.