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.

CommentFileSizeAuthor
#9 admin_jquery_update-1334804-9.patch3.73 KBidflood
#7 admin_jquery_update-1334804-7.patch3.24 KBidflood
#6 1334804-admin-jquery-updated-6.patch2.93 KBpopulist
#3 admin-js.patch3.12 KBartfulrobot
#2 compat_jq_1.6-1334804-2.patch1.88 KBpdrake
#1 compat_jq_1.6-1334804-1.patch2.09 KBMrMaksimize
compat_jq_1.6-issue-comment.patch0 bytesMrMaksimize

Comments

MrMaksimize’s picture

MrMaksimize commented
StatusFileSize
new compat_jq_1.6-1334804-1.patch2.09 KB

Oops that one came up blank. here's a new and improved one.

pdrake’s picture

pdrake commented
StatusFileSize
new compat_jq_1.6-1334804-2.patch1.88 KB

Here is a re-roll of this patch against the latest 6.x-2.x, relative to module root.

artfulrobot’s picture

artfulrobot commented
StatusFileSize
new admin-js.patch3.12 KB

Thanks! 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

dcyuri7’s picture

dcyuri7 commented

thanks this helped me :)

jackbravo’s picture

jackbravo commented
Version: 6.x-2.x-dev » 7.x-2.0-beta3
Status: Active » Reviewed & tested by the community

Yes this works for me with jquery_update and admin. Without this patch I get "Syntax error, unrecognized expression: [href=/admin]".

populist’s picture

populist commented
StatusFileSize
new 1334804-admin-jquery-updated-6.patch2.93 KB

Here is a quick reroll of #3 to cleanly apply. I also tested this and think its great.

idflood’s picture

idflood commented
Status: Reviewed & tested by the community » Needs review
StatusFileSize
new admin_jquery_update-1334804-7.patch3.24 KB

Patch 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.

idflood’s picture

idflood commented
Version: 7.x-2.0-beta3 » 7.x-2.x-dev
idflood’s picture

idflood commented
StatusFileSize
new admin_jquery_update-1334804-9.patch3.73 KB

New patch to make the modification compatible with jquery 1.4 and >= 1.6.

sickness29’s picture

sickness29 commented
Issue summary: View changes
Status: Needs review » Closed (outdated)

D7 reached its EOL back in January 2025, and there is no active release for D7 for this module anymore.
Development or support is not planned for D7. All D7-related issues are marked as outdated in a bunch.

Everyone can apply the patch above (not tested by the maintainers, tested by the community) to their D7 projects.
If the issue remains relevant for D10+ versions, merge requests with proposed solutions for a new module version (D10+) are welcome in a new follow-up issue.

Thanks!

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.