?
avatar joeforjoomla
joeforjoomla
6 Nov 2016

Steps to reproduce the issue

Use JApplicationSite getParams('com_mycomponent') method for a certain component in the frontend, having a valid routed menu item.

Expected result

All component params are merged with the specific menu item parameters in the same JRegistry object
If you have a menu item parameter 'myparam' and you call:
$registry = $this->app->getParams('com_mycomponent') ;

You always had:
$myParam = $registry->get('myparam');

Actual result

Menu item parameters are not merged as properties of the JRegistry object that can be obtained using the JRegistry 'get' method, but instead menu items parameters are all included in an array property named 'data'.
If you have a menu item parameter 'myparam' and you call:
$registry = $this->app->getParams('com_mycomponent') ;

You have:
$menuParameters = $registry->get('data');
$myParam = $menuParameters['myparam'];

System information (as much as possible)

Issue is due to the change at root/libraries/cms/application/site.php line 355 from:
$temp = new Registry;
$temp->loadString($menu->params);

to

$temp = new Registry($menu->params);

Additional comments

avatar joeforjoomla joeforjoomla - open - 6 Nov 2016
avatar joomla-cms-bot joomla-cms-bot - change - 6 Nov 2016
Labels Added: ?
avatar zero-24
zero-24 - comment - 6 Nov 2016

Please send changes as pull requests that made it easy for tests. Thanks..

avatar joeforjoomla
joeforjoomla - comment - 6 Nov 2016

Ok PR opened

avatar zero-24 zero-24 - change - 6 Nov 2016
Status New Closed
Closed_Date 0000-00-00 00:00:00 2016-11-06 16:21:55
Closed_By zero-24
avatar zero-24 zero-24 - close - 6 Nov 2016

Add a Comment

Login with GitHub to post a comment