Change the link that points to the current page in a Views [#2232083]

Problem/Motivation

Allow for a rendering to have its own target share url.

Proposed resolution

By adding a field that can contain the url and have a token inserted to be replaced.

Remaining tasks

#2232083-1: Change the link that points to the current page in a Views does not work against the latest HEAD.
Test it.
Does it work with already configured fields.

User interface changes

Additional field in the widget settings.

API changes

Original report by @ntsekov

I had an issue with Addthis facebook link when I tried to change the link that points to the current page in a Views.

Comment	File	Size	Author
#1	addthis_url_views_field.png	25.9 KB	ntsekov
#1	addthis-change_the_ link-2232083-1.patch	3.41 KB	ntsekov

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

ntsekov CreditAttribution: ntsekov commented 3 April 2014 at 07:57

File	Size
addthis-change_the_ link-2232083-1.patch	3.41 KB
addthis_url_views_field.png	25.9 KB

I made a patch that can specify where to point facebook link in a Vievs, also you can use tokens for this field.

Comment #2

matglas86 CreditAttribution: matglas86 commented 16 April 2014 at 07:12

Version:	7.x-4.0-alpha4	» 7.x-4.x-dev
Issue summary:	View changes
Status:	Active	» Needs work

I'd love to check this out. But the patch does not apply. Please rebase your patch against the latest HEAD.

Comment #3

SocialNicheGuru CreditAttribution: SocialNicheGuru commented 15 June 2015 at 14:31

patch applied cleanly for me against alpha6 (june 15)

patch -p1 < addthis-change_the_ link-2232083-1.patch
can't find file to patch at input line 5
Perhaps you used the wrong -p or --strip option?
The text leading up to this was:
--------------------------
|diff --git a/sites/all/modules/contrib/addthis/addthis_displays/addthis_displays.addthis.inc b/sites/all/modules/contrib/addthis/addthis_displays/addthis_displays.addthis.inc
|index 96b2b78..e6dc814 100644
|--- a/sites/all/modules/contrib/addthis/addthis_displays/addthis_displays.addthis.inc
|+++ b/sites/all/modules/contrib/addthis/addthis_displays/addthis_displays.addthis.inc
--------------------------
File to patch: ./addthis_displays/addthis_displays.addthis.inc
patching file ./addthis_displays/addthis_displays.addthis.inc
Hunk #2 succeeded at 35 with fuzz 2 (offset 4 lines).
Hunk #3 succeeded at 144 with fuzz 2 (offset 30 lines).
can't find file to patch at input line 39
Perhaps you used the wrong -p or --strip option?
The text leading up to this was:
--------------------------
|diff --git a/sites/all/modules/contrib/addthis/addthis_displays/addthis_displays.field.inc b/sites/all/modules/contrib/addthis/addthis_displays/addthis_displays.field.inc
|index aa0231b..f20ea78 100644
|--- a/sites/all/modules/contrib/addthis/addthis_displays/addthis_displays.field.inc
|+++ b/sites/all/modules/contrib/addthis/addthis_displays/addthis_displays.field.inc
--------------------------
File to patch: ./addthis_displays/addthis_displays.field.inc
patching file ./addthis_displays/addthis_displays.field.inc
Hunk #3 succeeded at 101 (offset 4 lines).