Project

General

Profile

Actions

Bug #16840

closed

menu.showAccessRestrictedPages does not work with hmenu.special = list

Added by Dirk Weise almost 18 years ago. Updated about 7 years ago.

Status:
Closed
Priority:
Should have
Assignee:
Category:
Content Rendering
Target version:
Start date:
2007-01-09
Due date:
% Done:

100%

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

Description

menu.showAccessRestrictedPages doesn't replace links in menus when HMENU.special = list is set.

(issue imported from #M4754)

Actions #1

Updated by Franz Koch about 17 years ago

hey - could this be fixed for 4.2?

Actions #2

Updated by Mathias Bolt Lesniak over 13 years ago

  • Target version deleted (0)

Hi!

It looks like this bug is still there in 4.5.2.

Actions #3

Updated by Fronzes Philippe about 13 years ago

Still present in version 4.5.7.

A patch could be:
Replace line 499 from file class.tslib_menu.php by this
$loadDB->additionalWhere['pages']=$this->sys_page->enableFields('pages',0,$this->mconf['showAccessRestrictedPages']?array('fe_group'=>1):array());

Actions #4

Updated by Benni Mack about 13 years ago

  • Category changed from Communication to Content Rendering
  • Status changed from New to Accepted
  • TYPO3 Version changed from 4.0 to 4.5
  • PHP Version deleted (5)

Hey Fronzes,

your approach seems reasonable.

care to create a patch that could be sent to our review server via Git/Gerrit?

http://wiki.typo3.org/Contribution_Walkthrough_Tutorials

All the best,
Benni.

Actions #5

Updated by Fronzes Philippe over 12 years ago

Hi Benni,

Just see your message.
I would of course did this patch, but I'm not using Git/Gerrit yet.
I will look at the link to a tutorial you gave, but i do not have a lot of time to do it these days
Hope i could do it reasonably fast.

Philippe FRONZES.

Actions #6

Updated by Alain over 12 years ago

Hello,

this issue is still present in TYPO3 4.6.10.

Best regards,

Alain

Actions #7

Updated by Mathias Schreiber almost 10 years ago

  • Assignee set to Benni Mack
  • Target version set to 7.2 (Frontend)
  • Is Regression set to No

@Benni Mack, maybe we can take a look at this

Actions #8

Updated by Benni Mack over 9 years ago

  • Target version changed from 7.2 (Frontend) to 7.4 (Backend)
  • Sprint Focus set to On Location Sprint
Actions #9

Updated by Gilles Fouet over 9 years ago

  • Status changed from Accepted to In Progress
  • Assignee changed from Benni Mack to Gilles Fouet
Actions #10

Updated by Gerrit Code Review over 9 years ago

  • Status changed from In Progress 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/40315

Actions #11

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

Actions #12

Updated by Gerrit Code Review over 9 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/40315

Actions #13

Updated by Gerrit Code Review over 9 years ago

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

Actions #14

Updated by Gilles Fouet over 9 years ago

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

Updated by Gerrit Code Review over 9 years ago

  • Status changed from Resolved to Under Review

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

Actions #16

Updated by Gilles Fouet over 9 years ago

  • Status changed from Under Review to Resolved
Actions #17

Updated by Mathias Schreiber over 9 years ago

  • Sprint Focus deleted (On Location Sprint)
Actions #18

Updated by Riccardo De Contardi about 7 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF