Needs review
Project:
DFP Small Business (Google Ad Manager)
Version:
7.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
3 Nov 2011 at 19:36 UTC
Updated:
15 Nov 2011 at 00:29 UTC
Jump to comment: Most recent file
Comments
Comment #1
13rac1 commentedHere is a quick implementation of a "Show ads for Administrator" checkbox in the administrative menus. Probably could/should be expanded to have more features, but I needed something so I don't lose my mind while working on this project.
Comment #2
13rac1 commentedI was too excited to disable the ads and missed some logic. This works as advertised...
Comment #3
jcisio commentedI prefer a "hide ads" permission rather than hard code it to uid 1.
User having this permission won't see ads. A step farther (but not indispensable now) is users with this permission will have a checkbox in their profile to choose whether they want to see ads.
Comment #4
13rac1 commentedThis was as first step, I don't think this version will be committed. I figured my quick fix could get the ball rolling.
Comment #5
13rac1 commentedThe reason it is "Active" and not "Needs Review"
Comment #6
13rac1 commentedThis can't be done with a standard permission since user_access() always returns TRUE for UID 1. Instead, I implemented it using code from the Google Analytics module to allow ad visibility selection by role. New section added to settings page at: /admin/config/system/google_admanager