Sortable AMI sets

Had to break the Listing Cache (documented like nowhere) to allow our theme + Sorts to actually change the Table (if not the rendered version got stuck)

Author: DiegoPino

src/Entity/Controller/amiSetEntityListBuilder.php

@@ -6,13 +6,13 @@
 use Drupal\Core\Entity\EntityInterface;
 use Drupal\Core\Entity\EntityListBuilder;
 use Drupal\ami\AmiUtilityService;
+use Drupal\Core\Entity\Query\QueryInterface;
 
-
-  /**
-   * Provides a list controller for the Ami Set entity.
-   *
-   * @ingroup ami
-   */
+/**
+ * Provides a list controller for the Ami Set entity.
+ *
+ * @ingroup ami
+ */
 class amiSetEntityListBuilder extends EntityListBuilder {
 
   /**
@@ -44,10 +44,32 @@ public function render() {
    */
   public function buildHeader() {
     $header['uuid'] = $this->t('Set UUID');
-    $header['id'] = $this->t('Set ID');
-    $header['name'] = $this->t('Name');
-    $header['last update'] = $this->t('Last update');
-    $header['status'] = $this->t('Status');
+    $header['id'] = [
+      'data' => $this->t('Set ID'),
+      'class' => [RESPONSIVE_PRIORITY_LOW],
+      'field' => 'id',
+      'specifier' => 'id',
+      'sort' => 'asc',
+    ];
+    $header['name'] = [
+      'data' => $this->t('Name'),
+      'class' => [RESPONSIVE_PRIORITY_MEDIUM],
+      'field' => 'name',
+      'specifier' => 'name',
+    ];
+    $header['changed'] =  [
+      'data' => $this->t('Last update'),
+      'class' => [RESPONSIVE_PRIORITY_LOW],
+      'field' => 'changed',
+      'specifier' => 'changed',
+    ];
+    $header['status'] = [
+      'data' => $this->t('Status'),
+      'class' => [RESPONSIVE_PRIORITY_LOW],
+      'field' => 'status',
+      'specifier' => 'status',
+    ];
+
     return $header + parent::buildHeader();
   }
 
@@ -56,15 +78,33 @@ public function buildHeader() {
    */
   public function buildRow(EntityInterface $entity) {
     /* @var $entity \Drupal\ami\Entity\amiSetEntity */
-    $row['uuid'] = $entity->uuid();
-    $row['id'] = $entity->id();
-    $row['name'] = $entity->toLink();
-    $row['last update'] = \Drupal::service('date.formatter')->format($entity->changed->value, 'custom', 'd/m/Y');
+    $row['uuid']['data'] = $entity->uuid();
+    $row['id']['data'] = $entity->id();
+    $row['name']['data'] = $entity->toLink();
+    $row['changed']['data'] = \Drupal::service('date.formatter')->format($entity->changed->value, 'custom', 'd/m/Y');
     $status_code = !empty($entity->getStatus()->first()->getValue()) ? $entity->getStatus()->first()->getValue()['value'] : amiSetEntity::STATUS_NOT_READY;
-    $row['status'] = amiSetEntity::STATUS[strtoupper($status_code)] ?? amiSetEntity::STATUS[amiSetEntity::STATUS_NOT_READY];
+    $row['status']['data'] = amiSetEntity::STATUS[strtoupper($status_code)] ?? amiSetEntity::STATUS[amiSetEntity::STATUS_NOT_READY];
     return $row + parent::buildRow($entity);
   }
 
+  /**
+   * @inheritDoc
+   */
+  protected function getEntityListQuery(): QueryInterface {
+    $query = $this->getStorage()->getQuery()
+      ->accessCheck(TRUE);
+    $header = $this->buildHeader();
+    $query->tableSort($header);
+    // Only add the pager if a limit is specified.
+    if ($this->limit) {
+      $query->pager($this->limit);
+    }
+    return $query;
+  }
+
+
+
+
   public function getOperations(EntityInterface $entity) {
     $operations = parent::getOperations($entity); // TODO: Change the autogenerated stub
     if ($entity->access('process') && $entity->hasLinkTemplate('process-form')) {

src/Entity/amiSetEntity.php

@@ -92,6 +92,7 @@
  *     "access" = "Drupal\ami\Entity\Controller\amiSetEntityAccessControlHandler",
  *   },
  *   base_table = "ami_setentity",
+ *   list_cache_contexts = {"user", "url"},
  *   admin_permission = "administer amiset entity",
  *   fieldable = TRUE,
  *   entity_keys = {