Project

General

Profile

Actions

Bug #67047

closed

Bug #85138: Shortcut in menu always is rendered as inactive

Cannot access shortcut target in menus

Added by Mathias Brodala almost 9 years ago. Updated about 4 years ago.

Status:
Closed
Priority:
Must have
Category:
Localization
Target version:
-
Start date:
2015-05-20
Due date:
% Done:

100%

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

Description

With #36822 links via overlaid shortcuts have been fixed but the original menu item data was kept unchanged.

This makes it impossible to e.g. mark active menu items with pure TypoScript. An override is not useful since both CONTENT as well as RECORDS perform overlays too.


Related issues 3 (0 open3 closed)

Related to TYPO3 Core - Bug #36822: Invalid shortcut target on translated pagesClosedOliver Hader2012-05-03

Actions
Related to TYPO3 Core - Feature #80841: MenuProcessor is missing an option to mark shorcuts as active or currentClosed2017-04-13

Actions
Related to TYPO3 Core - Bug #69454: Shortcut page ignores the configured url targetClosed2015-09-01

Actions
Actions #1

Updated by Gerrit Code Review almost 9 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 http://review.typo3.org/39630

Actions #2

Updated by Mathias Brodala almost 9 years ago

Example code to mark active menu items including shortcuts:

// For "Menu of subpages of selected pages" 
tt_content.menu.20.1.1 {
  // Disable link and add active class for menu items of the current page
  ACT < .NO
  ACT = 1
  ACT {
    doNotLinkIt = 1
    wrapItemAndSub = <li class="active"><a>|</a></li>
  }
  // Do the same for page shortcut menu items
  NO.wrapItemAndSub.stdWrap.override < .ACT.wrapItemAndSub
  NO.wrapItemAndSub.stdWrap.override.if {
    // 1st condition: value of field "doktype" must be 4 (shortcut)
    value.field = doktype
    equals = 4
    // 2nd condition: only true if nested condition is true
    isTrue = 1
    // Nested condition: value of field "shortcut" must be the current page
    isTrue.if {
      value.field = shortcut
      equals.data = TSFE:id
    }
  }
  NO.doNotLinkIt.stdWrap.override < .ACT.doNotLinkIt
  NO.doNotLinkIt.stdWrap.override.if < .NO.wrapItemAndSub.stdWrap.override.if
}
Actions #3

Updated by Gerrit Code Review almost 9 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/39630

Actions #4

Updated by Gerrit Code Review over 8 years ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/39630

Actions #5

Updated by Mathias Schreiber over 8 years ago

  • Target version deleted (next-patchlevel)
Actions #6

Updated by Gerrit Code Review about 8 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/39630

Actions #7

Updated by Gerrit Code Review about 8 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/39630

Actions #8

Updated by Gerrit Code Review almost 8 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/39630

Actions #9

Updated by Susanne Moog over 6 years ago

  • Category changed from Content Rendering to Localization
Actions #10

Updated by Simon Gilli almost 6 years ago

  • Parent task set to #85138
Actions #11

Updated by Riccardo De Contardi over 5 years ago

  • Related to Feature #80841: MenuProcessor is missing an option to mark shorcuts as active or current added
Actions #12

Updated by Riccardo De Contardi over 5 years ago

  • Related to Bug #69454: Shortcut page ignores the configured url target added
Actions #13

Updated by Gerrit Code Review over 5 years ago

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

Actions #14

Updated by Gerrit Code Review over 5 years ago

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

Actions #15

Updated by Gerrit Code Review over 4 years ago

Patch set 9 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/+/39630

Actions #16

Updated by Gerrit Code Review over 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/+/62477

Actions #17

Updated by Mathias Brodala over 4 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 50 to 100
Actions #18

Updated by Benni Mack about 4 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF