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

Index correct ghost content when deleting the locale of an article #652

Merged

Conversation

wachterjohannes
Copy link
Member

Q A
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Fixed tickets fixes #issuenum
Related issues/PRs #issuenum
License MIT

What's in this PR?

When deleting a locale of an article the behaviour was wrong which locale is the "ghosted" locale. This should be fixed with this PR.

This PR uses now an existing one. If the default one is available this one will be used else another one will be used.

Additionally this PR fixes the problem when a locale was remove which is the ghosted locale of another page.

Why?

Without this fix ALWAYS the default locale was the ghosted locale. Also if this locale was removed or another one but the default one was not created.

To Do

  • Fix: When a locale was remove which is the ghosted locale of another page
  • Tests

@wachterjohannes wachterjohannes force-pushed the hotfix/index-correct-ghost-content branch from 080a06f to 61ac120 Compare December 14, 2023 14:38
@wachterjohannes wachterjohannes changed the base branch from 2.5 to 2.4 December 14, 2023 14:39
@wachterjohannes wachterjohannes force-pushed the hotfix/index-correct-ghost-content branch 2 times, most recently from b47dbb2 to 665a5ef Compare December 14, 2023 14:40
@wachterjohannes wachterjohannes changed the base branch from 2.4 to 2.5 December 14, 2023 14:40
@wachterjohannes wachterjohannes force-pushed the hotfix/index-correct-ghost-content branch 6 times, most recently from fd1a97a to 12e25e9 Compare December 15, 2023 07:07
@wachterjohannes wachterjohannes marked this pull request as ready for review December 15, 2023 07:10
@wachterjohannes wachterjohannes force-pushed the hotfix/index-correct-ghost-content branch from 12e25e9 to 073e7e4 Compare December 15, 2023 07:33
@wachterjohannes wachterjohannes force-pushed the hotfix/index-correct-ghost-content branch from 073e7e4 to 02e3485 Compare December 15, 2023 09:01
@wachterjohannes wachterjohannes self-assigned this Dec 15, 2023
@chirimoya chirimoya merged commit a37febd into sulu:2.5 Dec 18, 2023
7 checks passed
@wachterjohannes wachterjohannes deleted the hotfix/index-correct-ghost-content branch December 18, 2023 19:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants