Feeling Lucky
[#20208] - Fix for: FileLayout::refreshIncludePaths is deprecated, use FileLayout::clearIncludePaths() instead.
- Closed
- 21 Apr 2018
- Medium
- Build: staging
- # 20208
- Diff
- NickSdot:fix-for-20207
User tests: Successful: Unsuccessful:
Pull Request for Issue #20207.
Summary of Changes
setComponent() and setClient() in \Joomla\CMS\Layout\FileLayout now using clearIncludePaths() instead of $this->refreshIncludePaths()
Testing Instructions
- Use LayoutHelper::render() to render a layout.
- Check your deprecated log before and after the patch is applied.
- Functionality should stay the same, but now without deprecated message in log.
Expected result
No more message 'FileLayout::refreshIncludePaths is deprecated, use FileLayout::clearIncludePaths() instead.' in /logs/deprecated.php
Actual result
Message 'FileLayout::refreshIncludePaths is deprecated, use FileLayout::clearIncludePaths() instead.' in /logs/deprecated.php
Documentation Changes Required
No.
| Status | New | ⇒ | Pending |
| Category | ⇒ | Libraries |
| Closed_By | Quy | ⇒ | joomla-cms-bot |
Quy
- | Status | Pending | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2018-04-21 09:34:28 |
| Closed_By | ⇒ | Quy |
joomla-cms-bot
- comment
- 21 Apr 2018
Set to "closed" on behalf of @Quy by The JTracker Application at issues.joomla.org/joomla-cms/20208
This change has already been proposed and reverted in 3.x because changing the method calls breaks subclasses expecting their overridden methods to be called. See #13164 for additional context.
It is already addressed in 4.0 and cannot be patched in 3.x due to B/C.