Bug #79647

Add hooks to enable extension using the old typoLinkHandler syntax to migrate data

Added by Markus Klein almost 4 years ago. Updated about 2 years ago.

Status:
Closed
Priority:
Must have
Assignee:
-
Category:
Link Handling, Site Handling & Routing
Target version:
-
Start date:
2017-02-06
Due date:
% Done:

100%

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

Description

This is a followup to the new link-syntax introduced with version 8.
Extensions using the typoLinkHandler syntax '<identifier>:...' currently do not work anymore and have no sane possibility to migrate old links.

The Core code contains todo annotations with this issue number attached.


Files


Related issues

Related to TYPO3 Core - Bug #80806: Upgrade Wizard / Execute database migrations on single rows / Error: The RecordLinkHandler expects identifier and uid as $parameter configurationClosedMarkus Klein2017-04-11

Actions
#1

Updated by Christian Kuhn over 3 years ago

  • Target version deleted (8 LTS)
  • Sprint Focus deleted (Stabilization Sprint)
#2

Updated by Mathias Schreiber almost 3 years ago

  • Is Regression changed from Yes to No
#3

Updated by Marc Bastian Heinrichs over 2 years ago

  • Related to Bug #80806: Upgrade Wizard / Execute database migrations on single rows / Error: The RecordLinkHandler expects identifier and uid as $parameter configuration added
#4

Updated by Marc Bastian Heinrichs over 2 years ago

Info: the todo annotation was deleted with https://review.typo3.org/#/c/52684/

#5

Updated by Marc Bastian Heinrichs over 2 years ago

Since the LegacyLinkNotationConverter already provides the key url with all info needed based on the old linkhandler notation

list($a['identifier'], $a['table'], $a['uid']) = explode(':', $linkHandlerValue);
$result['url'] = $a;

I would like to see this handled in RecordLinkHandler instead of having a simple hook. See attached patch.

#6

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

#7

Updated by Daniel Goerz over 2 years ago

I stumpled over this missing API today and had to XCLASS the LinkService. I pushed my proposal for an hook, although I am not entirely sure if I got this ticket right.

Here is an example extension that uses the newly introduced hook for testing and verifying purposes: https://github.com/ervaude/forge_links

#8

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

#9

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

#10

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

#11

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

#12

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

#13

Updated by Gerrit Code Review over 2 years ago

Patch set 1 for branch TYPO3_8-7 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/56790

#14

Updated by Daniel Goerz over 2 years ago

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

Updated by Benni Mack about 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF