Issue #2080131 by damiankloip: Disable 'Aggregation settings' for Views...

Issue #2080131 by damiankloip: Disable 'Aggregation settings' for Views handlers that should not support aggregation.

core/modules/comment/lib/Drupal/comment/Plugin/views/field/Comment.php

@@ -22,6 +22,13 @@
  */
 class Comment extends FieldPluginBase {
 
+  /**
+   * {@inheritdoc}
+   */
+  public function usesGroupBy() {
+    return FALSE;
+  }
+
   /**
    * Overrides \Drupal\views\Plugin\views\field\FieldPluginBase::init().
    *

core/modules/comment/lib/Drupal/comment/Plugin/views/field/NodeNewComments.php

@@ -24,6 +24,13 @@
  */
 class NodeNewComments extends Numeric {
 
+  /**
+   * {@inheritdoc}
+   */
+  public function usesGroupBy() {
+    return FALSE;
+  }
+
   /**
    * Database Service Object.
    *

core/modules/contextual/lib/Drupal/contextual/Plugin/views/field/ContextualLinks.php

@@ -20,6 +20,13 @@
  */
 class ContextualLinks extends FieldPluginBase {
 
+  /**
+   * {@inheritdoc}
+   */
+  public function usesGroupBy() {
+    return FALSE;
+  }
+
   protected function defineOptions() {
     $options = parent::defineOptions();
 

core/modules/history/lib/Drupal/history/Plugin/views/field/HistoryUserTimestamp.php

@@ -25,6 +25,13 @@
  */
 class HistoryUserTimestamp extends Node {
 
+  /**
+   * {@inheritdoc}
+   */
+  public function usesGroupBy() {
+    return FALSE;
+  }
+
   /**
    * Overrides \Drupal\node\Plugin\views\field\Node::init().
    */

core/modules/history/lib/Drupal/history/Plugin/views/filter/HistoryUserTimestamp.php

@@ -25,6 +25,13 @@ class HistoryUserTimestamp extends FilterPluginBase {
   // Don't display empty space where the operator would be.
   var $no_operator = TRUE;
 
+  /**
+   * {@inheritdoc}
+   */
+  public function usesGroupBy() {
+    return FALSE;
+  }
+
   public function buildExposeForm(&$form, &$form_state) {
     parent::buildExposeForm($form, $form_state);
     // @todo There are better ways of excluding required and multiple (object flags)

core/modules/node/lib/Drupal/node/Plugin/views/field/Link.php

@@ -20,6 +20,13 @@
  */
 class Link extends FieldPluginBase {
 
+  /**
+   * {@inheritdoc}
+   */
+  public function usesGroupBy() {
+    return FALSE;
+  }
+
   protected function defineOptions() {
     $options = parent::defineOptions();
     $options['text'] = array('default' => '', 'translatable' => TRUE);

core/modules/user/lib/Drupal/user/Plugin/views/field/Link.php

@@ -23,6 +23,13 @@
  */
 class Link extends FieldPluginBase {
 
+  /**
+   * {@inheritdoc}
+   */
+  public function usesGroupBy() {
+    return FALSE;
+  }
+
   /**
    * Overrides Drupal\views\Plugin\views\field\FieldPluginBase::init().
    */

core/modules/views/lib/Drupal/views/Plugin/views/field/Counter.php

@@ -19,6 +19,14 @@
  */
 class Counter extends FieldPluginBase {
 
+  /**
+   * {@inheritdoc}
+   */
+  public function usesGroupBy() {
+    return FALSE;
+  }
+
+
   protected function defineOptions() {
     $options = parent::defineOptions();
     $options['counter_start'] = array('default' => 1);

core/modules/views/lib/Drupal/views/Plugin/views/field/Custom.php

@@ -19,6 +19,13 @@
  */
 class Custom extends FieldPluginBase {
 
+  /**
+   * {@inheritdoc}
+   */
+  public function usesGroupBy() {
+    return FALSE;
+  }
+
   public function query() {
     // do nothing -- to override the parent query.
   }

core/modules/views/lib/Drupal/views/Plugin/views/field/Links.php

@@ -14,6 +14,13 @@
  */
 abstract class Links extends FieldPluginBase {
 
+  /**
+   * {@inheritdoc}
+   */
+  public function usesGroupBy() {
+    return FALSE;
+  }
+
   /**
    * Overrides \Drupal\views\Plugin\views\field\FieldPluginBase::defineOptions().
    */

core/modules/views/lib/Drupal/views/Plugin/views/sort/Random.php

@@ -16,6 +16,13 @@
  */
 class Random extends SortPluginBase {
 
+  /**
+   * {@inheritdoc}
+   */
+  public function usesGroupBy() {
+    return FALSE;
+  }
+
   public function query() {
     $this->query->addOrderBy('rand');
   }