Sitemap.js prevents use of /admin in subdirectory (was Re: H

Earl C. [email protected] 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.

Earl,

Could you submit a patch for that to the Trac?

Sean

Sean C. wrote:

Could you submit a patch for that to the Trac?

Submitted as http://dev.radiantcms.org/radiant/ticket/509

Earl

Earl wrote:

Sean C. wrote:

Could you submit a patch for that to the Trac?

Is there a page describing the pre-requisites and actions for doing
so?

Yes:

http://dev.radiantcms.org/radiant/wiki#Contributions


John L.
http://wiseheartdesign.com

Sean C. wrote:

Could you submit a patch for that to the Trac?

Is there a page describing the pre-requisites and actions for doing
so?

Earl