Closed (fixed)
Project:
AddThis
Version:
7.x-2.1-beta1
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
24 Jul 2008 at 16:38 UTC
Updated:
21 May 2012 at 18:40 UTC
Jump to comment: Most recent file
Comments
Comment #1
Richard Blackborder commentedHere's a patch that adds this feature. I've made it always on, because it seems to me that if you wanted a link to the current page, you'd use the Add This block. I think any user would expect the teaser buttons to add the node itself, not the page of the teaser.
Note: this patch is of version 6.x-2.0-beta3.
Comment #2
vesapalmu commentedThanks for the patch Likeless, it fixes the issue. I commited the patch to CVS and will roll out a new release soon.
Comment #3
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.
Comment #4
dagomar commentedIf I am cluttering the issue cue here, I am deeply sorry. I think this is not working on my drupal 7 install. It's the same issue, all addthis buttons in the teaser point to the front page. Looking into the issue.
** edit **
Tracked the issue to line 179 of addthis.module. It seems "$node" is not an object and it's not passed in the $variables
** edit 2 **
Got it working. I don't know how to make patches, and I am not sure if what I did is the right way. I'll post what I did step by step, it's quite simple (I'm replacing the lines, passing the node info along each step):
line 35:
'#markup' => _addthis_create_button(array('build_mode' => $build_mode), $node)line 148:
return theme('addthis_button', array('build_mode' => $build_mode, $node));line 155:
$build_mode = $variables['build_mode']['build_mode'];line 177,178
Comment #5
monican commentedI'm also on Drupal 7, and have the same issue. I will try the changes suggested in #4 as a workaround, but wesku might want to fixing the code so no hacking is needed for updates. Thanks dagomar.
Comment #6
kappaluppa commentedI tried the above changes and I am still not getting a link to the node listed in the teaser, but to the main page where the teasers are listed.
Will this issue be address in the upcoming 7.x-4.x release?
Thanks,
K
Comment #7
matglas86 commentedThe 4.x release is a complete new implementation. It handles this already indeed.
Comment #8
matglas86 commentedFixed in 4.x release.