In normal Views, the query plugin’s query() method seems to be optional, as it is only used internally by the Sql query plugin itself.
However, as other modules integrating with Views also sometimes use it, it can’t hurt to override it in out query plugin and try to provide a reasonable return value.

Comments

drunken monkey created an issue. See original summary.

drunken monkey’s picture

Status: Active » Needs review

This should do it.

(Work paid for by https://jura.ku.dk/icourts/.)

drunken monkey’s picture

StatusFileSize
new983 bytes
ressa’s picture

Status: Needs review » Reviewed & tested by the community

I have tested the patch (see #189), and it works perfectly. Great work!

  • drunken monkey committed fea11c5 on 8.x-1.x
    Issue #3024753 by drunken monkey, ressa: Added overridden query() method...
drunken monkey’s picture

Status: Reviewed & tested by the community » Fixed

Alright, great to hear! Thanks for testing!
Committed.

Status: Fixed » Closed (fixed)

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