Path in MODx config files

[eprochasson]

eprochasson created Redmine issue ID 7366

The config files generated by MODx (/config.core.php, /manager/config.core.php, /connectors/config.core.php and /core/config/config.inc.php) includes absolute path.

As it works perfectly with a new install, I assume I'm not the only one who have a local version for development and a live version on a different server, with different config files (because different paths).

Is there a reason to not use automatically generated path in those config files, assuming most of the developer use the standard MODx file structure anyway.

For example, /config.core.php can be:

That makes the deployment of any website way easier (you just have to change the database information). Power user with a specific file structure can still change those information manually if required.

[eprochasson]

eprochasson submitted:

I modified the template config file and the resolvers for building the transport package to have relative path config files. However, that will conflict with the new setup process (which allow to change paths) as it will always use the default, relative path. I therefore doubt that it can be merged, but in case, it's here.

In my opinion, the case where one moves a modx installation to a different server (I typically develop locally then deploy to a remote server with different paths) is way more common than someone having special path needs, but I might be wrong.

Commits are available on github, for the resolvers:
eprochasson@ac762ca
For the config file:
eprochasson@bc90ca6

Thank you !

[modxbot]

everettg_99 submitted:

Yes, simplified configs would be helpful.

1. Use relative paths when possible.
2. Update config.inc.php to avoid needless hard-coded paths. E.g. can't we always leave the MODX_CORE_PATH setting to be relative because the config.inc.php always lives in the core directory? Surely it can find itself:

if (!defined('MODX_CORE_PATH')) {
    $modx_core_path= dirname(dirname(__FILE__)).'/';
    define('MODX_CORE_PATH', $modx_core_path);
}

[whitebyte]

Core config now uses relative pathes AFAIK, and pull requests are no more actual, 2.3+ doesn't have _build/resolvers/resolve.connectors.php at all, so suggest closing

[eprochasson]

That, or the fact that the question was raised in 2012.

[whitebyte]

We just trying to sort out old issues that was imported from redmine

[OptimusCrime]

I would +1 for closing. No longer relevant.

[whitebyte]

Also, #9252 is possible duplicate and should be closed too

[adamwintle]

This is no longer relevant @modxbot close