Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Dispatch the CACHE_CLEAR_CONTENT events from the Legacy #57

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Dispatch the CACHE_CLEAR_CONTENT events from the Legacy #57

wants to merge 1 commit into from

Conversation

Plopix
Copy link
Contributor

@Plopix Plopix commented Jun 2, 2016

It seems that the CACHE_CLEAR_CONTENT event is not trigger from the legacy Bridge.
https://doc.ez.no/display/EZP/Smart+HttpCache+clearing

The configuration mapper calls purge
https://github.com/ezsystems/LegacyBridge/blob/master/bundle/LegacyMapper/Configuration.php#L205

And the purge does not dispatch the events:
https://github.com/ezsystems/ezpublish-kernel/blob/master/eZ/Publish/Core/MVC/Symfony/Cache/Http/InstantCachePurger.php#L54
Note sure about the if ($locationId != 1) but seems needed.

Let me kow!

@Plopix Plopix changed the title Triggers the events from the Legacy Dispatch the CACHE_CLEAR_CONTENT events from the Legacy Jun 2, 2016
@Plopix
Copy link
Contributor Author

Plopix commented Jun 2, 2016

Ok after few other tests it is prob not the good way to do it for 2 reasons:

It is a purely bridge issue, any idea @bdunogier @andrerom @lolautruche @emodric ?

@andrerom
Copy link
Contributor

andrerom commented Jun 2, 2016

There are other issue around, so actually the cache clearing system is kind of broken, it assume content still exists, which is not the case when you delete/trash it. So recently been investigating doing it differently and/or deprecating that system.

Also the reason legacy bridge uses purge is that it is supposed to triggered by cache clearing events in legacy that has already resolved affected nodes.


So in summary: @Plopix better if you describe the issue you have, as this does not seem like the right approach.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants