From f79a2ee5e7402419293796da4fd135837ec2d126 Mon Sep 17 00:00:00 2001
From: Rafael de Camargo <rafaelcamargo@MacBook-Air-de-Rafael.local>
Date: Thu, 10 Jan 2019 01:17:03 -0200
Subject: Fix permission check before "Assign to me"

Users who should not be able to change assignee cannot "assign to me"

Fixes #4121
---
 app/Template/task/dropdown.php | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'app/Template/task')

diff --git a/app/Template/task/dropdown.php b/app/Template/task/dropdown.php
index 1bad6615..cc97d1a1 100644
--- a/app/Template/task/dropdown.php
+++ b/app/Template/task/dropdown.php
@@ -2,7 +2,7 @@
     <a href="#" class="dropdown-menu dropdown-menu-link-icon"><strong>#<?= $task['id'] ?> <i class="fa fa-caret-down"></i></strong></a>
     <ul>
         <?php if ($this->projectRole->canUpdateTask($task)): ?>
-            <?php if (array_key_exists('owner_id', $task) && $task['owner_id'] != $this->user->getId()): ?>
+            <?php if ($this->projectRole->canChangeAssignee($task) && array_key_exists('owner_id', $task) && $task['owner_id'] != $this->user->getId()): ?>
             <li>
                 <?= $this->url->icon('hand-o-right', t('Assign to me'), 'TaskModificationController', 'assignToMe', ['task_id' => $task['id'], 'project_id' => $task['project_id'], 'redirect' => isset($redirect) ? $redirect : '']) ?>
             </li>
-- 
cgit v1.2.3