Project

General

Profile

Actions

Feature #80113

closed

f:link.page option for shortcuts

Added by Stephan Bauer over 7 years ago. Updated about 2 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Fluid
Target version:
-
Start date:
2017-03-03
Due date:
% Done:

100%

Estimated time:
PHP Version:
Tags:
Complexity:
Sprint Focus:

Description

f:link.page does not resolve shortcuts.

VHS menu viewhelper has a solution for this:
useShortcutData: Shortcut to set useShortcutTarget and useShortcutData simultaneously
useShortcutTarget: Optional param for using shortcut target instead of shortcut itself for current link
useShortcutUid: If TRUE, substitutes the link UID of a shortcut with the target page UID (and thus avoiding redirects) but does not change other data - which is done by using useShortcutData.
[[https://fluidtypo3.org/viewhelpers/vhs/master/Page/MenuViewHelper.html]]

Would it be possible to add an option like this to f:link.page also?


Related issues 1 (0 open1 closed)

Related to TYPO3 Core - Bug #96078: Shortcut Info-Message incorrect when page refers to frontend-restricted pageClosed2021-11-25

Actions
Actions #1

Updated by Riccardo De Contardi about 5 years ago

  • Status changed from New to Needs Feedback

Is this issue still present on 9.5.x? I tried this short test:

1) Create a page "Test shortcut" (UID=4)
2) Create a page "Test target" (UID=5)
3) Edit the page created at point 1), change the type to "shortcut" and set the shortcut target to the page created at point 2)
4) On a Fluid Template / Partial use:

<f:link.page pageUid="4">page link </f:link.page>

Test Results:

the generated link is:

<a href="/test-target">page link</a>

Is this sufficient? Or a different test is required?

Actions #2

Updated by Anonymous almost 5 years ago

Riccardo De Contardi wrote:

Is this issue still present on 9.5.x? I tried this short test:

1) Create a page "Test shortcut" (UID=4)
2) Create a page "Test target" (UID=5)
3) Edit the page created at point 1), change the type to "shortcut" and set the shortcut target to the page created at point 2)
4) On a Fluid Template / Partial use:

[...]

Test Results:

the generated link is:

[...]

Is this sufficient? Or a different test is required?

Did a similar test here and the generated link is not the expected one.

1) page 10 of doktype 1 with slug "foo"
2) page 20 of doktype 4 with slug "bar" and shortcut_mode=0 and shortcut=10

<f:link.page pageUid="20">Hallo</f:link.page>

Desired result:

<a href="/foo">Hallo</a>

Actual result:

<a href="/bar">Hallo</a>

TYPO3 9.5.13

In single-tree at least a 301 is done. But with multi-tree it leads to a 404.

Actions #3

Updated by Riccardo De Contardi almost 5 years ago

  • Status changed from Needs Feedback to New
Actions #4

Updated by Gerrit Code Review almost 3 years ago

  • Status changed from New to Under Review

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

Actions #5

Updated by Gerrit Code Review almost 3 years ago

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

Actions #6

Updated by Gerrit Code Review almost 3 years ago

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

Actions #7

Updated by Gerrit Code Review almost 3 years ago

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

Actions #8

Updated by Gerrit Code Review almost 3 years ago

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

Actions #9

Updated by Gerrit Code Review almost 3 years ago

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

Actions #10

Updated by Gerrit Code Review almost 3 years ago

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

Actions #11

Updated by Gerrit Code Review almost 3 years ago

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

Actions #12

Updated by Gerrit Code Review almost 3 years ago

Patch set 1 for branch 11.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/+/72772

Actions #13

Updated by Benni Mack almost 3 years ago

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

Updated by Benni Mack almost 3 years ago

  • Related to Bug #96078: Shortcut Info-Message incorrect when page refers to frontend-restricted page added
Actions #15

Updated by Benni Mack about 2 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF