Needs review
Project:
Adform
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
14 Feb 2013 at 15:31 UTC
Updated:
10 Mar 2013 at 20:44 UTC
Jump to comment: Most recent file
Comments
Comment #1
kasperg commentedThe javascript breaks for clients with ad blockers enabled as the adf object is not available. I have added a bit of defensive coding and cleaned the module up as well.
Comment #2
xen commentedI'm not sure about this. I feel it's to specific to warrant configuration in the module.
Will the configuration options really make it possible for someone to implement product tracking without writing any code? If not, I think it'll be better just to let an addon module add the JS for tracking products itself. I'd suggest a hook, but I can't even see the base for that, as there don't seem to be any interaction from the base tracking functions to the product tracking functions. At best, adform.module could add the JS and a helper JS function for tracking products for third party modules to call.