From 3b9ca530881c6b1f9398ad59a9f8d32273ca7b9c Mon Sep 17 00:00:00 2001
From: Thierry Bugier <tbugier@teclib.com>
Date: Tue, 28 May 2024 15:42:36 +0200
Subject: [PATCH] fix(formanswer): access for ticket valdiator

---
 inc/formanswer.class.php | 69 ++++++++++++++++++++++++++++++++++++++++
 1 file changed, 69 insertions(+)

diff --git a/inc/formanswer.class.php b/inc/formanswer.class.php
index e3c398c7f..678c3ea8d 100644
--- a/inc/formanswer.class.php
+++ b/inc/formanswer.class.php
@@ -167,6 +167,23 @@ public function canViewItem() {
          }
       }
 
+      if ($this->userIsTicketActor()) {
+         return true;
+      }
+
+
+      if ($this->userIsTicketValidator()) {
+         return true;
+      }
+
+      return false;
+   }
+
+   public function userIsTicketActor(): bool {
+      global $DB;
+
+      $currentUser = Session::getLoginUserID();
+
       // Check if the current user is a requester of a ticket linked to a form answer typed
       // Matches search option 42, 43 and 44 of PluginFormcreatorIssue (requester, watcher, assigned)
       $ticket_table = Ticket::getTable();
@@ -208,6 +225,52 @@ public function canViewItem() {
       return false;
    }
 
+   public function userIsTicketValidator(): bool {
+      global $DB;
+
+      $currentUser = Session::getLoginUserID();
+
+      //  Check if the current user is a validator of a ticket linked to a form answer typed
+      $ticket_table = Ticket::getTable();
+      $ticketvalidation_table = TicketValidation::getTable();
+      $item_ticket_table = Item_Ticket::getTable();
+      $request = [
+         'SELECT' => [
+            TicketValidation::getTableField(User::getForeignKeyField() . '_validate'),
+            Ticket::getTableField('id'),
+         ],
+         'FROM' => $ticketvalidation_table,
+         'INNER JOIN' => [
+            $ticket_table => [
+               'FKEY' => [
+                  $ticket_table => 'id',
+                  $ticketvalidation_table => 'tickets_id',
+                  ['AND' => [
+                     TicketValidation::getTableField(User::getForeignKeyField() . '_validate') => $currentUser,
+                  ]],
+               ],
+            ],
+            $item_ticket_table => [
+               'FKEY' => [
+                  $item_ticket_table => 'tickets_id',
+                  $ticket_table => 'id',
+                  ['AND' => [
+                     Item_Ticket::getTableField('itemtype') => self::getType(),
+                     Item_Ticket::getTableField('items_id') => $this->getID(),
+                  ]],
+               ],
+            ],
+         ]
+      ];
+
+      if ($DB->request($request)->count() > 0) {
+         return true;
+      }
+
+      return false;
+
+   }
+
    public static function canPurge() {
       return true;
    }
@@ -581,6 +644,12 @@ public function showForm($ID, $options = []) {
       if (!isset($ID) || !$this->getFromDB($ID)) {
          Html::displayNotFoundError();
       }
+
+      if ($this->canViewItem() && !$this->userIsTicketActor()) {
+         echo '<div class="alert alert-danger">' . __('You are not allowed to view this answer.') . '</div>';
+         return false;
+      }
+
       $options['canedit'] = false;
 
       // Print css media