Hi
I am using Advertisement 6.x-2.2 and I have tried both Ad Flash 6.x-2.7 and the latest .dev release dated 2011-12-21. Clicking on my flash advert takes me to a new browser tab with the URL saying "undefined" and the standard "page not found" displayed.
My advert was supplied by a large agency and is for a major corporate so I'm presuming it is fine.
Looking at the page source ( with the .dev version of this module ) I see the following as part of the object tag :
<param name="flashvars" value="clickTAG=http://mydomain.com/ad/redirect/6513/0?url=node/6505">
6513 is the nid of the advert node and 6505 is the nid of the page the advert is displayed on.
Any suggestions much appreciated.
Thanks
| Comment | File | Size | Author |
|---|---|---|---|
| #9 | ad_flash.module.patch | 1.24 KB | lazly |
Comments
Comment #1
El Bandito commentedQuick followup. If I enter the url from the clickTAG parameter ( i.e. http://mydomain.com/ad/redirect/6513/0?url=node/6505 ) directly into my browser address bar it redirects successfully to the correct external destination URL.
Comment #2
El Bandito commentedOK. This was a result of the infamous "clickTag" problem. This module is hardcoded to send the parameter as "clickTAG" but my advert was expecting "clickTag".
Changing line 605 in ad_flash.module to :
$output .= 'var flashvars = {clickTag:"'.$ad->redirect.'/@HOSTID___"};';fixed the problem for me.
Comment #3
Alex Andrascu commentedUmm...hardcoded ?
Maybe not.
Syntax is clickTAG not cLicKTag or whatever a major agency decides it's correct.
Maybe send this back to the major agency http://www.adobe.com/resources/richmedia/tracking/designers_guide/
Comment #4
El Bandito commentedAlex
I'm sure you're right and your link supports your assertion. However I can see from the issue queue that others have fallen into this trap.
Perhaps it would be an idea to make it default to "clickTAG" but with the option of specifying something else ?
Thanks for the great module.
Comment #5
El Bandito commentedIn fact even the Adobe doc you reference is inconsistent. It states use :
but the example dialog box image shows :
Ho Hummm
Comment #6
Alex Andrascu commentedDavid,
It might make sense to have this kind of feature in the end as i'm tired to see people falling into traps by now :) .
I'm willing to review and commit any sensible patches addressing this issue.
Hope it helps.
Alex
Comment #7
Alex Andrascu commentedComment #8
Alex Andrascu commentedComment #9
lazly commented#2 is work for me too, a made a patch, please commit to source. Its not a feature request, if its not commited to your code your clicks will be not work AND not counted too. Comment: Its looks like the flash vars are case sensitive.
Comment #10
Alex Andrascu commentedThanks for the patch ! Will commit.
Comment #11
Alex Andrascu commentedComment #12
lazly commentedYour welcome, thanks your work too! ;)