Amend reference in admin_menu.js to point to

Drupal.settings.admin_menu_margin_top
Drupal.settings.admin_menu_tweak_modules
Drupal.settings.admin_menu_tweak_menu

instead of the current

Drupal.settings.admin_menu.margin_top
Drupal.settings.admin_menu.tweak_modules
Drupal.settings.admin_menu.tweak_menu

I assume this is a mistake from different versions of the code being used. For the live 5.x-2.5 the above change needs to be made.

Comments

sun’s picture

Assigned: wsherliker » Unassigned
Priority: Critical » Normal
Status: Needs review » Postponed (maintainer needs more info)

Sorry, but I've double-checked all instances of this JS variable in the current code and was not able to find a misnamed one. Could you please elaborate further?

sun’s picture

Status: Postponed (maintainer needs more info) » Closed (won't fix)

Without further information I'm obliged to mark this as won't fix.

timrutter’s picture

Version: 5.x-2.5 » 6.x-1.x-dev

Tested with 6.x-1.0 aswell

Error: Drupal.settings.admin_menu is undefined
Source File: http://ajas.org.au/sites/all/modules/admin_menu/admin_menu.js?4
Line: 5

Is causing all javascript after not to load. e.g. interferes with gmap

jcnventura’s picture

Status: Closed (won't fix) » Active

It's not enough to simply change the version, as this was still marked won't fix.

sun’s picture

Status: Active » Postponed (maintainer needs more info)

Yes, but I also need more information, because I cannot replicate this bug.

sun’s picture

Status: Postponed (maintainer needs more info) » Fixed

It may be possible that this bug was not fixed in 6.x-1.0. However, 6.x-1.1 should contain the fix.

Anonymous’s picture

Status: Fixed » Closed (fixed)

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

marcoBauli’s picture

Version: 6.x-1.x-dev » 5.x-2.7

Just updated to latest 5-2.7 version on a website that makes extensive use of google maps, and now the message "Javascript is required to view this map" appears in place of any map.. reverting to older version (5-2.6) gets the maps back again.

Drupal 5.13 and GMap module 5.x-1 here, thank you

jcnventura’s picture

marcoBauli,

You shouldn't be submitting to this thread for two reasons:

1. It's closed
2. Your problem is only slightly related to the original (both involve Javascript)...

marcoBauli’s picture

@jcnventura: since #3 above calls to gmap module conflicts, my intention was to go bit deeper on this before opening a new issue now available at http://drupal.org/node/363091