Bug #90031

Wrong link calculation when languages in Site configuration are ordered non-ascendingly

Added by Mario Lubenka 27 days ago. Updated 8 days ago.

Status:
Under Review
Priority:
Should have
Assignee:
-
Category:
Link Handling, Site Handling & Routing
Start date:
2019-12-29
Due date:
% Done:

0%

TYPO3 Version:
9
PHP Version:
7.2
Tags:
site configuration,language
Complexity:
easy
Is Regression:
Sprint Focus:

Description

In the case where the languages in the Site configuration are not ordered by languageId it will not use the language with langId=0 as default page but the very first entry.

languages:
  -
    title: Japanese
    enabled: true
    languageId: '2'
    base: /jp
  -
    title: English
    enabled: true
    languageId: '0'
    base: /

This will cause wrong link calculation when opening a frontend page from the backend, as it will (in the above case) use the English page title with the Japanese base.

History

#1 Updated by Gerrit Code Review 27 days ago

  • Status changed from New to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/62779

#2 Updated by Gerrit Code Review 27 days ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/62779

#3 Updated by Mario Lubenka 27 days ago

  • Subject changed from Default language when languages in Site configuration are ordered differently to Wrong link calculation when languages in Site configuration are ordered non-ascendingly

#4 Updated by Riccardo De Contardi 27 days ago

Is this one related? #84671

#5 Updated by Mario Lubenka 27 days ago

Riccardo De Contardi wrote:

Is this one related? #84671

If languages become sortable via the backend the problem I described will occur more frequently.
Right now this only happens if one does that directly in the site configuration YAML file.

#6 Updated by Gerrit Code Review 9 days ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/62779

#7 Updated by Gerrit Code Review 9 days ago

Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/62779

#8 Updated by Gerrit Code Review 9 days ago

Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/62779

#9 Updated by Gerrit Code Review 9 days ago

Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/62779

#10 Updated by Gerrit Code Review 9 days ago

Patch set 7 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/62779

#11 Updated by Gerrit Code Review 8 days ago

Patch set 8 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/62779

Also available in: Atom PDF