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
FileSize
431 bytes
PASSED: [[SimpleTest]]: [MySQL] 1,603 pass(es). View
dawehner’s picture

FileSize
558 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
FileSize
1.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.