When multiple addthis buttons are on a single page they each come with their own script tag. This patch only includes the tag once in the head.
Not sure you want this one as the code isn't the cleanest, I took the google analytics module approach to workaround the lack of support for external js in drupal_add_js(). Works a treat on my site though as I have an addthis button next to each content item.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | addthis-single-javascript-include.patch | 2.23 KB | benpjohnson |
| 0001-fix-addthis-to-only-insert-one-script-tag.patch | 2.62 KB | benpjohnson |
Comments
Comment #1
benpjohnson commentedI've updated this patch for 6.28 after finally getting my patch management system in order.
Comment #2
vesapalmu commentedThank you for the patch. However we already have a counter variable $_addthis_counter for keeping track on how many buttons we have on a single page. I reimplemented your patch for latest dev version on slightly different way. Some changes are still not quite ready, but I will commit a new version soon.
Comment #3
benpjohnson commentedGreat thanks!
One more patch I don't need to worry about :)
Comment #4
vesapalmu commentedNext version with this issue fixed in it will be out on next week. Sorry about the delay.