Les balises secret ne sont plus dupliquées lors de la prévisualisation

[Situphen]

Q	R
Correction de bugs ?	oui
Nouvelle Fonctionnalité ?	non
Tickets (issues) concernés	#3070

Les balises secret ne sont plus dupliquées lors de la prévisualisation

QA :

• Générer le front avec npm run gulp -- build
• Puis, pour chaque test, vérifier que le nombre de lien correspond au nombre de [[secret]]
  • lors de la prévisualisation du message (cliquer sur "Aperçu")
  • lors de l'affichage du message (cliquer sur "Envoyer")

Tests :

Test avec 3 balises secret imbriquées

[[secret]]
| Test 1
| [[secret]]
| | Test 2
| | [[secret]]
| | | Test 3

Test avec 2 balises secret séparées

[[secret]]
| Test 1

[[secret]]
| Test 2

Test avec 1 seule balise secret

[[secret]]
| Test

Test sans balise secret

[francoisthire]

Afin d'être cohérent, il ne faudrait pas écrire $this.next au lieu de $(this).next ?

[Situphen]

Ce n'est pas le même this donc on ne peut pas (j'ai déjà essayé) ! ;)

[francoisthire]

Oui, je viens de voir. Mais du coup ce n'est pas vraiment cohérent.

[Situphen]

Ce n'est pas forcément harmonieux, mais c'est logique (on est dans une autre fonction).

[Axylium]

Juste pour préciser :

Test avec plusieurs balises secret

[[secret]]
| Test 1
| [[secret]]
| | Test 2
| | [[secret]]
| | | Test 3

En fait, même si on les imbrique pas, le bug est présent. Si tu fais :

[[secret]]
| ton texte

[[secret]]
| ton texte

À l'aperçu, il y aura 3 balises qui vont apparaître ! Je voulais juste le préciser, car tu as l'air d'être partis sur le fait qu'il faut qu'elles soient imbriquées entres elle, donc je ne sais pas si ta PR s'occupe seulement de ce cas là (je n'y ai connais rien en JS donc bon..) ! :)

[Situphen]

@Florianboux : Ah, j'avais oublié ce cas-là ! Ma PR le corrige bien (et je l'ai rajouté dans le premier post).

[pierre-24]

Rapport de QA : c'est OK =)