Project

General

Profile

Actions

Bug #89871

closed

Epic #89797: HrefLang / Canonical issues

HrefLangGenerator: Wrong links generated with page translations that use a shortcut

Added by Christian Eßl almost 5 years ago. Updated over 3 years ago.

Status:
Closed
Priority:
Should have
Category:
-
Target version:
-
Start date:
2019-12-06
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
9
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:
On Location Sprint

Description

Steps to reproduce:

- Create a site config with the languages "*Default*", "*German*", "*Spanish*"
- Create a page "Page A" and translate it to all languages
- Create a page "Page B" and translate it to all languages
- On "Page A", make the german translation to a "Shortcut" to "Page B"
- Now open the spanish version of "Page A".

Expected:
- The hreflang for "de" looks like https://www.sitename.com/de/page-b

Actual:
- The hreflang for "de" now looks like https://www.sitename.com/es/page-b
- This will also easily result in a 404 error, if the page title is different in the translation.

The HreflangGenerator incorrectly uses the language parameter of the current language instead of the one of the target language, if shortcuts are used.


Related issues 5 (0 open5 closed)

Related to TYPO3 Core - Bug #81657: TSFE->page contains default language page when content_fallback; 1,0 is usedClosed2017-06-21

Actions
Related to TYPO3 Core - Bug #86595: No Language FallbackClosedAnja Leichsenring2018-10-08

Actions
Related to TYPO3 Core - Bug #19114: sys_language_mode content_fallback with a defined fallback chain does not output expected fallback content Closed2008-07-16

Actions
Related to TYPO3 Core - Bug #94677: Menu-item of type "shortcut" is language-overlayed twiceClosed2021-07-30

Actions
Related to TYPO3 Core - Task #95966: Wrong tests for #89871Closed2021-11-12

Actions
Actions #1

Updated by Christian Eßl almost 5 years ago

  • Description updated (diff)
Actions #2

Updated by Christian Eßl almost 5 years ago

  • Description updated (diff)
Actions #3

Updated by Christian Eßl almost 5 years ago

Found the reason:
https://github.com/TYPO3/TYPO3.CMS/commit/8d61bef0865649bbe3d73bdf77a47c760c8d9ba4

In this commit, shortcut pages are now generated with the setting "'language' => 'current',", which overrides the L parameter.

Actions #4

Updated by Gerrit Code Review almost 5 years 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/+/62552

Actions #5

Updated by Mathias Brodala almost 5 years ago

  • Related to Bug #81657: TSFE->page contains default language page when content_fallback; 1,0 is used added
Actions #6

Updated by Mathias Brodala almost 5 years ago

  • Related to Bug #86595: No Language Fallback added
Actions #7

Updated by Mathias Brodala almost 5 years ago

  • Related to Bug #19114: sys_language_mode content_fallback with a defined fallback chain does not output expected fallback content added
Actions #8

Updated by Gerrit Code Review almost 5 years 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/+/62552

Actions #9

Updated by Christian Eßl almost 5 years ago

  • Assignee set to Christian Eßl
Actions #10

Updated by Susanne Moog almost 5 years ago

  • Sprint Focus set to On Location Sprint
Actions #11

Updated by Gerrit Code Review almost 5 years 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/+/62552

Actions #12

Updated by Gerrit Code Review almost 5 years 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/+/62552

Actions #13

Updated by Gerrit Code Review almost 5 years 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/+/62552

Actions #14

Updated by Gerrit Code Review over 4 years 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/+/62552

Actions #15

Updated by Gerrit Code Review over 4 years 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/+/62552

Actions #16

Updated by Gerrit Code Review over 4 years 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/+/62552

Actions #17

Updated by Richard Haeser almost 4 years ago

  • Parent task set to #89797
Actions #18

Updated by Gerrit Code Review almost 4 years ago

Patch set 9 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/+/62552

Actions #19

Updated by Gerrit Code Review almost 4 years ago

Patch set 10 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/+/62552

Actions #20

Updated by Gerrit Code Review almost 4 years ago

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

Actions #21

Updated by Christian Eßl almost 4 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
Actions #22

Updated by Benni Mack over 3 years ago

  • Status changed from Resolved to Closed
Actions #23

Updated by Nikita Hovratov about 3 years ago

  • Related to Bug #94677: Menu-item of type "shortcut" is language-overlayed twice added
Actions #24

Updated by Nikita Hovratov about 3 years ago

Actions

Also available in: Atom PDF