Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

parsing des dates à corriger pour les dates seules #13

Open
fphg opened this issue Sep 15, 2016 · 9 comments
Open

parsing des dates à corriger pour les dates seules #13

fphg opened this issue Sep 15, 2016 · 9 comments
Labels

Comments

@fphg
Copy link
Contributor

fphg commented Sep 15, 2016

Après 4093a2a (ou était-ce le cas avant ?)
dans les md les dates seules (cad ddmmaaaa) sont interprétées en null.

Les dates ddmmaaahhmmss sont OK

@fphg fphg added the bug label Sep 15, 2016
@bchartier
Copy link
Collaborator

désolé, si j'ai cassé des trucs pour les dates
de mémoire ma modification avait pour but de pouvoir trier les métadonnées par date.

C'est peut-être cette fonction qui coince avec tes métadonnées :

def parse_string_for_max_date(dates_as_str):

@bchartier
Copy link
Collaborator

@fphg : est-ce que tu as un exemple de fiche de métadonnées sur lequel je pourrais faire un test ?

@fphg
Copy link
Contributor Author

fphg commented Sep 15, 2016

Pas de problème. l'essentiel est d'extraire les bonnes dates. J'en vois 3 : date de mise à jour des données, date de mise à jour de la md, et moins important date de création de la donnée.

Exemple http://geobretagne.fr/geonetwork/apps/georchestra/?uuid=7bb2f770-07d7-4cf6-b56e-f544c73a0d11

Il faudra rajouter un test pour voir si les dates obligatoires INSPIRE sont bien présentes.

@fphg
Copy link
Contributor Author

fphg commented Sep 15, 2016

L'erreur provient du fait que
/gmd:MD_Metadata/gmd:identificationInfo/
gmd:MD_DataIdentification/gmd:citation/gmd:CI_Citation/
gmd:date/gmd:CI_Date/gmd:date/gco:DateTime/text()
retourne souvent null, du moins sur nos md

Parfois nous avons DateTime, parfois Date

@fphg
Copy link
Contributor Author

fphg commented Sep 15, 2016

Correctif rapide dans 0a25c18 on retrouve bien les dates de référence format date ou datetime

@fphg fphg closed this as completed Sep 15, 2016
fphg added a commit that referenced this issue Sep 15, 2016
@bchartier
Copy link
Collaborator

@fphg : t'es sûr que ça marche ?
En lisant rapidement le code, j'ai l'impression que si la fiche mélange les date et les datetime on ne récupère que les dates. Il faudrait fusionner dates_str et datetimes_str avant d'appeler parse_string_for_max_date, non ?

@fphg
Copy link
Contributor Author

fphg commented Sep 15, 2016

Je n'ai eu que des fiches avec soi l'un soit l'autre, et dans tous les cas une seule date (ou datetime). Je n'ai pas de cas de figure avec plusieurs dates. Tu en as eu besoin pour une fiche particulière ?

@fphg fphg reopened this Sep 15, 2016
@bchartier
Copy link
Collaborator

Je n'ai eu que des fiches avec soi l'un soit l'autre, et dans tous les cas une seule date (ou datetime). Je n'ai pas de cas de figure avec plusieurs dates. Tu en as eu besoin pour une fiche particulière ?

J'avoue que je n'ai pas tous les cas de figure en tête dans le catalogue GéoPicardie. C'est vrai que les cas doivent-être rares. En tout cas, je n'ai pas de besoin précis en tête. Je te propose qu'on laisse ouvert pour l'instant et qu'on corrige quand on aura le temps.

@bchartier
Copy link
Collaborator

qu'on corrige

"qu'on perfectionne" serait plus adapté

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants