It would be ridiculously useful for the Views bulk select field provided by Entity Browser to support Views relationships. By doing that, you could have a view that displays entities of one type, but actually selects related entities. This would be hugely useful in Lightning's case (where you need to be able to select file entities from a library of media entities), and I imagine in many other cases as well. And it's a simple change!

First pass at a patch attached.

CommentFileSizeAuthor
#2 entity-browser-2769401-2.patch516 bytessamuel.mortenson
entity_browser-views_bulk_select_relationship_support.patch1004 bytesphenaproxima

Comments

phenaproxima created an issue. See original summary.

samuel.mortenson’s picture

samuel.mortenson
he/him
Primary language English
commented
StatusFileSize
new entity-browser-2769401-2.patch516 bytes

Replicated the problem, and confirmed the patch fixes it. However, if a View doesn't use a relationship, this doesn't work because the logic checks if $this->options['relationship'] evaluates to TRUE, which it always will since if you don't use relationships $this->options['relationship'] equals "none". This patch calls the getEntity() method instead, which should handle the relationship logic for us.

phenaproxima’s picture

phenaproxima
he/him
Primary language English
Location Massachusetts
commented

OK, I am absolutely in love with @samuel.mortenson's patch in #2. I didn't even know there was a getEntity() method. I wish I could RTBC that patch. In fact, I want to marry it.

Whole-hearted +1 from me.

slashrsm’s picture

slashrsm commented
Status: Needs review » Fixed
Issue tags: +D8Media

This is great. Committed. Thanks!

Status: Fixed » Closed (fixed)

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