AFAICS It is currently impossible to have a video search integrated with the other core-provided searches: the only available search is a custom search on the playlists.
On d8, this would require implementing a @SearchPlugin .
| Comment | File | Size | Author |
|---|---|---|---|
| #8 | brightcove_add-search-integration-2879387-8.patch | 8.03 KB | yce |
| #6 | interdiff-5-6.txt | 9.43 KB | yce |
| #6 | brightcove_add-search-integration-2879387-6.patch | 8.03 KB | yce |
| #5 | 0001-Issue-2879387-add-search-integration.patch | 7.41 KB | fgm |
Comments
Comment #2
justkristin commentedWow - this would be a drawback if one wanted to expose each video to users as pages. :(
Comment #3
aaron.silberIf you're in a pinch, the Search API will allow you to index Brightcove entities.
Comment #4
fgmActually, I did implement a search plugin for Brightcove on the site where it was needed.
FWIW, here is an adaptation the code I used, in patch form
Comment #5
fgmGrrr, wrong patch.
Comment #6
yce commentedHi,
I think the BrightcoveVideo entities should be queried through the storage itself, so I've done some refactoring on the patch, please check.
I've also updated the BrightcoveVideoInterface with the status constants, so it represents the status more clearly.
Added/updated access check, users with "Administer Brightcove configuration" permission shouldn't get explicit access to (un)published videos in the search results.
Fixed some typos and coding standard issues.
Comment #7
fgmNot really dug in the patch, but regarding the wording changes, I think the changes from "and" to "or" are not correct: search will match all of the examples given, so it's really a "and", not a "or". The wording is copied from core, actually.
Comment #8
yce commentedYeah, sorry, I always mess this up with the logical and/or :)
Fixed.
Comment #10
yce commented