Skip to content
Snippets Groups Projects

Référentiels avec multiplicité

Merged Brendan Le Ny requested to merge referentiels_avec_multiplicite into master

Merge request reports

Pipeline #51714 passed

Pipeline passed for bd1e6476 on referentiels_avec_multiplicite

Merged by Philippe TcherniatinskyPhilippe Tcherniatinsky 3 years ago (Mar 2, 2022 8:16am UTC)

Loading

Pipeline #51744 passed

Pipeline passed for 2a245e08 on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
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;
  • Author Maintainer

    @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)

  • Please register or sign in to reply
  • Brendan Le Ny canceled the automatic merge

    canceled the automatic merge

  • Author Maintainer

    @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.

  • added 1 commit

    • bd1e6476 - Suppression de code Pro restant

    Compare with previous version

  • Philippe Tcherniatinsky enabled an automatic merge when the pipeline for bd1e6476 succeeds

    enabled an automatic merge when the pipeline for bd1e6476 succeeds

  • Philippe Tcherniatinsky canceled the automatic merge

    canceled the automatic merge

    • 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.
    • Author Maintainer

      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

    • Pas trouvé tous les accents sont corrects dans site donc ailleurs aussi.

    • Please register or sign in to reply
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading