From 4213582aabb1c149b33dcec61cb967df3e117d9f Mon Sep 17 00:00:00 2001
From: Pilar Candia <pilar@2600hz.com>
Date: Wed, 6 Mar 2024 13:12:58 -0500
Subject: [PATCH] [5.3] MSPB-360: Set 'start_date' and 'cycle=date' for non
 recurring holidays (#525)

* MSPB-360: Set 'start_date' and 'cycle=date' for non recurring holidays

* end_date should use the user/account timezone
---
 .../strategyHolidays/strategyHolidays.js      | 24 +++++++++++++++----
 1 file changed, 20 insertions(+), 4 deletions(-)

diff --git a/submodules/strategyHolidays/strategyHolidays.js b/submodules/strategyHolidays/strategyHolidays.js
index 897ac315..3b79b46a 100644
--- a/submodules/strategyHolidays/strategyHolidays.js
+++ b/submodules/strategyHolidays/strategyHolidays.js
@@ -997,6 +997,20 @@ define(function(require) {
 			return ordinalId === 5 ? _.last(datesArray) : datesArray[ordinalId];
 		},
 
+		/**
+		 * Returns Date
+		 * @param  {string} selectedYear
+		 * @param  {Object} holiday
+		 * @return {Date}
+		 */
+		strategyHolidaysGetStartDate: function(selectedYear, holiday) {
+			var self = this,
+				holidayData = holiday.holidayData,
+				startDate = new Date(selectedYear, holidayData.fromMonth - 1, holidayData.fromDay);
+
+			return monster.util.dateToBeginningOfGregorianDay(startDate, monster.util.getCurrentTimeZone());
+		},
+
 		/**
 		 * Returns Date
 		 * @param  {string} endYear
@@ -1044,7 +1058,7 @@ define(function(require) {
 				id = holidayData.id,
 				endDate = holidayData.recurring
 					? null
-					: monster.util.dateToEndOfGregorianDay(self.strategyHolidaysGetEndDate(holidayData.endYear, holiday)),
+					: monster.util.dateToEndOfGregorianDay(self.strategyHolidaysGetEndDate(holidayData.endYear, holiday), monster.util.getCurrentTimeZone()),
 				holidayRuleConfig = _.merge({
 					oldType: holidayData.set ? 'set' : 'rule'
 				}, id && {
@@ -1056,6 +1070,7 @@ define(function(require) {
 					'time_window_start',
 					'time_window_stop'
 				]), endDate && {
+					start_date: self.strategyHolidaysGetStartDate(holidayData.endYear, holiday),
 					end_date: endDate
 				}),
 				holidayRule = {};
@@ -1072,7 +1087,7 @@ define(function(require) {
 			} else {
 				holidayRule = _.merge({}, holidayRuleConfig, {
 					name: name,
-					cycle: 'yearly',
+					cycle: holidayData.recurring ? 'yearly' : 'date',
 					interval: 1,
 					month: month,
 					type: 'main_holidays'
@@ -1134,10 +1149,11 @@ define(function(require) {
 						days = _.range(fromDay, toDay + 1),
 						rule = {
 							name: name + '_' + month,
-							cycle: 'yearly',
+							cycle: data.start_date ? 'date' : 'yearly',
 							days: days,
 							interval: 1,
-							month: month
+							month: month,
+							start_date: data.start_date
 						};
 
 					if (data.exclude) {