Bug #82942

Wrong Links for pageids in rte_ckeditor in TYPO3 8.7.8

Added by Manfred Mirsch about 2 years ago. Updated 5 months ago.

Status:
New
Priority:
Should have
Assignee:
-
Category:
RTE (rtehtmlarea + ckeditor)
Target version:
-
Start date:
2017-11-07
Due date:
% Done:

0%

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

Description

When using pageIdSelector to enter TYPO3 page-ids directly, the generated links are not working any more.

Problem is, that the page-id itself is beeing used to create the typolink, which will lead to links like this:
www.my-domain.de/typo3/12345

I fixed this by patching "/typo3_src/typo3/sysext/rte_ckeditor/Resources/Public/JavaScript/RteLinkBrowser.js" to modify the link:

if (parseInt(link) == link) {
link = 't3://page?uid=' + link;
}

Greeting,
Manfred

RteLinkBrowser.js.patch View (863 Bytes) Manfred Mirsch, 2017-11-07 15:13


Related issues

Related to TYPO3 Core - Bug #82888: Missing label in link browser Closed 2017-10-31
Related to TYPO3 Core - Bug #83056: Cannot set pageIdSelector configuration on link browser (outside of rte) Under Review 2017-11-21

History

#1 Updated by Georg Ringer about 2 years ago

  • Related to Bug #82888: Missing label in link browser added

#2 Updated by Riccardo De Contardi almost 2 years ago

  • Related to Bug #83056: Cannot set pageIdSelector configuration on link browser (outside of rte) added

#3 Updated by Riccardo De Contardi over 1 year ago

  • Category set to RTE (rtehtmlarea + ckeditor)

#4 Updated by Riccardo De Contardi over 1 year ago

Manfred, thank you very much for your report.

Would you consider uploading your patch to Gerrit? Someone could do this for you, but I am thinking you might like the opportunity to contribute to TYPO3 yourself.

If in need, you can find a description of the TYPO3 contribution workflow here: https://docs.typo3.org/typo3cms/ContributionWorkflowGuide/

Thank you in advance and best regards!

#5 Updated by Riccardo De Contardi 5 months ago

This seems already solved on master, please test after applying https://review.typo3.org/c/Packages/TYPO3.CMS/+/61194

Also available in: Atom PDF