Bug #73083

Use API to check for "Hide default translation of page"

Added by Mathias Brodala almost 4 years ago. Updated about 1 year ago.

Status:
Closed
Priority:
Must have
Category:
-
Target version:
Start date:
2016-02-02
Due date:
% Done:

100%

TYPO3 Version:
8
PHP Version:
Tags:
Complexity:
Is Regression:
No
Sprint Focus:

Description

Throughout the core the logic to handle pages.l18n_cfg with value 1 (Hide default translation of page) is duplicated instead of calling GeneralUtility::hideIfDefaultLanguage() This should be fixed.


Related issues

Related to TYPO3 Core - Bug #73518: Disabled menu item state wrong for pages hidden in default language Closed 2016-02-17

Associated revisions

Revision e28c5654 (diff)
Added by Mathias Brodala over 3 years ago

[BUGFIX] Deduplicate "Hide default translation of page" logic

Resolves: #73083
Releases: master, 7.6, 6.2
Change-Id: I35dfbeb2034990b5746568a733c3e11240a4399d
Reviewed-on: https://review.typo3.org/46450
Reviewed-by: Andreas Fernandez <>
Tested-by: Andreas Fernandez <>
Reviewed-by: Markus Sommer <>
Tested-by: Markus Sommer <>
Reviewed-by: Morton Jonuschat <>
Tested-by: Morton Jonuschat <>

Revision 07b92e40 (diff)
Added by Mathias Brodala over 3 years ago

[BUGFIX] Deduplicate "Hide default translation of page" logic

Resolves: #73083
Releases: master, 7.6, 6.2
Change-Id: I35dfbeb2034990b5746568a733c3e11240a4399d
Reviewed-on: https://review.typo3.org/46450
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Tested-by: Andreas Fernandez <typo3@scripting-base.de>
Reviewed-by: Markus Sommer <markussom@posteo.de>
Tested-by: Markus Sommer <markussom@posteo.de>
Reviewed-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Tested-by: Morton Jonuschat <m.jonuschat@mojocode.de>
(cherry picked from commit e28c56540792cf19fc3782cd894eb5c9e0af6aa4)
Reviewed-on: https://review.typo3.org/46615

Revision ba303809 (diff)
Added by Morton Jonuschat over 3 years ago

[BUGFIX] Deduplicate "Hide default translation of page" logic

Resolves: #73083
Releases: master, 7.6, 6.2
Change-Id: I35dfbeb2034990b5746568a733c3e11240a4399d
Reviewed-on: https://review.typo3.org/46450
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Tested-by: Andreas Fernandez <typo3@scripting-base.de>
Reviewed-by: Markus Sommer <markussom@posteo.de>
Tested-by: Markus Sommer <markussom@posteo.de>
Reviewed-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Tested-by: Morton Jonuschat <m.jonuschat@mojocode.de>
(cherry picked from commit e28c56540792cf19fc3782cd894eb5c9e0af6aa4)
Reviewed-on: https://review.typo3.org/46616

Revision b3225d19 (diff)
Added by Mathias Brodala over 3 years ago

[BUGFIX] Fix disabled menu item state for l18n_cfg=1

This fixes an issue introduced with the backport of the
change in #73083 to TYPO3 6.2.

Resolves: #73518
Related: #73083
Releases: 6.2
Change-Id: I8fef570b15e8bbf94da124f47a5bd4b3158c1b9f
Reviewed-on: https://review.typo3.org/46743
Reviewed-by: Andreas Fernandez <>
Tested-by: Andreas Fernandez <>
Tested-by: Mario Rimann <>
Tested-by: David Hoeckele <>
Reviewed-by: Christian Kuhn <>
Tested-by: Christian Kuhn <>

History

#1 Updated by Gerrit Code Review almost 4 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/46450

#2 Updated by Gerrit Code Review over 3 years ago

Patch set 1 for branch TYPO3_7-6 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/46615

#3 Updated by Mathias Brodala over 3 years ago

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

#4 Updated by Gerrit Code Review over 3 years ago

  • Status changed from Resolved to Under Review

Patch set 1 for branch TYPO3_6-2 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/46616

#5 Updated by Mathias Brodala over 3 years ago

  • Status changed from Under Review to Resolved

#6 Updated by Benni Mack about 1 year ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF