From aa29fc591401efdf0ee15a805d2b523444b5b531 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jacobo=20Aragunde=20P=C3=A9rez?= <jaragunde@igalia.com>
Date: Fri, 18 Mar 2022 16:11:32 +0100
Subject: [PATCH] Reuse getVacationsProjectId() inside getVacationsDates().

We simplify our code and remove one use of the VACATIONS_PROJECT
config parameter, which we may want to get rid of later.
---
 model/dao/TaskDAO/PostgreSQLTaskDAO.php            | 4 +++-
 model/dao/TaskDAO/TaskDAO.php                      | 2 +-
 model/facade/action/GetScheduledHolidaysAction.php | 5 +----
 3 files changed, 5 insertions(+), 6 deletions(-)

diff --git a/model/dao/TaskDAO/PostgreSQLTaskDAO.php b/model/dao/TaskDAO/PostgreSQLTaskDAO.php
index a2c4d21f4..fd3e0a028 100644
--- a/model/dao/TaskDAO/PostgreSQLTaskDAO.php
+++ b/model/dao/TaskDAO/PostgreSQLTaskDAO.php
@@ -621,8 +621,10 @@ public function getVacations(UserVO $userVO, DateTime $initDate = NULL, DateTime
 
     }
 
-    public function getVacationsDates(UserVO $userVO, int $projectId = NULL, DateTime $initDate = NULL, DateTime $endDate = NULL): array
+    public function getVacationsDates(UserVO $userVO, DateTime $initDate = NULL, DateTime $endDate = NULL): array
     {
+        $projectId = $this->getVacationsProjectId();
+
         if (is_null($projectId)) {
             return [];
         }
diff --git a/model/dao/TaskDAO/TaskDAO.php b/model/dao/TaskDAO/TaskDAO.php
index 1de311302..517753279 100644
--- a/model/dao/TaskDAO/TaskDAO.php
+++ b/model/dao/TaskDAO/TaskDAO.php
@@ -258,7 +258,7 @@ public abstract function getProjectUserWeeklyWorkingHours(ProjectVO $project, Da
      */
     public abstract function getVacations(UserVO $userVO, DateTime $initDate = NULL, DateTime $endDate = NULL);
 
-    public abstract function getVacationsDates(UserVO $userVO, int $projectId = NULL, DateTime $initDate = NULL, DateTime $endDate = NULL): array;
+    public abstract function getVacationsDates(UserVO $userVO, DateTime $initDate = NULL, DateTime $endDate = NULL): array;
 
     /** Task partial updater for PostgreSQL.
      *
diff --git a/model/facade/action/GetScheduledHolidaysAction.php b/model/facade/action/GetScheduledHolidaysAction.php
index 0fa04e21f..b6b4d2a9a 100644
--- a/model/facade/action/GetScheduledHolidaysAction.php
+++ b/model/facade/action/GetScheduledHolidaysAction.php
@@ -45,7 +45,6 @@ protected function doExecute(): array
 
         $taskDao = DAOFactory::getTaskDAO();
         $userDao = DAOFactory::getUserDAO();
-        $projectDao = DAOFactory::getProjectDAO();
 
         if (is_null($this->user)) {
             return [];
@@ -58,12 +57,10 @@ protected function doExecute(): array
         } else
             if (is_null($this->user->getId()))
             $this->user = $userDao->getByUserLogin($this->user->getLogin());
-
-        $projectId = $projectDao->getByDescription(ConfigurationParametersManager::getParameter('VACATIONS_PROJECT'));
         $reportInit = $this->init;
         $reportEnd = $this->end;
 
-        $vacations = $taskDao->getVacationsDates($this->user, $projectId, $reportInit, $reportEnd);
+        $vacations = $taskDao->getVacationsDates($this->user, $reportInit, $reportEnd);
 
         return $vacations;
     }