Project

General

Profile

Actions

Bug #76760

closed

"Insert link" – missing page uid when linking a content element

Added by Dominik Hahn over 8 years ago. Updated about 6 years ago.

Status:
Closed
Priority:
-- undefined --
Category:
Site Handling, Site Sets & Routing
Target version:
Start date:
2016-06-23
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
8
PHP Version:
7.0
Tags:
Complexity:
no-brainer
Is Regression:
Yes
Sprint Focus:
Stabilization Sprint

Description

I came across this while trying to link a content element on a specific page with the "insert link" function of the RTE.

To reproduce:
  • Create an element with a RTE
  • Mark some text and click the "insert link" button
  • In the popup window (make sure you are on the "page" tab), click the little black arrow next to a page with some content on it
  • you will see a list of content elements
  • click one of the content elements, the popup will close
  • toggle 'text mode' on and
  • inspect the content of the RTE

You will notice that the Url is incomplete, the page uid of the page you are linking to is missing: http://typo3.dev/?id=#260
This broken links are marked as external as soon as the element is saved.
A working url would look like this: http://typo3.dev/?id=101#260

Upon inspection I have found out that the link to the content element within the "insert link" popup should normally know about the page uid it is on.

In short: the recordlist sysext has a bug in one of the templates and the page uid is missing, thus the link can not be constructed properly.

I have fixed the bug already and will push it in a few minutes.


Related issues 1 (0 open1 closed)

Related to TYPO3 Core - Task #76332: Use Fluid for rendering LinkBrowsersClosedBenni Mack2016-05-27

Actions
Actions #1

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

Actions #2

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

Actions #3

Updated by Markus Klein over 8 years ago

  • Complexity set to no-brainer
  • Is Regression changed from No to Yes
  • Sprint Focus set to Stabilization Sprint
Actions #4

Updated by Anonymous over 8 years ago

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

Updated by Benni Mack about 6 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF