Login!
Lost password?
 

MODx Bug/Feature Tracker and Feature Requests

Welcome to the MODx CMS Tracker. Please choose the appropriate project from the drop down menu and provide as much information as possible regarding your server environment and browser. Thanks!

FS#825 — base_url and site_url error

Attached to Project — MODx
Opened by Susan Ottwell (sottwell) - Wednesday, 28 March 2007, 10:35AM
Last edited by Jason Coward (opengeek) - Sunday, 15 April 2007, 07:38PM
Task Type Bug Report
Category Core Distribution
Status Closed
Assigned To Jason Coward (opengeek)
Operating System All
Severity Critical
Priority Urgent
Reported Version 0.9.6-RC1
Due in Version 0.9.6
Due Date Undecided
Percent Complete 100%

Details

In the latest SVN revision config.inc.php is using $_SERVER[SCRIPT_NAME] to set up the base_url. Problem is that my $_SERVER[SCRIPT_NAME] is /execscriptdir/php.cgi. So The 'execscriptdir' ends up getting added into the base_url and base_path.

Older version seem to have used PHP_SELF, which worked fine.
This task depends upon

This task blocks these from closing
Closed by  Jason Coward (opengeek)
Sunday, 23 September 2007, 01:13PM
Reason for closing:  Fixed
Project Manager action pending. See the History tab for details.
Comment by Susan Ottwell (sottwell) - Thursday, 29 March 2007, 09:56PM
Why can't this be done on installation and inserted into the database as part of the site configuration, instead of being calculated on each page load?

Comment by Jason Coward (opengeek) - Wednesday, 11 April 2007, 10:44AM
Because then you have to run the install anytime you want to relocate it. There is absolutely no reason this should not be auto-calculated in most cases. PHP_SELF should be used however, not SCRIPT_NAME.

Comment by Jason Coward (opengeek) - Sunday, 15 April 2007, 07:38PM
  • Field changed: Status (Assigned → Requires testing)
  • Field changed: Percent Complete (0% → 90%)
A new approach to automatic path configuration (provided by netnoise) that should handle more situations where certain path variables are not set properly, is now available in trunk as of revision 2545.