Projet

Général

Profil

Demande #28334

GAR : aaf2gar : prise en charge de la version 1.7 de la grammaire (GAR v3) : attention ! plusieurs mails possibles

Ajouté par arnaud grossir il y a presque 5 ans. Mis à jour il y a plus de 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
Début:
17/05/2019
Echéance:
% réalisé:

100%


Description

Bonjour, en ce qui concerne les mails, la grammaire 1.7 précise qu'il peut y avoir plusieurs mail MaxOccurs=unbounded (pour un enseignant par exemple)

Si cela vous parait pertinent, pourrait-on inclure toutes les adresses mail ? En l'état, vous ne prenez, si je ne me trompe pas, seulement une adresse
par exemple pour le fichier src/econnect-1.2/aaf2gar/xsl/GAR-ENT-Enseignant.xsl
Vous ecrivez:

...
        <xsl:variable name="courriel">
            <xsl:choose>
                <xsl:when test="$ANONYMAT = 'oui'"><xsl:value-of select="concat('noreply@',substring-after(normalize-space(attr[@name='mail']),'@'))"/></xsl:when>
                <xsl:otherwise><xsl:value-of select="normalize-space(attr[@name='mail'])"/></xsl:otherwise>
            </xsl:choose>
        </xsl:variable>
    </xsl:for-each>
...
    <men:GARPersonMail><xsl:value-of select="$courriel"/></men:GARPersonMail>
</men:GAREnseignant>
...

J'avais testé il y a quelques temps cette génération de mails multiples avec ces lignes (beaucoup moins élaborées que votre version puisque je ne prenais pas en compte l'anonymation.. ce n'était qu'un test)

 </xsl:if>
    </xsl:for-each>
    <xsl:for-each select="attr[@name='mail']/value">
        <men:GARPersonMail><xsl:value-of select="."/></men:GARPersonMail>
    </xsl:for-each>
</men:GAREnseignant>

Ce n'est qu'une suggestion, les mails multiples ne sont peut-être pas à prendre en compte...

Cordialement

Révisions associées

Révision effae798 (diff)
Ajouté par Pierre CADEOT il y a presque 5 ans

fixes #28334 prise en charge correcte de la grammaire v1.7 en cas de plusieurs mails enseignants

Historique

#1 Mis à jour par Arnaud FORNEROT il y a presque 5 ans

  • Projet changé de Envole à envole-connecteur

#2 Mis à jour par Pierre CADEOT il y a presque 5 ans

Merci Arnaud pour cette remarque (désolé je n'en prends connaissance qu'aujourd'hui :/ )

Tu as tout à fait raison, je m'occupe d'en tenir compte rapidement.
Pierre

#3 Mis à jour par Pierre CADEOT il y a presque 5 ans

  • Statut changé de Nouveau à Résolu
  • % réalisé changé de 0 à 100

#4 Mis à jour par Arnaud FORNEROT il y a plus de 4 ans

  • Version cible mis à Envole 5.15

#5 Mis à jour par Arnaud FORNEROT il y a plus de 4 ans

  • Statut changé de Résolu à Fermé

Formats disponibles : Atom PDF