From 8983fd79aa05647b16f814fcbe895b60dcb72a90 Mon Sep 17 00:00:00 2001
From: alaa-yahia <alaay873@live.com>
Date: Thu, 5 Dec 2024 01:36:14 +0200
Subject: [PATCH] fix: working list filters to use gregorian

---
 .../Date/DateFilter.component.js              | 28 +++++++++++++++----
 .../date/convertIsoToLocalCalendar.js         |  2 +-
 .../date/convertLocalToIsoCalendar.js         |  2 +-
 .../date/dateObjectToDateFormatString.js      |  2 +-
 .../DateField/Date.component.js               |  3 --
 5 files changed, 26 insertions(+), 11 deletions(-)

diff --git a/src/core_modules/capture-core/components/FiltersForTypes/Date/DateFilter.component.js b/src/core_modules/capture-core/components/FiltersForTypes/Date/DateFilter.component.js
index c326e4477b..5c995df0d1 100644
--- a/src/core_modules/capture-core/components/FiltersForTypes/Date/DateFilter.component.js
+++ b/src/core_modules/capture-core/components/FiltersForTypes/Date/DateFilter.component.js
@@ -1,6 +1,7 @@
 // @flow
 import React, { Component } from 'react';
 import classNames from 'classnames';
+import moment from 'moment';
 import { withStyles } from '@material-ui/core/styles';
 import i18n from '@dhis2/d2-i18n';
 import { isValidZeroOrPositiveInteger } from 'capture-core-utils/validators/form';
@@ -16,7 +17,8 @@ import './calendarFilterStyles.css';
 import { mainOptionKeys, mainOptionTranslatedTexts } from './options';
 import { getDateFilterData } from './dateFilterDataGetter';
 import { RangeFilter } from './RangeFilter.component';
-import { parseDate } from '../../../utils/converters/date';
+import { parseDate, convertLocalToIsoCalendar, convertIsoToLocalCalendar } from '../../../utils/converters/date';
+import { systemSettingsStore } from '../../../metaDataMemoryStores';
 
 const getStyles = (theme: Theme) => ({
     fromToContainer: {
@@ -257,12 +259,28 @@ class DateFilterPlain extends Component<Props, State> implements UpdatableFilter
         return !values || DateFilter.isFilterValid(values.main, values.from, values.to, values.start, values.end);
     }
 
-    getUpdatedValue(valuePart: { [key: string]: string }) {
-        // $FlowFixMe[cannot-spread-indexer] automated comment
+    // eslint-disable-next-line complexity
+    getUpdatedValue(valuePart: Object) {
         const valueObject = {
             ...this.props.value,
             ...valuePart,
         };
+        const dateFormat = systemSettingsStore.get().dateFormat;
+
+        if (valuePart.from && valueObject?.from?.value) {
+            valueObject.from = {
+                ...valueObject.from,
+                value: moment(convertLocalToIsoCalendar(valueObject.from.value)).format(dateFormat),
+            };
+        }
+
+        if (valuePart.to && valueObject?.to?.value) {
+            valueObject.to = {
+                ...valueObject.to,
+                value: moment(convertLocalToIsoCalendar(valueObject.to.value)).format(dateFormat),
+            };
+        }
+
         const isRelativeRangeValue = () => valueObject?.start || valuePart?.start || valuePart?.end;
         const isAbsoluteRangevalue = () => valueObject?.from || valuePart?.from || valuePart?.to;
 
@@ -358,7 +376,7 @@ class DateFilterPlain extends Component<Props, State> implements UpdatableFilter
                         {/* $FlowSuppress: Flow not working 100% with HOCs */}
                         {/* $FlowFixMe[prop-missing] automated comment */}
                         <FromDateFilter
-                            value={fromValue?.value}
+                            value={convertIsoToLocalCalendar(fromValue?.value)}
                             onBlur={this.handleFieldBlur}
                             onEnterKey={this.handleEnterKeyInFrom}
                             onDateSelectedFromCalendar={this.handleDateSelectedFromCalendarInFrom}
@@ -371,7 +389,7 @@ class DateFilterPlain extends Component<Props, State> implements UpdatableFilter
                         {/* $FlowSuppress: Flow not working 100% with HOCs */}
                         {/* $FlowFixMe[prop-missing] automated comment */}
                         <ToDateFilter
-                            value={toValue?.value}
+                            value={convertIsoToLocalCalendar(toValue?.value)}
                             onBlur={this.handleFieldBlur}
                             textFieldRef={this.setToD2DateTextFieldInstance}
                             onFocusUpdateButton={onFocusUpdateButton}
diff --git a/src/core_modules/capture-core/utils/converters/date/convertIsoToLocalCalendar.js b/src/core_modules/capture-core/utils/converters/date/convertIsoToLocalCalendar.js
index 3f9f99e34b..019872c197 100644
--- a/src/core_modules/capture-core/utils/converters/date/convertIsoToLocalCalendar.js
+++ b/src/core_modules/capture-core/utils/converters/date/convertIsoToLocalCalendar.js
@@ -12,7 +12,7 @@ import { padWithZeros } from './padWithZeros';
  * @returns {string}
  */
 
-export function convertIsoToLocalCalendar(isoDate: string): string {
+export function convertIsoToLocalCalendar(isoDate: ?string): string {
     if (!isoDate) {
         return '';
     }
diff --git a/src/core_modules/capture-core/utils/converters/date/convertLocalToIsoCalendar.js b/src/core_modules/capture-core/utils/converters/date/convertLocalToIsoCalendar.js
index 99b1215919..b3f323d178 100644
--- a/src/core_modules/capture-core/utils/converters/date/convertLocalToIsoCalendar.js
+++ b/src/core_modules/capture-core/utils/converters/date/convertLocalToIsoCalendar.js
@@ -12,7 +12,7 @@ import { padWithZeros } from './padWithZeros';
  * @param {string} localDate - date in local calendar format
  * @returns {string}
  */
-export function convertLocalToIsoCalendar(localDate: string): string {
+export function convertLocalToIsoCalendar(localDate: ?string): string {
     if (!localDate) {
         return '';
     }
diff --git a/src/core_modules/capture-core/utils/converters/date/dateObjectToDateFormatString.js b/src/core_modules/capture-core/utils/converters/date/dateObjectToDateFormatString.js
index bbe748a579..1f91c521cc 100644
--- a/src/core_modules/capture-core/utils/converters/date/dateObjectToDateFormatString.js
+++ b/src/core_modules/capture-core/utils/converters/date/dateObjectToDateFormatString.js
@@ -8,7 +8,7 @@ import { convertIsoToLocalCalendar } from './convertIsoToLocalCalendar';
  * @param {Date} dateValue: the date instance
  * @returns {string}
  */
-export function convertDateObjectToDateFormatString(dateValue: Date |  moment$Moment) {
+export function convertDateObjectToDateFormatString(dateValue: Date | moment$Moment) {
     const momentDate = moment(dateValue);
     const dateString = momentDate.format('YYYY-MM-DD');
     return convertIsoToLocalCalendar(dateString);
diff --git a/src/core_modules/capture-ui/DateAndTimeFields/DateField/Date.component.js b/src/core_modules/capture-ui/DateAndTimeFields/DateField/Date.component.js
index 82de3aa925..95bf289714 100644
--- a/src/core_modules/capture-ui/DateAndTimeFields/DateField/Date.component.js
+++ b/src/core_modules/capture-ui/DateAndTimeFields/DateField/Date.component.js
@@ -35,9 +35,6 @@ type State = {
     calendarError: ?Validation,
 };
 
-const formatDate = (date: any, dateFormat: string): ?string =>
-    (dateFormat === 'dd-MM-yyyy' ? date?.format('DD-MM-YYYY') : date?.format('YYYY-MM-DD'));
-
 export class DateField extends React.Component<Props, State> {
     handleDateSelected: (value: {calendarDateString: string}) => void;