Hi, i have the following problem with addthis, tested on 2 different 5.x-sites:
This is the result of sharing via mail:
--
addthis test
/node/1069?Array
[Message sent by foo@bar.com via AddThis]
--
And this is the url when i try to submit to myspace:
--
http:///node/1069?Array
--
| Comment | File | Size | Author |
|---|---|---|---|
| #10 | absoluteurl.patch | 668 bytes | kingandy |
Comments
Comment #1
rasmusmp commentedI have the same problem. I have outcommented:
/* addthis_offset_top = \'%d\';
addthis_offset_left = \'%d\'; */
But that should not have an effect. Any solution?
Comment #2
vesapalmu commentedThis has now been fixed in 2.1 version for D5.x.
Comment #4
ttotade commentedNo it has not been fixed, I am using 2.1 with drupal 5.x and I am still getting the same problems as explained above.
Comment #5
vesapalmu commentedThen it is a new issue. This issue was originally related to offset handling and that has now been disabled. Please provide more information about the issue. Does this happen always? In every page? With all browsers?
Comment #6
gaijinu commentedI second this, it happens in FF3 and IE6 when the button is set to appear in the teaser. When it's on the full node view the url is sent fine. When is on the teaser only the http://node_name/ is sent to the bookmarking services.
Please take a look here: http://muaythai.sg/
Using the 5.x-2.1 version.
Thank you.
Comment #7
bobbydonohue commentedI'm experiencing this also on several web sites. I can confirm that it works fine on the full node, but not the teaser.
Comment #8
gaijinu commentedAny news? Thanks.
Comment #9
Mindy Block commentedstill an issue with the teaser, but works so far as I see it, with the full node post...
Comment #10
kingandyThis looks like something that has been addressed in the 6.x version but was never backported ... on line 221 the URL is added using
url('node/'.$node->nid). We need to specifically tell url() to generate this as an absolute URL (ie, including the domain) rather than relative to the base path.In Drupal 6 this is done with a settings array, but in D5 we need to add a series of defaulted variables between the URL and the 'absolute' parameter. The line becomes:
$teaser ? url('node/'. $node->nid, NULL, NULL, TRUE) : '[URL]',Find attached a patch to apply this to the current 5.x-2.1 build.
Comment #11
kingandyHa ha! I just ran into this again and look, here's my fix from the last time. How handy. Thanks, past me!
Comment #12
vesapalmu commentedI think we need a new 5.x branch maintainer for the module. Sorry about this taking a loooooong time. I am trying to clean up the issue queue and get back to 5.x issues also.
Comment #13
edgartanaka commentedThis is a possible fix. I tried this on function addthis_link(). My version is D6.
It worked here.
Comment #14
gisleThis bug is no longer present in the current version (7.x-1.x) of the module.
I don't know if it has been fixed for D6, but that version is no longer supported (and I only maintain the D7 version).
If somebody want to see the D6 version of this module still maintained, they need to contact the project owner and sign on as co-maintainer.