From fa5cb8adf3a9c185cbfb594f7b560fac30c0e465 Mon Sep 17 00:00:00 2001
From: Kamil Tekiela <tekiela246@gmail.com>
Date: Wed, 22 Mar 2023 10:05:39 +0000
Subject: [PATCH] Refactor getBulkLinks

Signed-off-by: Kamil Tekiela <tekiela246@gmail.com>
---
 libraries/classes/Display/Results.php | 29 ++++++++++++++-------------
 psalm-baseline.xml                    |  4 ++++
 templates/display/results/table.twig  |  6 +++---
 3 files changed, 22 insertions(+), 17 deletions(-)

diff --git a/libraries/classes/Display/Results.php b/libraries/classes/Display/Results.php
index 34258881fef5..cf235e046906 100644
--- a/libraries/classes/Display/Results.php
+++ b/libraries/classes/Display/Results.php
@@ -3553,7 +3553,7 @@ public function getTable(
         $this->properties['display_params'] = null;
 
         // 4. ----- Prepares the link for multi-fields edit and delete
-        $bulkLinks = $this->getBulkLinks($dtResult, $statementInfo, $displayParts->deleteLink);
+        $isClauseUnique = $this->isClauseUnique($dtResult, $statementInfo, $displayParts->deleteLink);
 
         // 5. ----- Prepare "Query results operations"
         $operations = [];
@@ -3568,7 +3568,9 @@ public function getTable(
             'navigation' => $navigation,
             'headers' => $headers,
             'body' => $body,
-            'bulk_links' => $bulkLinks,
+            'has_bulk_links' => $displayParts->deleteLink === DisplayParts::DELETE_ROW,
+            'has_export_button' => $this->hasExportButton($statementInfo, $displayParts->deleteLink),
+            'clause_is_unique' => $isClauseUnique,
             'operations' => $operations,
             'db' => $this->properties['db'],
             'table' => $this->properties['table'],
@@ -3873,19 +3875,16 @@ private function setParamForLinkForeignKeyRelatedTables(array $map): array
      *
      * @see     getTable()
      *
-     * @param ResultInterface $dtResult the link id associated to the query which
-     *                          results have to be displayed
+     * @param ResultInterface $dtResult the link id associated to the query which results have to be displayed
      * @psalm-param DisplayParts::NO_DELETE|DisplayParts::DELETE_ROW|DisplayParts::KILL_PROCESS $deleteLink
-     *
-     * @psalm-return array{has_export_button:bool, clause_is_unique:mixed}|array<empty, empty>
      */
-    private function getBulkLinks(
+    private function isClauseUnique(
         ResultInterface $dtResult,
         StatementInfo $statementInfo,
         int $deleteLink,
-    ): array {
+    ): bool {
         if ($deleteLink !== DisplayParts::DELETE_ROW) {
-            return [];
+            return false;
         }
 
         // fetch last row of the result set
@@ -3894,7 +3893,7 @@ private function getBulkLinks(
 
         $expressions = [];
 
-        if (isset($statementInfo->statement) && $statementInfo->statement instanceof SelectStatement) {
+        if ($statementInfo->statement instanceof SelectStatement) {
             $expressions = $statementInfo->statement->expr;
         }
 
@@ -3914,10 +3913,12 @@ private function getBulkLinks(
         // reset to first row for the loop in getTableBody()
         $dtResult->seek(0);
 
-        return [
-            'has_export_button' => $statementInfo->queryType === 'SELECT',
-            'clause_is_unique' => $clauseIsUnique,
-        ];
+        return $clauseIsUnique;
+    }
+
+    private function hasExportButton(StatementInfo $statementInfo, int $deleteLink): bool
+    {
+        return $deleteLink === DisplayParts::DELETE_ROW && $statementInfo->queryType === 'SELECT';
     }
 
     /**
diff --git a/psalm-baseline.xml b/psalm-baseline.xml
index dc1a8320b505..f8409231aeb0 100644
--- a/psalm-baseline.xml
+++ b/psalm-baseline.xml
@@ -6576,6 +6576,7 @@
     </MixedAssignment>
     <MixedInferredReturnType>
       <code>Message</code>
+      <code>bool</code>
     </MixedInferredReturnType>
     <MixedMethodCall>
       <code>new $className()</code>
@@ -6602,6 +6603,9 @@
       <code><![CDATA[$statementInfo->statement->from[0]->table]]></code>
       <code><![CDATA[$statementInfo->statement->limit->offset]]></code>
     </MixedPropertyFetch>
+    <MixedReturnStatement>
+      <code>$clauseIsUnique</code>
+    </MixedReturnStatement>
     <MixedReturnTypeCoercion>
       <code>$map</code>
       <code><![CDATA[[
diff --git a/templates/display/results/table.twig b/templates/display/results/table.twig
index 3ae0c87215bd..09e01526251e 100644
--- a/templates/display/results/table.twig
+++ b/templates/display/results/table.twig
@@ -298,7 +298,7 @@
     </table>
   </div>
 
-{% if bulk_links is not empty %}
+{% if has_bulk_links %}
     <div class="d-print-none">
       <img class="selectallarrow" src="{{ image('arrow_' ~ text_dir ~ '.png') }}" width="38" height="22" alt="{% trans 'With selected:' %}">
       <input type="checkbox" id="resultsForm_{{ unique_id }}_checkall" class="checkall_box" title="{% trans 'Check all' %}">
@@ -317,14 +317,14 @@
         {{ get_icon('b_drop', 'Delete'|trans) }}
       </button>
 
-      {% if bulk_links.has_export_button %}
+      {% if has_export_button %}
         <button class="btn btn-link mult_submit" type="submit" name="submit_mult" value="export" title="{% trans 'Export' %}">
           {{ get_icon('b_tblexport', 'Export'|trans) }}
         </button>
       {% endif %}
     </div>
 
-    <input type="hidden" name="clause_is_unique" value="{{ bulk_links.clause_is_unique }}">
+    <input type="hidden" name="clause_is_unique" value="{{ clause_is_unique }}">
     <input type="hidden" name="sql_query" value="{{ sql_query }}">
   </form>
 {% endif %}