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

Fix EZP-22360: clear use role cache when moving locations from legacy #744

Merged

Conversation

5 participants
@joaoinacio
Copy link
Contributor

commented Feb 20, 2014

JIRA: https://jira.ez.no/browse/EZP-22360

When moving a user to a different group, in legacy stack, stash caches are not cleared, which means new permissions are not taken into account.
This clears the user role assignments for the given location when clearing it's content cache as well,
an alternative would be to attach the 'content/cache' event to a different handler as well.

@lolautruche

This comment has been minimized.

Copy link
Contributor

commented Feb 20, 2014

+1

Review ping @andrerom

@mloureiro

This comment has been minimized.

Copy link
Contributor

commented Feb 20, 2014

Tested and it's passing
+1

@yannickroger

This comment has been minimized.

Copy link
Contributor

commented Feb 21, 2014

Can you rebase against master. Travis test have been fixed.

@joaoinacio

This comment has been minimized.

Copy link
Contributor Author

commented Feb 21, 2014

Rebased to master.

@yannickroger

This comment has been minimized.

Copy link
Contributor

commented Feb 21, 2014

+1 ping @andrerom for a final call

@andrerom

This comment has been minimized.

Copy link
Member

commented Feb 21, 2014

+1

yannickroger added a commit that referenced this pull request Feb 24, 2014

Merge pull request #744 from joaoinacio/EZP-22360_usergroup_role_cache
Fix EZP-22360: clear use role cache when moving locations from legacy

@yannickroger yannickroger merged commit 158e9d9 into ezsystems:master Feb 24, 2014

1 check passed

default The Travis CI build passed
Details
@yannickroger

This comment has been minimized.

Copy link
Contributor

commented Feb 24, 2014

Thanks a lot @joaoinacio

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.