Project

General

Profile

Actions

Bug #20043

closed

typolink creates links to not translated pages in config.sys_language_mode=strict

Added by Alexander Bohndorf over 15 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Should have
Assignee:
Category:
Localization
Target version:
Start date:
2009-02-18
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
7
PHP Version:
Tags:
Complexity:
Is Regression:
No
Sprint Focus:

Description

The expected language handling does only work with MENU objects but not with links in RTE or Links rendered by FE-Plugins via typolink.

(issue imported from #M10476)


Files

class.tslib_content.patch (2.68 KB) class.tslib_content.patch Administrator Admin, 2009-12-10 00:14
class.tslib_content-repaired.patch (1.2 KB) class.tslib_content-repaired.patch Administrator Admin, 2011-01-26 17:24
ContentObjectRenderer.php.patch (700 Bytes) ContentObjectRenderer.php.patch please add patch to core. Marcos Fadul, 2012-11-15 10:59
ContentObjectRenderer.php.patch (782 Bytes) ContentObjectRenderer.php.patch Same patch but now includes a check for l18n_cfg Marcos Fadul, 2012-11-16 15:44
Actions #1

Updated by Ralf Hettinger over 15 years ago

I can confirm this issue with an early 4-3 trunk revision.
Imo a solution without breaking backwards "compatibility" could be to implement some switches within the typolink function that adds additional typoscript configuration... I remember that I had solved it that way once and might easily supply a patch if you guys think this sounds like a good idea.

Actions #2

Updated by Michael Bakonyi almost 15 years ago

Hi, I think it would definitely be a good idea to fix this as the editors might not keep this in mind when setting links within the RTE.

Actions #3

Updated by Marcos Fadul over 14 years ago

I found this problem today when I update to T3 4.3. I attached a patch to solve the problem. It just add a condition so when sys_language_mode== strict and there is no overlay, it return the same as if the web wouldn't exist.

I think this is a very important bug. I hope the developer team add this patch to the next release.

Best regards and please leave feedback if it works for you too.

Actions #4

Updated by Marcos Fadul over 14 years ago

In the patch there is a small mistake. It is missing a d at the end of "hideNonTranslate". Correct would be "hideNonTranslated".

Actions #5

Updated by Marcos Fadul over 13 years ago

I try today with the typo3 4.5rc1 and the path still doing his job. Links to pages that are not translated disable. I will upload a patch with no mistake as the one I upload before is missing a "d".

Actions #6

Updated by Jan Loderhose over 13 years ago

Post a patch request on typo3.teams.core mailling list to get your patch into the core. please follow the ml rules http://typo3.org/teams/core/core-mailinglist-rules/

Actions #7

Updated by Marcos Fadul over 11 years ago

This is still open in Typo3 6.0. I guest it is time to close this issue.

Here is the patch that I made years ago. Still working even in version 6.0. It just need someone to push it into the core. I used the option sys_language_mode to filter which links are correct and which should be unset.

Greetings.

Actions #8

Updated by Marcos Fadul over 11 years ago

Same patch but now includes a check for l18n_cfg.

Please push into Core

Actions #9

Updated by Claus Harup almost 11 years ago

This works perfectly and really a feature TYPO3 needs - any plans on adding it to the core?

Actions #10

Updated by Gerrit Code Review about 10 years ago

  • Status changed from Accepted 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/28581

Actions #11

Updated by Gerrit Code Review about 10 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/28581

Actions #12

Updated by Gerrit Code Review about 10 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/28581

Actions #13

Updated by Gerrit Code Review over 9 years ago

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

Actions #14

Updated by Gerrit Code Review almost 9 years ago

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

Actions #15

Updated by Alexander Opitz over 8 years ago

  • Description updated (diff)
  • Target version changed from 0 to 7.5
  • TYPO3 Version set to 7
  • Is Regression set to No
Actions #16

Updated by Benni Mack over 8 years ago

  • Target version changed from 7.5 to 7 LTS
Actions #17

Updated by Gerrit Code Review over 8 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/28581

Actions #18

Updated by Gerrit Code Review over 7 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/28581

Actions #19

Updated by Gerrit Code Review over 7 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/28581

Actions #20

Updated by Gerrit Code Review over 7 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/28581

Actions #21

Updated by Gerrit Code Review over 7 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/28581

Actions #22

Updated by Riccardo De Contardi about 7 years ago

  • Category set to Localization
Actions #23

Updated by Gerrit Code Review over 6 years ago

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

Actions #24

Updated by Gerrit Code Review over 6 years ago

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

Actions #25

Updated by Gerrit Code Review over 6 years ago

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

Actions #26

Updated by Christian Kuhn over 5 years ago

  • Status changed from Under Review to Closed

hey. i'm abandoning this stalled patch for now and will reject the issue: nothing really happened here again for quite a while, and the situation might also have changed a bit with the v9 site configuration and some other fallback modes a bit again. i may be wrong with this decision, so please open a new issue summarizing the available information again to start this a-new if further work happens in this area.

Actions

Also available in: Atom PDF