Skip to content
This repository was archived by the owner on Sep 3, 2026. It is now read-only.

Fix EZP-23421: Add support for static cache handler system in rss import cronjob part - #1086

Merged
yannickroger merged 1 commit into
ezsystems:masterfrom
brookinsconsulting:static-cache-handler-support
Oct 13, 2014
Merged

Fix EZP-23421: Add support for static cache handler system in rss import cronjob part#1086
yannickroger merged 1 commit into
ezsystems:masterfrom
brookinsconsulting:static-cache-handler-support

Conversation

@brookinsconsulting

Copy link
Copy Markdown
Contributor

Implemented: EZP-23421: Add support for static cache handler system in rss import cronjob part

The problem

Hello,

Recently we tried on a new project to implement delayed to cronjob static caching support for rssimport cronjob imported content. We also implemented a custom static cache handler class and settings changes which work well except in one regard, rssimport of content.

Results were when using rssimport cronjob part the requests to update static cache via delayed to cronjob feature using the staticcache_cleanup cronjob part were not being stored.

This is because the rssimport cronjob part does not support the standard static cache handler system that the rest of the kernel supports. We traced the problem in code to one line 'eZStaticCache::executeActions();' in the cronjobs/rssimport.php file.

The solution

We then wrote replacement code to enable support for the static cache handler system. See pull request file changes diff.

We have created a new issue ticket about this feature request, you can find it here: https://jira.ez.no/browse/EZP-23421

Please review and let us know what you think.

Thank you for your continued support!

Cheers,
Brookins Consulting

@andrerom

andrerom commented Oct 6, 2014

Copy link
Copy Markdown
Contributor

+1

@brookinsconsulting

Copy link
Copy Markdown
Contributor Author

Hello @andrerom

Thank you very much for your support. It really means a lot to us.

Please let us know what if anything we can do to help expedite getting these changes into eZ Publish.

Cheers,
Brookins Consulting

@lolautruche lolautruche changed the title Implemented: EZP-23421: Add support for static cache handler system in rss import cronjob part Fix EZP-23421: Add support for static cache handler system in rss import cronjob part Oct 7, 2014
Comment thread cronjobs/rssimport.php Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Extra empty line

@lolautruche

Copy link
Copy Markdown
Contributor

+1 with empty lines removed 😉

@brookinsconsulting
brookinsconsulting force-pushed the static-cache-handler-support branch from dba7a06 to 132d2ca Compare October 7, 2014 09:46
@brookinsconsulting
brookinsconsulting force-pushed the static-cache-handler-support branch from 132d2ca to cac8ca0 Compare October 7, 2014 09:49
@brookinsconsulting

Copy link
Copy Markdown
Contributor Author

Hello @lolautruche

Thank you very much for your feedback and review!

We have removed the extra lines, squashed the commits, changed the commit message to match your PR title change and pushed up our branch changes for final review and merging.

We think we did this correctly. Please let us know if we did anything wrong or need to do anything more. This is our second time squashing commits and our first time amending commits :)

Thank you again for your continued support!

Cheers,
Brookins Consulting

yannickroger added a commit that referenced this pull request Oct 13, 2014
…-support

Fix EZP-23421: Add support for static cache handler system in rss import cronjob part
@yannickroger
yannickroger merged commit d47c54b into ezsystems:master Oct 13, 2014
@yannickroger

Copy link
Copy Markdown
Contributor

Thanks @brookinsconsulting

@brookinsconsulting
brookinsconsulting deleted the static-cache-handler-support branch March 2, 2015 07:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants