From 1c3602a938c701bbbf27a93006629d62f40f85e4 Mon Sep 17 00:00:00 2001 From: Orka Arnest CRUZE Date: Tue, 26 Nov 2024 18:00:43 +0100 Subject: [PATCH] =?UTF-8?q?fix(publi-service-metadata):=20correction=20mes?= =?UTF-8?q?sage=20d'erreur=20type=20et=20g=C3=A9n=C3=A9alogie=20de=20resso?= =?UTF-8?q?urce=20#528?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- assets/entrepot/pages/service/common-schemas-validation.ts | 1 + assets/entrepot/pages/service/default-values.ts | 2 +- .../pages/service/metadatas/metadatas-validation-tr.ts | 3 +++ src/Dto/CommonDTO.php | 4 +++- src/Dto/Pyramid/PublishPyramidDTO.php | 4 +++- src/Dto/WfsAddDTO.php | 4 +++- translations/cartesgouvfr.fr.yml | 2 +- translations/validators/validators.fr.yml | 4 ++-- 8 files changed, 17 insertions(+), 7 deletions(-) diff --git a/assets/entrepot/pages/service/common-schemas-validation.ts b/assets/entrepot/pages/service/common-schemas-validation.ts index 6d603ae3..91590d67 100644 --- a/assets/entrepot/pages/service/common-schemas-validation.ts +++ b/assets/entrepot/pages/service/common-schemas-validation.ts @@ -86,6 +86,7 @@ export class CommonSchemasValidation { projection: yup.string().required(tValidMD("metadatas.projection_error")), // encoding: yup.string().required(tValidMD("metadatas.encoding_error")), resolution: yup.string(), + hierarchy_level: yup.string().required(tValidMD("metadatas.hierarchy_level_error")), }) .required(); } diff --git a/assets/entrepot/pages/service/default-values.ts b/assets/entrepot/pages/service/default-values.ts index a8b49d39..2de9dc87 100644 --- a/assets/entrepot/pages/service/default-values.ts +++ b/assets/entrepot/pages/service/default-values.ts @@ -15,7 +15,7 @@ const getMetadataFormDefaultValues = (metadata?: Metadata): MetadataFormValuesTy return { languages: metadata?.csw_metadata?.language ? [metadata?.csw_metadata?.language] : [DEFAULT_LANGUAGE], creation_date: metadata?.csw_metadata?.creation_date, - resource_genealogy: metadata?.csw_metadata?.resource_genealogy, + resource_genealogy: metadata?.csw_metadata?.resource_genealogy ?? "", hierarchy_level: metadata?.csw_metadata?.hierarchy_level ?? MetadataHierarchyLevel.Dataset, email_contact: metadata?.csw_metadata?.contact_email, organization: metadata?.csw_metadata?.organisation_name, diff --git a/assets/entrepot/pages/service/metadatas/metadatas-validation-tr.ts b/assets/entrepot/pages/service/metadatas/metadatas-validation-tr.ts index 68f01771..00481bab 100644 --- a/assets/entrepot/pages/service/metadatas/metadatas-validation-tr.ts +++ b/assets/entrepot/pages/service/metadatas/metadatas-validation-tr.ts @@ -21,6 +21,7 @@ export const { i18n } = declareComponentKeys< | "metadatas.organization_error" | "metadatas.organization_email_required_error" | "metadatas.organization_email_error" + | "metadatas.hierarchy_level_error" | "metadatas.language_error" | "metadatas.charset_error" | "metadatas.encoding_error" @@ -49,6 +50,7 @@ export const ValidationMetadatasFrTranslations: Translations<"fr">["ValidationMe "metadatas.organization_error": "l’organisme est obligatoire", "metadatas.organization_email_required_error": "l’email de l’organisme est obligatoire", "metadatas.organization_email_error": "l’email de l’organisme n'est pas correct", + "metadatas.hierarchy_level_error": "Le type de ressource doit être [dataset] ou [series]", "metadatas.language_error": "La langue est obligatoire", "metadatas.charset_error": "Le jeu de caractères est obligatoire", "metadatas.encoding_error": "l’encodage de la ressource est obligatoire", @@ -75,6 +77,7 @@ export const ValidationMetadatasEnTranslations: Translations<"en">["ValidationMe "metadatas.organization_error": "Organization is required", "metadatas.organization_email_required_error": "Organization email is required", "metadatas.organization_email_error": "Organization email is not correct", + "metadatas.hierarchy_level_error": undefined, "metadatas.language_error": "La langue est obligatoire", "metadatas.charset_error": "Dataset is required", "metadatas.encoding_error": "Encoding of the resource is required", diff --git a/src/Dto/CommonDTO.php b/src/Dto/CommonDTO.php index 68920714..06779301 100644 --- a/src/Dto/CommonDTO.php +++ b/src/Dto/CommonDTO.php @@ -60,8 +60,10 @@ public function __construct( #[Assert\Choice([ 'choices' => ['', 'dataset', 'series'], - 'message' => 'metadatas.resource_genealogy_error', + 'message' => 'metadatas.hierarchy_level_error', ])] + public readonly ?string $hierarchy_level, + public readonly string $resource_genealogy, #[Assert\Choice([ diff --git a/src/Dto/Pyramid/PublishPyramidDTO.php b/src/Dto/Pyramid/PublishPyramidDTO.php index 4db120f9..4cb8e7c8 100644 --- a/src/Dto/Pyramid/PublishPyramidDTO.php +++ b/src/Dto/Pyramid/PublishPyramidDTO.php @@ -72,8 +72,10 @@ public function __construct( #[Assert\Choice([ 'choices' => ['', 'dataset', 'series'], - 'message' => 'common.resource_genealogy_error', + 'message' => 'common.hierarchy_level_error', ])] + public readonly ?string $hierarchy_level, + public readonly ?string $resource_genealogy, #[Assert\Choice([ diff --git a/src/Dto/WfsAddDTO.php b/src/Dto/WfsAddDTO.php index 8b515608..d57911a1 100644 --- a/src/Dto/WfsAddDTO.php +++ b/src/Dto/WfsAddDTO.php @@ -74,8 +74,10 @@ public function __construct( #[Assert\Choice([ 'choices' => ['', 'dataset', 'series'], - 'message' => 'common.resource_genealogy_error', + 'message' => 'common.hierarchy_level_error', ])] + public readonly ?string $hierarchy_level, + public readonly ?string $resource_genealogy, #[Assert\NotBlank(['message' => 'common.frequency_code_error'])] diff --git a/translations/cartesgouvfr.fr.yml b/translations/cartesgouvfr.fr.yml index 94a593e1..a2d3973a 100644 --- a/translations/cartesgouvfr.fr.yml +++ b/translations/cartesgouvfr.fr.yml @@ -413,7 +413,7 @@ metadatas: description_error: "Le résumé est obligatoire" identifier_error: "L’identificateur de resource est obligatoire" identifier_regex: "L’identificateur de resource ne doit contenir que des lettres, chiffres, tirets (-), underscores (_), ou points (.)" - resource_genealogy_error: "La généalogie de la resource doit être [dataset] ou [series]" + hierarchy_level_error: "Le type de ressource doit être [dataset] ou [series]" organization_error: "Le nom de l’organisme est obligatoire" category_error: "La catégorie thématique est obligatoire" category_min_error: "Il doit y avoir au moins une catégorie thématique" diff --git a/translations/validators/validators.fr.yml b/translations/validators/validators.fr.yml index 3f6e4a98..bf430be3 100644 --- a/translations/validators/validators.fr.yml +++ b/translations/validators/validators.fr.yml @@ -31,7 +31,7 @@ pyramid_publish: description_error: "Le résumé est obligatoire" identifier_error: "L'identificateur de resource est obligatoire" identifier_regex: "L'identificateur de resource ne doit contenir que des lettres, chiffres, tirets (-), underscores (_), ou points (.)" - resource_genealogy_error: "La généalogie de la resource doit être [dataset] ou [series]" + hierarchy_level_error: "Le type de ressource doit être [dataset] ou [series]" organization_error: "Le nom de l'organisme est obligatoire" category_min_error: "Il doit y avoir au moins une catégorie thématique" projection_error: "La projection est obligatoire" @@ -82,7 +82,7 @@ common: description_error: "Le résumé est obligatoire" identifier_error: "L'identificateur de resource est obligatoire" identifier_regex: "L'identificateur de resource ne doit contenir que des lettres, chiffres, tirets (-), underscores (_), ou points (.)" - resource_genealogy_error: "La généalogie de la resource doit être [dataset] ou [series]" + hierarchy_level_error: "Le type de ressource doit être [dataset] ou [series]" frequency_code_error: "La fréquence de mise à jour est obligatoire" organization_error: "Le nom de l'organisme est obligatoire" category_error: "La catégorie thématique est obligatoire"