Missing handler: node term_node_tid relationship

Last time I ran the tests, that was a debug() though, not a PHP notice -- is that no longer the case?

If it's still a debug(), then I'd rather ignore it. We can add the dependency on taxonomy when we'll get that taxonomy view to work... ;)

this actually spits out various warning messages when editing the view admin_views_node as well:

Missing handler: taxonomy_term_data vid filter
Missing handler: node term_node_tid relationship
Missing handler: taxonomy_term_data vid filter
Missing handler: node term_node_tid relationship
Missing handler: node term_node_tid relationship

the problem is that taxonomy is not declared as a dependency of this module. i just came across this while testing something else on a site using the minimal install profile, which doesn't enable the taxonomy module.

trivial patch attached.

changing the title & status since i reproduced this outside the test suite.

@damiankloip: um, am i missing something here? you don't think the admin_views_node view is useful? isn't that what this project is for?

hmm, perhaps you just meant the vocabulary filter? in that case, no objection here.

Yes, because of the way the bundled views are constructed, the Taxonomy module is required.

Alternatively, how about only loading the relationship and filter if the Taxonomy module is enabled?

I like this method better, since it allows the admin_views to be used without taxonomy needing to be enabled.

I tested the patch and the notices disappeared from the tests, so I suppose they are gone from the UI as well.

Committed. Thanks.