it would be great to use the complete entity view processor with a Multiple types index.

At the moment it's not available.

CommentFileSizeAuthor
#9 entity_view_multi_type_indexes-2570117-9.patch2.92 KBr.van.doorn
#5 2570117-5--entity_view_multi_type_indexes.patch2.86 KBdrunken monkey
#5 2570117-5--entity_view_multi_type_indexes--interdiff.txt2.79 KBdrunken monkey
#4 search_api-2570117-complete_entity_view_multiple_types-4.patch4.23 KBjonmcl

Comments

miiimooo created an issue. See original summary.

drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
commented
Version: 7.x-1.16 » 7.x-1.x-dev

Yes, that indeed makes sense, as would some of the other processors (especially "Node access"). I'll see whether I get round to it, but I'm pretty busy with the D8 port at the moment. I'd be very glad about any patches in that direction, though!

miiimooo’s picture

miiimooo
Location Europe
commented

Feel free to postpone if no one else needs this. I'm knee deep in D8 now..

jonmcl’s picture

jonmcl commented
Status: Active » Needs review
StatusFileSize
new search_api-2570117-complete_entity_view_multiple_types-4.patch4.23 KB

I took a stab at this. Someone who is smarter than me might need to do a sanity check on my code. I'm not sure if it is possible for an entity to not support entity_view. If that is possible, not sure how well it would be handled here, but administrator should not enable that type anyway :)

drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
commented
Status: Needs review » Needs work
StatusFileSize
new 2570117-5--entity_view_multi_type_indexes--interdiff.txt2.79 KB
new 2570117-5--entity_view_multi_type_indexes.patch2.86 KB

Thanks, looks pretty good already! Attached are a few smaller adaptions.

However, one big issue remains to be fixed before I can commit this: entity types rarely have the same view modes defined, so having one selection for all types doesn't really make sense, I'd say. So, it would be great if you change the form builder for this case a bit more, to instead have one field per entity type. It should be pretty easy by just making a new protected helper method with most of the current configurationForm() code and just pass the entity type to that (and loop through all types for multi-type datasources).

jonmcl’s picture

jonmcl commented
Assigned: Unassigned » jonmcl

I need to work on this. Updating "assigned" field.

junaidpv’s picture

junaidpv
Location Kannur, Kerala
commented

We got same requirement to have index with both Node and User items in index. Created an index.
But I don't see "Rendered Entity" row plug-in available in views to show both Users and Nodes rendered in a view mode. Any idea?

drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
commented
Assigned: jonmcl » Unassigned

But I don't see "Rendered Entity" row plug-in available in views to show both Users and Nodes rendered in a view mode. Any idea?

Views is a different issue, this issue is about the processor/property.

r.van.doorn’s picture

r.van.doorn commented
StatusFileSize
new entity_view_multi_type_indexes-2570117-9.patch2.92 KB

I've recently changed to the latest develop commit for search_api (https://git.drupalcode.org/project/search_api/-/commit/6de9004798173315e...), because the have been a few bugdfixes commited since the last release that I need for my current project. The fix from #5 no longer works.

I did read the reason why patch from #5 hasn't been committed yet and I it would've been great If I had the time to change this, but I don't realy need the change since we did create the same view mode for all entity types for this project.

So this patch is the same as #5, but can be applied for the lasted dev version.