?
Referenced as Related to: # 12226
avatar izharaazmi
izharaazmi
5 Nov 2016

https://github.com/joomla/joomla-cms/blob/staging/components/com_config/controller/modules/save.php#L84

The variables $app and $data aren't defined in scope. I don't know the context hence could not attempt a fix. May be someone can do that.

PS: Code review based only.

avatar izharaazmi izharaazmi - open - 5 Nov 2016
avatar joomla-cms-bot joomla-cms-bot - change - 5 Nov 2016
Labels Added: ?
avatar nibra nibra - change - 16 Mar 2017
Category com_config
avatar nibra nibra - change - 16 Mar 2017
Status New Confirmed
avatar mbabker
mbabker - comment - 25 May 2017

$app should be $this->app.

As for $data, because of how this task chains into a com_modules controller and looking at that method's code, it looks like that should be replaced by $this->input->post->get('jform', array(), 'array');.

avatar franz-wohlkoenig franz-wohlkoenig - change - 8 Nov 2017
Status Confirmed Information Required
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 8 Nov 2017

@izharaazmi any Update on above Comment?


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/12768.

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 26 Nov 2017

If this Issue get no Response, it will be closed at 26th December 2017.

avatar franz-wohlkoenig franz-wohlkoenig - change - 6 Dec 2017
Status Information Required Closed
Closed_Date 0000-00-00 00:00:00 2017-12-06 15:56:21
Closed_By franz-wohlkoenig
avatar joomla-cms-bot joomla-cms-bot - change - 6 Dec 2017
Closed_By franz-wohlkoenig joomla-cms-bot
avatar joomla-cms-bot joomla-cms-bot - close - 6 Dec 2017
avatar joomla-cms-bot
joomla-cms-bot - comment - 6 Dec 2017
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 6 Dec 2017

closed as having Pull Request #19003

Add a Comment

Login with GitHub to post a comment