I have the code previously working fine for 7.x.2.12.
Then I saw the latest realease and updated my module to 7.x.2.13. But after updating to this version In views, I am getting "Broken missing handler" for the field on adding content shareThis.

I have attached image for reference.

CommentFileSizeAuthor
#13 After Patch screenshot.png97.21 KBkrina.addweb
#12 sharethis-views_broken_handler-2781649-12.patch1.03 KBwhthat
#7 sharethis-views_broken_handler-2781649-7.patch627 bytesAlex Bukach
#2 2781649-1-broken_handler.patch445 bytespradeep22saini
Screen Shot 2016-08-10 at 10.10.12 AM.png354.16 KBpradeep22saini
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

pradeep22saini created an issue. See original summary.

pradeep22saini’s picture

pradeep22saini CreditAttribution: pradeep22saini commented
FileSize
2781649-1-broken_handler.patch445 bytes

As 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.

hiramanpatil’s picture

hiramanpatil CreditAttribution: hiramanpatil at Clarion Technologies commented
Status: Active » Needs review

@pradeep22saini,

As you have resolved this issue and uploaded patch here. So status should be changed to 'Needs review'.

Marking as Needs review.

Thanks

criscom’s picture

criscom
Location Graz, Austria
CreditAttribution: criscom commented

I 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.

whthat’s picture

whthat CreditAttribution: whthat commented
Status: Needs review » Needs work

This 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.

phandolin’s picture

phandolin CreditAttribution: phandolin as a volunteer commented

I 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?

Alex Bukach’s picture

Alex Bukach CreditAttribution: Alex Bukach commented
Status: Needs work » Needs review
FileSize
sharethis-views_broken_handler-2781649-7.patch627 bytes

Here's the fix (method signature didn't correspond to the parent class method).

naveenvalecha’s picture

naveenvalecha
He/Him
Primary language Hindi
CreditAttribution: naveenvalecha as a volunteer commented
Version: 7.x-2.13 » 7.x-2.x-dev

Would anyone like to test it and RTBC it ?

wildfeed’s picture

wildfeed CreditAttribution: wildfeed as a volunteer commented

Sorry, no love on this patch. Still shows Broken/missing handler when adding link to View.

bananetheorique’s picture

bananetheorique CreditAttribution: bananetheorique commented

Hi! 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

whthat’s picture

whthat CreditAttribution: whthat commented
Status: Needs review » Needs work

Seeing 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.

whthat’s picture

whthat CreditAttribution: whthat commented
Status: Needs work » Needs review
FileSize
sharethis-views_broken_handler-2781649-12.patch1.03 KB

Included the call to include the proper JS when a view is rendered.

krina.addweb’s picture

krina.addweb CreditAttribution: krina.addweb at AddWeb Solution Pvt. Ltd. commented
Status: Needs review » Reviewed & tested by the community
FileSize
After Patch screenshot.png97.21 KB

Hello,

Patch works well with 7.x.2.13.

Thanks

webdrips’s picture

webdrips CreditAttribution: webdrips commented

Confirming #7 works (after clearing the caches).

criscom’s picture

criscom
Location Graz, Austria
CreditAttribution: criscom commented

Patch in #12 works against latest Dev. Thank you!

purushotam.rai’s picture

purushotam.rai CreditAttribution: purushotam.rai at QED42 commented
Status: Reviewed & tested by the community » Patch (to be ported)

Patch #12 is working fine.

  • purushotam.rai committed e224dbc on 7.x-2.x authored by whthat 
    Issue #2781649 by pradeep22saini, whthat, Alex Bukach, krina.addweb,...
purushotam.rai’s picture

purushotam.rai CreditAttribution: purushotam.rai at QED42 commented
Status: Patch (to be ported) » Fixed

Thanks a lot for all your efforts.

Status: Fixed » Closed (fixed)

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