Check access to the #nodes_by_type_link

Created on 28 May 2024, 6 months ago
Updated 29 May 2024, 6 months ago

Problem/Motivation

If the user does not have access to the content overview page, this link is still shown.

We should check for access before printing the link. Additionally it would be nice to change this url in the settings.

I know I can change it in hook_preprocess_type_tray_page() but it would be nice to specify it with all the other settings.

✨ Feature request
Status

Active

Version

1.2

Component

Code

Created by

πŸ‡ΊπŸ‡ΈUnited States loze Los Angeles

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

  • Issue created by @loze
  • πŸ‡ͺπŸ‡ΈSpain marcoscano Barcelona, Spain

    If the user does not have access to the content overview page, this link is still shown

    I don't think this would be a very common scenario, since I believe most sites allow editors to go to a place where they can see all content they have created? I have a hard time picturing editors with permissions to create content but not to see the list of content of that type. That said, I believe it's a good idea to not print 403 links if we can avoid. Let's just omit the link if the user doesn't have access to the admin content route.

    Additionally it would be nice to change this url in the settings

    On this one I am of the opinion that it's not worth adding the complexity for the few times this would be needed. It seems to me a legitimate scenario for custom code / preprocess tweaks, as you mention.

Production build 0.71.5 2024