According to Eloqua, the only tracking script they use now is elqCfg.min.js. This was switched with version 9 (currently on version 10). Would it be possible to make the module allow for this?

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

drupalninja99’s picture

Status: Active » Needs work

Looking at http://armorcastprod.com/wp-content/uploads/2012/10/Eloqua-Async-Trackin...

It appears we don't need to download a library now, right? We just add this snippet of code (in a Drupal behavor?) and that's all we need to do for the Eloqua tracking, right? I don't think this would affect the webform integration.

<script type="text/javascript">
var _elqQ = _elqQ || [];
_elqQ.push(['elqSetSiteId', 'siteId']);
_elqQ.push(['elqTrackPageView']);
(function () {
function async_load() {
var s = document.createElement('script'); s.type = 'text/javascript'; 
s.async = true; s.src = '//img.en25.com/i/elqCfg.min.js';
var x = document.getElementsByTagName('script')[0]; 
x.parentNode.insertBefore(s, x);
}
if (window.addEventListener) window.addEventListener('DOMContentLoaded', 
async_load, false);
else if (window.attachEvent) window.attachEvent('onload', async_load); 
})();
</script>

The only thing that's dynamic is 'siteId'.

drupalninja99’s picture

Status: Needs work » Needs review

Converted tracking to new method.

drupalninja99’s picture

Attaching patch

drupalninja99’s picture

NOTE: this patch has some whitespace cleanup as well so it may be a little tricky to sort through the diff

drupalninja99’s picture

Priority: Normal » Critical
drupalninja99’s picture

Made some additional fixes, I still don't think it's fully functional.

drupalninja99’s picture

I can confirm that the new tracking is working. I am still testing the webform integration as some javascript has changed there as well.

drupalninja99’s picture

I think the webform integration might be working as well but there is at least a 10 min delay if not longer to see stats show up in the Eloqua control panel so I am still testing.

drupalninja99’s picture

Status: Needs review » Reviewed & tested by the community

It takes a super long time for the data to show up but it does look like it's working.

drupalninja99’s picture

Status: Reviewed & tested by the community » Fixed

Committed to 7.x-1.x

Status: Fixed » Closed (fixed)

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