Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
sharethis_update_7002
was added in #2759221: Unable to select services on the configuration page to clean up the sharethis_service_option
variable. After updating to 7.x-2.x-dev, the services were still broken for me. I took a look at value of sharethis_service_option
prior to update hook running at it looked like this:
"Facebook:facebook
","Twitter:twitter","Email:email"
Here is a patch that will also strip out amp;
to fully make the changes in #2759221: Unable to select services on the configuration page work for all. Special thanks to @navneet0693 for helping me understand this on IRC.
Comment | File | Size | Author |
---|---|---|---|
#3 | remove_amp_in-2846432-3.patch | 1.13 KB | vinmassaro |
Comments
Comment #2
vinmassaro CreditAttribution: vinmassaro commentedComment #3
vinmassaro CreditAttribution: vinmassaro commentedNew patch to get the variable once, set the variable once.
Comment #4
navneet0693 CreditAttribution: navneet0693 as a volunteer and at QED42 commentedI am not sure, but only one service will get affected that is AT&T by this, which is saved like this
"AT&T:att"
. I cannot see much differences, AT&T still appears :) Good to go, thanks for the patch!Comment #5
navneet0693 CreditAttribution: navneet0693 as a volunteer and at QED42 commentedComment #7
purushotam.rai CreditAttribution: purushotam.rai at QED42 commented