?
avatar joomdonation
joomdonation
16 Dec 2015

The initialiseApp method of JApplicationCms class has this command (https://github.com/joomla/joomla-cms/blob/staging/libraries/cms/application/cms.php#L630)

$this->config = JFactory::getConfig();

I think it is a mistake because it will reset all the config data which is initialized before in JApplicationWeb using $this->set method (JApplicationCms extends JApplicationWeb). For example, with the above command, the entire code in this method is useless https://github.com/joomla/joomla-cms/blob/staging/libraries/joomla/application/web.php#L1103-L1191

I tried to remove the mentioned line of code and my local site still works well. Plus, we can now accessing to configuration data intialized in JApplicationWeb with these sample code:

$app = JFactory::getApplication();

echo $app->get('execution.datetime');
echo $app->get('uri.request');
echo $app->get('uri.base.full');

Before these commands return empty.

avatar joomdonation joomdonation - open - 16 Dec 2015
avatar brianteeman brianteeman - change - 16 Dec 2015
Labels Added: ?
avatar mbabker
mbabker - comment - 16 Dec 2015

Busted as it is, that's intentional. There was an attempt to merge both the local config and the config that gets generated when instantiating JApplicationWeb instances but that broke stuff too. So the preference was to keep only the local config. See 9445d5e and relevant links/issues.

avatar joomdonation joomdonation - change - 19 Dec 2015
Status New Closed
Closed_Date 0000-00-00 00:00:00 2015-12-19 10:29:38
Closed_By joomdonation
avatar joomdonation joomdonation - close - 19 Dec 2015

Add a Comment

Login with GitHub to post a comment