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

EZP-30493: Fix cache not cleared on node visibility change #1359

Merged
merged 2 commits into from
Apr 29, 2019

Conversation

ITernovtsii
Copy link
Contributor

@ITernovtsii ITernovtsii commented Apr 6, 2018

Fix issue with cache not cleared on node visibility change.
Search index takes wrong visibility status from cache.

Todo:

@andrerom
Copy link
Contributor

andrerom commented Aug 1, 2018

Updated PR here @diaborn19, would you be able to move the changes either with the two commits or just one by you to 2017.12 branch?

@andrerom andrerom requested a review from emodric August 2, 2018 09:42
@ITernovtsii ITernovtsii changed the base branch from master to 2018.09 April 11, 2019 05:26
@ITernovtsii
Copy link
Contributor Author

Hi @andrerom
Sorry, missed your comment, changed to 2018.09

@andrerom
Copy link
Contributor

andrerom commented Apr 11, 2019

I meant 2017.12 here, it's the release used with eZ Platform 1.13.x, so the oldest branch here that receives fixes for the time being. But if you prefer I can adapt this on merge.

But can you report JIRA issue? Or is there one already?

@ITernovtsii ITernovtsii changed the base branch from 2018.09 to 2017.12 April 23, 2019 07:51
@ITernovtsii
Copy link
Contributor Author

Hi, EZP-30493 created and base branch switched to 2017.12. Thanks.

@andrerom andrerom added this to the 2019.03 milestone Apr 29, 2019
@andrerom andrerom merged commit 078fa45 into ezsystems:2017.12 Apr 29, 2019
@andrerom andrerom changed the title FIX cache not cleared on node visibility change EZP-30493: Fix cache not cleared on node visibility change Apr 29, 2019
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.

3 participants