Closed (outdated)
Project:
Advertisement
Version:
6.x-2.2
Component:
ad_actions module
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
23 Dec 2010 at 11:22 UTC
Updated:
19 Sep 2024 at 11:40 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
apprayo commentedSame here. Subscribing.
Comment #2
apprayo commentedI tested it on 2.2-dev, but still no trigger related to ad-clicking.
Comment #3
mich5 commentedHi,
thanks for trying the dev version, I didn't, and it's not resolving the trigger 'problem' at least for us.
Comment #4
aitala commentedI'm having the same problem... however, it does appear that ad expiration email(s) are being sent.
Eric
Comment #5
mich5 commentedWhen I tried to access directly the ad_actions tab (ad_actions_hook_info() gave me the tab name)
with this url: example.com/admin/build/trigger/ad_actions
I 've a "strange" Access denied ! I 'm logged as user 1 ...
With new role and {module ad_actions >>> administer ad triggers} set to true, same story.
In fact Access rights don't seem to behave correctly for me. I don't know what to do now.
Comment #6
foxtrotcharlie commentedI solved this in ad_actions.module by removing the ad_actions_menu_alter function declaration and adding the following to function ad_actions_hook_info:
'access arguments' => array('administer ad triggers'),so that it looks like:Rebuild your menus, and you should now see the "Ad actions" tab on the admin/build/trigger page.
I'll create a patch and will post it shortly.
Comment #7
foxtrotcharlie commentedThe Ad actions trigger tab was not showing up as a tab on the trigger page. This patch moves the user access check from the hook_menu_alter function to the hook_hook_info function. I'm not sure why the original method didn't work.
After patching, you should see a tab titled "Ad actions" at the top of the admin/build/trigger page. This patch is for version 6.x-2.2 of the ad_actions module.
Comment #8
mich5 commentedthanks foxtrotcharlie !
I applied the patch and yes, ad_actions tab is there in the trigger section.
I need to test notifications now.
thanks again.
Michel
Comment #9
lrwebks commentedDrupal 6 is EOL and no longer supported. Closing this as outdated for that reason. Thanks for your contribution!