Feeling Lucky
?
[#6796] - Remove setting the lifetime value as it is already set.
- Closed
- 27 Mar 2016
- Medium
- Build: staging
- # 6796
- Diff
- roland-d:cache-lifetime
- Success The Travis CI build passed Details
Pull Request for
# 2539
Referenced as
Related to:
# 2539
User tests: Successful: Unsuccessful:
The lifetime was forced but not changed to minutes, so the cache time would become seconds. Since the lifetime is already set in the constructor, it is not needed to force it in the store() method.
This fixes #2539 as proposed by @ollyja
Test Instructions
Please see the test instructions provided by @ollyja here: http://issues.joomla.org/tracker/joomla-cms/2539#event-71926
After applying the patch the cache file e3ca4ca329e96b6bd2cae4c39f5bcc88-cache-com_test-6ab76d843e7ecfda29171a181cbcd76a.php should not have a changed timestamp.
| Rel_Number | ⇒ | 2539 | |
| Relation Type | ⇒ | Pull Request for |
zero-24
- | Category | ⇒ | Cache Libraries |
| Status | New | ⇒ | Pending |
| Status | Pending | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2016-03-27 19:15:34 |
| Closed_By | ⇒ | brianteeman |
Closing as this has been addressed in a larger rewrite of cache see #9622