I few thoughts on this I thought I’d mention.
- The browser support described for IE7 is not quite consistent between the roadmap, server requirements, and the design goals page.
- I didn’t see any mention of permissions for individual elements (maybe I missed it).
- In looking at the current code, it appears that the name of the root index.php is hardcoded in a number of places, is that likely to change? Someone might like to rename it. Also, it occurred to me that an Evo site and a Revo site might be able to coexist in the same directory if they had different index files, a moved or renamed manager directory (in Revo), and different table prefixes.
-
MODX Staff
- 10,725 Posts
That’s all Evo code. Isn’t this Revo we’re talking about?
Quote from: OpenGeek at May 14, 2010, 05:40 PM
That’s all Evo code. Isn’t this Revo we’re talking about?
Oops. That explains why I couldn’t find the ones I saw before.
Here are some Revo examples:
/* transport.core.php */
$attributes['resolve'][] = array (
'type' => 'file',
'source' => MODX_BASE_PATH . 'index.php',
'target' => "return MODX_BASE_PATH;",
);
require_once dirname(dirname(__FILE__)).'/index.php'; C:\xampp\htdocs\MODx097\core\packages\docmanager-2.1-beta6\modMenu\151749bd562431eec9f839f7eebd66c7\1\docmanager\processors\docperms\apply.php (16:43)
'handler' => 'new Function("this.redirect(\"index.php?a='.$actions['resource/create'].'&class_key=modStaticResource\");");', C:\xampp\htdocs\MODx097\core\model\modx\processors\resource\gettoolbar.php (46:53)
require_once dirname(dirname(dirname(__FILE__))).'/index.php'; C:\xampp\htdocs\MODx097\connectors\workspace\package\index.php (2:52)
require_once dirname(dirname(dirname(__FILE__))).'/index.php'; C:\xampp\htdocs\MODx097\connectors\system\import\html.php (2:52)
-
MODX Staff
- 10,725 Posts
And that looks like a component, not the core. If the component is referring to index.php in the root of the site, then that’s not valid and should be reported as a bug in the appropriate JIRA project. Referring to the index.php in the MODX_MANAGER_URL or MODX_MANAGER_PATH is valid however.
The first line is from DocManager, but I think the second one is part of the core and the last two are connectors that might be part of MODx.
-
MODX Staff
- 10,725 Posts
That DocManager one is a bug; the rest are valid and are not referring to the main request_controller.