?
avatar NickSdot
NickSdot
21 Apr 2018

Constructor in Joomla\CMS\Layout\FileLayout uses setComponent() and setClient() which both using the deprecated $this->refreshIncludePaths().

This is 'spamming' the /log/deprecated.php for no reason.

Both methods should use the new clearIncludePaths() which is already used by the old $this->refreshIncludePaths().

Steps to reproduce the issue

Use LayoutHelper::render()to render a layout.

Expected result

Use new clearIncludePaths() and get no deprecated messages in log.

Actual result

Use old refreshIncludePaths() which results in deprecated messages in log.

System information (as much as possible)

J3.8.7

Additional comments

I'll make a Pull Request in a bit.

avatar NickSdot NickSdot - open - 21 Apr 2018
avatar joomla-cms-bot joomla-cms-bot - change - 21 Apr 2018
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 21 Apr 2018
avatar franz-wohlkoenig franz-wohlkoenig - change - 21 Apr 2018
Status New Closed
Closed_Date 0000-00-00 00:00:00 2018-04-21 05:28:45
Closed_By franz-wohlkoenig
avatar joomla-cms-bot joomla-cms-bot - change - 21 Apr 2018
Closed_By franz-wohlkoenig joomla-cms-bot
avatar joomla-cms-bot joomla-cms-bot - close - 21 Apr 2018
avatar joomla-cms-bot
joomla-cms-bot - comment - 21 Apr 2018
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 21 Apr 2018

closed as having Pull Request #20208.

Add a Comment

Login with GitHub to post a comment