Needs review
Project:
Admin:hover
Version:
6.x-1.0
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
11 Oct 2011 at 23:56 UTC
Updated:
12 Oct 2011 at 15:09 UTC
Jump to comment: Most recent file
Was having difficulty getting admin:hover to show hover links on certain pages. Not sure what's special about my page, but I tracked it down to response.status being set to 0 after Drupal.parseJSON was called, despite the raw response clearing showing that status was true.
It looks like Drupal.parseJSON is deprecated anyway, so the attached patches (for both 1.0 and dev) simply use jQuery's internal json parser instead, which seems to fix the issue. The patches are both for admin_hover.js
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | incorrect_json_parsing-1306672-1.patch | 684 bytes | mdeltito |
| admin_hover_json-1.0.patch | 811 bytes | jimkeller | |
| admin_hover_json-dev.patch | 797 bytes | jimkeller |
Comments
Comment #1
mdeltito commentedChange pushed to dev, new patch attached
Comment #2
mdeltito commented