Title says it all, new issue created out of #879216: How to set up comments in display suite to show up on node pages

CommentFileSizeAuthor
#6 ds_extra-change_hook_field_attach_view_alter_to_hook_entity_view-1334128-6.patch2.5 KBseanb

Comments

zambrey’s picture

zambrey commented

I'm trying to use EVA module to display view attached to node in block region.
When I display it in main column it shows up okay but moving it into block region displays nothing.
I assume that this issue is related with my usecase.

swentel’s picture

swentel commented
Status: Active » Postponed

Ok, sadly enough I can't fix this at this moment :/ Postponing this to see in case there comes a 2 branch I can fix it (although I doubt it)

zambrey’s picture

zambrey commented

Damn, that's the bad news. Is it architectural problem or lack of time on your part?

swentel’s picture

swentel commented

The problem is that the preprocess of the node comes later than the block theming, so I can't possibly know what I need to pass on, it's a real bummer :/

swentel’s picture

swentel commented
Status: Postponed » Closed (won't fix)

Ok, this will never work, sorry :/

seanb’s picture

seanb
Primary language Dutch
Location Netherlands
commented
Issue summary: View changes
Status: Closed (won't fix) » Needs review
StatusFileSize
new ds_extra-change_hook_field_attach_view_alter_to_hook_entity_view-1334128-6.patch2.5 KB

Sorry to reopen this, but I think the issue could be fixed by using hook_entity_view() in stead of hook_field_attach_view_alter() to fetch the fields for the blocks. Is there a good reason not to do this?

Attached is a patch. Please review and let me know what you think.

*A fix for https://www.drupal.org/node/2221307 is included in the patch, since field groups are also not working. A cleaner patch for only the fieldgroups was added to the other issue.

kamenrs’s picture

kamenrs commented

Hello seanB,
for which version of DS is your patch?

seanb’s picture

seanb
Primary language Dutch
Location Netherlands
commented

I believe it was 7.x-2.6.

inversed’s picture

inversed commented

This applied cleanly for me to 7.x-2.6 but, unfortunately, did not have any effect for me. I'm still not able to see my preprocess field in the block region.

ArtuDrop’s picture

ArtuDrop commented
Version: 7.x-1.x-dev » 7.x-2.10
Status: Needs review » Needs work

I have same problem with DS 7.x.2.10 and EVA 7.x.1.2.

jmary’s picture

jmary commented

#6 patch worked for me.
EVA is not involved here though, but commerce fields from product variations, like picture and price.

amaisano’s picture

amaisano commented

Workaround for me:

1) Create a new DS view mode, 1-column layout, with ONLY the EVA field visible

2) In the view mode you ultimately want to display the EVA field in, use an existing block region or create a new block region

3) Create a new dynamic field and choose Node > Node Content.

4) Select the view mode you created in step 1 and adjust other settings as needed.

5) Move it to the region you want and save.

6) Enjoy!

aspilicious’s picture

aspilicious commented
Status: Needs work » Closed (won't fix)

Thnx for all the input, this will be an actual won't fix in D7.
You probably should use one of the workarounds described here if you encounter issues.

jmary’s picture

jmary commented

@aspilicious : Because that patch hasn't been applied, my site has broken when I update from ds 2.11 to 2.13 and I spent 8hours trying to figure out why.

If the patch doesn't break things, and actually fix things, there is no reason no to include it.

C'est une blague Belge :-D ?