Add roles permission Bypass ADs tracking (Track ADs exclude for whis roles.)

@Promo-IL could you prepare this as MR please?

dlevchik → changed the visibility of the branch 3414174-add-bypass-ad to hidden.

dlevchik → changed the visibility of the branch 3414174-add-bypass-ad to active.

Created MR porting a patch to 10.x, also changing it a little by moving permission check to trackEvent()

Question (mainly directed at @anybody): 📌 Integrate ad content type level permissions using "permission_callbacks" and "AccessControlHandler" Active will introduce a new permission called “View ads”. Only roles with that permission will see ads on the live site. Would this solve this issue? Or is a separate “Bypass ad tracking” permission sill necessary? I personally don't know why someone would want ads shown to them, but not being tracked, but what do I know.

Add "Bypass ad tracking" permission is still relevant, because certain roles should not be tracked. But I think it should be fairly simple. At the place where the counter is counting, simply skip counting with this permission. Everything else stays the same.

lrwebks → changed the visibility of the branch 4.0.x to hidden.

lrwebks → changed the visibility of the branch 11.x to hidden.

Everything should be operational now in !12. Unfortunately, click tracking is currently not working either way, even on 11.x, see 🐛 Click tracking doesn't work anymore Active .

anybody → closed merge request !4

@lrwebks: See comment

Re #18: Click tracking does work, so everything is working here. Click tracking is also not tracked for users with the bypass permission.

Perfect, thanks!! RTBC

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