Closed (fixed)
Project:
Search API
Version:
8.x-1.x-dev
Component:
Framework
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
25 Jan 2016 at 11:11 UTC
Updated:
6 Jan 2017 at 14:45 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
drunken monkeyComment #3
drunken monkeyThis should implement it, and at least the PhpUnit tests pass for me, locally.
The only non-trivial change is this place in
\Drupal\search_api\Plugin\search_api\datasource\ContentEntity::getIndexesForEntity():Without the extra
plugin_idkey, I don't think we can do this anymore. Or is there some entity query whiz who knows a solution?Like it is, I think we'll have to load all indexes and check them manually. (Not that this should matter much, I'd expect very few sites would have more than one or two indexes.)
Comment #4
drunken monkeyWanted to quickly get this in before, but now we need an upgrade path.
Comment #5
borisson_Tagging for upgrade path.
Comment #6
drunken monkeyComment #7
drunken monkeyRe-roll.
Comment #9
borisson_Yep, this all looks great! If we can get the tests to pass again, this has my stamp of approval.
Comment #11
drunken monkeyNot sure how this happened, but this should fix it.
Comment #13
drunken monkeyAh, finally green again!
Thanks also for your review, Joris!
Committed.