Feeling Lucky
?
[#13918] - Make JControllerLegacy::display() resilient to cache errors
- Fixed in Code Base
- 8 Feb 2017
- Medium
- Build: staging
- # 13918
- Diff
- mbabker:controller-display-cache
User tests: Successful: Unsuccessful:
Partial Pull Request for Issue #13357
Summary of Changes
Similar to #13524 this tries to make the JControllerLegacy class resilient to cache related issues.
- The call to display the component is wrapped in a try/catch so a cache error doesn't result in an error displaying the page, on a cache exception the view will directly be called similar to if caching isn't enabled.
Testing Instructions
If the cache store fails to connect or the cache configuration is bad, the exceptions thrown by the cache API should be caught and JControllerLegacy::display() won't cause an uncaught exception from the cache API to bubble up the call stack. When processing from the cache fails here, the view should still attempt to be displayed.
Documentation Changes Required
N/A
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Libraries |
laoneo
- comment
- 6 Feb 2017
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/13918.
| Milestone |
Added: |
||
| Status | Pending | ⇒ | Ready to Commit |
jeckodevelopment
- comment
- 6 Feb 2017
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/13918.
rdeutz
- | Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2017-02-08 14:07:42 |
| Closed_By | ⇒ | rdeutz | |
| Labels |
Added:
?
|
||
I have tested this item✅ successfully on 7e50fae
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/13918.