We have this interface, but it's empty at the moment. I think this is a good idea to implement, as stuff like efq will want to implement their own query class, and an interface will help with that.

postponing on #1941612: Tidy up view query plugin methods as that changes the method names.

There is nothing stopping the discussion of what methods should atleast be contained in this interface though.

Files: 
CommentFileSizeAuthor
#7 vdc-1941672-7.patch1.16 KBdawehner
PASSED: [[SimpleTest]]: [MySQL] 58,487 pass(es).
[ View ]
#2 1941672-2.patch11.08 KBpwolanin
PASSED: [[SimpleTest]]: [MySQL] 58,554 pass(es).
[ View ]

Comments

pwolanin’s picture

Status:Postponed» Active

Seems the plugin methods were already tidied as part of #1856630: [Change notice] [META] Rename Views methods to core standards, e.g. [##2003270]

pwolanin’s picture

StatusFileSize
new11.08 KB
PASSED: [[SimpleTest]]: [MySQL] 58,554 pass(es).
[ View ]

need some doxygen filled in yet, but here's a start.

pwolanin’s picture

Status:Active» Needs review
tim.plunkett’s picture

+++ b/core/modules/views/lib/Drupal/views/Plugin/views/query/QueryInterface.php
@@ -2,16 +2,173 @@
+  public function validateOptionsForm(&$form, &$form_state);
...
+  public function submitOptionsForm(&$form, &$form_state);

See, here's why these are hard to do. These methods should be on ViewsPluginInterface or something wayyyy higher up

dawehner’s picture

MHHH I don't think that views is in the state of having usable interfaces.

As written from tim in #4, that is d9 content.

damiankloip’s picture

Agreed, this issue is seriously old and outdated. It was probably forgotten about for a good reason :-) As with all other views plugins, you basically always extend the base class. That's how things are for views in d8. I think we pretty much all agreed they were a no go?

dawehner’s picture

StatusFileSize
new1.16 KB
PASSED: [[SimpleTest]]: [MySQL] 58,487 pass(es).
[ View ]

I also think that for core it does not make sense to write a search views query interface.

Here is a slightly different approach.

damiankloip’s picture

Title:Add methods to Drupal\views\Plugin\query\QueryInterface» Remove Drupal\views\Plugin\query\QueryInterface

So..

tim.plunkett’s picture

Title:Remove Drupal\views\Plugin\query\QueryInterface» Add methods to Drupal\views\Plugin\query\QueryInterface
Status:Needs review» Reviewed & tested by the community

Yeah, let's be honest here.

tim.plunkett’s picture

Title:Add methods to Drupal\views\Plugin\query\QueryInterface» Remove Drupal\views\Plugin\query\QueryInterface
klonos’s picture

Does that mean that #2083717: Convert Search Results to Views won't happen after all? :/

Status:Reviewed & tested by the community» Needs work

The last submitted patch, vdc-1941672-7.patch, failed testing.

damiankloip’s picture

Why not?

damiankloip’s picture

Status:Needs work» Needs review

#7: vdc-1941672-7.patch queued for re-testing.

damiankloip’s picture

Status:Needs review» Reviewed & tested by the community
dawehner’s picture

I think it will still happen: #2083717-6: Convert Search Results to Views, just different.

klonos’s picture

Ok, that's encouraging news.

catch’s picture

Status:Reviewed & tested by the community» Fixed

Committed/pushed to 8.x, thanks!

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

derhasi’s picture