From 136430defd9a555426256a2b715794ce797cdadd Mon Sep 17 00:00:00 2001 From: alaa-yahia Date: Thu, 5 Dec 2024 00:14:05 +0200 Subject: [PATCH 01/25] feat: enable non gregorian calendars in views and lists --- package.json | 3 +- src/components/AppLoader/init.js | 4 +-- .../configs/dateField/getDateFieldConfig.js | 3 +- .../getDateFieldConfigForCustomForm.js | 3 +- .../EnrollmentDataEntry.component.js | 7 +++-- .../DateAndTime/D2Date/D2Date.component.js | 4 +-- .../capture-core/converters/clientToForm.js | 11 ++++---- .../capture-core/converters/clientToList.js | 10 ++++--- .../capture-core/converters/clientToView.js | 12 +++++--- .../capture-core/converters/formToClient.js | 19 +++++++++---- .../metaData/SystemSettings/SystemSettings.js | 1 + .../systemSettings/cacheSystemSetttings.js | 6 +++- .../date/convertIsoToLocalCalendar.js | 28 +++++++++++++++++++ .../date/convertLocalToIsoCalendar.js | 26 +++++++++++++++++ .../date/dateObjectToDateFormatString.js | 10 +++---- .../utils/converters/date/index.js | 3 ++ .../utils/converters/date/padWithZeros.js | 12 ++++++++ .../DateField/Date.component.js | 9 ++---- yarn.lock | 2 +- 19 files changed, 129 insertions(+), 44 deletions(-) create mode 100644 src/core_modules/capture-core/utils/converters/date/convertIsoToLocalCalendar.js create mode 100644 src/core_modules/capture-core/utils/converters/date/convertLocalToIsoCalendar.js create mode 100644 src/core_modules/capture-core/utils/converters/date/padWithZeros.js diff --git a/package.json b/package.json index e4caffa666..1f22447823 100644 --- a/package.json +++ b/package.json @@ -10,7 +10,7 @@ "packages/rules-engine" ], "dependencies": { - "@dhis2/rules-engine-javascript": "101.20.3", + "@dhis2-ui/calendar": "^10.0.3", "@dhis2/app-runtime": "^3.9.3", "@dhis2/d2-i18n": "^1.1.0", "@dhis2/d2-icons": "^1.0.1", @@ -19,7 +19,6 @@ "@dhis2/d2-ui-rich-text": "^7.4.0", "@dhis2/d2-ui-sharing-dialog": "^7.3.3", "@dhis2/ui": "^9.10.1", - "@dhis2-ui/calendar": "^10.0.3", "@joakim_sm/react-infinite-calendar": "^2.4.2", "@material-ui/core": "3.9.4", "@material-ui/icons": "3", diff --git a/src/components/AppLoader/init.js b/src/components/AppLoader/init.js index f7106fabc4..de3a3153fc 100644 --- a/src/components/AppLoader/init.js +++ b/src/components/AppLoader/init.js @@ -131,7 +131,7 @@ async function initializeMetaDataAsync(dbLocale: string, onQueryApi: Function, m async function initializeSystemSettingsAsync( uiLocale: string, - systemSettings: { dateFormat: string, serverTimeZoneId: string }, + systemSettings: { dateFormat: string, serverTimeZoneId: string, calendar: string, }, ) { const systemSettingsCacheData = await cacheSystemSettings(uiLocale, systemSettings); await buildSystemSettingsAsync(systemSettingsCacheData); @@ -158,7 +158,7 @@ export async function initializeAsync( const systemSettings = await onQueryApi({ resource: 'system/info', params: { - fields: 'dateFormat,serverTimeZoneId', + fields: 'dateFormat,serverTimeZoneId,calendar', }, }); diff --git a/src/core_modules/capture-core/components/D2Form/field/configs/dateField/getDateFieldConfig.js b/src/core_modules/capture-core/components/D2Form/field/configs/dateField/getDateFieldConfig.js index 7c6f480ae7..7415bd7d5f 100644 --- a/src/core_modules/capture-core/components/D2Form/field/configs/dateField/getDateFieldConfig.js +++ b/src/core_modules/capture-core/components/D2Form/field/configs/dateField/getDateFieldConfig.js @@ -2,6 +2,7 @@ import moment from 'moment'; import { createFieldConfig, createProps } from '../base/configBaseDefaultForm'; import { DateFieldForForm } from '../../Components'; +import { convertDateObjectToDateFormatString } from '../../../../../../capture-core/utils/converters/date'; import type { DateDataElement } from '../../../../../metaData'; import type { QuerySingleResource } from '../../../../../utils/api/api.types'; @@ -15,7 +16,7 @@ export const getDateFieldConfig = (metaData: DateDataElement, options: Object, q maxWidth: options.formHorizontal ? 150 : 350, calendarWidth: options.formHorizontal ? 250 : 350, popupAnchorPosition: getCalendarAnchorPosition(options.formHorizontal), - calendarMaxMoment: !metaData.allowFutureDate ? moment() : undefined, + calendarMaxMoment: !metaData.allowFutureDate ? convertDateObjectToDateFormatString(moment()) : undefined, }, options, metaData); return createFieldConfig({ diff --git a/src/core_modules/capture-core/components/D2Form/field/configs/dateField/getDateFieldConfigForCustomForm.js b/src/core_modules/capture-core/components/D2Form/field/configs/dateField/getDateFieldConfigForCustomForm.js index b1b93fe119..8cb139fb73 100644 --- a/src/core_modules/capture-core/components/D2Form/field/configs/dateField/getDateFieldConfigForCustomForm.js +++ b/src/core_modules/capture-core/components/D2Form/field/configs/dateField/getDateFieldConfigForCustomForm.js @@ -2,6 +2,7 @@ import moment from 'moment'; import { createFieldConfig, createProps } from '../base/configBaseCustomForm'; import { DateFieldForCustomForm } from '../../Components'; +import { convertDateObjectToDateFormatString } from '../../../../../../capture-core/utils/converters/date'; import type { DateDataElement } from '../../../../../metaData'; import type { QuerySingleResource } from '../../../../../utils/api/api.types'; @@ -10,7 +11,7 @@ export const getDateFieldConfigForCustomForm = (metaData: DateDataElement, optio width: 350, maxWidth: 350, calendarWidth: 350, - calendarMaxMoment: !metaData.allowFutureDate ? moment() : undefined, + calendarMaxMoment: !metaData.allowFutureDate ? convertDateObjectToDateFormatString(moment()) : undefined, }, metaData); return createFieldConfig({ diff --git a/src/core_modules/capture-core/components/DataEntries/Enrollment/EnrollmentDataEntry.component.js b/src/core_modules/capture-core/components/DataEntries/Enrollment/EnrollmentDataEntry.component.js index 1444a98aea..76ed900ccc 100644 --- a/src/core_modules/capture-core/components/DataEntries/Enrollment/EnrollmentDataEntry.component.js +++ b/src/core_modules/capture-core/components/DataEntries/Enrollment/EnrollmentDataEntry.component.js @@ -42,6 +42,7 @@ import { withAOCFieldBuilder, withDataEntryFields, } from '../../DataEntryDhis2Helpers'; +import { convertDateObjectToDateFormatString } from '../../../../capture-core/utils/converters/date'; const overrideMessagePropNames = { errorMessage: 'validationError', @@ -111,7 +112,7 @@ const getEnrollmentDateSettings = () => { required: true, calendarWidth: props.formHorizontal ? 250 : 350, popupAnchorPosition: getCalendarAnchorPosition(props.formHorizontal), - calendarMaxMoment: !props.enrollmentMetadata.allowFutureEnrollmentDate ? moment() : undefined, + calendarMaxMoment: !props.enrollmentMetadata.allowFutureEnrollmentDate ? convertDateObjectToDateFormatString(moment()) : undefined, }), getPropName: () => 'enrolledAt', getValidatorContainers: getEnrollmentDateValidatorContainer, @@ -159,7 +160,9 @@ const getIncidentDateSettings = () => { required: true, calendarWidth: props.formHorizontal ? 250 : 350, popupAnchorPosition: getCalendarAnchorPosition(props.formHorizontal), - calendarMaxMoment: !props.enrollmentMetadata.allowFutureIncidentDate ? moment() : undefined, + calendarMaxMoment: !props.enrollmentMetadata.allowFutureIncidentDate ? + convertDateObjectToDateFormatString(moment()) : + undefined, }), getPropName: () => 'occurredAt', getPassOnFieldData: () => true, diff --git a/src/core_modules/capture-core/components/FormFields/DateAndTime/D2Date/D2Date.component.js b/src/core_modules/capture-core/components/FormFields/DateAndTime/D2Date/D2Date.component.js index eacca0a466..2adb875ce7 100644 --- a/src/core_modules/capture-core/components/FormFields/DateAndTime/D2Date/D2Date.component.js +++ b/src/core_modules/capture-core/components/FormFields/DateAndTime/D2Date/D2Date.component.js @@ -7,7 +7,6 @@ import { type DateValue } from '../../../FiltersForTypes/Date/types/date.types'; type Props = { label?: ?string, value: ?string, - calendar?: string, calendarWidth?: ?number, inputWidth?: ?number, onBlur: (value: DateValue) => void, @@ -50,7 +49,6 @@ export class D2Date extends React.Component { render() { const { - calendar, calendarWidth, inputWidth, classes, @@ -62,7 +60,7 @@ export class D2Date extends React.Component { ...passOnProps } = this.props; - const calendarType = calendar || 'gregory'; + const calendarType = systemSettingsStore.get().calendar || 'gregory'; const format = systemSettingsStore.get().dateFormat; return ( diff --git a/src/core_modules/capture-core/converters/clientToForm.js b/src/core_modules/capture-core/converters/clientToForm.js index f8e227b6d9..606b7e3f04 100644 --- a/src/core_modules/capture-core/converters/clientToForm.js +++ b/src/core_modules/capture-core/converters/clientToForm.js @@ -1,6 +1,6 @@ // @flow import moment from 'moment'; -import { convertMomentToDateFormatString } from '../utils/converters/date'; +import { convertMomentToDateFormatString, convertIsoToLocalCalendar } from '../utils/converters/date'; import { dataElementTypes } from '../metaData'; import { stringifyNumber } from './common/stringifyNumber'; @@ -23,16 +23,17 @@ type RangeValue = { } function convertDateForEdit(rawValue: string): string { - const momentInstance = moment(rawValue); - return convertMomentToDateFormatString(momentInstance); + const momentDate = moment(rawValue); + const dateString = momentDate.format('YYYY-MM-DD'); + return convertIsoToLocalCalendar(dateString); } function convertDateTimeForEdit(rawValue: string): DateTimeFormValue { const dateTime = moment(rawValue); - const dateString = convertMomentToDateFormatString(dateTime); + const dateString = dateTime.format('YYYY-MM-DD'); const timeString = dateTime.format('HH:mm'); return { - date: dateString, + date: convertIsoToLocalCalendar(dateString), time: timeString, }; } diff --git a/src/core_modules/capture-core/converters/clientToList.js b/src/core_modules/capture-core/converters/clientToList.js index e72b837179..6ee3cb78a7 100644 --- a/src/core_modules/capture-core/converters/clientToList.js +++ b/src/core_modules/capture-core/converters/clientToList.js @@ -5,21 +5,23 @@ import i18n from '@dhis2/d2-i18n'; import { Tag } from '@dhis2/ui'; import { PreviewImage } from 'capture-ui'; import { dataElementTypes, type DataElement } from '../metaData'; -import { convertMomentToDateFormatString } from '../utils/converters/date'; +import { convertIsoToLocalCalendar } from '../utils/converters/date'; import { stringifyNumber } from './common/stringifyNumber'; import { MinimalCoordinates } from '../components/MinimalCoordinates'; import { TooltipOrgUnit } from '../components/Tooltips/TooltipOrgUnit'; function convertDateForListDisplay(rawValue: string): string { const momentDate = moment(rawValue); - return convertMomentToDateFormatString(momentDate); + const dateString = momentDate.format('YYYY-MM-DD'); + return convertIsoToLocalCalendar(dateString); } function convertDateTimeForListDisplay(rawValue: string): string { const momentDate = moment(rawValue); - const dateString = convertMomentToDateFormatString(momentDate); + const dateString = momentDate.format('YYYY-MM-DD'); const timeString = momentDate.format('HH:mm'); - return `${dateString} ${timeString}`; + const localDate = convertIsoToLocalCalendar(dateString); + return `${localDate} ${timeString}`; } function convertTimeForListDisplay(rawValue: string): string { diff --git a/src/core_modules/capture-core/converters/clientToView.js b/src/core_modules/capture-core/converters/clientToView.js index fb7728f723..6f4e2b7e14 100644 --- a/src/core_modules/capture-core/converters/clientToView.js +++ b/src/core_modules/capture-core/converters/clientToView.js @@ -4,7 +4,7 @@ import moment from 'moment'; import i18n from '@dhis2/d2-i18n'; import { PreviewImage } from 'capture-ui'; import { dataElementTypes, type DataElement } from '../metaData'; -import { convertMomentToDateFormatString } from '../utils/converters/date'; +import { convertIsoToLocalCalendar } from '../utils/converters/date'; import { stringifyNumber } from './common/stringifyNumber'; import { MinimalCoordinates } from '../components/MinimalCoordinates'; import { TooltipOrgUnit } from '../components/Tooltips/TooltipOrgUnit'; @@ -12,14 +12,18 @@ import { TooltipOrgUnit } from '../components/Tooltips/TooltipOrgUnit'; function convertDateForView(rawValue: string): string { const momentDate = moment(rawValue); - return convertMomentToDateFormatString(momentDate); + const dateString = momentDate.format('YYYY-MM-DD'); + return convertIsoToLocalCalendar(dateString); } function convertDateTimeForView(rawValue: string): string { const momentDate = moment(rawValue); - const dateString = convertMomentToDateFormatString(momentDate); + const dateString = momentDate.format('YYYY-MM-DD'); const timeString = momentDate.format('HH:mm'); - return `${dateString} ${timeString}`; + + const localDate = convertIsoToLocalCalendar(dateString); + return `${localDate} ${timeString}`; } + function convertTimeForView(rawValue: string): string { const momentDate = moment(rawValue, 'HH:mm', true); return momentDate.format('HH:mm'); diff --git a/src/core_modules/capture-core/converters/formToClient.js b/src/core_modules/capture-core/converters/formToClient.js index 61c286d4d2..61b37923fd 100644 --- a/src/core_modules/capture-core/converters/formToClient.js +++ b/src/core_modules/capture-core/converters/formToClient.js @@ -1,8 +1,9 @@ // @flow +import moment from 'moment'; import isString from 'd2-utilizr/lib/isString'; import { parseNumber, parseTime } from 'capture-core-utils/parsers'; import { dataElementTypes } from '../metaData'; -import { parseDate } from '../utils/converters/date'; +import { parseDate, convertLocalToIsoCalendar } from '../utils/converters/date'; type DateTimeValue = { date: string, @@ -25,9 +26,11 @@ function convertDateTime(formValue: DateTimeValue): ?string { const minutes = momentTime.minute(); const parsedDate = editedDate ? parseDate(editedDate) : null; - if (!(parsedDate && parsedDate.isValid)) return null; - // $FlowFixMe[incompatible-type] automated comment - const momentDateTime: moment$Moment = parsedDate.momentDate; + if (!(parsedDate && parsedDate.isValid && parsedDate.momentDate)) return null; + + const formattedDate = parsedDate.momentDate.format('YYYY-MM-DD'); + const isoDate = convertLocalToIsoCalendar(formattedDate); + const momentDateTime = moment(isoDate); momentDateTime.hour(hours); momentDateTime.minute(minutes); return momentDateTime.toISOString(); @@ -35,8 +38,12 @@ function convertDateTime(formValue: DateTimeValue): ?string { function convertDate(dateValue: string) { const parsedDate = parseDate(dateValue); - // $FlowFixMe[incompatible-use] automated comment - return parsedDate.isValid ? parsedDate.momentDate.toISOString() : null; + if (!parsedDate.isValid || !parsedDate.momentDate) { + return null; + } + const formattedDate = parsedDate.momentDate.format('YYYY-MM-DD'); + + return convertLocalToIsoCalendar(formattedDate); } function convertTime(timeValue: string) { diff --git a/src/core_modules/capture-core/metaData/SystemSettings/SystemSettings.js b/src/core_modules/capture-core/metaData/SystemSettings/SystemSettings.js index 6231d3b637..ec8334b437 100644 --- a/src/core_modules/capture-core/metaData/SystemSettings/SystemSettings.js +++ b/src/core_modules/capture-core/metaData/SystemSettings/SystemSettings.js @@ -4,4 +4,5 @@ export class SystemSettings { dateFormat: string; dir: string; trackerAppRelativePath: string; + calendar: string; } diff --git a/src/core_modules/capture-core/metaDataStoreLoaders/systemSettings/cacheSystemSetttings.js b/src/core_modules/capture-core/metaDataStoreLoaders/systemSettings/cacheSystemSetttings.js index 7ff99e54aa..225c4261ab 100644 --- a/src/core_modules/capture-core/metaDataStoreLoaders/systemSettings/cacheSystemSetttings.js +++ b/src/core_modules/capture-core/metaDataStoreLoaders/systemSettings/cacheSystemSetttings.js @@ -10,7 +10,7 @@ function isLangRTL(code) { export async function cacheSystemSettings( uiLocale: string, - systemSettings: { dateFormat: string, serverTimeZoneId: string }, + systemSettings: { dateFormat: string, serverTimeZoneId: string, calendar: string, }, ) { const systemSettingsArray = [ { @@ -25,6 +25,10 @@ export async function cacheSystemSettings( id: 'serverTimeZoneId', value: systemSettings.serverTimeZoneId, }, + { + id: 'calendar', + value: systemSettings.calendar, + }, ]; const storageController = getMainStorageController(); diff --git a/src/core_modules/capture-core/utils/converters/date/convertIsoToLocalCalendar.js b/src/core_modules/capture-core/utils/converters/date/convertIsoToLocalCalendar.js new file mode 100644 index 0000000000..3f9f99e34b --- /dev/null +++ b/src/core_modules/capture-core/utils/converters/date/convertIsoToLocalCalendar.js @@ -0,0 +1,28 @@ +// @flow +import { + convertFromIso8601, +} from '@dhis2/multi-calendar-dates'; +import { systemSettingsStore } from '../../../../capture-core/metaDataMemoryStores'; +import { padWithZeros } from './padWithZeros'; + +/** + * Converts a date from ISO calendar to local calendar + * @export + * @param {string} isoDate - date in ISO format + * @returns {string} + */ + +export function convertIsoToLocalCalendar(isoDate: string): string { + if (!isoDate) { + return ''; + } + const calendar = systemSettingsStore.get().calendar; + const dateFormat = systemSettingsStore.get().dateFormat; + + const { year, eraYear, month, day } = convertFromIso8601(isoDate, calendar); + const localYear = calendar === 'ethiopian' ? eraYear : year; + + return dateFormat === 'DD-MM-YYYY' + ? `${padWithZeros(day, 2)}-${padWithZeros(month, 2)}-${padWithZeros(localYear, 4)}` + : `${padWithZeros(localYear, 4)}-${padWithZeros(month, 2)}-${padWithZeros(day, 2)}`; +} diff --git a/src/core_modules/capture-core/utils/converters/date/convertLocalToIsoCalendar.js b/src/core_modules/capture-core/utils/converters/date/convertLocalToIsoCalendar.js new file mode 100644 index 0000000000..99b1215919 --- /dev/null +++ b/src/core_modules/capture-core/utils/converters/date/convertLocalToIsoCalendar.js @@ -0,0 +1,26 @@ +// @flow +import moment from 'moment'; +import { + convertToIso8601, +} from '@dhis2/multi-calendar-dates'; +import { systemSettingsStore } from '../../../../capture-core/metaDataMemoryStores'; +import { padWithZeros } from './padWithZeros'; + +/** + * Converts a date from local calendar to ISO calendar + * @export + * @param {string} localDate - date in local calendar format + * @returns {string} + */ +export function convertLocalToIsoCalendar(localDate: string): string { + if (!localDate) { + return ''; + } + const calendar = systemSettingsStore.get().calendar; + + const { year, month, day } = convertToIso8601(localDate, calendar); + const dateString = `${padWithZeros(year, 4)}-${padWithZeros(month, 2)}-${padWithZeros(day, 2)}`; + const parsedMoment = moment(dateString); + + return parsedMoment.isValid() ? parsedMoment.toISOString() : ''; +} 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 de59f62d24..bbe748a579 100644 --- a/src/core_modules/capture-core/utils/converters/date/dateObjectToDateFormatString.js +++ b/src/core_modules/capture-core/utils/converters/date/dateObjectToDateFormatString.js @@ -1,6 +1,6 @@ // @flow import moment from 'moment'; -import { systemSettingsStore } from '../../../metaDataMemoryStores'; +import { convertIsoToLocalCalendar } from './convertIsoToLocalCalendar'; /** * Converts a date instance to a string based on the system date format @@ -8,8 +8,8 @@ import { systemSettingsStore } from '../../../metaDataMemoryStores'; * @param {Date} dateValue: the date instance * @returns {string} */ -export function convertDateObjectToDateFormatString(dateValue: Date) { - const dateFormat = systemSettingsStore.get().dateFormat; - const formattedDateString = moment(dateValue).format(dateFormat); - return formattedDateString; +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-core/utils/converters/date/index.js b/src/core_modules/capture-core/utils/converters/date/index.js index f7e46c2971..511298b89d 100644 --- a/src/core_modules/capture-core/utils/converters/date/index.js +++ b/src/core_modules/capture-core/utils/converters/date/index.js @@ -3,3 +3,6 @@ export { parseDate } from './parser'; export { convertDateObjectToDateFormatString } from './dateObjectToDateFormatString'; export { convertMomentToDateFormatString } from './momentToDateFormatString'; export { convertStringToDateFormat } from './stringToMomentDateFormat'; +export { padWithZeros } from './padWithZeros'; +export { convertIsoToLocalCalendar } from './convertIsoToLocalCalendar'; +export { convertLocalToIsoCalendar } from './convertLocalToIsoCalendar'; diff --git a/src/core_modules/capture-core/utils/converters/date/padWithZeros.js b/src/core_modules/capture-core/utils/converters/date/padWithZeros.js new file mode 100644 index 0000000000..5edc85befd --- /dev/null +++ b/src/core_modules/capture-core/utils/converters/date/padWithZeros.js @@ -0,0 +1,12 @@ +// @flow + +/** + * Pads a string or number with zeros at the start to reach a minimum length + * @export + * @param {string|number} value - the value to pad + * @param {number} length - length required + * @returns {string} + */ +export function padWithZeros(value: string | number, length: number): string { + return String(value).padStart(length, '0'); +} 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 b48ee77ae5..95bf289714 100644 --- a/src/core_modules/capture-ui/DateAndTimeFields/DateField/Date.component.js +++ b/src/core_modules/capture-ui/DateAndTimeFields/DateField/Date.component.js @@ -18,7 +18,6 @@ type Props = { onBlur: (value: Object, options: ValidationOptions) => void, onFocus?: ?() => void, onDateSelectedFromCalendar?: () => void, - calendar?: string, placeholder?: string, label?: string, calendarMaxMoment?: any, @@ -36,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 { handleDateSelected: (value: {calendarDateString: string}) => void; @@ -65,7 +61,6 @@ export class DateField extends React.Component { maxWidth, calendarWidth, inputWidth, - calendar, calendarMaxMoment, value, innerMessage, @@ -73,7 +68,7 @@ export class DateField extends React.Component { const calculatedInputWidth = inputWidth || width; const calculatedCalendarWidth = calendarWidth || width; - const calendarType = calendar || 'gregory'; + const calendarType = systemSettingsStore.get().calendar || 'gregory'; const format = systemSettingsStore.get().dateFormat; const errorProps = innerMessage && innerMessage.messageType === 'error' ? { error: !!innerMessage.message?.dateInnerErrorMessage, @@ -99,7 +94,7 @@ export class DateField extends React.Component { onFocus={this.props.onFocus} disabled={this.props.disabled} {...errorProps} - maxDate={calendarMaxMoment && formatDate(calendarMaxMoment, format)} + maxDate={calendarMaxMoment} /> ); diff --git a/yarn.lock b/yarn.lock index 4c2c7b3910..9a8bd281de 100644 --- a/yarn.lock +++ b/yarn.lock @@ -2718,7 +2718,7 @@ recompose "^0.26.0" rxjs "^5.5.7" -"@dhis2/multi-calendar-dates@2.0.0": +"@dhis2/multi-calendar-dates@2.0.0", "@dhis2/multi-calendar-dates@^2.0.0": version "2.0.0" resolved "https://registry.yarnpkg.com/@dhis2/multi-calendar-dates/-/multi-calendar-dates-2.0.0.tgz#febf04f873670960804d38c9ebaa1cadf8050db3" integrity sha512-pxu81kkkh70tB+CyAub41ulpNJPHyxDGwH2pdcc+NUqrKu4OTQr5ScdCBL2MndShrEKj9J6qj9zKVagvvymH5w== From b8fd9411c421044c6738e0aa2bcef4895e88e9e8 Mon Sep 17 00:00:00 2001 From: alaa-yahia Date: Thu, 5 Dec 2024 01:36:14 +0200 Subject: [PATCH 02/25] fix: working list filters to use gregorian --- .../Date/DateFilter.component.js | 28 +++++++++++++++---- .../date/convertIsoToLocalCalendar.js | 2 +- .../date/convertLocalToIsoCalendar.js | 2 +- .../date/dateObjectToDateFormatString.js | 2 +- 4 files changed, 26 insertions(+), 8 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 b0fa8f967e..20fc8fe9fd 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: { @@ -265,12 +267,28 @@ class DateFilterPlain extends Component 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; @@ -366,7 +384,7 @@ class DateFilterPlain extends Component implements UpdatableFilter {/* $FlowSuppress: Flow not working 100% with HOCs */} {/* $FlowFixMe[prop-missing] automated comment */} implements UpdatableFilter {/* $FlowSuppress: Flow not working 100% with HOCs */} {/* $FlowFixMe[prop-missing] automated comment */} Date: Tue, 10 Dec 2024 14:18:33 +0200 Subject: [PATCH 03/25] fix: working list filter runtime error when dd-mm-yyyy format is passed --- .../converters/date/convertIsoToLocalCalendar.js | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-) 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 019872c197..442ad4a60c 100644 --- a/src/core_modules/capture-core/utils/converters/date/convertIsoToLocalCalendar.js +++ b/src/core_modules/capture-core/utils/converters/date/convertIsoToLocalCalendar.js @@ -1,4 +1,5 @@ // @flow +import moment from 'moment'; import { convertFromIso8601, } from '@dhis2/multi-calendar-dates'; @@ -16,10 +17,18 @@ export function convertIsoToLocalCalendar(isoDate: ?string): string { if (!isoDate) { return ''; } + + const momentDate = moment(isoDate); + if (!momentDate.isValid()) { + return ''; + } + + const formattedIsoDate = momentDate.format('YYYY-MM-DD'); + const calendar = systemSettingsStore.get().calendar; const dateFormat = systemSettingsStore.get().dateFormat; - const { year, eraYear, month, day } = convertFromIso8601(isoDate, calendar); + const { year, eraYear, month, day } = convertFromIso8601(formattedIsoDate, calendar); const localYear = calendar === 'ethiopian' ? eraYear : year; return dateFormat === 'DD-MM-YYYY' From 8e44c3cad99d9ddc07f7f7a6dc7efc747b716ebf Mon Sep 17 00:00:00 2001 From: alaa-yahia Date: Wed, 11 Dec 2024 11:27:52 +0200 Subject: [PATCH 04/25] fix: missing rules engine in dependencies --- package.json | 1 + yarn.lock | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/package.json b/package.json index 1f22447823..1178b6e2e1 100644 --- a/package.json +++ b/package.json @@ -10,6 +10,7 @@ "packages/rules-engine" ], "dependencies": { + "@dhis2/rules-engine-javascript": "101.19.1", "@dhis2-ui/calendar": "^10.0.3", "@dhis2/app-runtime": "^3.9.3", "@dhis2/d2-i18n": "^1.1.0", diff --git a/yarn.lock b/yarn.lock index 9a8bd281de..4c2c7b3910 100644 --- a/yarn.lock +++ b/yarn.lock @@ -2718,7 +2718,7 @@ recompose "^0.26.0" rxjs "^5.5.7" -"@dhis2/multi-calendar-dates@2.0.0", "@dhis2/multi-calendar-dates@^2.0.0": +"@dhis2/multi-calendar-dates@2.0.0": version "2.0.0" resolved "https://registry.yarnpkg.com/@dhis2/multi-calendar-dates/-/multi-calendar-dates-2.0.0.tgz#febf04f873670960804d38c9ebaa1cadf8050db3" integrity sha512-pxu81kkkh70tB+CyAub41ulpNJPHyxDGwH2pdcc+NUqrKu4OTQr5ScdCBL2MndShrEKj9J6qj9zKVagvvymH5w== From 1f2518d9be14690500b64a8a5a3b0954fa62d130 Mon Sep 17 00:00:00 2001 From: alaa-yahia Date: Wed, 11 Dec 2024 13:13:00 +0200 Subject: [PATCH 05/25] fix: working list filter not displaying local date --- .../buttonTextBuilder/converters/dateConverter.js | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/core_modules/capture-core/components/ListView/Filters/FilterButton/buttonTextBuilder/converters/dateConverter.js b/src/core_modules/capture-core/components/ListView/Filters/FilterButton/buttonTextBuilder/converters/dateConverter.js index 4dce8ba7a9..adbc7c367d 100644 --- a/src/core_modules/capture-core/components/ListView/Filters/FilterButton/buttonTextBuilder/converters/dateConverter.js +++ b/src/core_modules/capture-core/components/ListView/Filters/FilterButton/buttonTextBuilder/converters/dateConverter.js @@ -6,6 +6,8 @@ import { convertMomentToDateFormatString } from '../../../../../../utils/convert import type { DateFilterData, AbsoluteDateFilterData } from '../../../../../FiltersForTypes'; import { areRelativeRangeValuesSupported } from '../../../../../../utils/validation/validators/areRelativeRangeValuesSupported'; +import { convertClientToView } from '../../../../../../../capture-core/converters'; +import { dataElementTypes } from '../../../../../../metaData'; const periods = { TODAY: 'TODAY', @@ -37,8 +39,8 @@ const convertToViewValue = (filterValue: string) => pipe( function translateAbsoluteDate(filter: AbsoluteDateFilterData) { let appliedText = ''; - const fromValue = filter.ge; - const toValue = filter.le; + const fromValue = convertClientToView(filter.ge, dataElementTypes.DATE); + const toValue = convertClientToView(filter.le, dataElementTypes.DATE); if (fromValue && toValue) { const momentFrom = moment(fromValue); From 9a3d60960dc8c8228b6fd8cf1446bf0953e3fc60 Mon Sep 17 00:00:00 2001 From: alaa-yahia Date: Wed, 11 Dec 2024 13:27:26 +0200 Subject: [PATCH 06/25] fix: reduce the complexity of getUpdatedValue --- .../FiltersForTypes/Date/DateFilter.component.js | 7 ++++--- 1 file changed, 4 insertions(+), 3 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 20fc8fe9fd..a38d9cd704 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 @@ -267,22 +267,23 @@ class DateFilterPlain extends Component implements UpdatableFilter return !values || DateFilter.isFilterValid(values.main, values.from, values.to, values.start, values.end); } - // eslint-disable-next-line complexity getUpdatedValue(valuePart: Object) { const valueObject = { ...this.props.value, ...valuePart, }; const dateFormat = systemSettingsStore.get().dateFormat; + const hasFromValue = () => valuePart.from && valueObject?.from?.value; + const hasToValue = () => valuePart.to && valueObject?.to?.value; - if (valuePart.from && valueObject?.from?.value) { + if (hasFromValue()) { valueObject.from = { ...valueObject.from, value: moment(convertLocalToIsoCalendar(valueObject.from.value)).format(dateFormat), }; } - if (valuePart.to && valueObject?.to?.value) { + if (hasToValue()) { valueObject.to = { ...valueObject.to, value: moment(convertLocalToIsoCalendar(valueObject.to.value)).format(dateFormat), From 3a3d0b2c320e2c6d48071b1ec3909450a6e9bc48 Mon Sep 17 00:00:00 2001 From: alaa-yahia Date: Wed, 11 Dec 2024 23:37:38 +0200 Subject: [PATCH 07/25] chore: refactor working lists filter component --- .../Date/DateFilter.component.js | 53 +++++++++++-------- .../converters/dateConverter.js | 24 +++------ 2 files changed, 39 insertions(+), 38 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 a38d9cd704..80b002c049 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 @@ -273,22 +273,18 @@ class DateFilterPlain extends Component implements UpdatableFilter ...valuePart, }; const dateFormat = systemSettingsStore.get().dateFormat; - const hasFromValue = () => valuePart.from && valueObject?.from?.value; - const hasToValue = () => valuePart.to && valueObject?.to?.value; - if (hasFromValue()) { - valueObject.from = { - ...valueObject.from, - value: moment(convertLocalToIsoCalendar(valueObject.from.value)).format(dateFormat), - }; - } - - if (hasToValue()) { - valueObject.to = { - ...valueObject.to, - value: moment(convertLocalToIsoCalendar(valueObject.to.value)).format(dateFormat), - }; - } + ['from', 'to'].forEach((key) => { + if (valuePart[key] && valueObject[key]?.value) { + const isValidDate = valuePart[key]?.isValid; + valueObject[key] = { + ...valueObject[key], + value: isValidDate ? + moment(convertLocalToIsoCalendar(valueObject[key].value)).format(dateFormat) : + valueObject[key].value, + }; + } + }); const isRelativeRangeValue = () => valueObject?.start || valuePart?.start || valuePart?.end; const isAbsoluteRangevalue = () => valueObject?.from || valuePart?.from || valuePart?.to; @@ -362,10 +358,23 @@ class DateFilterPlain extends Component implements UpdatableFilter render() { const { value, classes, onFocusUpdateButton } = this.props; - const fromValue = value?.from; - const toValue = value?.to; const { startValueError, endValueError, dateLogicError, bufferLogicError } = - this.getErrors(); + this.getErrors(); + + const dateFormat = systemSettingsStore.get().dateFormat; + + const formatDate = (dateValue) => { + if (!dateValue) return ''; + const momentValue = moment(dateValue, dateFormat); + if (!momentValue.isValid()) return dateValue; + const isoDate = momentValue.format('YYYY-MM-DD'); + return convertIsoToLocalCalendar(isoDate); + }; + + const from = value?.from; + const to = value?.to; + const fromDate = formatDate(from?.value); + const toDate = formatDate(to?.value); return (
@@ -385,11 +394,11 @@ class DateFilterPlain extends Component implements UpdatableFilter {/* $FlowSuppress: Flow not working 100% with HOCs */} {/* $FlowFixMe[prop-missing] automated comment */}
@@ -398,11 +407,11 @@ class DateFilterPlain extends Component implements UpdatableFilter {/* $FlowSuppress: Flow not working 100% with HOCs */} {/* $FlowFixMe[prop-missing] automated comment */} diff --git a/src/core_modules/capture-core/components/ListView/Filters/FilterButton/buttonTextBuilder/converters/dateConverter.js b/src/core_modules/capture-core/components/ListView/Filters/FilterButton/buttonTextBuilder/converters/dateConverter.js index adbc7c367d..35c60d8d2e 100644 --- a/src/core_modules/capture-core/components/ListView/Filters/FilterButton/buttonTextBuilder/converters/dateConverter.js +++ b/src/core_modules/capture-core/components/ListView/Filters/FilterButton/buttonTextBuilder/converters/dateConverter.js @@ -1,13 +1,10 @@ // @flow import i18n from '@dhis2/d2-i18n'; -import { pipe } from 'capture-core-utils'; import moment from 'moment'; -import { convertMomentToDateFormatString } from '../../../../../../utils/converters/date'; +import { convertIsoToLocalCalendar } from '../../../../../../utils/converters/date'; import type { DateFilterData, AbsoluteDateFilterData } from '../../../../../FiltersForTypes'; import { areRelativeRangeValuesSupported } from '../../../../../../utils/validation/validators/areRelativeRangeValuesSupported'; -import { convertClientToView } from '../../../../../../../capture-core/converters'; -import { dataElementTypes } from '../../../../../../metaData'; const periods = { TODAY: 'TODAY', @@ -32,32 +29,27 @@ const translatedPeriods = { [periods.RELATIVE_RANGE]: i18n.t('Relative range'), }; -const convertToViewValue = (filterValue: string) => pipe( - value => moment(value), - momentDate => convertMomentToDateFormatString(momentDate), -)(filterValue); - function translateAbsoluteDate(filter: AbsoluteDateFilterData) { let appliedText = ''; - const fromValue = convertClientToView(filter.ge, dataElementTypes.DATE); - const toValue = convertClientToView(filter.le, dataElementTypes.DATE); + const fromValue = filter.ge; + const toValue = filter.le; if (fromValue && toValue) { const momentFrom = moment(fromValue); const momentTo = moment(toValue); if (momentFrom.isSame(momentTo)) { - appliedText = convertMomentToDateFormatString(momentFrom); + appliedText = convertIsoToLocalCalendar(fromValue); } else { - const appliedTextFrom = convertMomentToDateFormatString(momentFrom); - const appliedTextTo = convertMomentToDateFormatString(momentTo); + const appliedTextFrom = convertIsoToLocalCalendar(fromValue); + const appliedTextTo = convertIsoToLocalCalendar(toValue); appliedText = i18n.t('{{fromDate}} to {{toDate}}', { fromDate: appliedTextFrom, toDate: appliedTextTo }); } } else if (fromValue) { - const appliedTextFrom = convertToViewValue(fromValue); + const appliedTextFrom = convertIsoToLocalCalendar(fromValue); appliedText = i18n.t('after or equal to {{date}}', { date: appliedTextFrom }); } else { // $FlowFixMe[incompatible-call] automated comment - const appliedTextTo = convertToViewValue(toValue); + const appliedTextTo = convertIsoToLocalCalendar(toValue); appliedText = i18n.t('before or equal to {{date}}', { date: appliedTextTo }); } return appliedText; From 53ef00edf71809d50e44332171ffdebeca1af7a9 Mon Sep 17 00:00:00 2001 From: alaa-yahia Date: Wed, 11 Dec 2024 23:45:12 +0200 Subject: [PATCH 08/25] fix: convert age values to local date --- src/core_modules/capture-core/converters/clientToForm.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/core_modules/capture-core/converters/clientToForm.js b/src/core_modules/capture-core/converters/clientToForm.js index 606b7e3f04..c0b0d10948 100644 --- a/src/core_modules/capture-core/converters/clientToForm.js +++ b/src/core_modules/capture-core/converters/clientToForm.js @@ -1,6 +1,6 @@ // @flow import moment from 'moment'; -import { convertMomentToDateFormatString, convertIsoToLocalCalendar } from '../utils/converters/date'; +import { convertIsoToLocalCalendar } from '../utils/converters/date'; import { dataElementTypes } from '../metaData'; import { stringifyNumber } from './common/stringifyNumber'; @@ -56,7 +56,7 @@ function convertAgeForEdit(rawValue: string): AgeFormValue { const days = now.diff(age, 'days'); return { - date: convertMomentToDateFormatString(moment(rawValue)), + date: convertIsoToLocalCalendar(rawValue), years: years.toString(), months: months.toString(), days: days.toString(), From 1d3faa6538664fd29c9143751d2e24cf1c7f6369 Mon Sep 17 00:00:00 2001 From: Alaa Yahia <6881345+alaa-yahia@users.noreply.github.com> Date: Mon, 16 Dec 2024 10:44:21 +0200 Subject: [PATCH 09/25] feat: [DHIS2 15466] typing the date when editing enrollment and incident date (#3905) feat: typing the date when editing enrollment and incident date --- .../WidgetEnrollment/Date/Date.component.js | 45 ++++++++++++------- 1 file changed, 28 insertions(+), 17 deletions(-) diff --git a/src/core_modules/capture-core/components/WidgetEnrollment/Date/Date.component.js b/src/core_modules/capture-core/components/WidgetEnrollment/Date/Date.component.js index 0fdba91aea..1b8dc31ee1 100644 --- a/src/core_modules/capture-core/components/WidgetEnrollment/Date/Date.component.js +++ b/src/core_modules/capture-core/components/WidgetEnrollment/Date/Date.component.js @@ -1,9 +1,8 @@ // @flow import React, { useState, useCallback } from 'react'; -import moment from 'moment'; +import { DateField } from 'capture-core/components/FormFields/New'; import { Button, - CalendarInput, IconCalendar16, IconEdit16, colors, @@ -12,8 +11,11 @@ import { import i18n from '@dhis2/d2-i18n'; import { withStyles } from '@material-ui/core'; import { convertValue as convertValueClientToView } from '../../../converters/clientToView'; +import { convertValue as convertValueFormToClient } from '../../../converters/formToClient'; +import { convertValue as convertValueClientToServer } from '../../../converters/clientToServer'; import { dataElementTypes } from '../../../metaData'; + type Props = { date: string, dateLabel: string, @@ -24,7 +26,7 @@ type Props = { ...CssClasses, } -const styles = { +const styles = (theme: Theme) => ({ editButton: { display: 'inline-flex', alignItems: 'center', @@ -62,7 +64,11 @@ const styles = { fontSize: '12px', color: colors.grey700, }, -}; + error: { + ...theme.typography.caption, + color: theme.palette.error.main, + }, +}); const DateComponentPlain = ({ date, @@ -75,21 +81,23 @@ const DateComponentPlain = ({ }: Props) => { const [editMode, setEditMode] = useState(false); const [selectedDate, setSelectedDate] = useState(); - const dateChangeHandler = useCallback(({ calendarDateString }) => { - setSelectedDate(calendarDateString); + const [validation, setValidation] = useState(); + + const dateChangeHandler = useCallback((dateString, internalComponentError) => { + setSelectedDate(dateString); + setValidation(internalComponentError); }, [setSelectedDate]); const displayDate = String(convertValueClientToView(date, dataElementTypes.DATE)); const onOpenEdit = () => { - // CalendarInput component only supports the YYYY-MM-DD format - setSelectedDate(moment(date).format('YYYY-MM-DD')); + setSelectedDate(String(convertValueClientToView(date, dataElementTypes.DATE))); setEditMode(true); }; const saveHandler = () => { - // CalendarInput component only supports the YYYY-MM-DD format if (selectedDate) { - const newDate = moment.utc(selectedDate, 'YYYY-MM-DD').format('YYYY-MM-DDTHH:mm:ss.SSS'); - if (newDate !== date) { + const newClientDate = convertValueFormToClient(selectedDate, dataElementTypes.DATE); + const newDate = convertValueClientToServer(newClientDate, dataElementTypes.DATE); + if (typeof newDate === 'string' && newDate !== date) { onSave(newDate); } } @@ -99,21 +107,24 @@ const DateComponentPlain = ({ return editMode ? (
- +
+ {validation && validation.error ? i18n.t('Please provide a valid date') : ''} +
From 7968ea734503ae0e14d3c13a61baead5e67161f7 Mon Sep 17 00:00:00 2001 From: alaa-yahia Date: Mon, 16 Dec 2024 11:45:42 +0200 Subject: [PATCH 10/25] fix: age values not filled correctly --- .../capture-ui/AgeField/AgeField.component.js | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/src/core_modules/capture-ui/AgeField/AgeField.component.js b/src/core_modules/capture-ui/AgeField/AgeField.component.js index a125ea8900..78abcf4a21 100644 --- a/src/core_modules/capture-ui/AgeField/AgeField.component.js +++ b/src/core_modules/capture-ui/AgeField/AgeField.component.js @@ -1,6 +1,8 @@ // @flow import React, { Component } from 'react'; import { isValidPositiveInteger } from 'capture-core-utils/validators/form'; +import { convertDateObjectToDateFormatString } from 'capture-core/utils/converters/date'; +import { systemSettingsStore } from 'capture-core/metaDataMemoryStores'; import i18n from '@dhis2/d2-i18n'; import classNames from 'classnames'; import { IconButton } from 'capture-ui'; @@ -72,7 +74,8 @@ function getCalculatedValues( days: '', }; } - const now = moment(); + const dateFormat = systemSettingsStore.get().dateFormat; + const now = moment(convertDateObjectToDateFormatString(moment()), dateFormat); const age = moment(parseData.momentDate); const years = now.diff(age, 'years'); @@ -133,8 +136,8 @@ class D2AgeFieldPlain extends Component { this.props.onBlur({ ...values, date: '' }); return; } - - const momentDate = moment(undefined, undefined, true); + const dateFormat = systemSettingsStore.get().dateFormat; + const momentDate = moment(convertDateObjectToDateFormatString(moment(undefined, undefined, true)), dateFormat); momentDate.subtract(D2AgeFieldPlain.getNumberOrZero(values.years), 'years'); momentDate.subtract(D2AgeFieldPlain.getNumberOrZero(values.months), 'months'); momentDate.subtract(D2AgeFieldPlain.getNumberOrZero(values.days), 'days'); From e8d226f9b13a93abee91e2c0f30a09bb81e88c9c Mon Sep 17 00:00:00 2001 From: alaa-yahia Date: Sun, 22 Dec 2024 23:51:24 +0200 Subject: [PATCH 11/25] fix: keep local calendar date in state in working list filters --- .../Date/DateFilter.component.js | 82 +++++++------------ .../Date/DateFilterManager.component.js | 7 +- .../Date/dateFilterDataGetter.js | 6 +- .../date/stringToMomentDateFormat.js | 4 +- 4 files changed, 36 insertions(+), 63 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 80b002c049..1e03d9167e 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,9 +1,9 @@ // @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 { Temporal } from '@js-temporal/polyfill'; import { isValidZeroOrPositiveInteger } from 'capture-core-utils/validators/form'; import { SelectBoxes, orientations } from '../../FormFields/Options/SelectBoxes'; import { OptionSet } from '../../../metaData/OptionSet/OptionSet'; @@ -17,8 +17,7 @@ import './calendarFilterStyles.css'; import { mainOptionKeys, mainOptionTranslatedTexts } from './options'; import { getDateFilterData } from './dateFilterDataGetter'; import { RangeFilter } from './RangeFilter.component'; -import { parseDate, convertLocalToIsoCalendar, convertIsoToLocalCalendar } from '../../../utils/converters/date'; -import { systemSettingsStore } from '../../../metaDataMemoryStores'; +import { convertStringToDateFormat } from '../../../utils/converters/date'; const getStyles = (theme: Theme) => ({ fromToContainer: { @@ -119,24 +118,27 @@ const getRelativeRangeErrors = (startValue, endValue, submitAttempted) => { return errors; }; +// eslint-disable-next-line complexity const isAbsoluteRangeFilterValid = (from, to) => { - if (!from?.value && !to?.value) { - return false; - } const fromValue = from?.value; const toValue = to?.value; - const parseResultFrom = fromValue ? parseDate(fromValue) : { isValid: true, moment: null }; - const parseResultTo = toValue ? parseDate(toValue) : { isValid: true, moment: null }; - if (!(parseResultFrom.isValid && parseResultTo.isValid)) { + if (!fromValue && !toValue) { + return false; + } + + const isFromValueValid = from ? from.isValid : true; + const isToValueValid = to ? to.isValid : true; + + if (!isFromValueValid || !isToValueValid) { return false; } - const isValidMomentDate = () => - parseResultFrom.momentDate && - parseResultTo.momentDate && - parseResultFrom.momentDate.isAfter(parseResultTo.momentDate); - return !isValidMomentDate(); + if ((!fromValue && toValue) || (fromValue && !toValue)) { + return true; + } + + return !DateFilter.isFromAfterTo(fromValue, toValue); }; const isRelativeRangeFilterValid = (startValue, endValue) => { @@ -188,11 +190,9 @@ class DateFilterPlain extends Component implements UpdatableFilter } static isFromAfterTo(valueFrom: string, valueTo: string) { - const momentFrom = parseDate(valueFrom).momentDate; - const momentTo = parseDate(valueTo).momentDate; - // $FlowFixMe[incompatible-use] automated comment - // $FlowFixMe[incompatible-call] automated comment - return momentFrom.isAfter(momentTo); + const formattedFrom = convertStringToDateFormat(valueFrom, 'YYYY-MM-DD'); + const fromattedTo = convertStringToDateFormat(valueTo, 'YYYY-MM-DD'); + return Temporal.PlainDate.compare(formattedFrom, fromattedTo) > 0; } toD2DateTextFieldInstance: any; @@ -267,25 +267,12 @@ class DateFilterPlain extends Component implements UpdatableFilter return !values || DateFilter.isFilterValid(values.main, values.from, values.to, values.start, values.end); } - getUpdatedValue(valuePart: Object) { + getUpdatedValue(valuePart: { [key: string]: string }) { + // $FlowFixMe[cannot-spread-indexer] automated comment const valueObject = { ...this.props.value, ...valuePart, }; - const dateFormat = systemSettingsStore.get().dateFormat; - - ['from', 'to'].forEach((key) => { - if (valuePart[key] && valueObject[key]?.value) { - const isValidDate = valuePart[key]?.isValid; - valueObject[key] = { - ...valueObject[key], - value: isValidDate ? - moment(convertLocalToIsoCalendar(valueObject[key].value)).format(dateFormat) : - valueObject[key].value, - }; - } - }); - const isRelativeRangeValue = () => valueObject?.start || valuePart?.start || valuePart?.end; const isAbsoluteRangevalue = () => valueObject?.from || valuePart?.from || valuePart?.to; @@ -358,23 +345,10 @@ class DateFilterPlain extends Component implements UpdatableFilter render() { const { value, classes, onFocusUpdateButton } = this.props; + const fromValue = value?.from; + const toValue = value?.to; const { startValueError, endValueError, dateLogicError, bufferLogicError } = - this.getErrors(); - - const dateFormat = systemSettingsStore.get().dateFormat; - - const formatDate = (dateValue) => { - if (!dateValue) return ''; - const momentValue = moment(dateValue, dateFormat); - if (!momentValue.isValid()) return dateValue; - const isoDate = momentValue.format('YYYY-MM-DD'); - return convertIsoToLocalCalendar(isoDate); - }; - - const from = value?.from; - const to = value?.to; - const fromDate = formatDate(from?.value); - const toDate = formatDate(to?.value); + this.getErrors(); return (
@@ -394,11 +368,11 @@ class DateFilterPlain extends Component implements UpdatableFilter {/* $FlowSuppress: Flow not working 100% with HOCs */} {/* $FlowFixMe[prop-missing] automated comment */}
@@ -407,11 +381,11 @@ class DateFilterPlain extends Component implements UpdatableFilter {/* $FlowSuppress: Flow not working 100% with HOCs */} {/* $FlowFixMe[prop-missing] automated comment */}
diff --git a/src/core_modules/capture-core/components/FiltersForTypes/Date/DateFilterManager.component.js b/src/core_modules/capture-core/components/FiltersForTypes/Date/DateFilterManager.component.js index 928d9fd754..c0464eb8a6 100644 --- a/src/core_modules/capture-core/components/FiltersForTypes/Date/DateFilterManager.component.js +++ b/src/core_modules/capture-core/components/FiltersForTypes/Date/DateFilterManager.component.js @@ -1,8 +1,7 @@ // @flow import * as React from 'react'; -import moment from 'moment'; import log from 'loglevel'; -import { convertMomentToDateFormatString } from '../../../utils/converters/date'; +import { convertIsoToLocalCalendar } from '../../../utils/converters/date'; import { DateFilter } from './DateFilter.component'; import { mainOptionKeys } from './options'; import { dateFilterTypes } from './constants'; @@ -22,8 +21,8 @@ type State = { export class DateFilterManager extends React.Component { static convertDateForEdit(rawValue: string) { - const momentInstance = moment(rawValue); - return convertMomentToDateFormatString(momentInstance); + const localDate = convertIsoToLocalCalendar(rawValue); + return localDate; } static calculateAbsoluteRangeValueState(filter: DateFilterData) { return { diff --git a/src/core_modules/capture-core/components/FiltersForTypes/Date/dateFilterDataGetter.js b/src/core_modules/capture-core/components/FiltersForTypes/Date/dateFilterDataGetter.js index f7758cd1b1..6be16601ed 100644 --- a/src/core_modules/capture-core/components/FiltersForTypes/Date/dateFilterDataGetter.js +++ b/src/core_modules/capture-core/components/FiltersForTypes/Date/dateFilterDataGetter.js @@ -2,7 +2,7 @@ import { parseNumber } from 'capture-core-utils/parsers'; import { mainOptionKeys } from './options'; import { dateFilterTypes } from './constants'; -import { parseDate } from '../../../utils/converters/date'; +import { convertLocalToIsoCalendar } from '../../../utils/converters/date'; import { type AbsoluteDateFilterData, type RelativeDateFilterData, type DateValue } from './types'; type Value = { @@ -20,13 +20,13 @@ function convertAbsoluteDate(fromValue: ?string, toValue: ?string) { if (fromValue) { // $FlowFixMe[incompatible-type] automated comment - const fromClientValue: string = parseDate(fromValue).momentDate; + const fromClientValue: string = convertLocalToIsoCalendar(fromValue); rangeData.ge = fromClientValue; } if (toValue) { // $FlowFixMe[incompatible-type] automated comment - const toClientValue: string = parseDate(toValue).momentDate; + const toClientValue: string = convertLocalToIsoCalendar(toValue); rangeData.le = toClientValue; } diff --git a/src/core_modules/capture-core/utils/converters/date/stringToMomentDateFormat.js b/src/core_modules/capture-core/utils/converters/date/stringToMomentDateFormat.js index eed1df7957..1e718a5119 100644 --- a/src/core_modules/capture-core/utils/converters/date/stringToMomentDateFormat.js +++ b/src/core_modules/capture-core/utils/converters/date/stringToMomentDateFormat.js @@ -8,9 +8,9 @@ import { systemSettingsStore } from '../../../metaDataMemoryStores'; * @param {*} string - the string instance * @returns {string} */ -export function convertStringToDateFormat(date: string) { +export function convertStringToDateFormat(date: string, format) { if (!date || !date.length) { return ''; } - const dateFormat = systemSettingsStore.get().dateFormat; + const dateFormat = format || systemSettingsStore.get().dateFormat; const formattedDateString = moment(date, dateFormat).format(dateFormat); return formattedDateString; } From 8ce68dd3714d1201b1bacc94055c03e58539de3c Mon Sep 17 00:00:00 2001 From: alaa-yahia Date: Sun, 22 Dec 2024 23:55:07 +0200 Subject: [PATCH 12/25] fix: rename calendarMaxMoment to calendarMax --- .../D2Form/field/configs/dateField/getDateFieldConfig.js | 2 +- .../configs/dateField/getDateFieldConfigForCustomForm.js | 2 +- .../Enrollment/EnrollmentDataEntry.component.js | 4 ++-- .../DateAndTimeFields/DateField/Date.component.js | 8 ++++---- 4 files changed, 8 insertions(+), 8 deletions(-) diff --git a/src/core_modules/capture-core/components/D2Form/field/configs/dateField/getDateFieldConfig.js b/src/core_modules/capture-core/components/D2Form/field/configs/dateField/getDateFieldConfig.js index 7415bd7d5f..355346d87d 100644 --- a/src/core_modules/capture-core/components/D2Form/field/configs/dateField/getDateFieldConfig.js +++ b/src/core_modules/capture-core/components/D2Form/field/configs/dateField/getDateFieldConfig.js @@ -16,7 +16,7 @@ export const getDateFieldConfig = (metaData: DateDataElement, options: Object, q maxWidth: options.formHorizontal ? 150 : 350, calendarWidth: options.formHorizontal ? 250 : 350, popupAnchorPosition: getCalendarAnchorPosition(options.formHorizontal), - calendarMaxMoment: !metaData.allowFutureDate ? convertDateObjectToDateFormatString(moment()) : undefined, + calendarMax: !metaData.allowFutureDate ? convertDateObjectToDateFormatString(moment()) : undefined, }, options, metaData); return createFieldConfig({ diff --git a/src/core_modules/capture-core/components/D2Form/field/configs/dateField/getDateFieldConfigForCustomForm.js b/src/core_modules/capture-core/components/D2Form/field/configs/dateField/getDateFieldConfigForCustomForm.js index 8cb139fb73..efeea1412c 100644 --- a/src/core_modules/capture-core/components/D2Form/field/configs/dateField/getDateFieldConfigForCustomForm.js +++ b/src/core_modules/capture-core/components/D2Form/field/configs/dateField/getDateFieldConfigForCustomForm.js @@ -11,7 +11,7 @@ export const getDateFieldConfigForCustomForm = (metaData: DateDataElement, optio width: 350, maxWidth: 350, calendarWidth: 350, - calendarMaxMoment: !metaData.allowFutureDate ? convertDateObjectToDateFormatString(moment()) : undefined, + calendarMax: !metaData.allowFutureDate ? convertDateObjectToDateFormatString(moment()) : undefined, }, metaData); return createFieldConfig({ diff --git a/src/core_modules/capture-core/components/DataEntries/Enrollment/EnrollmentDataEntry.component.js b/src/core_modules/capture-core/components/DataEntries/Enrollment/EnrollmentDataEntry.component.js index 76ed900ccc..5ca70266c2 100644 --- a/src/core_modules/capture-core/components/DataEntries/Enrollment/EnrollmentDataEntry.component.js +++ b/src/core_modules/capture-core/components/DataEntries/Enrollment/EnrollmentDataEntry.component.js @@ -112,7 +112,7 @@ const getEnrollmentDateSettings = () => { required: true, calendarWidth: props.formHorizontal ? 250 : 350, popupAnchorPosition: getCalendarAnchorPosition(props.formHorizontal), - calendarMaxMoment: !props.enrollmentMetadata.allowFutureEnrollmentDate ? convertDateObjectToDateFormatString(moment()) : undefined, + calendarMax: !props.enrollmentMetadata.allowFutureEnrollmentDate ? convertDateObjectToDateFormatString(moment()) : undefined, }), getPropName: () => 'enrolledAt', getValidatorContainers: getEnrollmentDateValidatorContainer, @@ -160,7 +160,7 @@ const getIncidentDateSettings = () => { required: true, calendarWidth: props.formHorizontal ? 250 : 350, popupAnchorPosition: getCalendarAnchorPosition(props.formHorizontal), - calendarMaxMoment: !props.enrollmentMetadata.allowFutureIncidentDate ? + calendarMax: !props.enrollmentMetadata.allowFutureIncidentDate ? convertDateObjectToDateFormatString(moment()) : undefined, }), 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 95bf289714..c8ba299197 100644 --- a/src/core_modules/capture-ui/DateAndTimeFields/DateField/Date.component.js +++ b/src/core_modules/capture-ui/DateAndTimeFields/DateField/Date.component.js @@ -20,7 +20,7 @@ type Props = { onDateSelectedFromCalendar?: () => void, placeholder?: string, label?: string, - calendarMaxMoment?: any, + calendarMax?: any, innerMessage?: any }; @@ -61,11 +61,11 @@ export class DateField extends React.Component { maxWidth, calendarWidth, inputWidth, - calendarMaxMoment, + calendarMax, value, innerMessage, } = this.props; - +console.log(calendarMax,"calendarMax") const calculatedInputWidth = inputWidth || width; const calculatedCalendarWidth = calendarWidth || width; const calendarType = systemSettingsStore.get().calendar || 'gregory'; @@ -94,7 +94,7 @@ export class DateField extends React.Component { onFocus={this.props.onFocus} disabled={this.props.disabled} {...errorProps} - maxDate={calendarMaxMoment} + maxDate={calendarMax} />
); From b0dc55c8b770280b8ee0165ce64c06235db4fb59 Mon Sep 17 00:00:00 2001 From: alaa-yahia Date: Mon, 23 Dec 2024 00:52:08 +0200 Subject: [PATCH 13/25] fix: remove formating lines --- .../capture-core/converters/clientToForm.js | 7 ++----- .../capture-core/converters/clientToList.js | 7 ++----- .../capture-core/converters/clientToView.js | 7 ++----- .../capture-core/converters/formToClient.js | 8 ++------ .../utils/converters/date/convertLocalToIsoCalendar.js | 10 +++++++++- .../utils/converters/date/stringToMomentDateFormat.js | 3 ++- .../DateAndTimeFields/DateField/Date.component.js | 1 - 7 files changed, 19 insertions(+), 24 deletions(-) diff --git a/src/core_modules/capture-core/converters/clientToForm.js b/src/core_modules/capture-core/converters/clientToForm.js index c0b0d10948..d8cec38fd3 100644 --- a/src/core_modules/capture-core/converters/clientToForm.js +++ b/src/core_modules/capture-core/converters/clientToForm.js @@ -23,17 +23,14 @@ type RangeValue = { } function convertDateForEdit(rawValue: string): string { - const momentDate = moment(rawValue); - const dateString = momentDate.format('YYYY-MM-DD'); - return convertIsoToLocalCalendar(dateString); + return convertIsoToLocalCalendar(rawValue); } function convertDateTimeForEdit(rawValue: string): DateTimeFormValue { const dateTime = moment(rawValue); - const dateString = dateTime.format('YYYY-MM-DD'); const timeString = dateTime.format('HH:mm'); return { - date: convertIsoToLocalCalendar(dateString), + date: convertIsoToLocalCalendar(rawValue), time: timeString, }; } diff --git a/src/core_modules/capture-core/converters/clientToList.js b/src/core_modules/capture-core/converters/clientToList.js index 6ee3cb78a7..f026191a4e 100644 --- a/src/core_modules/capture-core/converters/clientToList.js +++ b/src/core_modules/capture-core/converters/clientToList.js @@ -11,16 +11,13 @@ import { MinimalCoordinates } from '../components/MinimalCoordinates'; import { TooltipOrgUnit } from '../components/Tooltips/TooltipOrgUnit'; function convertDateForListDisplay(rawValue: string): string { - const momentDate = moment(rawValue); - const dateString = momentDate.format('YYYY-MM-DD'); - return convertIsoToLocalCalendar(dateString); + return convertIsoToLocalCalendar(rawValue); } function convertDateTimeForListDisplay(rawValue: string): string { const momentDate = moment(rawValue); - const dateString = momentDate.format('YYYY-MM-DD'); const timeString = momentDate.format('HH:mm'); - const localDate = convertIsoToLocalCalendar(dateString); + const localDate = convertIsoToLocalCalendar(rawValue); return `${localDate} ${timeString}`; } diff --git a/src/core_modules/capture-core/converters/clientToView.js b/src/core_modules/capture-core/converters/clientToView.js index 6f4e2b7e14..0eb7efc21e 100644 --- a/src/core_modules/capture-core/converters/clientToView.js +++ b/src/core_modules/capture-core/converters/clientToView.js @@ -11,16 +11,13 @@ import { TooltipOrgUnit } from '../components/Tooltips/TooltipOrgUnit'; function convertDateForView(rawValue: string): string { - const momentDate = moment(rawValue); - const dateString = momentDate.format('YYYY-MM-DD'); - return convertIsoToLocalCalendar(dateString); + return convertIsoToLocalCalendar(rawValue); } function convertDateTimeForView(rawValue: string): string { const momentDate = moment(rawValue); - const dateString = momentDate.format('YYYY-MM-DD'); const timeString = momentDate.format('HH:mm'); - const localDate = convertIsoToLocalCalendar(dateString); + const localDate = convertIsoToLocalCalendar(rawValue); return `${localDate} ${timeString}`; } diff --git a/src/core_modules/capture-core/converters/formToClient.js b/src/core_modules/capture-core/converters/formToClient.js index 61b37923fd..d0a4460e24 100644 --- a/src/core_modules/capture-core/converters/formToClient.js +++ b/src/core_modules/capture-core/converters/formToClient.js @@ -38,12 +38,8 @@ function convertDateTime(formValue: DateTimeValue): ?string { function convertDate(dateValue: string) { const parsedDate = parseDate(dateValue); - if (!parsedDate.isValid || !parsedDate.momentDate) { - return null; - } - const formattedDate = parsedDate.momentDate.format('YYYY-MM-DD'); - - return convertLocalToIsoCalendar(formattedDate); + // $FlowFixMe[incompatible-use] automated comment + return parsedDate.isValid ? convertLocalToIsoCalendar(parsedDate.momentDate.toISOString()) : null; } function convertTime(timeValue: string) { 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 b3f323d178..2c2ce3d192 100644 --- a/src/core_modules/capture-core/utils/converters/date/convertLocalToIsoCalendar.js +++ b/src/core_modules/capture-core/utils/converters/date/convertLocalToIsoCalendar.js @@ -16,9 +16,17 @@ export function convertLocalToIsoCalendar(localDate: ?string): string { if (!localDate) { return ''; } + + const momentDate = moment(localDate); + if (!momentDate.isValid()) { + return ''; + } + + const formattedIsoDate = momentDate.format('YYYY-MM-DD'); + const calendar = systemSettingsStore.get().calendar; - const { year, month, day } = convertToIso8601(localDate, calendar); + const { year, month, day } = convertToIso8601(formattedIsoDate, calendar); const dateString = `${padWithZeros(year, 4)}-${padWithZeros(month, 2)}-${padWithZeros(day, 2)}`; const parsedMoment = moment(dateString); diff --git a/src/core_modules/capture-core/utils/converters/date/stringToMomentDateFormat.js b/src/core_modules/capture-core/utils/converters/date/stringToMomentDateFormat.js index 1e718a5119..29c7b2c929 100644 --- a/src/core_modules/capture-core/utils/converters/date/stringToMomentDateFormat.js +++ b/src/core_modules/capture-core/utils/converters/date/stringToMomentDateFormat.js @@ -6,9 +6,10 @@ import { systemSettingsStore } from '../../../metaDataMemoryStores'; * Converts a string date to a string date with default format based on the system date format * @export * @param {*} string - the string instance + * @param {string} [format] - optional date format. If not provided, the function uses system date format * @returns {string} */ -export function convertStringToDateFormat(date: string, format) { +export function convertStringToDateFormat(date: string, format?: string) { if (!date || !date.length) { return ''; } const dateFormat = format || systemSettingsStore.get().dateFormat; const formattedDateString = moment(date, dateFormat).format(dateFormat); 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 c8ba299197..1c4b2ee4b0 100644 --- a/src/core_modules/capture-ui/DateAndTimeFields/DateField/Date.component.js +++ b/src/core_modules/capture-ui/DateAndTimeFields/DateField/Date.component.js @@ -65,7 +65,6 @@ export class DateField extends React.Component { value, innerMessage, } = this.props; -console.log(calendarMax,"calendarMax") const calculatedInputWidth = inputWidth || width; const calculatedCalendarWidth = calendarWidth || width; const calendarType = systemSettingsStore.get().calendar || 'gregory'; From 2cc70be916491eb0988cd81d2110ec77f299a939 Mon Sep 17 00:00:00 2001 From: alaa-yahia Date: Mon, 23 Dec 2024 02:57:17 +0200 Subject: [PATCH 14/25] fix: day value not giving correct result in age field --- .../New/Fields/AgeField/AgeField.component.js | 5 -- .../date/convertStringToTemporal.js | 39 +++++++++ .../date/convertTemporalToString.js | 29 +++++++ .../utils/converters/date/index.js | 2 + .../capture-ui/AgeField/AgeField.component.js | 85 +++++++------------ 5 files changed, 102 insertions(+), 58 deletions(-) create mode 100644 src/core_modules/capture-core/utils/converters/date/convertStringToTemporal.js create mode 100644 src/core_modules/capture-core/utils/converters/date/convertTemporalToString.js diff --git a/src/core_modules/capture-core/components/FormFields/New/Fields/AgeField/AgeField.component.js b/src/core_modules/capture-core/components/FormFields/New/Fields/AgeField/AgeField.component.js index 2306b0854d..c160c34475 100644 --- a/src/core_modules/capture-core/components/FormFields/New/Fields/AgeField/AgeField.component.js +++ b/src/core_modules/capture-core/components/FormFields/New/Fields/AgeField/AgeField.component.js @@ -2,8 +2,6 @@ import * as React from 'react'; import { withStyles, withTheme } from '@material-ui/core/styles'; import { AgeField as UIAgeField } from 'capture-ui'; -import moment from 'moment'; -import { parseDate, convertMomentToDateFormatString } from '../../../../../utils/converters/date'; import { systemSettingsStore } from '../../../../../metaDataMemoryStores'; const getStyles = (theme: Theme) => ({ @@ -50,9 +48,6 @@ const AgeFieldPlain = (props: Props) => { return ( // $FlowFixMe[cannot-spread-inexact] automated comment diff --git a/src/core_modules/capture-core/utils/converters/date/convertStringToTemporal.js b/src/core_modules/capture-core/utils/converters/date/convertStringToTemporal.js new file mode 100644 index 0000000000..5eee8cc7a1 --- /dev/null +++ b/src/core_modules/capture-core/utils/converters/date/convertStringToTemporal.js @@ -0,0 +1,39 @@ +// @flow +import { Temporal } from '@js-temporal/polyfill'; +import { systemSettingsStore } from '../../../metaDataMemoryStores'; + +/** + * Converts a date string into a Temporal.PlainDate object using the system set calendar + * @export + * @param {*} string - dateString + * @returns {(Temporal.PlainDate | null)} + */ + +export function convertStringToTemporal(dateString: string): Temporal.PlainDate | null { + if (!dateString) { + return null; + } + try { + const dateWithHyphen = dateString.replace(/[\/\.]/g, '-'); + + const calendar = systemSettingsStore.get().calendar; + const dateFormat = systemSettingsStore.get().dateFormat; + + let year; let month; let day; + + if (dateFormat === 'YYYY-MM-DD') { + [year, month, day] = dateWithHyphen.split('-').map(Number); + } + if (dateFormat === 'DD-MM-YYYY') { + [day, month, year] = dateWithHyphen.split('-').map(Number); + } + return Temporal.PlainDate.from({ + year, + month, + day, + calendar, + }); + } catch (error) { + return ''; + } +} diff --git a/src/core_modules/capture-core/utils/converters/date/convertTemporalToString.js b/src/core_modules/capture-core/utils/converters/date/convertTemporalToString.js new file mode 100644 index 0000000000..a295b7984d --- /dev/null +++ b/src/core_modules/capture-core/utils/converters/date/convertTemporalToString.js @@ -0,0 +1,29 @@ +// @flow +import { Temporal } from '@js-temporal/polyfill'; +import { padWithZeros } from './padWithZeros'; +import { systemSettingsStore } from '../../../../capture-core/metaDataMemoryStores'; + +/** + * Converts a Temporal.PlainDate to a formatted date string (YYYY-MM-DD || DD-MM-YYYY) + * @param {Temporal.PlainDate} temporalDate - The Temporal date to convert + * @returns {string} Formatted date string, or empty string if invalid + */ + +export function convertTemporalToString(temporalDate: Temporal.PlainDate | null): string { + if (!temporalDate) { + return ''; + } + const dateFormat = systemSettingsStore.get().dateFormat; + + try { + const year = temporalDate.year; + const month = temporalDate.month; + const day = temporalDate.day; + + return dateFormat === 'YYYY-MM-DD' ? + `${padWithZeros(year, 4)}-${padWithZeros(month, 2)}-${padWithZeros(day, 2)}` : + `${padWithZeros(day, 2)}-${padWithZeros(month, 2)}-${padWithZeros(year, 4)}`; + } catch (error) { + return ''; + } +} diff --git a/src/core_modules/capture-core/utils/converters/date/index.js b/src/core_modules/capture-core/utils/converters/date/index.js index 511298b89d..11f200fb34 100644 --- a/src/core_modules/capture-core/utils/converters/date/index.js +++ b/src/core_modules/capture-core/utils/converters/date/index.js @@ -6,3 +6,5 @@ export { convertStringToDateFormat } from './stringToMomentDateFormat'; export { padWithZeros } from './padWithZeros'; export { convertIsoToLocalCalendar } from './convertIsoToLocalCalendar'; export { convertLocalToIsoCalendar } from './convertLocalToIsoCalendar'; +export { convertStringToTemporal } from './convertStringToTemporal'; +export { convertTemporalToString } from './convertTemporalToString'; diff --git a/src/core_modules/capture-ui/AgeField/AgeField.component.js b/src/core_modules/capture-ui/AgeField/AgeField.component.js index 78abcf4a21..b04b755613 100644 --- a/src/core_modules/capture-ui/AgeField/AgeField.component.js +++ b/src/core_modules/capture-ui/AgeField/AgeField.component.js @@ -1,7 +1,7 @@ // @flow import React, { Component } from 'react'; +import { Temporal } from '@js-temporal/polyfill'; import { isValidPositiveInteger } from 'capture-core-utils/validators/form'; -import { convertDateObjectToDateFormatString } from 'capture-core/utils/converters/date'; import { systemSettingsStore } from 'capture-core/metaDataMemoryStores'; import i18n from '@dhis2/d2-i18n'; import classNames from 'classnames'; @@ -12,6 +12,7 @@ import { AgeDateInput } from '../internal/AgeInput/AgeDateInput.component'; import defaultClasses from './ageField.module.css'; import { orientations } from '../constants/orientations.const'; import { withInternalChangeHandler } from '../HOC/withInternalChangeHandler'; +import { convertStringToTemporal, convertTemporalToString } from '../../capture-core/utils/converters/date'; type AgeValues = { date?: ?string, @@ -47,9 +48,6 @@ type Props = { inputMessageClasses: ?InputMessageClasses, inFocus?: ?boolean, shrinkDisabled?: ?boolean, - onParseDate: DateParser, - onGetFormattedDateStringFromMoment: DateStringFromMomentFormatter, - moment: any, dateCalendarTheme: Object, dateCalendarWidth?: ?any, datePopupAnchorPosition?: ?string, @@ -59,38 +57,26 @@ type Props = { datePlaceholder?: ?string, disabled?: ?boolean, }; -function getCalculatedValues( - dateValue: ?string, - onParseDate: DateParser, - onGetFormattedDateStringFromMoment: DateStringFromMomentFormatter, - moment: any, -): AgeValues { - const parseData = dateValue && onParseDate(dateValue); - if (!parseData || !parseData.isValid) { - return { - date: dateValue, - years: '', - months: '', - days: '', - }; - } - const dateFormat = systemSettingsStore.get().dateFormat; - const now = moment(convertDateObjectToDateFormatString(moment()), dateFormat); - const age = moment(parseData.momentDate); - const years = now.diff(age, 'years'); - age.add(years, 'years'); +function getCalculatedValues(dateValue: ?string): AgeValues { + const calendar = systemSettingsStore.get().calendar; + + const now = Temporal.Now.plainDateISO().withCalendar(calendar); - const months = now.diff(age, 'months'); - age.add(months, 'months'); + const age = convertStringToTemporal(dateValue); - const days = now.diff(age, 'days'); + const diff = now.since(age, { + largestUnit: 'years', + smallestUnit: 'days', + }); + + const date = convertTemporalToString(age); return { - date: onGetFormattedDateStringFromMoment(parseData.momentDate), - years: years.toString(), - months: months.toString(), - days: days.toString(), + date, + years: diff.years.toString(), + months: diff.months.toString(), + days: diff.days.toString(), }; } @@ -124,7 +110,7 @@ class D2AgeFieldPlain extends Component { } handleNumberBlur = (values: AgeValues) => { - const { onParseDate, onGetFormattedDateStringFromMoment, onRemoveFocus, moment } = this.props; + const { onRemoveFocus } = this.props; onRemoveFocus && onRemoveFocus(); if (D2AgeFieldPlain.isEmptyNumbers(values)) { @@ -136,28 +122,25 @@ class D2AgeFieldPlain extends Component { this.props.onBlur({ ...values, date: '' }); return; } - const dateFormat = systemSettingsStore.get().dateFormat; - const momentDate = moment(convertDateObjectToDateFormatString(moment(undefined, undefined, true)), dateFormat); - momentDate.subtract(D2AgeFieldPlain.getNumberOrZero(values.years), 'years'); - momentDate.subtract(D2AgeFieldPlain.getNumberOrZero(values.months), 'months'); - momentDate.subtract(D2AgeFieldPlain.getNumberOrZero(values.days), 'days'); - const calculatedValues = getCalculatedValues( - onGetFormattedDateStringFromMoment(momentDate), - onParseDate, - onGetFormattedDateStringFromMoment, - moment, - ); + + const calendar = systemSettingsStore.get().calendar; + + const now = Temporal.Now.plainDateISO().withCalendar(calendar); + + const calculatedDate = now.subtract({ + years: D2AgeFieldPlain.getNumberOrZero(values.years), + months: D2AgeFieldPlain.getNumberOrZero(values.months), + days: D2AgeFieldPlain.getNumberOrZero(values.days), + }); + + const calculatedValues = getCalculatedValues(convertTemporalToString(calculatedDate)); this.props.onBlur(calculatedValues); } handleDateBlur = (date: ?string, options: ?ValidationOptions) => { - const { onParseDate, onGetFormattedDateStringFromMoment, onRemoveFocus, moment } = this.props; + const { onRemoveFocus } = this.props; onRemoveFocus && onRemoveFocus(); - const calculatedValues = date ? getCalculatedValues( - date, - onParseDate, - onGetFormattedDateStringFromMoment, - moment) : null; + const calculatedValues = date ? getCalculatedValues(date) : null; this.props.onBlur(calculatedValues, options); } @@ -184,8 +167,6 @@ class D2AgeFieldPlain extends Component { datePopupAnchorPosition, dateCalendarTheme, dateCalendarLocale, - moment, - onParseDate, ...passOnProps } = this.props; return (
@@ -211,8 +192,6 @@ class D2AgeFieldPlain extends Component { shrinkDisabled, dateCalendarWidth, datePlaceholder, - moment, - onParseDate, ...passOnProps } = this.props; From 383c66fc571b6b4220fc0abf43f748744b62191f Mon Sep 17 00:00:00 2001 From: alaa-yahia Date: Mon, 23 Dec 2024 03:13:11 +0200 Subject: [PATCH 15/25] fix: flow errors --- .../utils/converters/date/convertStringToTemporal.js | 10 ++++++++-- .../utils/converters/date/convertTemporalToString.js | 9 +++++++-- .../capture-ui/AgeField/AgeField.component.js | 4 ---- 3 files changed, 15 insertions(+), 8 deletions(-) diff --git a/src/core_modules/capture-core/utils/converters/date/convertStringToTemporal.js b/src/core_modules/capture-core/utils/converters/date/convertStringToTemporal.js index 5eee8cc7a1..a50c92be11 100644 --- a/src/core_modules/capture-core/utils/converters/date/convertStringToTemporal.js +++ b/src/core_modules/capture-core/utils/converters/date/convertStringToTemporal.js @@ -9,7 +9,13 @@ import { systemSettingsStore } from '../../../metaDataMemoryStores'; * @returns {(Temporal.PlainDate | null)} */ -export function convertStringToTemporal(dateString: string): Temporal.PlainDate | null { +type PlainDate = { + year: number, + month: number, + day: number +}; + +export function convertStringToTemporal(dateString: ?string): PlainDate | null { if (!dateString) { return null; } @@ -34,6 +40,6 @@ export function convertStringToTemporal(dateString: string): Temporal.PlainDate calendar, }); } catch (error) { - return ''; + return null; } } diff --git a/src/core_modules/capture-core/utils/converters/date/convertTemporalToString.js b/src/core_modules/capture-core/utils/converters/date/convertTemporalToString.js index a295b7984d..e413468c66 100644 --- a/src/core_modules/capture-core/utils/converters/date/convertTemporalToString.js +++ b/src/core_modules/capture-core/utils/converters/date/convertTemporalToString.js @@ -1,5 +1,4 @@ // @flow -import { Temporal } from '@js-temporal/polyfill'; import { padWithZeros } from './padWithZeros'; import { systemSettingsStore } from '../../../../capture-core/metaDataMemoryStores'; @@ -9,7 +8,13 @@ import { systemSettingsStore } from '../../../../capture-core/metaDataMemoryStor * @returns {string} Formatted date string, or empty string if invalid */ -export function convertTemporalToString(temporalDate: Temporal.PlainDate | null): string { +type PlainDate = { + year: number, + month: number, + day: number +}; + +export function convertTemporalToString(temporalDate: PlainDate | null): string { if (!temporalDate) { return ''; } diff --git a/src/core_modules/capture-ui/AgeField/AgeField.component.js b/src/core_modules/capture-ui/AgeField/AgeField.component.js index b04b755613..fccf96d0e9 100644 --- a/src/core_modules/capture-ui/AgeField/AgeField.component.js +++ b/src/core_modules/capture-ui/AgeField/AgeField.component.js @@ -28,10 +28,6 @@ type InputMessageClasses = { validating?: ?string, } -type DateParser = (value: string) => { isValid: boolean, momentDate: any }; - -type DateStringFromMomentFormatter = (momentValue: Object) => string; - type ValidationOptions = { error?: ?string, errorCode?: ?string, From 013db2ccccfea8aae6363a471820c3bec70193d3 Mon Sep 17 00:00:00 2001 From: alaa-yahia Date: Mon, 6 Jan 2025 12:37:16 +0200 Subject: [PATCH 16/25] fix: display local time in tooltips --- package.json | 4 +- .../WidgetEnrollment.component.js | 3 +- .../WidgetNote/NoteSection/NoteSection.js | 41 +++++++++++-------- .../StageOverview/StageOverview.component.js | 5 ++- 4 files changed, 31 insertions(+), 22 deletions(-) diff --git a/package.json b/package.json index 1178b6e2e1..e4caffa666 100644 --- a/package.json +++ b/package.json @@ -10,8 +10,7 @@ "packages/rules-engine" ], "dependencies": { - "@dhis2/rules-engine-javascript": "101.19.1", - "@dhis2-ui/calendar": "^10.0.3", + "@dhis2/rules-engine-javascript": "101.20.3", "@dhis2/app-runtime": "^3.9.3", "@dhis2/d2-i18n": "^1.1.0", "@dhis2/d2-icons": "^1.0.1", @@ -20,6 +19,7 @@ "@dhis2/d2-ui-rich-text": "^7.4.0", "@dhis2/d2-ui-sharing-dialog": "^7.3.3", "@dhis2/ui": "^9.10.1", + "@dhis2-ui/calendar": "^10.0.3", "@joakim_sm/react-infinite-calendar": "^2.4.2", "@material-ui/core": "3.9.4", "@material-ui/icons": "3", diff --git a/src/core_modules/capture-core/components/WidgetEnrollment/WidgetEnrollment.component.js b/src/core_modules/capture-core/components/WidgetEnrollment/WidgetEnrollment.component.js index 1cd597f02d..66b9321c2a 100644 --- a/src/core_modules/capture-core/components/WidgetEnrollment/WidgetEnrollment.component.js +++ b/src/core_modules/capture-core/components/WidgetEnrollment/WidgetEnrollment.component.js @@ -74,6 +74,7 @@ export const WidgetEnrollmentPlain = ({ }: PlainProps) => { const [open, setOpenStatus] = useState(true); const { fromServerDate } = useTimeZoneConversion(); + const localDateTime: string = (convertValue(enrollment?.updatedAt, dataElementTypes.DATETIME): any); const geometryType = getGeometryType(enrollment?.geometry?.type); const { displayName: orgUnitName, ancestors } = useOrgUnitNameWithAncestors(enrollment?.orgUnit); const { displayName: ownerOrgUnitName, ancestors: ownerAncestors } = useOrgUnitNameWithAncestors(ownerOrgUnit?.id); @@ -157,7 +158,7 @@ export const WidgetEnrollmentPlain = ({ {i18n.t('Last updated')} - + {moment(fromServerDate(enrollment.updatedAt)).fromNow()}
diff --git a/src/core_modules/capture-core/components/WidgetNote/NoteSection/NoteSection.js b/src/core_modules/capture-core/components/WidgetNote/NoteSection/NoteSection.js index f908d1d3ed..825d27d527 100644 --- a/src/core_modules/capture-core/components/WidgetNote/NoteSection/NoteSection.js +++ b/src/core_modules/capture-core/components/WidgetNote/NoteSection/NoteSection.js @@ -9,6 +9,8 @@ import { colors, spacersNum, Button, Tooltip } from '@dhis2/ui'; import moment from 'moment'; import { useTimeZoneConversion } from '@dhis2/app-runtime'; import { TextField } from '../../FormFields/New'; +import { convertValue as convertValueClientToView } from '../../../converters/clientToView'; +import { dataElementTypes } from '../../../metaData'; const FocusTextField = withFocusSaver()(TextField); @@ -97,26 +99,29 @@ const NoteSectionPlain = ({ setEditing(false); }, [handleAddNote, newNoteValue]); - const NoteItem = ({ value, storedAt, createdBy }) => ( -
- {/* TODO: add avatar */} -
-
- {createdBy && - {createdBy.firstName} {' '} {createdBy.surname} - } - - - {moment(fromServerDate(storedAt)).fromNow()} - - -
-
- {value} + const NoteItem = ({ value, storedAt, createdBy }) => { + const localDateTime: string = (convertValueClientToView(storedAt, dataElementTypes.DATETIME): any); + return ( +
+ {/* TODO: add avatar */} +
+
+ {createdBy && + {createdBy.firstName} {' '} {createdBy.surname} + } + + + {moment(fromServerDate(storedAt)).fromNow()} + + +
+
+ {value} +
-
- ); + ); + }; return ( diff --git a/src/core_modules/capture-core/components/WidgetStagesAndEvents/Stages/Stage/StageOverview/StageOverview.component.js b/src/core_modules/capture-core/components/WidgetStagesAndEvents/Stages/Stage/StageOverview/StageOverview.component.js index 907e049840..5c72a76670 100644 --- a/src/core_modules/capture-core/components/WidgetStagesAndEvents/Stages/Stage/StageOverview/StageOverview.component.js +++ b/src/core_modules/capture-core/components/WidgetStagesAndEvents/Stages/Stage/StageOverview/StageOverview.component.js @@ -12,6 +12,8 @@ import { statusTypes } from 'capture-core/events/statusTypes'; import { NonBundledDhis2Icon } from '../../../../NonBundledDhis2Icon'; import type { Props } from './stageOverview.types'; import { isEventOverdue } from '../StageDetail/hooks/helpers'; +import { convertValue as convertValueClientToView } from '../../../../../converters/clientToView'; +import { dataElementTypes } from '../../../../../metaData'; const styles = { container: { @@ -71,11 +73,12 @@ const getLastUpdatedAt = (events, fromServerDate) => { if (lastEventUpdated) { const { updatedAt } = lastEventUpdated; + const localDateTime: string = (convertValueClientToView(updatedAt, dataElementTypes.DATETIME): any); return lastEventUpdated?.updatedAt && moment(updatedAt).isValid() ? ( <> {i18n.t('Last updated')}  - + {moment(fromServerDate(updatedAt)).fromNow()} From 8dd111092a4f17190c4c1cd9de8b2d00cca89989 Mon Sep 17 00:00:00 2001 From: alaa-yahia Date: Mon, 6 Jan 2025 15:22:02 +0200 Subject: [PATCH 17/25] fix: date is not valid error not displayed --- .../capture-ui/AgeField/AgeField.component.js | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/src/core_modules/capture-ui/AgeField/AgeField.component.js b/src/core_modules/capture-ui/AgeField/AgeField.component.js index fccf96d0e9..65085efe6b 100644 --- a/src/core_modules/capture-ui/AgeField/AgeField.component.js +++ b/src/core_modules/capture-ui/AgeField/AgeField.component.js @@ -7,6 +7,7 @@ import i18n from '@dhis2/d2-i18n'; import classNames from 'classnames'; import { IconButton } from 'capture-ui'; import { IconCross24 } from '@dhis2/ui'; +import { parseDate } from 'capture-core/utils/converters/date'; import { AgeNumberInput } from '../internal/AgeInput/AgeNumberInput.component'; import { AgeDateInput } from '../internal/AgeInput/AgeDateInput.component'; import defaultClasses from './ageField.module.css'; @@ -55,6 +56,16 @@ type Props = { }; function getCalculatedValues(dateValue: ?string): AgeValues { + const parseData = dateValue && parseDate(dateValue); + if (!parseData || !parseData.isValid) { + return { + date: dateValue, + years: '', + months: '', + days: '', + }; + } + const calendar = systemSettingsStore.get().calendar; const now = Temporal.Now.plainDateISO().withCalendar(calendar); From 209860c630fce98c981aff7413513b7bda13676d Mon Sep 17 00:00:00 2001 From: alaa-yahia Date: Thu, 9 Jan 2025 13:42:31 +0200 Subject: [PATCH 18/25] fix: remove app specific objects --- .../capture-core-utils/date/index.js | 3 + .../date/padWithZeros.js | 0 .../date/stringToTemporal.js | 45 +++++++++++++++ .../date/temporalToString.js | 33 +++++++++++ .../configs/ageField/getAgeFieldConfig.js | 3 + .../configs/dateField/getDateFieldConfig.js | 3 + .../date/convertIsoToLocalCalendar.js | 2 +- .../date/convertLocalToIsoCalendar.js | 2 +- .../date/convertStringToTemporal.js | 28 ++-------- .../date/convertTemporalToString.js | 15 +---- .../utils/converters/date/index.js | 1 - .../capture-ui/AgeField/AgeField.component.js | 55 ++++++++++--------- .../DateField/Date.component.js | 13 +++-- 13 files changed, 131 insertions(+), 72 deletions(-) rename src/core_modules/{capture-core/utils/converters => capture-core-utils}/date/padWithZeros.js (100%) create mode 100644 src/core_modules/capture-core-utils/date/stringToTemporal.js create mode 100644 src/core_modules/capture-core-utils/date/temporalToString.js diff --git a/src/core_modules/capture-core-utils/date/index.js b/src/core_modules/capture-core-utils/date/index.js index 8af83e8b8d..f29e3a8760 100644 --- a/src/core_modules/capture-core-utils/date/index.js +++ b/src/core_modules/capture-core-utils/date/index.js @@ -1,2 +1,5 @@ // @flow export { getFormattedStringFromMomentUsingEuropeanGlyphs } from './date.utils'; +export { padWithZeros } from './padWithZeros'; +export { temporalToString } from './temporalToString'; +export { stringToTemporal } from './stringToTemporal'; diff --git a/src/core_modules/capture-core/utils/converters/date/padWithZeros.js b/src/core_modules/capture-core-utils/date/padWithZeros.js similarity index 100% rename from src/core_modules/capture-core/utils/converters/date/padWithZeros.js rename to src/core_modules/capture-core-utils/date/padWithZeros.js diff --git a/src/core_modules/capture-core-utils/date/stringToTemporal.js b/src/core_modules/capture-core-utils/date/stringToTemporal.js new file mode 100644 index 0000000000..8063305d0f --- /dev/null +++ b/src/core_modules/capture-core-utils/date/stringToTemporal.js @@ -0,0 +1,45 @@ +// @flow +import { Temporal } from '@js-temporal/polyfill'; + +/** + * Converts a date string into a Temporal.PlainDate object using the specified calendar + * @export + * @param {?string} dateString - The date string to convert + * @param {?string} calendarType - The calendar type to use + * @param {?string} dateFormat - The current system date format ('YYYY-MM-DD' or 'DD-MM-YYYY') + * @returns {(Temporal.PlainDate | null)} + */ + +type PlainDate = { + year: number, + month: number, + day: number +}; + +export function stringToTemporal(dateString: ?string, + calendarType: ?string, + dateFormat: ?string): PlainDate | null { + if (!dateString) { + return null; + } + try { + const dateWithHyphen = dateString.replace(/[\/\.]/g, '-'); + + let year; let month; let day; + + if (dateFormat === 'YYYY-MM-DD') { + [year, month, day] = dateWithHyphen.split('-').map(Number); + } + if (dateFormat === 'DD-MM-YYYY') { + [day, month, year] = dateWithHyphen.split('-').map(Number); + } + return Temporal.PlainDate.from({ + year, + month, + day, + calendarType, + }); + } catch (error) { + return null; + } +} diff --git a/src/core_modules/capture-core-utils/date/temporalToString.js b/src/core_modules/capture-core-utils/date/temporalToString.js new file mode 100644 index 0000000000..72f650b149 --- /dev/null +++ b/src/core_modules/capture-core-utils/date/temporalToString.js @@ -0,0 +1,33 @@ +// @flow +import { padWithZeros } from './padWithZeros'; + +/** + * Converts a Temporal.PlainDate to a formatted date string (YYYY-MM-DD or DD-MM-YYYY) + * @param {Temporal.PlainDate | null} temporalDate - The Temporal date to convert + * @param {?string} dateFormat - The current system date format ('YYYY-MM-DD' or 'DD-MM-YYYY') + * @returns {string} Formatted date string, or empty string if invalid + */ + +type PlainDate = { + year: number, + month: number, + day: number +}; + +export function temporalToString(temporalDate: PlainDate | null, dateFormat: ?string): string { + if (!temporalDate) { + return ''; + } + + try { + const year = temporalDate.year; + const month = temporalDate.month; + const day = temporalDate.day; + + return dateFormat === 'YYYY-MM-DD' ? + `${padWithZeros(year, 4)}-${padWithZeros(month, 2)}-${padWithZeros(day, 2)}` : + `${padWithZeros(day, 2)}-${padWithZeros(month, 2)}-${padWithZeros(year, 4)}`; + } catch (error) { + return ''; + } +} diff --git a/src/core_modules/capture-core/components/D2Form/field/configs/ageField/getAgeFieldConfig.js b/src/core_modules/capture-core/components/D2Form/field/configs/ageField/getAgeFieldConfig.js index 4c55b92527..e9ffbf41b6 100644 --- a/src/core_modules/capture-core/components/D2Form/field/configs/ageField/getAgeFieldConfig.js +++ b/src/core_modules/capture-core/components/D2Form/field/configs/ageField/getAgeFieldConfig.js @@ -2,6 +2,7 @@ import { orientations } from '../../../../FormFields/New'; import { createFieldConfig, createProps } from '../base/configBaseDefaultForm'; import { AgeFieldForForm } from '../../Components'; +import { systemSettingsStore } from '../../../../../metaDataMemoryStores'; import { type DataElement } from '../../../../../metaData'; import type { QuerySingleResource } from '../../../../../utils/api/api.types'; @@ -15,6 +16,8 @@ export const getAgeFieldConfig = (metaData: DataElement, options: Object, queryS shrinkDisabled: options.formHorizontal, dateCalendarWidth: options.formHorizontal ? 250 : 350, datePopupAnchorPosition: getCalendarAnchorPosition(options.formHorizontal), + calendarType: systemSettingsStore.get().calendar, + dateFormat: systemSettingsStore.get().dateFormat, }, options, metaData); return createFieldConfig({ diff --git a/src/core_modules/capture-core/components/D2Form/field/configs/dateField/getDateFieldConfig.js b/src/core_modules/capture-core/components/D2Form/field/configs/dateField/getDateFieldConfig.js index 355346d87d..4af9e783c5 100644 --- a/src/core_modules/capture-core/components/D2Form/field/configs/dateField/getDateFieldConfig.js +++ b/src/core_modules/capture-core/components/D2Form/field/configs/dateField/getDateFieldConfig.js @@ -3,6 +3,7 @@ import moment from 'moment'; import { createFieldConfig, createProps } from '../base/configBaseDefaultForm'; import { DateFieldForForm } from '../../Components'; import { convertDateObjectToDateFormatString } from '../../../../../../capture-core/utils/converters/date'; +import { systemSettingsStore } from '../../../../../metaDataMemoryStores'; import type { DateDataElement } from '../../../../../metaData'; import type { QuerySingleResource } from '../../../../../utils/api/api.types'; @@ -17,6 +18,8 @@ export const getDateFieldConfig = (metaData: DateDataElement, options: Object, q calendarWidth: options.formHorizontal ? 250 : 350, popupAnchorPosition: getCalendarAnchorPosition(options.formHorizontal), calendarMax: !metaData.allowFutureDate ? convertDateObjectToDateFormatString(moment()) : undefined, + calendarType: systemSettingsStore.get().calendar, + dateFormat: systemSettingsStore.get().dateFormat, }, options, metaData); return createFieldConfig({ 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 442ad4a60c..fa09f77c53 100644 --- a/src/core_modules/capture-core/utils/converters/date/convertIsoToLocalCalendar.js +++ b/src/core_modules/capture-core/utils/converters/date/convertIsoToLocalCalendar.js @@ -4,7 +4,7 @@ import { convertFromIso8601, } from '@dhis2/multi-calendar-dates'; import { systemSettingsStore } from '../../../../capture-core/metaDataMemoryStores'; -import { padWithZeros } from './padWithZeros'; +import { padWithZeros } from '../../../../capture-core-utils/date'; /** * Converts a date from ISO calendar to local calendar 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 2c2ce3d192..2323f01370 100644 --- a/src/core_modules/capture-core/utils/converters/date/convertLocalToIsoCalendar.js +++ b/src/core_modules/capture-core/utils/converters/date/convertLocalToIsoCalendar.js @@ -4,7 +4,7 @@ import { convertToIso8601, } from '@dhis2/multi-calendar-dates'; import { systemSettingsStore } from '../../../../capture-core/metaDataMemoryStores'; -import { padWithZeros } from './padWithZeros'; +import { padWithZeros } from '../../../../capture-core-utils/date'; /** * Converts a date from local calendar to ISO calendar diff --git a/src/core_modules/capture-core/utils/converters/date/convertStringToTemporal.js b/src/core_modules/capture-core/utils/converters/date/convertStringToTemporal.js index a50c92be11..57c0dec668 100644 --- a/src/core_modules/capture-core/utils/converters/date/convertStringToTemporal.js +++ b/src/core_modules/capture-core/utils/converters/date/convertStringToTemporal.js @@ -1,6 +1,6 @@ // @flow -import { Temporal } from '@js-temporal/polyfill'; import { systemSettingsStore } from '../../../metaDataMemoryStores'; +import { stringToTemporal } from '../../../../capture-core-utils/date'; /** * Converts a date string into a Temporal.PlainDate object using the system set calendar @@ -19,27 +19,7 @@ export function convertStringToTemporal(dateString: ?string): PlainDate | null { if (!dateString) { return null; } - try { - const dateWithHyphen = dateString.replace(/[\/\.]/g, '-'); - - const calendar = systemSettingsStore.get().calendar; - const dateFormat = systemSettingsStore.get().dateFormat; - - let year; let month; let day; - - if (dateFormat === 'YYYY-MM-DD') { - [year, month, day] = dateWithHyphen.split('-').map(Number); - } - if (dateFormat === 'DD-MM-YYYY') { - [day, month, year] = dateWithHyphen.split('-').map(Number); - } - return Temporal.PlainDate.from({ - year, - month, - day, - calendar, - }); - } catch (error) { - return null; - } + const calendar = systemSettingsStore.get().calendar; + const dateFormat = systemSettingsStore.get().dateFormat; + return stringToTemporal(dateString, calendar, dateFormat); } diff --git a/src/core_modules/capture-core/utils/converters/date/convertTemporalToString.js b/src/core_modules/capture-core/utils/converters/date/convertTemporalToString.js index e413468c66..9ba772b23a 100644 --- a/src/core_modules/capture-core/utils/converters/date/convertTemporalToString.js +++ b/src/core_modules/capture-core/utils/converters/date/convertTemporalToString.js @@ -1,6 +1,6 @@ // @flow -import { padWithZeros } from './padWithZeros'; import { systemSettingsStore } from '../../../../capture-core/metaDataMemoryStores'; +import { temporalToString } from '../../../../capture-core-utils/date'; /** * Converts a Temporal.PlainDate to a formatted date string (YYYY-MM-DD || DD-MM-YYYY) @@ -19,16 +19,5 @@ export function convertTemporalToString(temporalDate: PlainDate | null): string return ''; } const dateFormat = systemSettingsStore.get().dateFormat; - - try { - const year = temporalDate.year; - const month = temporalDate.month; - const day = temporalDate.day; - - return dateFormat === 'YYYY-MM-DD' ? - `${padWithZeros(year, 4)}-${padWithZeros(month, 2)}-${padWithZeros(day, 2)}` : - `${padWithZeros(day, 2)}-${padWithZeros(month, 2)}-${padWithZeros(year, 4)}`; - } catch (error) { - return ''; - } + return temporalToString(temporalDate, dateFormat); } diff --git a/src/core_modules/capture-core/utils/converters/date/index.js b/src/core_modules/capture-core/utils/converters/date/index.js index 11f200fb34..8f878a421d 100644 --- a/src/core_modules/capture-core/utils/converters/date/index.js +++ b/src/core_modules/capture-core/utils/converters/date/index.js @@ -3,7 +3,6 @@ export { parseDate } from './parser'; export { convertDateObjectToDateFormatString } from './dateObjectToDateFormatString'; export { convertMomentToDateFormatString } from './momentToDateFormatString'; export { convertStringToDateFormat } from './stringToMomentDateFormat'; -export { padWithZeros } from './padWithZeros'; export { convertIsoToLocalCalendar } from './convertIsoToLocalCalendar'; export { convertLocalToIsoCalendar } from './convertLocalToIsoCalendar'; export { convertStringToTemporal } from './convertStringToTemporal'; diff --git a/src/core_modules/capture-ui/AgeField/AgeField.component.js b/src/core_modules/capture-ui/AgeField/AgeField.component.js index 65085efe6b..f2b96b19a5 100644 --- a/src/core_modules/capture-ui/AgeField/AgeField.component.js +++ b/src/core_modules/capture-ui/AgeField/AgeField.component.js @@ -2,18 +2,16 @@ import React, { Component } from 'react'; import { Temporal } from '@js-temporal/polyfill'; import { isValidPositiveInteger } from 'capture-core-utils/validators/form'; -import { systemSettingsStore } from 'capture-core/metaDataMemoryStores'; import i18n from '@dhis2/d2-i18n'; import classNames from 'classnames'; import { IconButton } from 'capture-ui'; import { IconCross24 } from '@dhis2/ui'; -import { parseDate } from 'capture-core/utils/converters/date'; import { AgeNumberInput } from '../internal/AgeInput/AgeNumberInput.component'; import { AgeDateInput } from '../internal/AgeInput/AgeDateInput.component'; import defaultClasses from './ageField.module.css'; import { orientations } from '../constants/orientations.const'; import { withInternalChangeHandler } from '../HOC/withInternalChangeHandler'; -import { convertStringToTemporal, convertTemporalToString } from '../../capture-core/utils/converters/date'; +import { stringToTemporal, temporalToString } from '../../capture-core-utils/date'; type AgeValues = { date?: ?string, @@ -53,31 +51,21 @@ type Props = { dateCalendarOnConvertValueOut: (value: string) => string, datePlaceholder?: ?string, disabled?: ?boolean, + dateFormat: ?string, + calendarType: ?string, }; -function getCalculatedValues(dateValue: ?string): AgeValues { - const parseData = dateValue && parseDate(dateValue); - if (!parseData || !parseData.isValid) { - return { - date: dateValue, - years: '', - months: '', - days: '', - }; - } - - const calendar = systemSettingsStore.get().calendar; - - const now = Temporal.Now.plainDateISO().withCalendar(calendar); +function getCalculatedValues(dateValue: ?string, calendarType: ?string, dateFormat: ?string): AgeValues { + const now = Temporal.Now.plainDateISO().withCalendar(calendarType); - const age = convertStringToTemporal(dateValue); + const age = stringToTemporal(dateValue, calendarType, dateFormat); const diff = now.since(age, { largestUnit: 'years', smallestUnit: 'days', }); - const date = convertTemporalToString(age); + const date = temporalToString(age, dateFormat); return { date, @@ -117,7 +105,7 @@ class D2AgeFieldPlain extends Component { } handleNumberBlur = (values: AgeValues) => { - const { onRemoveFocus } = this.props; + const { onRemoveFocus, calendarType = 'gregory', dateFormat = 'YYYY-MM-DD' } = this.props; onRemoveFocus && onRemoveFocus(); if (D2AgeFieldPlain.isEmptyNumbers(values)) { @@ -130,24 +118,37 @@ class D2AgeFieldPlain extends Component { return; } - const calendar = systemSettingsStore.get().calendar; - - const now = Temporal.Now.plainDateISO().withCalendar(calendar); + const now = Temporal.Now.plainDateISO().withCalendar(calendarType); const calculatedDate = now.subtract({ years: D2AgeFieldPlain.getNumberOrZero(values.years), months: D2AgeFieldPlain.getNumberOrZero(values.months), days: D2AgeFieldPlain.getNumberOrZero(values.days), }); - - const calculatedValues = getCalculatedValues(convertTemporalToString(calculatedDate)); + const dateString = temporalToString(calculatedDate, dateFormat); + const calculatedValues = getCalculatedValues(dateString, calendarType, dateFormat); this.props.onBlur(calculatedValues); } handleDateBlur = (date: ?string, options: ?ValidationOptions) => { - const { onRemoveFocus } = this.props; + const { onRemoveFocus, calendarType = 'gregory', dateFormat = 'YYYY-MM-DD' } = this.props; onRemoveFocus && onRemoveFocus(); - const calculatedValues = date ? getCalculatedValues(date) : null; + const isDateValid = options && !options.error; + if (!date) { + this.props.onBlur(null, options); + return; + } + if (!isDateValid) { + const calculatedValues = { + date, + years: '', + months: '', + days: '', + }; + this.props.onBlur(calculatedValues, options); + return; + } + const calculatedValues = getCalculatedValues(date, calendarType, dateFormat); this.props.onBlur(calculatedValues, options); } 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 1c4b2ee4b0..8fa5d01b6e 100644 --- a/src/core_modules/capture-ui/DateAndTimeFields/DateField/Date.component.js +++ b/src/core_modules/capture-ui/DateAndTimeFields/DateField/Date.component.js @@ -1,7 +1,6 @@ // @flow import React from 'react'; import { CalendarInput } from '@dhis2/ui'; -import { systemSettingsStore } from '../../../capture-core/metaDataMemoryStores'; type ValidationOptions = { error?: ?string, @@ -21,7 +20,9 @@ type Props = { placeholder?: string, label?: string, calendarMax?: any, - innerMessage?: any + innerMessage?: any, + dateFormat: ?string, + calendarType: ?string, }; type Validation = {| @@ -64,11 +65,13 @@ export class DateField extends React.Component { calendarMax, value, innerMessage, + calendarType, + dateFormat, } = this.props; const calculatedInputWidth = inputWidth || width; const calculatedCalendarWidth = calendarWidth || width; - const calendarType = systemSettingsStore.get().calendar || 'gregory'; - const format = systemSettingsStore.get().dateFormat; + const calendar = calendarType || 'gregory'; + const format = dateFormat || 'YYYY-MM-DD'; const errorProps = innerMessage && innerMessage.messageType === 'error' ? { error: !!innerMessage.message?.dateInnerErrorMessage, validationText: innerMessage.message?.dateInnerErrorMessage } @@ -86,7 +89,7 @@ export class DateField extends React.Component { placeholder={this.props.placeholder} format={format} onDateSelect={this.handleDateSelected} - calendar={calendarType} + calendar={calendar} date={value} width={String(calculatedCalendarWidth)} inputWidth={String(calculatedInputWidth)} From c8d1fbd06d059a4eac71255c0812b4988b85523c Mon Sep 17 00:00:00 2001 From: alaa-yahia Date: Mon, 13 Jan 2025 11:21:00 +0200 Subject: [PATCH 19/25] refactor: simplify converters logic --- .../capture-core-utils/date/stringToTemporal.js | 4 ++-- .../configs/dateTimeField/getDateTimeFieldConfig.js | 3 +++ .../Enrollment/EnrollmentDataEntry.component.js | 9 ++++++++- .../EditEventDataEntry.component.js | 3 +++ .../capture-core/converters/formToClient.js | 13 +++++-------- .../converters/date/convertLocalToIsoCalendar.js | 9 +-------- .../converters/date/convertStringToTemporal.js | 8 ++++---- .../converters/date/convertTemporalToString.js | 4 ++-- 8 files changed, 28 insertions(+), 25 deletions(-) diff --git a/src/core_modules/capture-core-utils/date/stringToTemporal.js b/src/core_modules/capture-core-utils/date/stringToTemporal.js index 8063305d0f..3be34f0775 100644 --- a/src/core_modules/capture-core-utils/date/stringToTemporal.js +++ b/src/core_modules/capture-core-utils/date/stringToTemporal.js @@ -17,7 +17,7 @@ type PlainDate = { }; export function stringToTemporal(dateString: ?string, - calendarType: ?string, + calendar: ?string, dateFormat: ?string): PlainDate | null { if (!dateString) { return null; @@ -37,7 +37,7 @@ export function stringToTemporal(dateString: ?string, year, month, day, - calendarType, + calendar, }); } catch (error) { return null; diff --git a/src/core_modules/capture-core/components/D2Form/field/configs/dateTimeField/getDateTimeFieldConfig.js b/src/core_modules/capture-core/components/D2Form/field/configs/dateTimeField/getDateTimeFieldConfig.js index d7a24cb7f3..939efefb95 100644 --- a/src/core_modules/capture-core/components/D2Form/field/configs/dateTimeField/getDateTimeFieldConfig.js +++ b/src/core_modules/capture-core/components/D2Form/field/configs/dateTimeField/getDateTimeFieldConfig.js @@ -2,6 +2,7 @@ import { orientations } from '../../../../FormFields/New'; import { createFieldConfig, createProps } from '../base/configBaseDefaultForm'; import { DateTimeFieldForForm } from '../../Components'; +import { systemSettingsStore } from '../../../../../metaDataMemoryStores'; import type { DataElement as MetaDataElement } from '../../../../../metaData'; import type { QuerySingleResource } from '../../../../../utils/api/api.types'; @@ -17,6 +18,8 @@ export const getDateTimeFieldConfig = (metaData: MetaDataElement, options: Objec shrinkDisabled: options.formHorizontal, calendarWidth: options.formHorizontal ? '250px' : '350px', popupAnchorPosition: getCalendarAnchorPosition(options.formHorizontal), + calendarType: systemSettingsStore.get().calendar, + dateFormat: systemSettingsStore.get().dateFormat, }, options, metaData); return createFieldConfig({ diff --git a/src/core_modules/capture-core/components/DataEntries/Enrollment/EnrollmentDataEntry.component.js b/src/core_modules/capture-core/components/DataEntries/Enrollment/EnrollmentDataEntry.component.js index 5ca70266c2..6f9a52fb42 100644 --- a/src/core_modules/capture-core/components/DataEntries/Enrollment/EnrollmentDataEntry.component.js +++ b/src/core_modules/capture-core/components/DataEntries/Enrollment/EnrollmentDataEntry.component.js @@ -43,6 +43,7 @@ import { withDataEntryFields, } from '../../DataEntryDhis2Helpers'; import { convertDateObjectToDateFormatString } from '../../../../capture-core/utils/converters/date'; +import { systemSettingsStore } from '../../../metaDataMemoryStores'; const overrideMessagePropNames = { errorMessage: 'validationError', @@ -112,7 +113,11 @@ const getEnrollmentDateSettings = () => { required: true, calendarWidth: props.formHorizontal ? 250 : 350, popupAnchorPosition: getCalendarAnchorPosition(props.formHorizontal), - calendarMax: !props.enrollmentMetadata.allowFutureEnrollmentDate ? convertDateObjectToDateFormatString(moment()) : undefined, + calendarMax: !props.enrollmentMetadata.allowFutureEnrollmentDate ? + convertDateObjectToDateFormatString(moment()) : + undefined, + calendarType: systemSettingsStore.get().calendar, + dateFormat: systemSettingsStore.get().dateFormat, }), getPropName: () => 'enrolledAt', getValidatorContainers: getEnrollmentDateValidatorContainer, @@ -163,6 +168,8 @@ const getIncidentDateSettings = () => { calendarMax: !props.enrollmentMetadata.allowFutureIncidentDate ? convertDateObjectToDateFormatString(moment()) : undefined, + calendarType: systemSettingsStore.get().calendar, + dateFormat: systemSettingsStore.get().dateFormat, }), getPropName: () => 'occurredAt', getPassOnFieldData: () => true, diff --git a/src/core_modules/capture-core/components/WidgetEventEdit/EditEventDataEntry/EditEventDataEntry.component.js b/src/core_modules/capture-core/components/WidgetEventEdit/EditEventDataEntry/EditEventDataEntry.component.js index e958a69644..96b1e4ff0a 100644 --- a/src/core_modules/capture-core/components/WidgetEventEdit/EditEventDataEntry/EditEventDataEntry.component.js +++ b/src/core_modules/capture-core/components/WidgetEventEdit/EditEventDataEntry/EditEventDataEntry.component.js @@ -47,6 +47,7 @@ import { withAOCFieldBuilder, withDataEntryFields, } from '../../DataEntryDhis2Helpers/'; +import { systemSettingsStore } from '../../../metaDataMemoryStores'; import type { UserFormField } from '../../FormFields/UserField'; const tabMode = Object.freeze({ @@ -137,6 +138,8 @@ const buildReportDateSettingsFn = () => { calendarWidth: 350, label: props.formFoundation.getLabel('occurredAt'), required: true, + calendarType: systemSettingsStore.get().calendar, + dateFormat: systemSettingsStore.get().dateFormat, }), getPropName: () => 'occurredAt', getValidatorContainers: () => getEventDateValidatorContainers(), diff --git a/src/core_modules/capture-core/converters/formToClient.js b/src/core_modules/capture-core/converters/formToClient.js index d0a4460e24..187549d140 100644 --- a/src/core_modules/capture-core/converters/formToClient.js +++ b/src/core_modules/capture-core/converters/formToClient.js @@ -3,7 +3,7 @@ import moment from 'moment'; import isString from 'd2-utilizr/lib/isString'; import { parseNumber, parseTime } from 'capture-core-utils/parsers'; import { dataElementTypes } from '../metaData'; -import { parseDate, convertLocalToIsoCalendar } from '../utils/converters/date'; +import { convertLocalToIsoCalendar } from '../utils/converters/date'; type DateTimeValue = { date: string, @@ -25,21 +25,18 @@ function convertDateTime(formValue: DateTimeValue): ?string { const hours = momentTime.hour(); const minutes = momentTime.minute(); - const parsedDate = editedDate ? parseDate(editedDate) : null; - if (!(parsedDate && parsedDate.isValid && parsedDate.momentDate)) return null; + const isoDate = convertDate(editedDate); + if (!isoDate) return null; - const formattedDate = parsedDate.momentDate.format('YYYY-MM-DD'); - const isoDate = convertLocalToIsoCalendar(formattedDate); const momentDateTime = moment(isoDate); + if (!momentDateTime.isValid()) return null; momentDateTime.hour(hours); momentDateTime.minute(minutes); return momentDateTime.toISOString(); } function convertDate(dateValue: string) { - const parsedDate = parseDate(dateValue); - // $FlowFixMe[incompatible-use] automated comment - return parsedDate.isValid ? convertLocalToIsoCalendar(parsedDate.momentDate.toISOString()) : null; + return convertLocalToIsoCalendar(dateValue); } function convertTime(timeValue: string) { 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 2323f01370..408116d011 100644 --- a/src/core_modules/capture-core/utils/converters/date/convertLocalToIsoCalendar.js +++ b/src/core_modules/capture-core/utils/converters/date/convertLocalToIsoCalendar.js @@ -17,16 +17,9 @@ export function convertLocalToIsoCalendar(localDate: ?string): string { return ''; } - const momentDate = moment(localDate); - if (!momentDate.isValid()) { - return ''; - } - - const formattedIsoDate = momentDate.format('YYYY-MM-DD'); - const calendar = systemSettingsStore.get().calendar; - const { year, month, day } = convertToIso8601(formattedIsoDate, calendar); + const { year, month, day } = convertToIso8601(localDate, calendar); const dateString = `${padWithZeros(year, 4)}-${padWithZeros(month, 2)}-${padWithZeros(day, 2)}`; const parsedMoment = moment(dateString); diff --git a/src/core_modules/capture-core/utils/converters/date/convertStringToTemporal.js b/src/core_modules/capture-core/utils/converters/date/convertStringToTemporal.js index 57c0dec668..1c53443cc6 100644 --- a/src/core_modules/capture-core/utils/converters/date/convertStringToTemporal.js +++ b/src/core_modules/capture-core/utils/converters/date/convertStringToTemporal.js @@ -15,11 +15,11 @@ type PlainDate = { day: number }; -export function convertStringToTemporal(dateString: ?string): PlainDate | null { +export function convertStringToTemporal(dateString: ?string, calendar: ?string, format: ?string): PlainDate | null { if (!dateString) { return null; } - const calendar = systemSettingsStore.get().calendar; - const dateFormat = systemSettingsStore.get().dateFormat; - return stringToTemporal(dateString, calendar, dateFormat); + const calendarType = calendar || systemSettingsStore.get().calendar; + const dateFormat = format || systemSettingsStore.get().dateFormat; + return stringToTemporal(dateString, calendarType, dateFormat); } diff --git a/src/core_modules/capture-core/utils/converters/date/convertTemporalToString.js b/src/core_modules/capture-core/utils/converters/date/convertTemporalToString.js index 9ba772b23a..0030ce0d68 100644 --- a/src/core_modules/capture-core/utils/converters/date/convertTemporalToString.js +++ b/src/core_modules/capture-core/utils/converters/date/convertTemporalToString.js @@ -14,10 +14,10 @@ type PlainDate = { day: number }; -export function convertTemporalToString(temporalDate: PlainDate | null): string { +export function convertTemporalToString(temporalDate: PlainDate | null, format: ?string): string { if (!temporalDate) { return ''; } - const dateFormat = systemSettingsStore.get().dateFormat; + const dateFormat = format || systemSettingsStore.get().dateFormat; return temporalToString(temporalDate, dateFormat); } From 98054358fe0936f29471ea47e7ce2aae444f246f Mon Sep 17 00:00:00 2001 From: alaa-yahia Date: Mon, 13 Jan 2025 11:52:52 +0200 Subject: [PATCH 20/25] fix: logic of dateRange & dateTimeRange validators --- .../capture-core-utils/parsers/date.parser.js | 88 ------------------- .../capture-core-utils/parsers/index.js | 1 - .../utils/converters/date/index.js | 1 - .../utils/converters/date/parser.js | 14 --- .../date/stringToMomentDateFormat.js | 2 +- .../validators/form/getDateRangeValidator.js | 8 +- .../form/getDateTimeRangeValidator.js | 38 ++++---- 7 files changed, 25 insertions(+), 127 deletions(-) delete mode 100644 src/core_modules/capture-core-utils/parsers/date.parser.js delete mode 100644 src/core_modules/capture-core/utils/converters/date/parser.js diff --git a/src/core_modules/capture-core-utils/parsers/date.parser.js b/src/core_modules/capture-core-utils/parsers/date.parser.js deleted file mode 100644 index 98bbef55f0..0000000000 --- a/src/core_modules/capture-core-utils/parsers/date.parser.js +++ /dev/null @@ -1,88 +0,0 @@ -// @flow -import moment from 'moment'; - -const getReturnObject = (momentDate: ?moment$Moment) => ({ - momentDate, - isValid: momentDate ? momentDate.isValid() : false, -}); - -function manipulateFormatAndParseWithSeparator(dateString: string, inputFormat: string, separator: string) { - const dateSplitted = dateString.split(separator); - const formatSplitted = inputFormat.split(separator); - if (dateSplitted.length === formatSplitted.length) { - const newLocaleFormat = formatSplitted - .map((formatPart, index) => { - let newFormatPart = ''; - const datePart = dateSplitted[index]; - if (formatPart && datePart) { - const partKey = formatPart.charAt(0); - if (partKey === 'M') { - if (datePart.length === 1) { - newFormatPart = 'M'; - } else if (datePart.length === 2) { - newFormatPart = 'MM'; - } - } else if (partKey === 'D') { - if (datePart.length === 1) { - newFormatPart = 'D'; - } else if (datePart.length === 2) { - newFormatPart = 'DD'; - } - } else if (partKey === 'Y') { - if (datePart.length === 2) { - newFormatPart = 'YY'; - } else if (datePart.length === 4) { - newFormatPart = 'YYYY'; - } - } - } - return newFormatPart || formatPart; - }) - .join(separator); - const momentDate = moment(dateString, newLocaleFormat, true); - return getReturnObject(momentDate); - } - - return getReturnObject(null); -} - -function parseWithSeparator(dateString: string, localeFormat: string, separatorPattern: RegExp) { - const specialCharactersInLocaleFormat = localeFormat.match(separatorPattern); - // $FlowFixMe[incompatible-type] automated comment - const separator: string = specialCharactersInLocaleFormat && specialCharactersInLocaleFormat[0]; - const dateStringWithLocaleSeparator = dateString.replace(separatorPattern, separator); - const localeFormatSameSeparator = localeFormat.replace(separatorPattern, separator); - - const momentDate = moment(dateStringWithLocaleSeparator, localeFormatSameSeparator, true); - if (momentDate.isValid()) { - return getReturnObject(momentDate); - } - - const parseData = manipulateFormatAndParseWithSeparator( - dateStringWithLocaleSeparator, - localeFormatSameSeparator, - separator, - ); - return parseData; -} - -function parseWithoutSeparator(dateString: string, localeFormat: string, separatorPattern: RegExp) { - const dateStringWithoutSeparator = dateString.replace(separatorPattern, ''); - const localeFormatWithoutSeparator = localeFormat.replace(separatorPattern, ''); - - const momentDate = moment(dateStringWithoutSeparator, localeFormatWithoutSeparator, true); - if (momentDate.isValid()) { - return getReturnObject(momentDate); - } - - return getReturnObject(null); -} - -export function parseDate(dateString: string, format: string) { - const separatorPattern = /[.,\-_/\\]/g; - if (separatorPattern.test(dateString) && separatorPattern.test(format)) { - return parseWithSeparator(dateString, format, separatorPattern); - } - - return parseWithoutSeparator(dateString, format, separatorPattern); -} diff --git a/src/core_modules/capture-core-utils/parsers/index.js b/src/core_modules/capture-core-utils/parsers/index.js index e28120d581..46502500df 100644 --- a/src/core_modules/capture-core-utils/parsers/index.js +++ b/src/core_modules/capture-core-utils/parsers/index.js @@ -1,4 +1,3 @@ // @flow -export { parseDate } from './date.parser'; export { parseNumber } from './number.parser'; export { parseTime } from './time.parser'; diff --git a/src/core_modules/capture-core/utils/converters/date/index.js b/src/core_modules/capture-core/utils/converters/date/index.js index 8f878a421d..6c755de6c3 100644 --- a/src/core_modules/capture-core/utils/converters/date/index.js +++ b/src/core_modules/capture-core/utils/converters/date/index.js @@ -1,5 +1,4 @@ // @flow -export { parseDate } from './parser'; export { convertDateObjectToDateFormatString } from './dateObjectToDateFormatString'; export { convertMomentToDateFormatString } from './momentToDateFormatString'; export { convertStringToDateFormat } from './stringToMomentDateFormat'; diff --git a/src/core_modules/capture-core/utils/converters/date/parser.js b/src/core_modules/capture-core/utils/converters/date/parser.js deleted file mode 100644 index a487155ca0..0000000000 --- a/src/core_modules/capture-core/utils/converters/date/parser.js +++ /dev/null @@ -1,14 +0,0 @@ -// @flow -import { parseDate as parseDateCore } from 'capture-core-utils/parsers'; -import { systemSettingsStore } from '../../../metaDataMemoryStores'; - -/** - * Parse a string in date format - * @export - * @param {string} value - the string in date format - * @returns {date} - */ -export function parseDate(value: string) { - const format = systemSettingsStore.get().dateFormat; - return parseDateCore(value, format); -} diff --git a/src/core_modules/capture-core/utils/converters/date/stringToMomentDateFormat.js b/src/core_modules/capture-core/utils/converters/date/stringToMomentDateFormat.js index 29c7b2c929..f75fb68255 100644 --- a/src/core_modules/capture-core/utils/converters/date/stringToMomentDateFormat.js +++ b/src/core_modules/capture-core/utils/converters/date/stringToMomentDateFormat.js @@ -9,7 +9,7 @@ import { systemSettingsStore } from '../../../metaDataMemoryStores'; * @param {string} [format] - optional date format. If not provided, the function uses system date format * @returns {string} */ -export function convertStringToDateFormat(date: string, format?: string) { +export function convertStringToDateFormat(date: ?string, format?: string) { if (!date || !date.length) { return ''; } const dateFormat = format || systemSettingsStore.get().dateFormat; const formattedDateString = moment(date, dateFormat).format(dateFormat); diff --git a/src/core_modules/capture-core/utils/validation/validators/form/getDateRangeValidator.js b/src/core_modules/capture-core/utils/validation/validators/form/getDateRangeValidator.js index c76d738184..f54f241d7d 100644 --- a/src/core_modules/capture-core/utils/validation/validators/form/getDateRangeValidator.js +++ b/src/core_modules/capture-core/utils/validation/validators/form/getDateRangeValidator.js @@ -1,6 +1,7 @@ // @flow +import { Temporal } from '@js-temporal/polyfill'; import { isValidDate } from './dateValidator'; -import { parseDate } from '../../../converters/date'; +import { convertStringToDateFormat } from '../../../converters/date'; /** * * @export @@ -30,6 +31,9 @@ export const getDateRangeValidator = (invalidDateMessage: string) => errorMessage: errorResult.reduce((map, error) => ({ ...map, ...error }), {}), }; } + const { from, to } = value; // $FlowFixMe - return parseDate(value.from).momentDate <= parseDate(value.to).momentDate; + const formattedFrom = convertStringToDateFormat(from, 'YYYY-MM-DD'); + const fromattedTo = convertStringToDateFormat(to, 'YYYY-MM-DD'); + return Temporal.PlainDate.compare(formattedFrom, fromattedTo) <= 0; }; diff --git a/src/core_modules/capture-core/utils/validation/validators/form/getDateTimeRangeValidator.js b/src/core_modules/capture-core/utils/validation/validators/form/getDateTimeRangeValidator.js index 154749b666..f238469b0f 100644 --- a/src/core_modules/capture-core/utils/validation/validators/form/getDateTimeRangeValidator.js +++ b/src/core_modules/capture-core/utils/validation/validators/form/getDateTimeRangeValidator.js @@ -1,34 +1,32 @@ // @flow +import { Temporal } from '@js-temporal/polyfill'; import { isValidDateTime } from './dateTimeValidator'; -import { parseDate } from '../../../converters/date'; function isValidDateTimeWithEmptyCheck(value: ?Object) { return value && isValidDateTime(value); } -const convertDateTimeToMoment = (value: Object) => { - const date = value.date; - const time = value.time; +const convertDateTimeToTemporal = (value: Object) => { + const { date, time } = value; + + const [year, month, day] = date.split('-').map(Number); + let hour; let minutes; if (/[:.]/.test(time)) { - [hour, minutes] = time.split(/[:.]/); + [hour, minutes] = time.split(/[:.]/).map(Number); } else if (time.length === 3) { - hour = time.substring(0, 1); - minutes = time.substring(2, 3); + hour = Number(time.substring(0, 1)); + minutes = Number(time.substring(2, 3)); } else { - hour = time.substring(0, 2); - minutes = time.substring(3, 4); + hour = Number(time.substring(0, 2)); + minutes = Number(time.substring(3, 4)); } - const momentDateTime = parseDate(date).momentDate; - // $FlowFixMe[incompatible-use] automated comment - momentDateTime.hour(hour); - // $FlowFixMe[incompatible-use] automated comment - momentDateTime.minute(minutes); - return momentDateTime; + + return new Temporal.PlainDateTime(year, month, day, hour, minutes); }; export const getDateTimeRangeValidator = (invalidDateTimeMessage: string) => - (value: { from?: ?Object, to?: ?Object}) => { + (value: { from?: ?Object, to?: ?Object }) => { const errorResult = []; if (!isValidDateTimeWithEmptyCheck(value.from)) { errorResult.push({ from: invalidDateTimeMessage }); @@ -46,8 +44,8 @@ export const getDateTimeRangeValidator = (invalidDateTimeMessage: string) => }; } - const fromDateTime = convertDateTimeToMoment(value.from); - const toDateTime = convertDateTimeToMoment(value.to); - // $FlowFixMe[invalid-compare] automated comment - return fromDateTime <= toDateTime; + const fromDateTime = convertDateTimeToTemporal(value.from); + const toDateTime = convertDateTimeToTemporal(value.to); + + return Temporal.PlainDateTime.compare(fromDateTime, toDateTime) <= 0; }; From d67810bedf6a73c62ebe17735231f5be4c60366e Mon Sep 17 00:00:00 2001 From: alaa-yahia Date: Mon, 13 Jan 2025 12:54:15 +0200 Subject: [PATCH 21/25] chore: pass format and calendar as props --- .../dateField/getDateFieldConfigForCustomForm.js | 3 +++ .../dateRangeField/getDateRangeFieldConfig.js | 3 +++ .../getDateTimeFieldConfigForCustomForm.js | 3 +++ .../getDateTimeRangeFieldConfig.js | 3 +++ .../EnrollmentWithFirstStageDataEntry.component.js | 3 +++ .../DataEntry/DataEntry.component.js | 3 +++ .../DataEntry/DataEntry.component.js | 3 +++ .../EditEventDataEntry.component.js | 2 ++ .../ScheduleDate/ScheduleDate.component.js | 3 +++ .../FormComponents/DateFieldForRelatedStages.js | 6 +++++- .../validators/form/getDateTimeRangeValidator.js | 12 ++++++++++-- 11 files changed, 41 insertions(+), 3 deletions(-) diff --git a/src/core_modules/capture-core/components/D2Form/field/configs/dateField/getDateFieldConfigForCustomForm.js b/src/core_modules/capture-core/components/D2Form/field/configs/dateField/getDateFieldConfigForCustomForm.js index efeea1412c..9f5b057d0d 100644 --- a/src/core_modules/capture-core/components/D2Form/field/configs/dateField/getDateFieldConfigForCustomForm.js +++ b/src/core_modules/capture-core/components/D2Form/field/configs/dateField/getDateFieldConfigForCustomForm.js @@ -3,6 +3,7 @@ import moment from 'moment'; import { createFieldConfig, createProps } from '../base/configBaseCustomForm'; import { DateFieldForCustomForm } from '../../Components'; import { convertDateObjectToDateFormatString } from '../../../../../../capture-core/utils/converters/date'; +import { systemSettingsStore } from '../../../../../metaDataMemoryStores'; import type { DateDataElement } from '../../../../../metaData'; import type { QuerySingleResource } from '../../../../../utils/api/api.types'; @@ -12,6 +13,8 @@ export const getDateFieldConfigForCustomForm = (metaData: DateDataElement, optio maxWidth: 350, calendarWidth: 350, calendarMax: !metaData.allowFutureDate ? convertDateObjectToDateFormatString(moment()) : undefined, + calendarType: systemSettingsStore.get().calendar, + dateFormat: systemSettingsStore.get().dateFormat, }, metaData); return createFieldConfig({ diff --git a/src/core_modules/capture-core/components/D2Form/field/configs/dateRangeField/getDateRangeFieldConfig.js b/src/core_modules/capture-core/components/D2Form/field/configs/dateRangeField/getDateRangeFieldConfig.js index 283da64708..90fec23265 100644 --- a/src/core_modules/capture-core/components/D2Form/field/configs/dateRangeField/getDateRangeFieldConfig.js +++ b/src/core_modules/capture-core/components/D2Form/field/configs/dateRangeField/getDateRangeFieldConfig.js @@ -1,6 +1,7 @@ // @flow import { createFieldConfig, createProps } from '../base/configBaseDefaultForm'; import { DateRangeFieldForForm } from '../../Components'; +import { systemSettingsStore } from '../../../../../metaDataMemoryStores'; import type { DataElement as MetaDataElement } from '../../../../../metaData'; import type { QuerySingleResource } from '../../../../../utils/api/api.types'; @@ -14,6 +15,8 @@ export const getDateRangeFieldConfig = (metaData: MetaDataElement, options: Obje maxWidth: options.formHorizontal ? 150 : 350, calendarWidth: options.formHorizontal ? 250 : 350, popupAnchorPosition: getCalendarAnchorPosition(options.formHorizontal), + calendarType: systemSettingsStore.get().calendar, + dateFormat: systemSettingsStore.get().dateFormat, }, options, metaData); return createFieldConfig({ diff --git a/src/core_modules/capture-core/components/D2Form/field/configs/dateTimeField/getDateTimeFieldConfigForCustomForm.js b/src/core_modules/capture-core/components/D2Form/field/configs/dateTimeField/getDateTimeFieldConfigForCustomForm.js index 86024056fc..c061ea81e7 100644 --- a/src/core_modules/capture-core/components/D2Form/field/configs/dateTimeField/getDateTimeFieldConfigForCustomForm.js +++ b/src/core_modules/capture-core/components/D2Form/field/configs/dateTimeField/getDateTimeFieldConfigForCustomForm.js @@ -2,6 +2,7 @@ import { orientations } from '../../../../FormFields/New'; import { createFieldConfig, createProps } from '../base/configBaseCustomForm'; import { DateTimeFieldForCustomForm } from '../../Components'; +import { systemSettingsStore } from '../../../../../metaDataMemoryStores'; import type { DataElement as MetaDataElement } from '../../../../../metaData'; import type { QuerySingleResource } from '../../../../../utils/api/api.types'; @@ -12,6 +13,8 @@ export const getDateTimeFieldConfigForCustomForm = (metaData: MetaDataElement, o calendarWidth: '350px', orientation: orientations.HORIZONTAL, shrinkDisabled: false, + calendarType: systemSettingsStore.get().calendar, + dateFormat: systemSettingsStore.get().dateFormat, }, metaData); return createFieldConfig({ diff --git a/src/core_modules/capture-core/components/D2Form/field/configs/dateTimeRangeField/getDateTimeRangeFieldConfig.js b/src/core_modules/capture-core/components/D2Form/field/configs/dateTimeRangeField/getDateTimeRangeFieldConfig.js index cc6e6c1a1a..b4919fd991 100644 --- a/src/core_modules/capture-core/components/D2Form/field/configs/dateTimeRangeField/getDateTimeRangeFieldConfig.js +++ b/src/core_modules/capture-core/components/D2Form/field/configs/dateTimeRangeField/getDateTimeRangeFieldConfig.js @@ -2,6 +2,7 @@ import { orientations } from '../../../../FormFields/New'; import { createFieldConfig, createProps } from '../base/configBaseDefaultForm'; import { DateTimeRangeFieldForForm } from '../../Components'; +import { systemSettingsStore } from '../../../../../metaDataMemoryStores'; import type { DataElement as MetaDataElement } from '../../../../../metaData'; import type { QuerySingleResource } from '../../../../../utils/api/api.types'; @@ -17,6 +18,8 @@ export const getDateTimeRangeFieldConfig = (metaData: MetaDataElement, options: shrinkDisabled: options.formHorizontal, calendarWidth: options.formHorizontal ? 250 : 350, popupAnchorPosition: getCalendarAnchorPosition(options.formHorizontal), + calendarType: systemSettingsStore.get().calendar, + dateFormat: systemSettingsStore.get().dateFormat, }, options, metaData); return createFieldConfig({ diff --git a/src/core_modules/capture-core/components/DataEntries/Enrollment/EnrollmentWithFirstStageDataEntry/EnrollmentWithFirstStageDataEntry.component.js b/src/core_modules/capture-core/components/DataEntries/Enrollment/EnrollmentWithFirstStageDataEntry/EnrollmentWithFirstStageDataEntry.component.js index cff88329d0..e547c41eaf 100644 --- a/src/core_modules/capture-core/components/DataEntries/Enrollment/EnrollmentWithFirstStageDataEntry/EnrollmentWithFirstStageDataEntry.component.js +++ b/src/core_modules/capture-core/components/DataEntries/Enrollment/EnrollmentWithFirstStageDataEntry/EnrollmentWithFirstStageDataEntry.component.js @@ -25,6 +25,7 @@ import { withCleanUp } from './withCleanUp'; import { getEventDateValidatorContainers } from './fieldValidators/eventDate.validatorContainersGetter'; import { stageMainDataIds } from './getDataEntryPropsToInclude'; import { withTransformPropName } from '../../../../HOC'; +import { systemSettingsStore } from '../../../../metaDataMemoryStores'; const overrideMessagePropNames = { errorMessage: 'validationError', @@ -219,6 +220,8 @@ const getReportDateSettingsFn = () => { required: true, calendarWidth: props.formHorizontal ? 250 : 350, popupAnchorPosition: getCalendarAnchorPosition(props.formHorizontal), + calendarType: systemSettingsStore.get().calendar, + dateFormat: systemSettingsStore.get().dateFormat, }), getPropName: () => stageMainDataIds.OCCURRED_AT, getValidatorContainers: () => getEventDateValidatorContainers(), diff --git a/src/core_modules/capture-core/components/DataEntries/SingleEventRegistrationEntry/DataEntryWrapper/DataEntry/DataEntry.component.js b/src/core_modules/capture-core/components/DataEntries/SingleEventRegistrationEntry/DataEntryWrapper/DataEntry/DataEntry.component.js index 3743fa5dc4..ac5fb4c65d 100644 --- a/src/core_modules/capture-core/components/DataEntries/SingleEventRegistrationEntry/DataEntryWrapper/DataEntry/DataEntry.component.js +++ b/src/core_modules/capture-core/components/DataEntries/SingleEventRegistrationEntry/DataEntryWrapper/DataEntry/DataEntry.component.js @@ -52,6 +52,7 @@ import { attributeOptionsKey, getCategoryOptionsValidatorContainers, withAOCFieldBuilder, withDataEntryFields, } from '../../../../DataEntryDhis2Helpers'; +import { systemSettingsStore } from '../../../../../metaDataMemoryStores'; const getStyles = theme => ({ savingContextContainer: { @@ -162,6 +163,8 @@ const buildReportDateSettingsFn = () => { required: true, calendarWidth: props.formHorizontal ? 250 : 350, popupAnchorPosition: getCalendarAnchorPosition(props.formHorizontal), + calendarType: systemSettingsStore.get().calendar, + dateFormat: systemSettingsStore.get().dateFormat, }), getPropName: () => 'occurredAt', getValidatorContainers: () => getEventDateValidatorContainers(), diff --git a/src/core_modules/capture-core/components/WidgetEnrollmentEventNew/DataEntry/DataEntry.component.js b/src/core_modules/capture-core/components/WidgetEnrollmentEventNew/DataEntry/DataEntry.component.js index aa4520ca1c..3954899320 100644 --- a/src/core_modules/capture-core/components/WidgetEnrollmentEventNew/DataEntry/DataEntry.component.js +++ b/src/core_modules/capture-core/components/WidgetEnrollmentEventNew/DataEntry/DataEntry.component.js @@ -42,6 +42,7 @@ import { attributeOptionsKey, getCategoryOptionsValidatorContainers, } from '../../DataEntryDhis2Helpers'; +import { systemSettingsStore } from '../../../metaDataMemoryStores'; const getStyles = theme => ({ savingContextContainer: { @@ -149,6 +150,8 @@ const buildReportDateSettingsFn = () => { required: true, calendarWidth: props.formHorizontal ? 250 : 350, popupAnchorPosition: getCalendarAnchorPosition(props.formHorizontal), + calendarType: systemSettingsStore.get().calendar, + dateFormat: systemSettingsStore.get().dateFormat, }), getPropName: () => 'occurredAt', getValidatorContainers: () => getEventDateValidatorContainers(), diff --git a/src/core_modules/capture-core/components/WidgetEventEdit/EditEventDataEntry/EditEventDataEntry.component.js b/src/core_modules/capture-core/components/WidgetEventEdit/EditEventDataEntry/EditEventDataEntry.component.js index 96b1e4ff0a..5a112438b1 100644 --- a/src/core_modules/capture-core/components/WidgetEventEdit/EditEventDataEntry/EditEventDataEntry.component.js +++ b/src/core_modules/capture-core/components/WidgetEventEdit/EditEventDataEntry/EditEventDataEntry.component.js @@ -187,6 +187,8 @@ const buildScheduleDateSettingsFn = () => { calendarWidth: 350, label: props.formFoundation.getLabel('scheduledAt'), disabled: true, + calendarType: systemSettingsStore.get().calendar, + dateFormat: systemSettingsStore.get().dateFormat, }), getIsHidden: (props: Object) => props.id !== dataEntryIds.ENROLLMENT_EVENT || props.hideDueDate, getPropName: () => 'scheduledAt', diff --git a/src/core_modules/capture-core/components/WidgetEventSchedule/ScheduleDate/ScheduleDate.component.js b/src/core_modules/capture-core/components/WidgetEventSchedule/ScheduleDate/ScheduleDate.component.js index ca3f806f2d..cd5d2ab5f1 100644 --- a/src/core_modules/capture-core/components/WidgetEventSchedule/ScheduleDate/ScheduleDate.component.js +++ b/src/core_modules/capture-core/components/WidgetEventSchedule/ScheduleDate/ScheduleDate.component.js @@ -4,6 +4,7 @@ import { spacersNum } from '@dhis2/ui'; import withStyles from '@material-ui/core/styles/withStyles'; import { DateField } from 'capture-core/components/FormFields/New'; import { InfoBox } from '../InfoBox'; +import { systemSettingsStore } from '../../../metaDataMemoryStores'; import type { Props } from './scheduleDate.types'; const styles = { @@ -42,6 +43,8 @@ const ScheduleDatePlain = ({ } setScheduleDate(e); }} + calendarType={systemSettingsStore.get().calendar} + dateFormat={systemSettingsStore.get().dateFormat} />
} ); }; diff --git a/src/core_modules/capture-core/utils/validation/validators/form/getDateTimeRangeValidator.js b/src/core_modules/capture-core/utils/validation/validators/form/getDateTimeRangeValidator.js index f238469b0f..45b497a108 100644 --- a/src/core_modules/capture-core/utils/validation/validators/form/getDateTimeRangeValidator.js +++ b/src/core_modules/capture-core/utils/validation/validators/form/getDateTimeRangeValidator.js @@ -1,5 +1,6 @@ // @flow import { Temporal } from '@js-temporal/polyfill'; +import { convertStringToTemporal } from 'capture-core/utils/converters/date'; import { isValidDateTime } from './dateTimeValidator'; function isValidDateTimeWithEmptyCheck(value: ?Object) { @@ -8,7 +9,12 @@ function isValidDateTimeWithEmptyCheck(value: ?Object) { const convertDateTimeToTemporal = (value: Object) => { const { date, time } = value; - const [year, month, day] = date.split('-').map(Number); + const dateInTemporal = convertStringToTemporal(date); + + if (!dateInTemporal) { + return null; + } + const { year, month, day } = dateInTemporal; let hour; let minutes; @@ -46,6 +52,8 @@ export const getDateTimeRangeValidator = (invalidDateTimeMessage: string) => const fromDateTime = convertDateTimeToTemporal(value.from); const toDateTime = convertDateTimeToTemporal(value.to); - + if (!fromDateTime || !toDateTime) { + return false; + } return Temporal.PlainDateTime.compare(fromDateTime, toDateTime) <= 0; }; From 068c3ef7ef988110ed3aff212ae853209aff6c6e Mon Sep 17 00:00:00 2001 From: alaa-yahia Date: Thu, 16 Jan 2025 01:36:54 +0200 Subject: [PATCH 22/25] fix: status label in the StagesAndEvents widget display ISO date --- .../Stages/Stage/StageDetail/hooks/helpers.js | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/core_modules/capture-core/components/WidgetStagesAndEvents/Stages/Stage/StageDetail/hooks/helpers.js b/src/core_modules/capture-core/components/WidgetStagesAndEvents/Stages/Stage/StageDetail/hooks/helpers.js index 9d7f31f63f..eb5c2726b7 100644 --- a/src/core_modules/capture-core/components/WidgetStagesAndEvents/Stages/Stage/StageDetail/hooks/helpers.js +++ b/src/core_modules/capture-core/components/WidgetStagesAndEvents/Stages/Stage/StageDetail/hooks/helpers.js @@ -3,13 +3,14 @@ import React from 'react'; import moment from 'moment'; import i18n from '@dhis2/d2-i18n'; import { statusTypes, translatedStatusTypes } from 'capture-core/events/statusTypes'; -import { convertMomentToDateFormatString } from '../../../../../../utils/converters/date'; +import { convertClientToList } from '../../../../../../converters'; import { getSubValues } from '../../getEventDataWithSubValue'; import type { StageDataElement } from '../../../../types/common.types'; import { Notes } from '../Notes.component'; import type { QuerySingleResource } from '../../../../../../utils/api/api.types'; import { isEventOverdue } from '../../../../../../utils/isEventOverdue'; import { TooltipOrgUnit } from '../../../../../Tooltips/TooltipOrgUnit/TooltipOrgUnit.component'; +import { dataElementTypes } from '../../../../../../metaData'; const getEventStatus = (event: ApiEnrollmentEvent) => { const today = moment().startOf('day'); @@ -37,7 +38,7 @@ const getEventStatus = (event: ApiEnrollmentEvent) => { if (daysUntilDueDate < 14) { return { status: statusTypes.SCHEDULE, options: dueDateFromNow }; } - return { status: statusTypes.SCHEDULE, options: convertMomentToDateFormatString(dueDate) }; + return { status: statusTypes.SCHEDULE, options: convertClientToList(dueDate, dataElementTypes.DATE) }; } return { status: event.status, options: undefined }; }; From cda3dab8b651e6dbf5d69d16e4b8fb60ce593be1 Mon Sep 17 00:00:00 2001 From: alaa-yahia Date: Thu, 16 Jan 2025 02:05:34 +0200 Subject: [PATCH 23/25] fix: currentSearchTerms are not converted correctly --- src/core_modules/capture-core/converters/clientToList.js | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/core_modules/capture-core/converters/clientToList.js b/src/core_modules/capture-core/converters/clientToList.js index f026191a4e..c64c30db9e 100644 --- a/src/core_modules/capture-core/converters/clientToList.js +++ b/src/core_modules/capture-core/converters/clientToList.js @@ -62,7 +62,7 @@ function convertImageForDisplay(clientValue: ImageClientValue) { return ; } -function convertRangeForDisplay(parser: any, clientValue: any) { +function convertRangeForDisplay(parser: any = (value: any) => value, clientValue: any) { return ( {parser(clientValue.from)} {'->'} {parser(clientValue.to)} @@ -106,9 +106,9 @@ const valueConvertersForType = { [dataElementTypes.INTEGER_NEGATIVE_RANGE]: value => convertRangeForDisplay(stringifyNumber, value), [dataElementTypes.PERCENTAGE]: (value: number) => `${stringifyNumber(value)} %`, [dataElementTypes.DATE]: convertDateForListDisplay, - [dataElementTypes.DATE_RANGE]: value => convertRangeForDisplay(convertDateForListDisplay, value), + [dataElementTypes.DATE_RANGE]: value => convertRangeForDisplay(undefined, value), [dataElementTypes.DATETIME]: convertDateTimeForListDisplay, - [dataElementTypes.DATETIME_RANGE]: value => convertRangeForDisplay(convertDateTimeForListDisplay, value), + [dataElementTypes.DATETIME_RANGE]: value => convertRangeForDisplay(undefined, value), [dataElementTypes.TIME]: convertTimeForListDisplay, [dataElementTypes.TIME_RANGE]: value => convertRangeForDisplay(convertTimeForListDisplay, value), [dataElementTypes.TRUE_ONLY]: () => i18n.t('Yes'), From 217dc2509ee7eb5814eff48ce6d46e7ede6b6fa1 Mon Sep 17 00:00:00 2001 From: alaa-yahia Date: Thu, 16 Jan 2025 02:19:37 +0200 Subject: [PATCH 24/25] fix: enrollment and incident date display in iso --- .../components/WidgetEnrollment/Date/Date.component.js | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/core_modules/capture-core/components/WidgetEnrollment/Date/Date.component.js b/src/core_modules/capture-core/components/WidgetEnrollment/Date/Date.component.js index 1b8dc31ee1..5d4f6e83b8 100644 --- a/src/core_modules/capture-core/components/WidgetEnrollment/Date/Date.component.js +++ b/src/core_modules/capture-core/components/WidgetEnrollment/Date/Date.component.js @@ -10,6 +10,7 @@ import { } from '@dhis2/ui'; import i18n from '@dhis2/d2-i18n'; import { withStyles } from '@material-ui/core'; +import { systemSettingsStore } from '../../../metaDataMemoryStores'; import { convertValue as convertValueClientToView } from '../../../converters/clientToView'; import { convertValue as convertValueFormToClient } from '../../../converters/formToClient'; import { convertValue as convertValueClientToServer } from '../../../converters/clientToServer'; @@ -114,6 +115,8 @@ const DateComponentPlain = ({ label={dateLabel} dense locale={locale} + calendarType={systemSettingsStore.get().calendar} + dateFormat={systemSettingsStore.get().dateFormat} />
{validation && validation.error ? i18n.t('Please provide a valid date') : ''} From d05011a73441b58cd3310a659d50fbdb9db9e7b8 Mon Sep 17 00:00:00 2001 From: alaa-yahia Date: Thu, 16 Jan 2025 03:01:51 +0200 Subject: [PATCH 25/25] fix: runtime error when selecting a date in age field in ethiopian calendar --- .../date/temporalToString.js | 4 +++- .../capture-ui/AgeField/AgeField.component.js | 23 +++++++++++-------- 2 files changed, 16 insertions(+), 11 deletions(-) diff --git a/src/core_modules/capture-core-utils/date/temporalToString.js b/src/core_modules/capture-core-utils/date/temporalToString.js index 72f650b149..69caedbe7a 100644 --- a/src/core_modules/capture-core-utils/date/temporalToString.js +++ b/src/core_modules/capture-core-utils/date/temporalToString.js @@ -1,4 +1,5 @@ // @flow +import { systemSettingsStore } from 'capture-core/metaDataMemoryStores'; import { padWithZeros } from './padWithZeros'; /** @@ -20,7 +21,8 @@ export function temporalToString(temporalDate: PlainDate | null, dateFormat: ?st } try { - const year = temporalDate.year; + const calendar = systemSettingsStore.get().calendar; + const year = calendar === 'ethiopian' ? temporalDate.eraYear : temporalDate.year; const month = temporalDate.month; const day = temporalDate.day; diff --git a/src/core_modules/capture-ui/AgeField/AgeField.component.js b/src/core_modules/capture-ui/AgeField/AgeField.component.js index f2b96b19a5..1684c97f8a 100644 --- a/src/core_modules/capture-ui/AgeField/AgeField.component.js +++ b/src/core_modules/capture-ui/AgeField/AgeField.component.js @@ -1,6 +1,10 @@ // @flow import React, { Component } from 'react'; import { Temporal } from '@js-temporal/polyfill'; +import { + convertFromIso8601, + convertToIso8601, +} from '@dhis2/multi-calendar-dates'; import { isValidPositiveInteger } from 'capture-core-utils/validators/form'; import i18n from '@dhis2/d2-i18n'; import classNames from 'classnames'; @@ -11,7 +15,7 @@ import { AgeDateInput } from '../internal/AgeInput/AgeDateInput.component'; import defaultClasses from './ageField.module.css'; import { orientations } from '../constants/orientations.const'; import { withInternalChangeHandler } from '../HOC/withInternalChangeHandler'; -import { stringToTemporal, temporalToString } from '../../capture-core-utils/date'; +import { temporalToString } from '../../capture-core-utils/date'; type AgeValues = { date?: ?string, @@ -56,19 +60,17 @@ type Props = { }; function getCalculatedValues(dateValue: ?string, calendarType: ?string, dateFormat: ?string): AgeValues { - const now = Temporal.Now.plainDateISO().withCalendar(calendarType); + const nowIso = Temporal.Now.plainDateISO(); - const age = stringToTemporal(dateValue, calendarType, dateFormat); + const ageIso = convertToIso8601(dateValue, calendarType); - const diff = now.since(age, { + const diff = nowIso.since(ageIso, { largestUnit: 'years', smallestUnit: 'days', }); - const date = temporalToString(age, dateFormat); - return { - date, + date: dateValue, years: diff.years.toString(), months: diff.months.toString(), days: diff.days.toString(), @@ -118,14 +120,15 @@ class D2AgeFieldPlain extends Component { return; } - const now = Temporal.Now.plainDateISO().withCalendar(calendarType); + const nowIso = Temporal.Now.plainDateISO(); - const calculatedDate = now.subtract({ + const calculatedDateIso = nowIso.subtract({ years: D2AgeFieldPlain.getNumberOrZero(values.years), months: D2AgeFieldPlain.getNumberOrZero(values.months), days: D2AgeFieldPlain.getNumberOrZero(values.days), }); - const dateString = temporalToString(calculatedDate, dateFormat); + const localCalculatedDate = convertFromIso8601(calculatedDateIso.toString(), calendarType); + const dateString = temporalToString(localCalculatedDate, dateFormat); const calculatedValues = getCalculatedValues(dateString, calendarType, dateFormat); this.props.onBlur(calculatedValues); }