Description
Currently, there is only minimal Views integration implemented in the D8 version – you can display rendered entities/items, you can add a fulltext search filter – but that's more or less it. I think it's obvious that this needs to be fixed as soon as possible.
I'm creating a single issue for now, but probably we should create child issues for the actual work, to not have one gigantic patch, get better attribution, and make it easier for several people to collaborate on this. (Either just issues for "fields", "filters", etc., or even finer-grained.)
A rewrite of the query should probably come first, then extending search_api.views.inc
to have a skeleton for actually including the different plugins, then porting of those other plugins.
Estimated Value and Story Points
This issue was identified as a Beta Blocker for Drupal 8. We sat down and figured out the value proposition and amount of work (story points) for this issue.
Value and Story points are in the scale of fibonacci. Our minimum is 1, our maximum is 21. The higher, the more value or work a certain issue has.
Value : 21
Story Points: 21
Comment | File | Size | Author |
---|---|---|---|
#6 | 2387017-6--add_ensure_table.patch | 1.58 KB | drunken monkey |
|
Comments
Comment #1
drunken monkeyThis could potentially help: #1792828: [META] Provide entity integration with Views.
Would need to look at it, though, and Berdir is pessimistic. (On the other hand, Bojan in the issue uses the Search API as the prime example for who this would help.)
Comment #2
Nick_vhComment #3
drunken monkeyComment #4
tstoecklerOpened #2484565: Views is broken with non-Sql query plugins, which I *think* is a blocker for this, although it would be very easy to work around from the Search API side. Not providing a patch for that, though, because the Search API views query plugin and views query plugins in general are a bit over my head.
Comment #5
ChristianAdamski CreditAttribution: ChristianAdamski as a volunteer commentedI do not see #2484565 changing any time soon. I am also constantly stumbling over ensureTable() function missing in the SearhApiQuery. Maybe we could include a stub for now?
Comment #6
drunken monkeyYou're right, we might as well already add this to make it easier for others to add custom plugins until our own integration is done.
Patch attached, please test/review!
Comment #7
Nick_vhComment #8
ChristianAdamski CreditAttribution: ChristianAdamski as a volunteer commentedWorks fine, please commit :)
Comment #9
drunken monkeyOK, committed.
Setting back to Active.
Comment #11
ohthehugemanatee CreditAttribution: ohthehugemanatee at Forum One commentedWhat work remains here? All the child Issues are closed, and there's a comment on the related Views Issue, which says that we've worked around it already in search_api.
So what further work is required to finish Views integration?
Comment #12
drunken monkeyGood question. I think we don't yet support all the contextual filters we did before, but other than that I'd say the port is more or less done.
There's still some open issues, but there always will be. The basic Views integration, as it should be for a beta or stable release, is done, I'd say.
So, marking this as "Fixed". Thanks for pointing it out!