@cnamts/synapse 0.0.12-alpha → 0.0.14-alpha

package/src/components/PeriodField/PeriodField.vue

@@ -1,7 +1,9 @@
 <script lang="ts" setup>
-	import { ref, watch, computed } from 'vue'
+	/* eslint-disable @typescript-eslint/no-explicit-any -- Nécessaire pour gérer différents types d'entrée */
+	import { ref, watch, computed, onMounted } from 'vue'
 	import DatePicker from '@/components/DatePicker/DatePicker.vue'
-	import { type RuleOptions, useFieldValidation } from '@/composables'
+	import { useFieldValidation } from '@/composables'
+	import { useValidation, type ValidationRule } from '@/composables/validation/useValidation'
 
 	const { parseDate } = useFieldValidation()
 
@@ -18,13 +20,15 @@
 		required?: boolean
 		displayIcon?: boolean
 		displayAppendIcon?: boolean
-		isDisabled?: boolean
+		disabled?: boolean
 		noIcon?: boolean
 		noCalendar?: boolean
 		isOutlined?: boolean
 		showSuccessMessages?: boolean
-		customRules?: { type: string, options: RuleOptions }[]
-		customWarningRules?: { type: string, options: RuleOptions }[]
+		customRules?: ValidationRule[]
+		customWarningRules?: ValidationRule[]
+		disableErrorHandling?: boolean
+		readOnly?: boolean
 	}>(), {
 		modelValue: () => ({ from: null, to: null }),
 		placeholderFrom: 'Début',
@@ -35,99 +39,45 @@
 		required: false,
 		displayIcon: true,
 		displayAppendIcon: false,
-		isDisabled: false,
+		disabled: false,
 		noIcon: false,
 		noCalendar: false,
 		isOutlined: true,
-		showSuccessMessages: false,
+		showSuccessMessages: true,
 		customRules: () => [],
 		customWarningRules: () => [],
+		disableErrorHandling: false,
+		readOnly: false,
 	})
 
 	const emit = defineEmits(['update:modelValue'])
 
+	// Références aux composants DatePicker
+	const fromDateRef = ref()
+	const toDateRef = ref()
+
+	// Valeurs internes pour les dates
 	const internalFromDate = ref<string | null>(null)
 	const internalToDate = ref<string | null>(null)
 
-	// Règles de validation pour la date de début
-	const fromDateRules = [
-		{
-			type: 'custom',
-			options: {
-				validate: (value: Date | null) => {
-					if (value === null) return true
-					if (tempToDate.value === undefined || tempToDate.value === null) return true
-					return value <= tempToDate.value
-				},
-				message: 'La date de début ne peut pas être supérieure à la date de fin.',
-				successMessage: 'La date de début est valide.',
-				fieldIdentifier: 'fromDateRef',
-			},
-		},
-		...(props.required
-			? [{
-				type: 'required',
-				options: {
-					validate: (value: Date | null) => {
-						// Si les deux champs sont vides, on affiche l'erreur sur les deux
-						if (!value && !tempToDate.value) {
-							return false
-						}
-						// Si l'autre champ est rempli, on force la validation de celui-ci
-						if (!value && tempToDate.value) {
-							return false
-						}
-						return true
-					},
-					message: 'La date de début est requise.',
-					successMessage: 'La date de début est renseignée.',
-					fieldIdentifier: 'fromDateRef',
-				},
-			}]
-			: []),
-		...props.customRules,
-	]
+	// Utiliser le composable de validation
+	const fromDateValidation = useValidation({
+		showSuccessMessages: props.showSuccessMessages,
+		fieldIdentifier: 'fromDate',
+		disableErrorHandling: props.disableErrorHandling,
+	})
 
-	// Règles de validation pour la date de fin
-	const toDateRules = [
-		{
-			type: 'custom',
-			options: {
-				validate: (value: Date | null) => {
-					if (value === null) return true
-					if (tempFromDate.value === undefined || tempFromDate.value === null) return true
-					return value >= tempFromDate.value
-				},
-				message: 'La date de fin ne peut pas être inférieure à la date de début.',
-				successMessage: 'La date de fin est valide.',
-				fieldIdentifier: 'toDate',
-			},
-		},
-		...(props.required
-			? [{
-				type: 'required',
-				options: {
-					validate: (value: Date | null) => {
-						// Si les deux champs sont vides, on affiche l'erreur sur les deux
-						if (!value && !tempFromDate.value) {
-							return false
-						}
-						// Si l'autre champ est rempli, on force la validation de celui-ci
-						if (!value && tempFromDate.value) {
-							return false
-						}
-						return true
-					},
-					message: 'La date de fin est requise.',
-					successMessage: 'La date de fin est renseignée.',
-					fieldIdentifier: 'toDate',
-				},
-			}]
-			: []),
-		...props.customRules,
-	]
+	const toDateValidation = useValidation({
+		showSuccessMessages: props.showSuccessMessages,
+		fieldIdentifier: 'toDate',
+		disableErrorHandling: props.disableErrorHandling,
+	})
 
-	// eslint-disable-next-line @typescript-eslint/no-explicit-any -- This is a generic type
+	/**
+	 * Formate une valeur de date en chaîne de caractères au format spécifié
+	 * @param value - La valeur de date à formater
+	 * @returns La date formatée ou null
+	 */
 	function formatDateValue(value: any): string | null {
 		if (!value) return null
 		if (typeof value === 'string') return value
@@ -145,25 +95,102 @@
 	const formattedFromDate = computed(() => formatDateValue(internalFromDate.value))
 	const formattedToDate = computed(() => formatDateValue(internalToDate.value))
 
-	// Computed properties pour les dates temporaires
-	const tempFromDate = computed(() => formattedFromDate.value ? parseDate(formattedFromDate.value, props.format) : undefined)
-	const tempToDate = computed(() => formattedToDate.value ? parseDate(formattedToDate.value, props.format) : undefined)
-
-	// Sets pour optimiser la recherche des erreurs et succès
-	const fromDateErrorsSet = computed(() => new Set(errors.value.filter(error => error.includes('fromDate'))))
-	const toDateErrorsSet = computed(() => new Set(errors.value.filter(error => error.includes('toDate'))))
-	const fromDateSuccessesSet = computed(() => new Set(successes.value.filter(success => success.includes('fromDate'))))
-	const toDateSuccessesSet = computed(() => new Set(successes.value.filter(success => success.includes('toDate'))))
+	// Computed properties pour les dates parsées
+	const parsedFromDate = computed(() => formattedFromDate.value ? parseDate(formattedFromDate.value, props.format) : null)
+	const parsedToDate = computed(() => formattedToDate.value ? parseDate(formattedToDate.value, props.format) : null)
 
-	const hasFromDateErrors = computed(() => fromDateErrorsSet.value.size > 0)
-	const hasToDateErrors = computed(() => toDateErrorsSet.value.size > 0)
-	const hasFromDateSuccesses = computed(() => fromDateSuccessesSet.value.size > 0)
-	const hasToDateSuccesses = computed(() => toDateSuccessesSet.value.size > 0)
-
-	const errors = ref<string[]>([])
-	const successes = ref<string[]>([])
+	// Règles de validation pour la date de début
+	const fromDateRules = computed<ValidationRule[]>(() =>
+		!props.disableErrorHandling
+			? [
+				{
+					type: 'custom',
+					options: {
+						validate: (value: Date | null) => {
+							if (value === null) return true
+							if (parsedToDate.value === null) return true
+							return value <= parsedToDate.value
+						},
+						message: 'La date de début ne peut pas être supérieure à la date de fin.',
+						successMessage: 'La date de début est valide.',
+						fieldIdentifier: 'fromDate',
+					},
+				},
+				...(props.required && !props.disableErrorHandling
+					? [{
+						type: 'required',
+						options: {
+							validate: (value: Date | null) => {
+								// Si les deux champs sont vides, on affiche l'erreur sur les deux
+								if (!value && !parsedToDate.value) {
+									return false
+								}
+								// Si l'autre champ est rempli, on force la validation de celui-ci
+								if (!value && parsedToDate.value) {
+									return false
+								}
+								return true
+							},
+							message: 'La date de début est requise.',
+							successMessage: 'La date de début est renseignée.',
+							fieldIdentifier: 'fromDate',
+						},
+					}]
+					: []),
+				...(!props.disableErrorHandling ? props.customRules : []),
+			]
+			: [],
+	)
+
+	// Variable réactive pour contrôler l'affichage des messages de succès
+	const showSuccessMessagesActual = computed(() =>
+		props.disableErrorHandling ? false : props.showSuccessMessages,
+	)
 
-	// Computed property pour vérifier si le formulaire est valide
+	// Règles de validation pour la date de fin
+	const toDateRules = computed<ValidationRule[]>(() =>
+		!props.disableErrorHandling
+			? [
+				{
+					type: 'custom',
+					options: {
+						validate: (value: Date | null) => {
+							if (value === null) return true
+							if (parsedFromDate.value === null) return true
+							return value >= parsedFromDate.value
+						},
+						message: 'La date de fin ne peut pas être inférieure à la date de début.',
+						successMessage: 'La date de fin est valide.',
+						fieldIdentifier: 'toDate',
+					},
+				},
+				...(props.required
+					? [{
+						type: 'required',
+						options: {
+							validate: (value: Date | null) => {
+								// Si les deux champs sont vides, on affiche l'erreur sur les deux
+								if (!value && !parsedFromDate.value) {
+									return false
+								}
+								// Si l'autre champ est rempli, on force la validation de celui-ci
+								if (!value && parsedFromDate.value) {
+									return false
+								}
+								return true
+							},
+							message: 'La date de fin est requise.',
+							successMessage: 'La date de fin est renseignée.',
+							fieldIdentifier: 'toDate',
+						},
+					}]
+					: []),
+				...props.customRules,
+			]
+			: [],
+	)
+
+	// Vérification de la validité du formulaire en utilisant les validations
 	const isValid = computed(() => {
 		// Si aucune date n'est renseignée et que ce n'est pas required, c'est valide
 		if (!props.required && !formattedFromDate.value && !formattedToDate.value) {
@@ -184,35 +211,57 @@
 		}
 
 		// Si les deux dates sont renseignées, vérifier qu'elles sont cohérentes
-		if (formattedFromDate.value && formattedToDate.value) {
-			const fromDate = parseDate(formattedFromDate.value, props.format)
-			const toDate = parseDate(formattedToDate.value, props.format)
-			if (!fromDate || !toDate || fromDate > toDate) {
+		if (formattedFromDate.value && formattedToDate.value && parsedFromDate.value && parsedToDate.value) {
+			if (parsedFromDate.value > parsedToDate.value) {
 				return false
 			}
 		}
 
-		// Vérifier qu'il n'y a pas d'erreurs
-		return errors.value.length === 0
+		// Vérifier que les deux validations ne signalent pas d'erreurs
+		return !fromDateValidation.hasError.value && !toDateValidation.hasError.value
 	})
 
-	// Watch pour les changements de la date de début
+	// Synchronisation lorsque l'une des dates change
+	function validateBothDates() {
+		if (fromDateRef.value) {
+			fromDateRef.value.validateOnSubmit()
+		}
+		if (toDateRef.value) {
+			toDateRef.value.validateOnSubmit()
+		}
+	}
+
+	// Validation complète du PeriodField
+	function validateFields() {
+		fromDateValidation.validateField(parsedFromDate.value, fromDateRules.value, props.customWarningRules)
+		toDateValidation.validateField(parsedToDate.value, toDateRules.value, props.customWarningRules)
+	}
+
+	// Gestionnaires d'événements closed
+	function handleFromDateClosed() {
+		validateBothDates()
+	}
+
+	function handleToDateClosed() {
+		validateBothDates()
+	}
+
+	// Watch pour les changements des dates - validation croisée
 	watch(formattedFromDate, () => {
-		// Si la date de fin existe, on revalide
+		validateFields()
 		if (formattedToDate.value && toDateRef.value) {
 			toDateRef.value.validateOnSubmit()
 		}
 	})
 
-	// Watch pour les changements de la date de fin
 	watch(formattedToDate, () => {
-		// Si la date de début existe, on revalide
+		validateFields()
 		if (formattedFromDate.value && fromDateRef.value) {
 			fromDateRef.value.validateOnSubmit()
 		}
 	})
 
-	// Watch pour les changements internes
+	// Watch pour les changements internes - Mise à jour du modèle
 	watch([internalFromDate, internalToDate], () => {
 		emit('update:modelValue', {
 			from: formattedFromDate.value,
@@ -220,7 +269,7 @@
 		})
 	})
 
-	// Watch pour les changements externes avec immediate pour synchroniser l'état initial
+	// Watch pour les changements externes - Synchronisation
 	watch(() => props.modelValue, (newValue) => {
 		if (!newValue) return
 
@@ -233,52 +282,52 @@
 		if (internalToDate.value !== newToDate) {
 			internalToDate.value = newToDate
 		}
+		// Valider les champs après la mise à jour des valeurs
+		validateFields()
 	}, { deep: true, immediate: true })
 
-	// Initialisation
-	internalFromDate.value = formatDateValue(props.modelValue?.from)
-	internalToDate.value = formatDateValue(props.modelValue?.to)
-
-	const fromDateRef = ref()
-	const toDateRef = ref()
-
-	// Gestionnaires d'événements closed
-	const handleFromDateClosed = () => {
-		if (fromDateRef.value) {
-			fromDateRef.value.validateOnSubmit()
-		}
-	}
-
-	const handleToDateClosed = () => {
-		if (toDateRef.value) {
-			toDateRef.value.validateOnSubmit()
-		}
-	}
-
+	// Fonction publique de validation
 	const validateOnSubmit = (): boolean => {
 		// Valider les deux DatePicker
 		const fromDateValid = fromDateRef.value?.validateOnSubmit() ?? true
 		const toDateValid = toDateRef.value?.validateOnSubmit() ?? true
 
-		// Retourner true seulement si tout est valide
-		const result = fromDateValid && toDateValid && isValid.value
+		// Valider avec les règles personnalisées
+		validateFields()
 
-		return result
+		// Retourner true seulement si tout est valide
+		return fromDateValid && toDateValid && isValid.value
 	}
 
+	// Initialisation
+	onMounted(() => {
+		internalFromDate.value = formatDateValue(props.modelValue?.from)
+		internalToDate.value = formatDateValue(props.modelValue?.to)
+		// Validation initiale
+		validateFields()
+	})
+
 	defineExpose({
 		validateOnSubmit,
-		errors,
-		successes,
+		errors: {
+			fromDate: fromDateValidation.errors,
+			toDate: toDateValidation.errors,
+		},
+		successes: {
+			fromDate: fromDateValidation.successes,
+			toDate: toDateValidation.successes,
+		},
+		warnings: {
+			fromDate: fromDateValidation.warnings,
+			toDate: toDateValidation.warnings,
+		},
 		isValid,
 	})
 </script>
 
 <template>
 	<div class="period-field">
-		<div
-			class="period-field__col"
-		>
+		<div class="period-field__col">
 			<DatePicker
 				ref="fromDateRef"
 				v-model="internalFromDate"
@@ -287,22 +336,22 @@
 				:date-format-return="props.dateFormatReturn"
 				:display-append-icon="props.displayAppendIcon"
 				:display-icon="props.displayIcon"
-				:error-message="hasFromDateErrors"
+				:error-message="fromDateValidation.hasError"
 				:format="props.format"
-				:is-disabled="props.isDisabled"
+				:disabled="props.disabled"
 				:is-outlined="props.isOutlined"
 				:no-calendar="props.noCalendar"
 				:no-icon="props.noIcon"
 				:placeholder="props.placeholderFrom"
-				:required="props.required"
+				:required="props.disableErrorHandling ? false : props.required"
 				:show-week-number="props.showWeekNumber"
-				:success-message="hasFromDateSuccesses"
+				:show-success-messages="showSuccessMessagesActual"
+				:success-message="fromDateValidation.hasSuccess"
+				:read-only="props.readOnly"
 				@closed="handleFromDateClosed"
 			/>
 		</div>
-		<div
-			class="period-field__col"
-		>
+		<div class="period-field__col">
 			<DatePicker
 				ref="toDateRef"
 				v-model="internalToDate"
@@ -311,16 +360,18 @@
 				:date-format-return="props.dateFormatReturn"
 				:display-append-icon="props.displayAppendIcon"
 				:display-icon="props.displayIcon"
-				:error-message="hasToDateErrors"
+				:error-message="toDateValidation.hasError"
 				:format="props.format"
-				:is-disabled="props.isDisabled"
+				:disabled="props.disabled"
 				:is-outlined="props.isOutlined"
 				:no-calendar="props.noCalendar"
 				:no-icon="props.noIcon"
 				:placeholder="props.placeholderTo"
-				:required="props.required"
+				:required="props.disableErrorHandling ? false : props.required"
 				:show-week-number="props.showWeekNumber"
-				:success-message="hasToDateSuccesses"
+				:show-success-messages="showSuccessMessagesActual"
+				:success-message="toDateValidation.hasSuccess"
+				:read-only="props.readOnly"
 				@closed="handleToDateClosed"
 			/>
 		</div>

package/src/components/PeriodField/tests/PeriodField.spec.ts

@@ -254,6 +254,152 @@ describe('PeriodField.vue', () => {
 			await wrapper.vm.$nextTick()
 			expect(wrapper.vm.isValid).toBe(true)
 		})
+
+		it('validates correctly the required rule when fields are empty or partially filled', async () => {
+			// Cas 1: Tester la validation quand les deux champs sont vides
+			const wrapper1 = mount(PeriodField, {
+				global: {
+					plugins: [vuetify],
+				},
+				props: {
+					required: true,
+					modelValue: {
+						from: null,
+						to: null,
+					},
+				},
+			})
+
+			await wrapper1.vm.validateOnSubmit()
+			await wrapper1.vm.$nextTick()
+
+			// Vérifier que la fonction de validation renvoie false quand les deux champs sont vides
+			const fromDatePicker1 = wrapper1.findAllComponents({ name: 'DatePicker' })[0]
+			const requiredRule1 = fromDatePicker1.props('customRules').find(rule => rule.type === 'required')
+			expect(requiredRule1.options.validate(null)).toBe(false)
+			expect(wrapper1.vm.isValid).toBe(false)
+
+			// Cas 2: Tester la validation quand un champ est rempli mais l'autre est vide
+			const wrapper2 = mount(PeriodField, {
+				global: {
+					plugins: [vuetify],
+				},
+				props: {
+					required: true,
+					modelValue: {
+						from: null,
+						to: '20/12/2023',
+					},
+				},
+			})
+
+			await wrapper2.vm.validateOnSubmit()
+			await wrapper2.vm.$nextTick()
+
+			// Vérifier que la fonction de validation renvoie false quand from est vide mais to est rempli
+			const fromDatePicker2 = wrapper2.findAllComponents({ name: 'DatePicker' })[0]
+			const requiredRule2 = fromDatePicker2.props('customRules').find(rule => rule.type === 'required')
+			// Simuler le scénario où parsedToDate.value est non-null
+			expect(requiredRule2.options.validate(null)).toBe(false)
+			expect(wrapper2.vm.isValid).toBe(false)
+
+			// Cas 3: Tester que la validation renvoie true quand le champ n'est pas vide
+			const wrapper3 = mount(PeriodField, {
+				global: {
+					plugins: [vuetify],
+				},
+				props: {
+					required: true,
+					modelValue: {
+						from: '15/12/2023',
+						to: '20/12/2023',
+					},
+				},
+			})
+
+			await wrapper3.vm.validateOnSubmit()
+			await wrapper3.vm.$nextTick()
+
+			// Vérifier que la fonction de validation renvoie true quand le champ a une valeur
+			const fromDatePicker3 = wrapper3.findAllComponents({ name: 'DatePicker' })[0]
+			const requiredRule3 = fromDatePicker3.props('customRules').find(rule => rule.type === 'required')
+			const mockDate = new Date('2023-12-15')
+			expect(requiredRule3.options.validate(mockDate)).toBe(true)
+			expect(wrapper3.vm.isValid).toBe(true)
+		})
+
+		it('validates correctly the to-date required rule when fields are empty or partially filled', async () => {
+			// Cas 1: Tester la validation quand les deux champs sont vides
+			const wrapper1 = mount(PeriodField, {
+				global: {
+					plugins: [vuetify],
+				},
+				props: {
+					required: true,
+					modelValue: {
+						from: null,
+						to: null,
+					},
+				},
+			})
+
+			await wrapper1.vm.validateOnSubmit()
+			await wrapper1.vm.$nextTick()
+
+			// Vérifier que la fonction de validation renvoie false quand les deux champs sont vides
+			const toDatePicker1 = wrapper1.findAllComponents({ name: 'DatePicker' })[1]
+			const requiredRule1 = toDatePicker1.props('customRules').find(rule => rule.type === 'required')
+			expect(requiredRule1.options.validate(null)).toBe(false)
+			expect(wrapper1.vm.isValid).toBe(false)
+
+			// Cas 2: Tester la validation quand un champ est rempli mais l'autre est vide
+			const wrapper2 = mount(PeriodField, {
+				global: {
+					plugins: [vuetify],
+				},
+				props: {
+					required: true,
+					modelValue: {
+						from: '15/12/2023',
+						to: null,
+					},
+				},
+			})
+
+			await wrapper2.vm.validateOnSubmit()
+			await wrapper2.vm.$nextTick()
+
+			// Vérifier que la fonction de validation renvoie false quand to est vide mais from est rempli
+			const toDatePicker2 = wrapper2.findAllComponents({ name: 'DatePicker' })[1]
+			const requiredRule2 = toDatePicker2.props('customRules').find(rule => rule.type === 'required')
+			// Simuler le scénario où parsedFromDate.value est non-null
+			expect(requiredRule2.options.validate(null)).toBe(false)
+			expect(wrapper2.vm.isValid).toBe(false)
+
+			// Cas 3: Tester que la validation renvoie true quand le champ n'est pas vide
+			const wrapper3 = mount(PeriodField, {
+				global: {
+					plugins: [vuetify],
+				},
+				props: {
+					required: true,
+					modelValue: {
+						from: '15/12/2023',
+						to: '20/12/2023',
+					},
+				},
+			})
+
+			await wrapper3.vm.validateOnSubmit()
+			await wrapper3.vm.$nextTick()
+
+			// Vérifier que la fonction de validation renvoie true quand le champ a une valeur
+			const toDatePicker3 = wrapper3.findAllComponents({ name: 'DatePicker' })[1]
+			const requiredRule3 = toDatePicker3.props('customRules').find(rule => rule.type === 'required')
+			const mockDate = new Date('2023-12-20')
+			expect(requiredRule3.options.validate(mockDate)).toBe(true)
+			expect(wrapper3.vm.isValid).toBe(true)
+		})
 	})
 
 	describe('Utils', () => {