ShareThis has recently enabled its new CopyNShare functionality, which now makes copy/pasting text from a site using ShareThis to result not in "this is the text I copied", but in "this is the text I copied - See more at: http://example.com/page#sthash.xxxxxxx.dpuf" (which is not the behavior I need for my site).
CopyNShare settings are described here:
http://support.sharethis.com/customer/portal/articles/961937-copynshare-...
with some new options:
doNotCopy - Disables the pasting of “See more: yourURL.com#SThashtag” after a user copies-and-pastes text.
hashAddressBar - Enables the tracking of URL copy and paste shares by adding a special #hashtag at the end our your URL.

By default, 'doNotCopy' is presumed to be 'false'.
To disable the "See more" function, I guess the script should look like this:
<script type="text/javascript">stLight.options({publisher: "INSERTYOURPUBKEY", doNotCopy:true});</script>
How should it be implemented in Drupal's ShareThis module?

CommentFileSizeAuthor
#5 sharethis-copynshare-1931224-5.patch1.74 KBguschilds
#1 sharethis-copynshare-1931224-2.patch1.43 KBmksweet

Comments

mksweet’s picture

mksweet commented
Status: Active » Needs review
StatusFileSize
new sharethis-copynshare-1931224-2.patch1.43 KB

Here is a patch for 7.x-2x that adds the option to turn off CopyNShare from the module configuration page.

GN’s picture

GN commented

Thanks a lot Meghan!

andrew perriman’s picture

andrew perriman commented

I couldn't get the patch to work. Which version is it for? It doesn't seem to match 7.x-2.5.

Status: Needs review » Needs work

The last submitted patch, sharethis-copynshare-1931224-2.patch, failed testing.

guschilds’s picture

guschilds commented
StatusFileSize
new sharethis-copynshare-1931224-5.patch1.74 KB

This patch applies cleanly to 7.x-2.5 and fixes an issue that was causing the first patch to not work for me.

The setting can be found in the Advanced fieldset in the ShareThis module settings.

guschilds’s picture

guschilds commented
Status: Needs work » Needs review
ryanrain’s picture

ryanrain commented

worked perfectly. thank you!

guschilds’s picture

guschilds commented
Status: Needs review » Reviewed & tested by the community

Marking as RTBC per ryanrain's successful experience.

Gaofengzzz’s picture

Gaofengzzz commented
Status: Reviewed & tested by the community » Fixed

Thanks very much @guschilds for your patch.
Got it committed against the 7.x-2.x branch at 8099c66.

I allowed myself to mark this issue as fixed for now, but feel free to re-open it, or post a new ticket, at any time if you have any further objections with this patch (we would surely be happy to hear your feedback).

Please let me know if you would have any further comments, feedback, questions, issues, objections, suggestions or concerns on any aspects in the commits or this ticket in general, I would be glad to provide more information or explain in more details.

Special thanks to @mksweet, @GN, @Andrew Perriman, @ryanrain, for re-implementing, Re-rolling, testing this patch.

Thanks again to everyone for your help, reviews, feedback and comments on this issue.

Cheers!

Status: Fixed » Closed (fixed)

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

kerasai’s picture

kerasai commented

This does not seem to be working. I've opened #2039245: CopyNShare Functionality Still Not Working as a follow up, including a patch.