Permalink
Browse files

dont use tracker name for _link function

  • Loading branch information...
1 parent 08027ff commit b36b622e59de2962983f719d199f7ce57a2348b6 @eduardocereto eduardocereto committed Aug 1, 2012
Showing with 1 addition and 2 deletions.
  1. +1 −2 src/core.js
View
@@ -186,9 +186,8 @@ GAS.prototype._execute = function() {
if (foo === '_link' || foo === '_linkByPost' || foo === '_require' ||
foo === '_anonymizeIp')
{
- acc_foo = _build_acct_name(acct_name) + foo;
args = slice.call(sub);
- args.unshift(acc_foo);
+ args.unshift(foo);
return _gaq_push(args);
}

2 comments on commit b36b622

@analyticsewolff

Hi Eduardo,

Does that mean something like this would not work?

_gas.push(['master._setAccount', 'UA-XXXXX-1']);
_gas.push(['master._setAllowLinker', true]);
_gas.push(['master._setAllowAnchor', true]);
_gas.push(['master._setDomainName', 'mysite.com']);
_gas.push(['master._setDomainName', 'myothersite.com']);
_gas.push(['master._gasMultiDomain', 'click']);
_gas.push(['master._trackPageview']);
_gas.push(['_setAccount', 'UA-YYYYY-1']);
_gas.push(['_setAllowLinker', true]);
_gas.push(['_setAllowAnchor', true]);
_gas.push(['_setDomainName', 'mysite.com']);
_gas.push(['_setDomainName', 'myothersite.com']);
_gas.push(['_gasMultiDomain', 'click']);
_gas.push(['_trackPageview']);

Your feedback is highly appreciated!

Cheers,

Sebastian

@eduardocereto
Contributor

I see nothing wrong with it. It should work.

Please sign in to comment.