User tests: Successful: Unsuccessful:
Partial Pull Request for Issue #13357
Similar to #13524 this tries to make the JComponentHelper
class resilient to cache related issues.
The database query is now fully stored and executed in a lambda function, this lambda function is passed to the cache controller instead of only instructing it to run JDatabaseDriver::loadObjectList()
Cache exceptions are caught and the lambda function will attempt to be run directly instead.
On a database failure, we're out of luck; sorry.
If the cache store fails to connect or the cache configuration is bad, the exceptions thrown by the cache API should be caught and JComponentHelper::load()
try to manually execute the lambda function to load the data. The method currently catches all thrown RuntimeException
objects but will now only catch JCacheException
objects; in the case of a database error (all the database exceptions extend RuntimeException
), we've got a bigger issue and this method shouldn't try to act as an error handler for that type of failure and let the application keep running.
N/A
Status | New | ⇒ | Pending |
Category | ⇒ | Libraries |
Labels |
Added:
?
|
I have tested this item
Milestone |
Added: |
||
Status | Pending | ⇒ | Ready to Commit |
Labels |
Added:
?
|
Status | Ready to Commit | ⇒ | Fixed in Code Base |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2017-02-08 14:06:39 |
Closed_By | ⇒ | rdeutz | |
Labels |
I have tested this item✅ successfully on 2a90389
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/13916.