Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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
Comment #1
Les LimAssigning to myself to keep track of it. If this weekend is productive I'll target a new release for around then.
Comment #2
r.aubin CreditAttribution: r.aubin commentedAny 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!
Comment #3
djschoone CreditAttribution: djschoone commentedIf i read and understand the module and documentation right, this should be the patch:
Comment #4
Les LimThanks, 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".Comment #5
Les LimComment #6
djschoone CreditAttribution: djschoone commentedI 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.
Comment #7
Les LimThe 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.
Comment #8
Les LimFix committed. New release should be available shortly.
Comment #9
djschoone CreditAttribution: djschoone commentedThanks 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.