The Facet API dependency system allows for progressive disclosure of facets. For example, you can configure things like taxonomy facets to only be displayed when content types the field is attached to are active. Refer to #551620: Type dependent facet blocks for the use case.

For some reason the dependency system is not working with Search API. Specifically, the dependency form is not accessible. The path to the dependency form shoudl be admin/config/search/facetapi/%facetapi_adapter/%facetapi_realm/%facetapi_dependencies/dependencies and should show up as a contextual link. Navigating to that page manually shows a "page not found".

CommentFileSizeAuthor
#18 1215526--facetapi-bundle-dependency-18.patch1.47 KBdrunken monkey
#17 search_api-1215526-17.patch1.09 KBcpliakas
#1 dependency-link.png127.6 KBcpliakas
#1 dependency-form.png84.89 KBcpliakas

Comments

cpliakas’s picture

cpliakas commented
StatusFileSize
new dependency-form.png84.89 KB
new dependency-link.png127.6 KB

The attached screenshots show the dependency link and form respectively.

cpliakas’s picture

cpliakas commented

Also, very open to this being a Facet API bug.

drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
commented

I could imagine this not working for nested fields (e.g., taxonomy terms on the author) as Facet API probably wouldn't know that those are fields, too. However, no real idea what could cause this for normal fields, as I don't know how this is constructed in the first place.

cpliakas’s picture

cpliakas commented

Well, Facet API does know about the fields it is attached to based on the "field api name" key. There is an isssue posted at #1215520: The "field api name" key should only be set for Field API fields, not all properties. But even if that key isn't present or valid, then the "A content type this field is attached to must be active" option isn't presented and you still have the option to select the content types manually. Regardless, the form isn't even showing up to select the options. I suspect that the dependency plugins aren't being set in the facet info.

cpliakas’s picture

cpliakas commented
Status: Active » Needs work

Looks like the dependency plugins aren't getting set. I added them in commit 43f3220. The "A content type this field is attached to must be active" option will start working once #1215520: The "field api name" key should only be set for Field API fields, not all properties is resolved. The only thing that needs work is that we don't want a "bundle" dependency for "bundle" facets. Not sure how to check for that. Marking thread as "needs work" for that reason.

drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
commented

The only thing that needs work is that we don't want a "bundle" dependency for "bundle" facets.

What do you mean by that?

cpliakas’s picture

cpliakas commented

So let's take a simple example. Vocabulary "A" is attached to content type "B" via a term reference field. That field is exposed as a facet. We would use the dependency system to say, only show the facet for Vocabulary A when content type B is selected as an active item, and not content types C, D, E, and so on.

If bundles can be dependent on bundles, then the facet would never be displayed. In other words, the facet would only be shown if one of it's own values was selected, which makes no sense because how would you select it in the first place?

drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
commented

Ah, OK, now I get the whole system! Thanks for clearing that up!

However, how does the Facet API know whether a bundle facet is active? This currently doesn't seem to work with the Search API – do we have to do something for that?

cpliakas’s picture

cpliakas commented

Thinking about this further, there may be some assumptions I have to battle. Dependencies are simply "plugins", and all of the logic can be found in the execute() methods of the appropriate plugins. Take a look at the FacetapiDependencyBundle::execute() method for a working example. An evil troll must have come down in the middle of the night and hard coded "bundle" in there. Then he left and I never saw him again.

drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
commented

Oh, OK, that's nasty … Then I guess you'll really have to change that somehow. Or we could just use "bundle" as the key for bundle fields as a work-around. If that would work … (Hm, it wouldn't, without us then re-converting when doing the actual filtering.)
Or we just remove "bundle" from the dependency plugins for now.

cpliakas’s picture

cpliakas commented

Posted an issue at #1222754: Resolve assumptions in the "bundle" dependency plugin to resolve this assumption. Also gave credit to Search API in the "Attribution" section of the Facet API project page for helping to identify such assumptions.

drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
commented
Title: Dependency system not working » Add support for the "Bundle" dependency plugin
Category: bug » feature
Status: Needs work » Active

I removed the declaration of the "bundle" dependency plugin support for now, so users won't be confused. Supporting it will then be a feature request, I guess.

cpliakas’s picture

cpliakas commented

I don't mind working on this. Adding support should be pretty straight forward, we just need to know the machine readable names of the Field API fields and whether is contains bundle data or not. I ended up resolving the remaining assumptions in the Facet API module, so HEAD should be ready for Search API integration.

cpliakas’s picture

cpliakas commented
Assigned: Unassigned » cpliakas

Assigning to myself, just as a reminder to work on it. Of course, anyone who wants to take this on is more than welcome.

cpliakas’s picture

cpliakas commented
Project: » Search API
Version: » 7.x-1.x-dev
Component: Code » Facets

Moving to Search API module.

cpliakas’s picture

cpliakas commented
Issue tags: +Facet API integration

Tagging with "Facet API Integration".

cpliakas’s picture

cpliakas commented
Status: Active » Needs review
StatusFileSize
new search_api-1215526-17.patch1.09 KB

The attached patch adds the "field api bundles" key for fields that contain bundle data. For all other fields, the bundle dependency is activated.

drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
commented
StatusFileSize
new 1215526--facetapi-bundle-dependency-18.patch1.47 KB

OK, that seems easy enough. Had to fix some notices and Entity API weirdnesses, though. Revised patch attached.

However, as long as dependencies don't work at all at the moment (#1337292: Facet dependency system doesn't work) it's probably rather pointless to add this now. As well as hard to test.

cpliakas’s picture

cpliakas commented

Agreed. My work on this issue is what helped discover the issue, and clearly this one is dependent on the other.

Thanks for reviewing! Like your changes. At least we have this on the back burner.
~Chris

drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
commented

Everything seems to work fine now with #1337292: Facet dependency system doesn't work committed and the patch in #18.
Unless there are any objections, I'm gonna commit that on Monday or Tuesday.

cpliakas’s picture

cpliakas commented
Status: Needs review » Reviewed & tested by the community

Confirming that the patch in #18 works.

drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
commented
Status: Reviewed & tested by the community » Fixed

Thanks for testing!
Committed.

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