It seems like a publisher id is required for analytics stats in addthis now. Can you please add the publisher id to the default settings (via admin settings form) so that analytics data is linked properly to an account? I haven't tested this and I'm just going on the following documentation: http://www.addthis.com/help/default-code

pubid being the relavent paramater that needs adding. I couldn't find any references in the addthis documentation to the username parameter so I'm not sure if its required anymore.

Comments

Les Lim’s picture

Assigned: Unassigned » Les Lim
Category: feature » task

Assigning to myself to keep track of it. If this weekend is productive I'll target a new release for around then.

r.aubin’s picture

Any movement on this? We're having the same problem where we really need statistics integration, but nothing is being tracked.

If no update is coming soon, we may have to embed the button without the module, so an update to this thread would be a great help so we know what direction to go.

Thanks!

djschoone’s picture

If i read and understand the module and documentation right, this should be the patch:

diff -r 922998603179 -r 2416507ea297 sites/all/modules/contrib/addthis/addthis.admin.inc
--- a/sites/all/modules/contrib/addthis/addthis.admin.inc	Mon Jul 18 13:10:03 2011 +0200
+++ b/sites/all/modules/contrib/addthis/addthis.admin.inc	Mon Jul 18 13:19:19 2011 +0200
@@ -176,6 +176,12 @@
     '#description' => t('If you are using AddThis Analytics, enter the username of the AddThis account where you are collecting AddThis click data.'),
     '#default_value' => $addthis_config['username'],
   );
+  $form['addthis_config']['pubid'] = array(
+      '#type' => 'textfield',
+      '#title' => t('AddThis Profile ID'),
+      '#description' => t('Your AddThis Profile ID to collect AddThis click data'),
+      '#default_value' => $addthis_config['pubid'],
+  );
   $form['addthis_config']['data_ga_tracker'] = array(
     '#type' => 'checkbox',
     '#title' => t('Integrate share data with Google Analytics'),
Les Lim’s picture

Thanks, dmedia! This should be correct, but it's only half of what the patch needs. The other part is to include a hook_update_N() function to change the set values of addthis_config in the variables table to change "username" to "pubid".

Les Lim’s picture

Status: Active » Needs work
djschoone’s picture

I added the field, because i hadn't the time to look further into the field username and if it is needed somewhere else. Do you overview if we need this anymore?

I think it should be better to remove the "username" value and add the "pubid" value, because keeping previously saved usernames won't work as pubid.

Les Lim’s picture

The old username values actually should work. When AddThis switched to pubid instead of username, they transitioned the old accounts to user the username value for the pubid.

Working on this a little later today.

Les Lim’s picture

Status: Needs work » Fixed

Fix committed. New release should be available shortly.

djschoone’s picture

Thanks for catching up with this. I found out the switch from username to pubid didn't work for me, so the fix was very important for us.
Now i see what you've done, i understand your question. The username is now outputed as 'pubid' for backwards compatibility at AddThis.

Status: Fixed » Closed (fixed)

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