The AJAX cache request url clashes with urls for the JS module, since JS intercepts all requests to paths starting with js/. This currently results in a 404 and no admin menu displayed. Adding hook_js() solves this issue (and you get a speedup for free ;).

CommentFileSizeAuthor
#1 admin_menu-no_js.patch1.99 KBsmk-ka
admin_menu.module-js.patch808 bytessmk-ka

Comments

smk-ka’s picture

smk-ka commented
Title: Compatibility with js.module » Clashes with js.module URLs
StatusFileSize
new admin_menu-no_js.patch1.99 KB

This actually cries for trouble because you never know which additional dependencies exist to build the admin menu anew. Better get rid of the js/ path prefix altogether.

sun’s picture

sun
Primary language German
Location Karlsruhe
commented

The idea in the existing issue #420812: Implement hook_js() was that we can serve the menu very fast via hook_js() if the client forgot about its cache, but the cached menu is still available on the server-side - which certainly can happen often.

Only if the menu needs to be rebuild, we do a full bootstrap. That approach worked flawlessly in one of our recent projects (the dark one ;).

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Status: Needs review » Closed (duplicate)