Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

EZP-31768: Fixed cache invalidation during url alias reassignment #3062

Merged
merged 4 commits into from
Aug 28, 2020
Merged

EZP-31768: Fixed cache invalidation during url alias reassignment #3062

merged 4 commits into from
Aug 28, 2020

Conversation

barw4
Copy link
Member

@barw4 barw4 commented Aug 27, 2020 •
edited
Loading

Question Answer
JIRA issue EZP-31768
Bug yes
New feature no
Target version 7.5
BC breaks no
Tests pass yes
Doc needed no

During reassigning of the URL Alias belonging to a different Content cached alias would still point to that previous Content.

TODO:

  • Implement feature / fix a bug.
  • Fix new code according to Coding Standards ($ composer fix-cs).
  • Ask for Code Review.

@barw4 barw4 added the Bug label Aug 27, 2020
@barw4 barw4 self-assigned this Aug 27, 2020
@barw4 barw4 changed the title EZP-31768: Fixed cache invalidation during url alias reassignment [WIP] EZP-31768: Fixed cache invalidation during url alias reassignment Aug 27, 2020
@barw4 barw4 changed the title [WIP] EZP-31768: Fixed cache invalidation during url alias reassignment EZP-31768: Fixed cache invalidation during url alias reassignment Aug 27, 2020
@barw4 barw4 requested a review from andrerom August 27, 2020 13:19
@tomaszszopinski tomaszszopinski self-assigned this Aug 28, 2020
tomaszszopinski
tomaszszopinski approved these changes Aug 28, 2020
View reviewed changes
Copy link

@tomaszszopinski tomaszszopinski left a comment

Choose a reason for hiding this comment

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

QA approved on eZPlatform-ee 2.5.12 with path & diff.

@lserwatka lserwatka merged commit 06fb92a into 7.5 Aug 28, 2020
@lserwatka lserwatka deleted the EZP-31768 branch August 28, 2020 15:27
@lserwatka
Copy link
Member

You can merge it up.

@barw4
Copy link
Member Author

barw4 commented Aug 28, 2020 •
edited
Loading

Merged into 1.1: ezsystems/ezplatform-kernel@70fd43d
Merged into master: ezsystems/ezplatform-kernel@5eb031a

# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@tomaszszopinski tomaszszopinski tomaszszopinski approved these changes

@ViniTou ViniTou ViniTou approved these changes

@andrerom andrerom andrerom approved these changes

@adamwojs adamwojs adamwojs approved these changes

@lserwatka lserwatka lserwatka approved these changes

@alongosz alongosz Awaiting requested review from alongosz

Assignees

@barw4 barw4

@tomaszszopinski tomaszszopinski

Labels
Bug QA approved
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@barw4 @lserwatka @adamwojs @andrerom @ViniTou @tomaszszopinski