Bug #20043
closed
typolink creates links to not translated pages in config.sys_language_mode=strict
Added by Alexander Bohndorf about 15 years ago. Updated over 5 years ago.
0%
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 |
Updated by Ralf Hettinger about 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.
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.
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.
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".
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".
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/
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.
Updated by Marcos Fadul over 11 years ago
Same patch but now includes a check for l18n_cfg.
Please push into Core
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?
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
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
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
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
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
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
Updated by Benni Mack over 8 years ago
- Target version changed from 7.5 to 7 LTS
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
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
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
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
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
Updated by
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
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
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
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.