[Radiant] sitemap.js prevents use of /admin in subdirectory (was Re: HTTP_X_FORWARDED_HOST breaks Apache proxying)

[Earl Chew]

Earl Chew <earl_chew at yahoo.com> wrote: 
There is a patch for Apache to add ProxyAddXHeaders:

http://mail-archives.apache.org/mod_mbox/httpd-dev/200702.mbox/browser

Setting this option Off causes Apache to refrain from inserting the HTTP_X_* headers making the proxying action transparent.
After applying this, I was stumped for a while when trying to use /admin to edit the site. Things appear to work properly except that clicking on the page expander (the + to see the child pages) would result in file not found /admin/ui/pages/children/...

I finally found this in public/javascripts/sitemap.js where:

    new Ajax.Updater(
      row,
      '/admin/ui/pages/children/' + id + '/' + level,

assumes the script is in the toplevel directory. I couldn't think of any way of fixing this without modifying the file. My fix was to make this path relative:

    new Ajax.Updater(
       row,
       '../admin/ui/pages/children/' + id + '/' + level,
 
I also see document.cookie saves the path=/admin, but I can't see this used anywhere.

       
---------------------------------
Building a website is a piece of cake. 
Yahoo! Small Business gives you all the tools to get online.