Currently the admin view (admin/content/advertisements) is open for all to see. I'd suggest that we should require "administer advertisement blocks" to access the view.

See attached patch

CommentFileSizeAuthor
simpleads-admin-view-permission.patch702 bytesAngry Dan
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Angry Dan’s picture

Angry Dan CreditAttribution: Angry Dan commented
Title: Advertisments admin view needs a permission » Advertisements admin view needs a permission
auworks’s picture

auworks CreditAttribution: auworks commented

+1

SocialNicheGuru’s picture

SocialNicheGuru CreditAttribution: SocialNicheGuru commented
Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, simpleads-admin-view-permission.patch, failed testing.

minnur’s picture

minnur CreditAttribution: minnur as a volunteer commented
Status: Needs work » Closed (won't fix)