Add an interface for operators() on views

Rerolled 3320569-2.patch against 11.x in new MR.

All patches after that show that c/p-ing rough example code isn't really helping, not even when tried repeatingly.

Updated IS and MR and added draft CR.

That's enough two-lettered capitalized words for me to put it on NR.

CR reads well, IS is clear and matches MR.

So I believe this is OK.

Couldn't think of anyone two letter word to use haha.

Added some feedback on the MR. Maybe we should split this into two issues.

Resolved all threads.

2 small nitpicky stuff.

Rest looks fine though

Probably good to self RTBC after those 2.

I'm triaging RTBC issues → . I read the IS, the comments and MR. I didn't find any unanswered questions or other work to do.

The change record does need to be updated. It should state that using the operator method without the interface will throw an exception in Drupal 11.

And the title of the CR could be more specific about where the operators method is used. Perhaps this is better, "Classes using operators() when extending \Drupal\views\Plugin\views\filter\FilterPluginBase must implement FilterOperatorsInterface'?

Thanks @quietone, updated CR.

I updated the deprecation to throwing an exception in 12.0.0 instead of 11.0.0, otherwise looks great. Committed/pushed to 11.x and cherry-picked to 10.3.x, thanks!

catch → closed merge request !4250

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