Closed (fixed)
Project:
ShareThis
Version:
7.x-2.x-dev
Component:
Code
Priority:
Major
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
9 Aug 2016 at 22:13 UTC
Updated:
18 Jan 2017 at 15:44 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
pradeep22saini commentedAs I checked issue is related with name of the handler class. If that is changed similar to the file name the error for broken missing handler is gone and it works fine.
Comment #3
hiramanpatil@pradeep22saini,
As you have resolved this issue and uploaded patch here. So status should be changed to 'Needs review'.
Marking as Needs review.
Thanks
Comment #4
criscomI have the same problem but patch in #2 didn't resolve it. I reverted to version 7.x-2.12 in order to fix it.
Comment #5
whthat commentedThis did fix the broken handler name in the view but the ShareThis links are not appearing as they used to in views, needs some additional work. Revert to 2.12 for views functionality.
Comment #6
phandolin commentedI can second this issue, and have the same experience with the patch as @whthat "This did fix the broken handler name in the view but the ShareThis links are not appearing as they used to in views"
This seems urgent, any word on a fix?
Comment #7
alex.bukach commentedHere's the fix (method signature didn't correspond to the parent class method).
Comment #8
naveenvalechaWould anyone like to test it and RTBC it ?
Comment #9
wildfeed commentedSorry, no love on this patch. Still shows Broken/missing handler when adding link to View.
Comment #10
bananetheorique commentedHi! I have the same problem. Where I am suppose to put the patch? I copy-paste this text in some file? I copy this file in some directory? Thanks
Comment #11
whthat commentedSeeing the proper field name in views, but ShareThis buttons are not displayed when field is used in views.
It does appear that the some button html is getting applied, but the ShareThis's button.js is not getting inserted into the page, where all the magic happens.
Comment #12
whthat commentedIncluded the call to include the proper JS when a view is rendered.
Comment #13
krina.addweb commentedHello,
Patch works well with 7.x.2.13.
Thanks
Comment #14
webdrips commentedConfirming #7 works (after clearing the caches).
Comment #15
criscomPatch in #12 works against latest Dev. Thank you!
Comment #16
purushotam.rai commentedPatch #12 is working fine.
Comment #18
purushotam.rai commentedThanks a lot for all your efforts.