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

[BUGFIX] Do not index translations on default language fallbackType: free #3785

Merged
merged 2 commits into from
Sep 19, 2023

Conversation

schliesser
Copy link
Contributor

What this pr does

With fallbackType: free the index queue contains an entry for each available language. Without this check all the entries are additionally indexed in the default language. They should be only indexed for the language, the record has defined as sys_language_uid.
See extensive debuggin in the issue.

How to test

Have a record based indexing configuration and change fallbackType to free in your site config. Do not set any fallbacks

Fixes #3560

Copy link

@abohntek abohntek left a comment

Choose a reason for hiding this comment

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

works now as expected! Even with fallbacks set.

Copy link
Collaborator

@dkd-kaehm dkd-kaehm left a comment

Choose a reason for hiding this comment

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

Thanks for the fix.

Current GH-Actions build fails, because of new rule single_line_empty_body,
which I'll fix soonish.

I'll merge this PR after port for release-11.5.x is available.

@dkd-kaehm dkd-kaehm added the BACKPORTABLE The changes SHOULD be backported label Sep 18, 2023
dkd-kaehm pushed a commit to dkd-kaehm/ext-solr that referenced this pull request Sep 18, 2023
…ree mode

With fallbackType: free the index queue contains an entry for each available language. Without this check all the entries are additionally indexed in the default language. They should be only indexed for the language, the record has defined as sys_language_uid.

Fixes TYPO3-Solr#3560
Ports: TYPO3-Solr#3785
dkd-kaehm and others added 2 commits September 18, 2023 17:10
…ree mode

With fallbackType: free the index queue contains an entry for each available language. Without this check all the entries are additionally indexed in the default language. They should be only indexed for the language, the record has defined as sys_language_uid.

Fixes TYPO3-Solr#3560
@dkd-kaehm dkd-kaehm merged commit a4013bd into TYPO3-Solr:main Sep 19, 2023
8 checks passed
dkd-kaehm pushed a commit that referenced this pull request Sep 19, 2023
…ree mode

With fallbackType: free the index queue contains an entry for each available language. Without this check all the entries are additionally indexed in the default language. They should be only indexed for the language, the record has defined as sys_language_uid.

Fixes #3560
Ports: #3785
dkd-friedrich pushed a commit to dkd-friedrich/ext-solr that referenced this pull request Sep 20, 2023
…guages free mode

With fallbackType: free the index queue contains an entry for each available language. Without this check all the entries are additionally indexed in the default language. They should be only indexed for the language, the record has defined as sys_language_uid.

Fixes TYPO3-Solr#3560
Ports: TYPO3-Solr#3785
@dkd-kaehm dkd-kaehm mentioned this pull request Oct 13, 2023
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BACKPORTABLE The changes SHOULD be backported
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] wrong indexing behaviour with fallbackType: free
3 participants