We ran into issues on our site concerning the redirect on test pages. Since the control script is added via drupal_add_js() it's placed AFTER the other javascript files, which was causing issues as the redirect was executed after our analytics scipts tracked that page.

I patched just this issue and am using drupal_set_html_head to include it on the page.

CommentFileSizeAuthor
order_of_scripts.patch2.19 KBdevkinetic
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

aufumy’s picture

aufumy CreditAttribution: aufumy commented
Status: Active » Reviewed & tested by the community

works for me.

jay_N’s picture

jay_N CreditAttribution: jay_N commented

I would recommend this patch to be included in a new release, too.

Webster619’s picture

Webster619 CreditAttribution: Webster619 commented

Any update on this being added to a stable release?

devkinetic’s picture

devkinetic CreditAttribution: devkinetic commented

I can confirm that this still works and has caused no issues since implimentation.

Schoonzie’s picture

Schoonzie CreditAttribution: Schoonzie commented
Issue summary: View changes
Status: Reviewed & tested by the community » Closed (won't fix)

GWO is no longer around. Closing old issues.