Joomla! Issue Tracker - CMS

Download
Launch
Feeling Lucky

[#22448] - [4.0] Use the injected application and db in the update notification plugin

? Pending

User tests: Successful: Unsuccessful:

avatar laoneo
laoneo
2 Oct 2018

Use the injected application and db in the update notification plugin.

avatar laoneo laoneo - open - 2 Oct 2018
avatar laoneo laoneo - change - 2 Oct 2018
Status New ⇒ Pending
avatar joomla-cms-bot joomla-cms-bot - change - 2 Oct 2018
Category ⇒ Front End Plugins
avatar laoneo laoneo - change - 2 Oct 2018
Labels Added: ?
avatar SharkyKZ
SharkyKZ - comment - 2 Oct 2018

Remove Factory::getConfig() here too please:

joomla-cms/plugins/system/updatenotification/updatenotification.php

Lines 247 to 250 in b7f34ad

$jConfig = Factory::getConfig();
$sitename = $jConfig->get('sitename');
$mailFrom = $jConfig->get('mailfrom');
$fromName = $jConfig->get('fromname');

avatar laoneo
laoneo - comment - 2 Oct 2018

@SharkyKZ good catch, changed it.

avatar wilsonge wilsonge - change - 14 Oct 2018
Status Pending ⇒ Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 ⇒ 2018-10-14 22:58:51
Closed_By ⇒ wilsonge
avatar wilsonge wilsonge - close - 14 Oct 2018
avatar wilsonge wilsonge - merge - 14 Oct 2018
avatar wilsonge
wilsonge - comment - 14 Oct 2018

Thanks!

Add a Comment

Login with GitHub to post a comment