Référentiels avec multiplicité
Merge request reports
Activity
assigned to @bleny
added 46 commits
-
3c8ee6a5...b2896f19 - 44 commits from branch
to_master
- 1802799c - Merge branch 'to_master' into referentiels_avec_multiplicite
- 892ec624 - Type le champs refValues de l'entité ReferenceValue avec ReferenceDatum
-
3c8ee6a5...b2896f19 - 44 commits from branch
added 1 commit
- 0bd3dbba - Pousse désormais en base avec multiplicité le cas échéant
added 9 commits
- a8b99173 - Ajout du type SQL COMPOSITE_DATE
- 1395f091 - Correction : checker date not required -> pas de transformation; on garde la chaine vide
- 2543da55 - Merge remote-tracking branch 'origin/master' into ajout_type_date
- e8169f8e - Merge remote-tracking branch 'origin/referentiels_avec_multiplicite' into ajout_type_date
- ff7bd7e4 - Correction d'une ligne perdue
- 3ca7eec0 - Renforce le type dans DateValidationCheckResult pour faciliter la...
- 3e991589 - Supprime du code mort
- cafdc771 - Dans les vue relationnelles, intègre aussi les colonnes non-déclarées
- 31b85ca1 - Supprime les tableaux JSON des vues relationnelles et utilise LTREE[]
Toggle commit list@ptcherniati à part l'inconnue signalée dans cette MR, je pense que c'est OK pour les multiplicités "simples".
Peux-tu remplacer les fichiers de tests par ceux en Unicode corrects ? Je ne les ai pas.
J'espère que la documentation est compréhensible pour les utilisateurs.
assigned to @ptcherniati and unassigned @bleny
enabled an automatic merge when the pipeline for a6cc46c4 succeeds
113 120 return result; 114 121 } 115 122 116 public ImmutableMap<String, ApplicationResult.Reference.ReferenceUUIDAndDisplay> getReferenceIdAndDisplayPerKeys(String referenceType, String locale) { 123 public ImmutableMap<Ltree, ApplicationResult.Reference.ReferenceUUIDAndDisplay> getReferenceIdAndDisplayPerKeys(String referenceType, String locale) { 124 Function<ReferenceValue, ApplicationResult.Reference.ReferenceUUIDAndDisplay> referenceValueToReferenceUuidAndDisplayFunction = 125 result -> { 126 ReferenceDatum referenceDatum = result.getRefValues(); 127 ReferenceColumn referenceColumnForDisplay = ReferenceColumn.forDisplay(locale); 128 String display; 129 if (referenceDatum.contains(referenceColumnForDisplay)) { 130 ReferenceColumnValue referenceColumnValueForDisplay = referenceDatum.get(referenceColumnForDisplay); 131 Preconditions.checkState(referenceColumnValueForDisplay instanceof ReferenceColumnSingleValue); 132 display = ((ReferenceColumnSingleValue) referenceColumnValueForDisplay).getValue(); 133 } else { 134 display = null; @ptcherniati je ne suis pas certains que ce cas soit censé arrivé et, si ça doit arrivé, ce qu'il faut faire (même si juste conserver null ou une collection vide peut suffire, il faudrait le confirmer)
@ptcherniati j'ai annulé la fusion automatique, je crois que Gitlab n'avait pas enregistrer ma revue de code, il y a des remarques à traiter listées.
enabled an automatic merge when the pipeline for bd1e6476 succeeds
Je ne vois pas les remarques à traité
- j'ai vérifié la documentation
- tous les fichiers csv sont en UTF-8
- pas vu : @ptcherniati à part l'inconnue signalée dans cette MR, je pense que c'est OK pour les multiplicités "simples". je ne sais pas à quoi cela fait référence.
- J'ai viré pro (il est dans sa propre forge et pour le moment les fichiers sont "pourris" et pro ne contient aucun cas d'usage qui ne soit traité par ailleurs. Je remettrai Pro éventuellement quand ses développements seront plus avancés. MAis on pourra aussi s'appuyer sur des versions stables des tests des différents ORE pour les jouer dans si-ore-v2; et ce quand on aura des évolutions importantes à faire depuis une version release de si-ore-v2.
j'ai vérifié la documentation
Merci
tous les fichiers csv sont en UTF-8
Ils sont faux, ce sont des fichiers ISO mal convertis (par exemple « Estr?es-Mons »), il serait plus sûr de partir des fichiers de prod.
pas vu : @ptcherniati à part l'inconnue signalée dans cette MR je ne sais pas à quoi cela fait référence.
Ici : !42 (comment 83049)
J'ai viré pro
OK