Project

General

Profile

Actions

Bug #82942

closed

Wrong Links for pageids in rte_ckeditor in TYPO3 8.7.8

Added by Manfred Mirsch over 6 years ago. Updated almost 4 years ago.

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

100%

Estimated time:
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


Files

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

Related issues 2 (0 open2 closed)

Related to TYPO3 Core - Bug #82888: Missing label in link browserClosed2017-10-31

Actions
Related to TYPO3 Core - Bug #83056: Cannot set pageIdSelector configuration on link browser (outside of rte)Closed2017-11-21

Actions
Actions #1

Updated by Georg Ringer over 6 years ago

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

Updated by Riccardo De Contardi about 6 years ago

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

Updated by Riccardo De Contardi almost 6 years ago

  • Category set to RTE (rtehtmlarea + ckeditor)
Actions #4

Updated by Riccardo De Contardi over 5 years 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!

Actions #5

Updated by Riccardo De Contardi over 4 years ago

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

Actions #6

Updated by Benni Mack about 4 years ago

  • Status changed from New to Closed

I will close this now, as Riccardo confirmed it is fixed, and we didn't get any feedback. If the issue still exists, let me know and I will re-open the ticket.

Actions #7

Updated by Markus Klein almost 4 years ago

  • Status changed from Closed to Accepted
  • Assignee set to Markus Klein

This is not fixed. The referenced ticket is about the field itself being visible and not about the value not being handled correctly.

This issue persists v8 to v10.

See also https://github.com/FriendsOfTYPO3/rtehtmlarea/issues/5

Actions #8

Updated by Gerrit Code Review almost 4 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/c/Packages/TYPO3.CMS/+/64064

Actions #9

Updated by Gerrit Code Review almost 4 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/+/64064

Actions #10

Updated by Gerrit Code Review almost 4 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/+/64064

Actions #11

Updated by Gerrit Code Review almost 4 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/+/64064

Actions #12

Updated by Gerrit Code Review almost 4 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/+/64064

Actions #13

Updated by Gerrit Code Review almost 4 years ago

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

Actions #14

Updated by Markus Klein almost 4 years ago

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

Updated by Gerrit Code Review almost 4 years ago

  • Status changed from Resolved to Under Review

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

Actions #16

Updated by Markus Klein almost 4 years ago

  • Status changed from Under Review to Resolved
Actions #17

Updated by Benni Mack almost 4 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF