Bug #79647
closed
Add hooks to enable extension using the old typoLinkHandler syntax to migrate data
100%
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
Updated by Christian Kuhn about 7 years ago
- Target version deleted (
8 LTS) - Sprint Focus deleted (
Stabilization Sprint)
Updated by Mathias Schreiber over 6 years ago
- Is Regression changed from Yes to No
Updated by Marc Bastian Heinrichs about 6 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
Updated by Marc Bastian Heinrichs about 6 years ago
Info: the todo annotation was deleted with https://review.typo3.org/#/c/52684/
Updated by Marc Bastian Heinrichs about 6 years ago
- File BUGFIX_Convert_legacy_parameters_in_record_link_handler.patch BUGFIX_Convert_legacy_parameters_in_record_link_handler.patch added
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.
Updated by Gerrit Code Review about 6 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
Updated by Daniel Goerz about 6 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
Updated by Gerrit Code Review about 6 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
Updated by Gerrit Code Review about 6 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
Updated by Gerrit Code Review about 6 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
Updated by Gerrit Code Review about 6 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
Updated by Gerrit Code Review almost 6 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
Updated by Gerrit Code Review almost 6 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
Updated by Daniel Goerz almost 6 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 1f54e70484901d3799e01e1c166c2df23934ad8e.
Updated by