It would be nice to do that.

CommentFileSizeAuthor
#4 drupal-1920278-4.patch1.37 KBdawehner
#2 drupal-1920278-2.patch558 bytesdawehner
#1 1920278_make_admin_views_restrict_access.patch431 bytesgreggles

Comments

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Status: Active » Needs review
StatusFileSize
new 1920278_make_admin_views_restrict_access.patch431 bytes
dawehner’s picture

dawehner
Primary language German
commented
StatusFileSize
new drupal-1920278-2.patch558 bytes

What about maing the other one restricted as well?

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented

Heh, yeah. That is a good idea.

dawehner’s picture

dawehner
Primary language German
commented
Title: Indicate that "administer views" is kind of a big deal by making it "restrict access" -> true » Indicate that "administer views" and "access all views" is kind of a big deal by making it "restrict access" -> true
Project: Views (for Drupal 7) » Drupal core
Version: 7.x-3.x-dev » 8.x-dev
Component: Code » views_ui.module
StatusFileSize
new drupal-1920278-4.patch1.37 KB

Thanks for the patch!

Let's get it in and move to Drupal 8. Here we should actually move the permission from views.module to views_ui.module.

dawehner’s picture

dawehner
Primary language German
commented
Issue tags: +VDC

adding tag

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
commented
Status: Needs review » Reviewed & tested by the community

Nice, glad to see this split up.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.x. Thanks!

Automatically closed -- issue fixed for 2 weeks with no activity.