I've been trying to figure out what's the issue, but it shows up in the layout configuration form, yet doesn't show up at the node display. Checked content types in SL configuration, permissions.

CommentFileSizeAuthor
#14 1789332-ds-compatability.patch749 bytesboznik
#1 1789332-service_links-display_suite-compatibility.patch933 bytesbarraponto

Comments

barraponto’s picture

barraponto
he / him
commented
StatusFileSize
new 1789332-service_links-display_suite-compatibility.patch933 bytes

This patch solves the issue for me, however it does need some work: it needs to upgrade the current layouts and needs to address the service_link-per-field case (didn't even look at it).

DS is not supposed to be upgraded from 7.x-1.x to 7.x-2.x, so we don't need to care about that, but we do need to care about updating the layouts configured previously to this patch. And of course, we should care about supporting both DS 7.x-2.x and 7.x-1.x

BEWARE: Using this work-in-progress patch requires re-adding the service links field to the layouts. Please see #1789332-4: Service Links Displays stopped working with DS 7.x-2.x

7wonders’s picture

7wonders commented
Status: Active » Needs work

patch applied cleanly but I still dont seem to be able to get service links to show up through ds. I have tried setting the Service Links Group field to all options and still nothing :(

danny englander’s picture

danny englander
(he/him)
commented

Same issue here, patch applied cleanly but no joy with Display Suite pages. I cleared cache as well. Works as expected on other non-Display Suite pages. When I removed Display Suite from a given content type, Service Links then worked as expected.

barraponto’s picture

barraponto
he / him
commented

Let me explain: I've not implemented an update function (I should, someone should). But if you want this to work right now, remove the service links field from the display BEFORE applying the patch. Then apply the patch and add it again.

xtfer’s picture

xtfer commented
Status: Needs work » Reviewed & tested by the community

This patch works fine, however you will need to re-add the Display Suite reference to the Service Links group after applying, as the key Display Suite looks for has changed.

barraponto’s picture

barraponto
he / him
commented
Status: Reviewed & tested by the community » Needs work

This is not RTBC if there's no upgrade path (re-adding stuff). Also, the patch doesn't even check what version of DS you're running, it just assumes 2.x.

Taxoman’s picture

Taxoman commented
Priority: Normal » Major
TheCrow’s picture

TheCrow commented
Category: bug » support
Priority: Major » Normal
Status: Needs work » Active

Installed DS 1, enabled SLD, configured some service and added to a content type, then updated DS to the 2.x version, runned update.php and cleaned the cache, everything is working fine.

Can someone helps to reproduce the error?

The patch above doesn't mean so much to me, being the id of the field not following any standard for DS...

simon georges’s picture

simon georges commented
Status: Active » Postponed (maintainer needs more info)
dddbbb’s picture

dddbbb commented

I just bumped into this problem as well: Service links not appearing on nodes where a Display Suite layout has been enabled. What are people doing to get around this if the patch above is no longer receiving any attention?

dddbbb’s picture

dddbbb commented

I'm a muppet. Forgot about the block that service links creates for you. I'll use that.

Would be nice if the DS feature worked though.

maxperience’s picture

maxperience commented

I'm new to Drupal and thought I had the same issue. This is what solved it for me.

Go to /admin/config/services/service-links/services , there you'll see a column "Displays" next to the column "Show"
Select the services you need under the "Displays" column and Service Links Displays will display. At least that's what worked for me.

peterwgnd’s picture

peterwgnd commented
Issue summary: View changes

Followed the advice in #12, which worked for getting the links to appear again, but I've lost the header. Any idea how to recover that bit?

boznik’s picture

boznik commented
StatusFileSize
new 1789332-ds-compatability.patch749 bytes

I just ran into this problem and fixed it with this patch.

john bickar’s picture

john bickar commented
Status: Postponed (maintainer needs more info) » Needs review

Setting to "needs review" for @boznik and testbot.