Bug #3865
ferméRégressions balises html mal gérées dans les descriptions; perte de la sélection du matin/après midi
0%
Description
Pendant mon analyse du problème de date en anglais(#3792), j'ai vu que des régressions se sont introduites par rapport à la version sur laquelle j'avais travaillé initialement:
1) Nicolas Esseyric à fait quelques modifications (https://github.com/Silverpeas/Silverpeas-Components/tree/3391b29bf45ce9f28cfa2e18ca5d029564a01e60) avant a mise en production du composant, il y a un an:
- Dans la classe OptionDateVO, la fonction setPartOfDayFromHour(int hour) n'utilise plus le paramètre hour pour déterminer si le matin, l'après-midi ou les 2 ont été sélectionnés. Du coup, la sélection est perdue pour les cas suivants:
- Aller/retour entre l'étape 3 et 4
- Aller/retour entre l'étape 3 et 2 - Le deuxième effet de bord fait que la page de notification ne grise plus les demi-journées non sélectionnées.
Je ne connais pas la raison pour laquelle Nicolas a fait cette modification peut-être pour activer par défaut les matins et après-midi. Si c'est cela, il serait préférable de modifier les valeurs par défaut dans le constructeur de la classe OptionDateVO.
2) Nicolas a remplacé les méthodes d'affichage de titre (getHtmlParagraphTitle) et de description (getHtmlParagraphDescription) du composant ScheduleEventVO par les taglibs de Silverpeas. Mais il semble qu'il y ai eu des oublis lors du refactoring (cf. l'illustration faite avec dans le texte: (<h1>></h1>)
Cordialement,
Fichiers
Mis à jour par Stéphanie Fariello il y a plus de 11 ans
- Statut changé de New à Qualified
- Votre version de Silverpeas changé de 5.11-SNAPSHOT à 5.11.1
Problèmes reproduit en 5.11.1 :
- perte du décochage des matins ou après-midi dans le cas de retour de l'étape 4 vers l'étape 3 (idem pour le retour de l'étape 3 vers l'étape 2, mais dans ce cas, ce n'est pas forcement en bug du fait que la validation des demi-journées n'est pas faite)
- sur l'étape 4, le choix des demi-journées n'apparait pas (les non sélectionnées ne sont pas grisées)
- le problème des balise < et > est reproduit aussi : elles apparaissent dans le fil d’Ariane mais non dans le titre.
Mis à jour par David Lesimple il y a presque 11 ans
- Statut changé de Qualified à Rejected
- Priorité changé de Normal à Low
Je ne reproduis pas le point 1)
Pour 2) le champ description'est pas prévu pour du contenu avec balise html