Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Attached is a patch. This does actually break compatibility with earlier jquery versions because it uses a method called .prop. I'm working on a backwards compatible patch.
Comment | File | Size | Author |
---|---|---|---|
#9 | admin_jquery_update-1334804-9.patch | 3.73 KB | idflood |
#7 | admin_jquery_update-1334804-7.patch | 3.24 KB | idflood |
#6 | 1334804-admin-jquery-updated-6.patch | 2.93 KB | populist |
#3 | admin-js.patch | 3.12 KB | artfulrobot |
#2 | compat_jq_1.6-1334804-2.patch | 1.88 KB | pdrake |
Comments
Comment #1
MrMaksimize CreditAttribution: MrMaksimize commentedOops that one came up blank. here's a new and improved one.
Comment #2
pdrake CreditAttribution: pdrake commentedHere is a re-roll of this patch against the latest 6.x-2.x, relative to module root.
Comment #3
artfulrobot CreditAttribution: artfulrobot commentedThanks! I'd found and fixed the missing quotes around the href selector, but missed the prop().
Here's my patch which includes the href fix, the prop fix, and adds a fix for the breadcrumbs mechanism (which used on a workaround for jquery 1.2 that didn't seem necessary with 1.7)
Against: 7.x-2.0-beta3
Comment #4
dcyuri7 CreditAttribution: dcyuri7 commentedthanks this helped me :)
Comment #5
jackbravo CreditAttribution: jackbravo commentedYes this works for me with jquery_update and admin. Without this patch I get "Syntax error, unrecognized expression: [href=/admin]".
Comment #6
populist CreditAttribution: populist commentedHere is a quick reroll of #3 to cleanly apply. I also tested this and think its great.
Comment #7
idflood CreditAttribution: idflood commentedPatch in #6 doesn't apply cleanly on 7.x-2.x so I manually rerolled it. There was two lines about the href which were already ok and I added a fix in admin.devel.js.
Comment #8
idflood CreditAttribution: idflood commentedComment #9
idflood CreditAttribution: idflood commentedNew patch to make the modification compatible with jquery 1.4 and >= 1.6.