📄 Composant Copier la publication

[keegan (Keegan George)]

Merci d’avoir identifié cela @Moin. J’ai appliqué un correctif :

FIX: Unable to set allowed groups to everyone group

[Shauny (Shaun Robinson)]

Je reçois ce message d’erreur chaque fois que je clique sur le bouton Copier :

[Arkshine]

Voyez-vous une erreur dans la console du navigateur ?

[Don]

J’ai mentionné ce problème sur l’autre sujet.

Full quote button in editor missing

I think the issue is that on iOS the copy only works if it comes from user interaction… It fails when it try to copy the quote. I will look for a solution if there are. There is the same issue in Copy Post Component.

Full quote button in editor missing

I tried a lots of way to fix this on iOS but it always fails on copy to clipboard because that wasn’t a real user interaction. So I’ve separated the actions and switches buttons between a prepare quote (which is fetching the quote) and copy quote (which is copy the quote to clipboard). This works fine. I also added a tooltip when the copy button appears.

[Arkshine]

J’ai remarqué une erreur sur un site qui utilise le composant de manière anonyme :

J’ai créé une PR pour corriger cela :

FIX: Restore anonymous access when `allowed_groups` is set to `everyone` group

[Arkshine]

La RP est maintenant fusionnée ; merci, Keegan !

[Ethsim2 (Ethan)]

les utilisateurs de ce composant reçoivent-ils cette erreur plus récemment

[Johnny_McIvor]

Je viens de l’installer, voici une idée :

Cliquez sur le bouton de copie de plusieurs messages pour « construire » votre presse-papiers. De cette façon, vous pouvez copier rapidement une section entière d’un fil de discussion de manière sélective.

Disons qu’il y a cinq messages :

Message A
Message B
Message C
Message D
Message E

Vous copiez D, puis B, puis E. Ce qui se trouve dans votre presse-papiers est en fait :

B
D
E

Ainsi, le presse-papiers est organisé en fonction de leur chronologie dans la conversation, et non de l’ordre dans lequel vous les copiez.

[Ethsim2 (Ethan)]

J’ai eu une erreur sur ce point, et comme c’est une erreur JS côté client, je ne peux pas la trouver dans \logs. Ma version de Discourse est 2026.7.0-latest +188

[Ethsim2 (Ethan)]

J’ai approfondi l’analyse en me basant sur la révision exacte de Discourse au moment où j’ai initialement constaté le problème, ainsi que sur la branche main actuelle.

Sur une configuration historique de juillet reconstituée, j’ai pu reproduire un échec lié au presse-papiers : la fonction « Copier le message » pouvait afficher l’indication de succès (coche) alors que le contenu du presse-papiers n’était pas réellement remplacé. Je ne peux pas confirmer qu’il s’agissait de la même erreur sous-jacente que l’erreur de thème/composant visible dans ma capture d’écran d’origine de juillet.

Cependant, la même implémentation de « Copier le message » fonctionne désormais sur mon instance de production actuelle, y compris dans une PWA sur Safari iOS.

J’ai comparé le code pertinent entre la révision de Discourse de juillet et la branche main actuelle. L’implémentation du presse-papiers de « Copier le message » elle-même n’a pas changé, et je n’ai pas trouvé de modification pertinente dans :

• clipboardCopy / clipboardCopyAsync
• la distribution des actions de DButton, y compris le parcours iOS
• le shim legacy de DButton
• le parcours du transformateur du menu de message
• le parcours GET dans discourse/lib/ajax

J’ai également écrit une spécification système expérimentale qui met en pause la requête du message brut et vérifie si navigator.clipboard.write() a démarré avant la fin de la requête. Ce test réussit si « Copier le message » est modifié pour utiliser clipboardCopyAsync, et échoue avec l’implémentation actuelle.

Cependant, « Copier le message » fonctionne actuellement dans les navigateurs/appareils que j’ai testés, y compris Safari iOS PWA, de sorte que ce test semble imposer une contrainte d’implémentation plus stricte plutôt que de démontrer une régression visible par l’utilisateur à l’heure actuelle.

Pour cette raison, je ne propose pas actuellement ce test ni la modification vers clipboardCopyAsync comme correctif. L’échec historique a peut-être dépendu du comportement du navigateur/WebKit plutôt que d’une modification du composant « Copier le message » lui-même.

[Ethsim2 (Ethan)]

Pour faire suite à cela, j’ai ouvert une petite PR qui modifie l’indication de succès/vérification afin qu’elle ne s’affiche qu’après que l’écriture dans le presse-papiers a réussi, avec une spécification de système de régression :

FIX: Show copy success after clipboard write completes (#39)

Le workflow GitHub Actions est actuellement en attente d’approbation par le mainteneur avant que les tâches CI puissent s’exécuter.