Upgrade to new Piwik v0.4 JS Tracking API

Looks like a stolen API from GA... :-)

drupal_to_js adds double quotes automatically: ("'. drupal_to_js($trackfiles_extensions) .'"). And + $script .= 'piwikTracker.setCustomVars({ '. implode(', ', $piwik_vars_fields) .' });'; looks code wise wrong, too. Do not forget that $piwik_vars_fields could be empty...

I keep this open until the 0.4 you referenced becomes available. Than create a new release for Piwik 0.4 very soon.

Is this a fake? Where can I find the info about this 0.4 version? The latest version from SVN still have the old code style... ???

Ah... found it. Do you know if there is a reason why enableLinkTracking() comes after trackPageView()? If this is a requirement than the patch still needs work.

</script><script type="text/javascript">
try {
  var piwikTracker = Piwik.getTracker(pkBaseURL + "piwik.php", {$idSite});
  piwikTracker.trackPageView();
  piwikTracker.enableLinkTracking();
} catch( err ) {}
</script>

Great, thank you very much for the links! This patch needs some work... there are many new features I waited for a long time... we could add some more features - GA already have and I was made to remove in piwik... Internal site search tracking may be possible, too. We need to investigate more on this.

Ok, but enableLinkTracking() need to come after pagetracker if I reed the dogs correctly. Isn't it?

Have matt explained you something about the order? If required - why? My idea is the binding to all elements may takes time and shouldn't block the page tracking process. Therefore it is done as the very last step.

Have you tested your code??? if (!($track_outgoing = variable_get('piwik_track', 1))) { looks wrong to me. enableLinkTracking() sounds like - link tracking is not enabled by default...

I've committed the patch with a few changes and a bugfix for #15 to D5 and D6