It would be nice to do that.

Files: 
CommentFileSizeAuthor
#4 drupal-1920278-4.patch1.37 KBdawehner
PASSED: [[SimpleTest]]: [MySQL] 51,346 pass(es).
[ View ]
#2 drupal-1920278-2.patch558 bytesdawehner
PASSED: [[SimpleTest]]: [MySQL] 1,621 pass(es).
[ View ]
#1 1920278_make_admin_views_restrict_access.patch431 bytesgreggles
PASSED: [[SimpleTest]]: [MySQL] 1,603 pass(es).
[ View ]

Comments

greggles’s picture

Status:Active» Needs review
StatusFileSize
new431 bytes
PASSED: [[SimpleTest]]: [MySQL] 1,603 pass(es).
[ View ]
dawehner’s picture

StatusFileSize
new558 bytes
PASSED: [[SimpleTest]]: [MySQL] 1,621 pass(es).
[ View ]

What about maing the other one restricted as well?

greggles’s picture

Heh, yeah. That is a good idea.

dawehner’s picture

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» Drupal core
Version:7.x-3.x-dev» 8.x-dev
Component:Code» views_ui.module
StatusFileSize
new1.37 KB
PASSED: [[SimpleTest]]: [MySQL] 51,346 pass(es).
[ View ]

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

Issue tags:+VDC

adding tag

tim.plunkett’s picture

Status:Needs review» Reviewed & tested by the community

Nice, glad to see this split up.

webchick’s picture

Status:Reviewed & tested by the community» Fixed

Committed and pushed to 8.x. Thanks!

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