From 3877038a70300f0e7df0ea91c859792754de0026 Mon Sep 17 00:00:00 2001
From: mdickson-adbe <95774602+mdickson-adbe@users.noreply.github.com>
Date: Fri, 9 Aug 2024 09:56:06 -0400
Subject: [PATCH] Assets 98992 (#153)

* add requested changes

- update start/end date properties
- change tooltip on 'end date' pill
- update filter to show current view
- disable increment/decrement if no add'l years to show

* add enhanced date checking

* fix typo, condense code

* remove intentional wrong property

* fix bug when calendar is refreshed

* disable increment/decrement instead of hide
---
 .../gmo-program-details.css                   |  7 +++++
 .../gmo-program-details.js                    |  2 +-
 scripts/program-calendar.js                   | 31 +++++++++++++------
 3 files changed, 30 insertions(+), 10 deletions(-)

diff --git a/blocks/gmo-program-details/gmo-program-details.css b/blocks/gmo-program-details/gmo-program-details.css
index e0ddd88..aa20114 100644
--- a/blocks/gmo-program-details/gmo-program-details.css
+++ b/blocks/gmo-program-details/gmo-program-details.css
@@ -224,10 +224,17 @@ body {
                         transform: rotate(180deg);
                     }
                 }
+                &.disabled {
+                    background-color: #c6c6c6;
+                    cursor: default;
+                }
             }
             & .current-year {
                 line-height: 32px;
                 margin-left: 20px;
+                &.single {
+                    margin-left: unset;
+                }
             }
         }
         & .right-controls {
diff --git a/blocks/gmo-program-details/gmo-program-details.js b/blocks/gmo-program-details/gmo-program-details.js
index 88cc9a0..4ed3901 100644
--- a/blocks/gmo-program-details/gmo-program-details.js
+++ b/blocks/gmo-program-details/gmo-program-details.js
@@ -195,7 +195,7 @@ export default async function decorate(block) {
                     <div class="today-button">Today</div>
                     <div class="filter-dropdown-wrapper">
                         <div class="filter-dropdown-button">
-                            <div class="label">Filter</div>
+                            <div class="label">Selected View: Year</div>
                             <span class="icon icon-chevronDown"></span>
                             <span class="icon icon-chevronUp inactive"></span>
                         </div>
diff --git a/scripts/program-calendar.js b/scripts/program-calendar.js
index 5cc2665..7bb73b1 100644
--- a/scripts/program-calendar.js
+++ b/scripts/program-calendar.js
@@ -3,8 +3,9 @@ import { searchAsset } from '../../scripts/assets.js';
 
 let deliverables, deliverableMapping;
 let viewStart, viewEnd;
-const startDateProp = 'deliverableProjectStartDate';
-const endDateProp = 'deliverableProjectEndDate';
+
+const startDateProp = 'taskPlannedStartDate';
+const endDateProp = 'taskPlannedEndDate';
 const taskStatusMappings = await getMappingArray('taskStatus');
 
 // Helper function to get task status mapping
@@ -30,7 +31,7 @@ export async function buildCalendar(dataObj, block, type, mappingArray, period)
 
     // get start of the view
     viewStart = getTimeBounds(deliverables, "start", startDateProp);
-    viewStart = (viewStart.getFullYear() === 1969) ? programLaunchDate : viewStart;    
+    viewStart = (!(isValidDate(viewStart)) || viewStart.getFullYear() === 1969) ? programLaunchDate : viewStart;   
     const viewStartYear = viewStart.getUTCFullYear();
 
     const displayYear = period ? period.year : viewStartYear;
@@ -42,7 +43,7 @@ export async function buildCalendar(dataObj, block, type, mappingArray, period)
 
     // get end of the view
     viewEnd = getTimeBounds(deliverables, "end", endDateProp);
-    if (viewEnd.getFullYear() === 1969) {
+    if (!(isValidDate(viewEnd)) || viewEnd.getFullYear() === 1969) {
         viewEnd = new Date(viewStart);
         viewEnd.setMonth(viewStart.getMonth() + 1);
     }
@@ -51,6 +52,11 @@ export async function buildCalendar(dataObj, block, type, mappingArray, period)
     // get array of all years to be included
     let years = calendarYears(viewStartYear, viewEndYear);
 
+    // disable increment/decrement if only one year in view
+    if (years.length === 1) {
+        document.querySelector('.inc-dec-wrapper > .year-switch').classList.add('disabled');
+    }
+
     // build the calendar background here as we already know the period and style
     let calendarEl;
     if (type === "year") {
@@ -82,10 +88,9 @@ export async function buildCalendar(dataObj, block, type, mappingArray, period)
 
         // find the earliest date- this is how we set the position for the group against the calendar
         let earliestStartDate = getTimeBounds(matchedItems, "start", startDateProp);
-        earliestStartDate = (earliestStartDate.getFullYear() === 1969) ? viewStart : earliestStartDate;
+        earliestStartDate = (!(isValidDate(earliestStartDate)) || earliestStartDate.getFullYear() === 1969) ? new Date(viewStart) : earliestStartDate;
         let latestEndDate = getTimeBounds(matchedItems, "end", endDateProp);
-        latestEndDate = (latestEndDate.getFullYear() === 1969) ? viewEnd : latestEndDate;
-
+        latestEndDate = (!(isValidDate(latestEndDate)) || latestEndDate.getFullYear() === 1969) ? new Date(viewEnd) : latestEndDate;
         const startMonth = (earliestStartDate.getUTCMonth()); // getMonth returns 0-11 but this is desirable
         const startDay = (earliestStartDate.getUTCDate() - 1); // if at start of month, we don't want to add any more margin
         const endMonth = (latestEndDate.getUTCMonth());
@@ -153,7 +158,7 @@ export async function buildCalendar(dataObj, block, type, mappingArray, period)
                         </div>
                     </div>
                     <div class="content-row bottom">
-                        ${itemEndDateStr ? '<div class="start-date" title="Task End Date: ' + itemEndDateStr + '">End Date: ' + itemEndDateStr + '</div>' : ''}
+                        ${itemEndDateStr ? '<div class="start-date" title="Task Planned End Date: ' + itemEndDateStr + '">End Date: ' + itemEndDateStr + '</div>' : ''}
                         <div class="link">
                             <a href="${item.reviewLink}">QA Files</a>
                         </div>
@@ -256,7 +261,9 @@ export async function buildCalendar(dataObj, block, type, mappingArray, period)
     document.querySelector('.gmo-program-details.block').addEventListener('click', dismissDropdown);
     block.querySelectorAll('.year-switch > .year-toggle').forEach((control) => {
         control.removeEventListener('click', changePeriod);
-        control.addEventListener('click', changePeriod);
+        if (years.length > 1) {
+            control.addEventListener('click', changePeriod);
+        }
     });
     block.querySelector('.right-controls .today-button').addEventListener('click', () => {
         const calendarWrapper = document.querySelector('.calendar-wrapper')
@@ -410,6 +417,8 @@ function filterDropdownSelection(event, viewStartYear, numYears) {
         const calendarWrapper = document.querySelector('.calendar-wrapper');
         scrollToPosition(calendarWrapper, scrollPct);
     } else {
+        const viewStr = view.charAt(0).toUpperCase() + view.slice(1);
+        document.querySelector('.filter-dropdown-button > .label').textContent = `Selected View: ${viewStr}`;
         refreshCalendar(period, view);
     }
     dismissDropdown();
@@ -617,3 +626,7 @@ function calculateScroll(type, viewStartYear, displayYear, displayQuarter, numYe
         return (yearWidthOffsetPct).toFixed(2);
     }
 }
+
+function isValidDate(dateObj) {
+    return dateObj instanceof Date && !isNaN(dateObj);
+}
\ No newline at end of file