[#5744] - Articles Related Items - missing "Cache Time"
- Closed
- 17 Jan 2015
- Medium
- Build: staging
- # 5744
- Diff
- pe7er:patch-5
- Success The Travis CI build passed Details
User tests: Successful: Unsuccessful:
See: Extensions > Module Manager > Articles Related Items > Advanced
There's a "Caching" option available but "Cache Time" field is missing.
This PR adds a "Cache Time" field.
jissues-bot
- | Labels |
Added:
?
|
||
brianteeman
- | Category | ⇒ | Modules |
| Category | Modules | ⇒ | Cache Modules |
Maybe a dumb question. When adding this field, is the use of this field ensured?
Thanks for your response. Not a dump question at all...
The caching of Modules is handled by /libraries/cms/module/helper.php
Another Module, Articles Categories, had the same "<field name="owncache" for "Caching" drop down option AND "<field name="cache_time" for "Cache Time".
See /modules/mod_articles_categories/mod_articles_categories.xml
This Articles Related Items Module had the "<field name="owncache" for "Caching" drop down option but was missing the field for "Cache Time". This patch uses the same code as Articles Categories, so it should work. I haven't tested the actual caching (of the other Articles Categories and Articles Related Items plus this patch) myself.
When testing the patch, please also test the caching itself,
and not only the appearance of the field in the back-end.
| Status | Pending | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2015-01-17 16:03:59 |
| Milestone |
Added: |
||
"Cache Time" field is missing
This PR adds a "Cache Time" field.
