Re: [Spip-zone-commit] r120910 - in _plugins_/mailsubscribers/trunk

Dans les faits la notion de « liste de diffusion » est une notion informatique qui parle certes aux webmestre et aux geeks, mais pas au grand public.
Le terme de « newsletter » est celui utilisé conventionnellement sur tout le web et accepté comme tel.
Le wording du plugin avait fait l’objet de discussions collective et c’est de là qu’il avait été choisi de ne pas faire apparaitre le terme de « liste de diffusion » sur la partie publique du site puisque ça ne parle pas au grand public, au contraire de l’espace privé qui s’adresse au webmestre et qui lui peut comprendre la différence.

Concernant le supposé bug, il y a peut-être une subtilité liée à la version PHP, je vais vérifier, mais en effet on a une chaine vide, qui devrait produire un tableau vide, qui fait qu’on ne passe pas dans la selection sql, et donc on envoie aucune liste à l’appel à « unsubscribe » ce qui veut bien dire se désinscrire de toutes les listes.
Je vais retester, et il manque peut-être juste un array_filter le cas échéant

Concernant la divulgation des emails aux redacteurs, en effet ça m’était sorti de la tête, je vais regarder ça dès que possible.

--
Cédric
Le 3 févr. 2020 à 11:17 +0100, Patrice Vanneufville <[email protected]>, a écrit :
> Bonjour Cédric,
>
> Merci pour ton retour.
>
> 1/ Newsletter étant un mot anglais qui veut dire lettre d'info, ça n'a pas beaucoup de sens de le garder ainsi.
>
> Dans les faits, les inscrits se désinscrivent bien d'une liste, et non d'une lettre d'info qui peut très bien être envoyée à plusieurs listes.
>
> Le constat est flagrant lorsque la désinscription concerne plusieurs listes, voire toutes les listes.
>
> Il s'agit donc de mettre en adéquation la partie privée et la partie publique, tout en restant clair avec tout le monde, sans anglicisme inutile.
>
> Enfin, les paramètres des chaînes de langue, notamment des mails envoyés par le serveur, sont bien des listes et non des newsletters. La lecture du mail de confirmation reçu est franchement bizarre, surtout si le webmaster appelle sa liste genre "Liste des anciens adhérents à Toto".
>
> Au passage, ce serait utile d'expliquer que le titre public d'une liste de diffusion est un titre utilisé à l'attention des abonnés, notamment dans les mails de notification d'inscription ou de désinscription.
>
> 2/ Après tests, il s'avère que la désinscriptions à "toutes" les listes ne fonctionnait pas puisque l'explode sur une chaîne vide renvoyait un array avec une clé [0] vide. Rien n'était donc sorti de la base et aucune info n'était fournie au "vain futur ex-inscrit" .
>
> Le résultat du clic sur le bouton "se retirer de TOUTES les listes" ne donne en retour qu'un message inapproprié genre : "vous êtes bien désinscrit de cette (?) newsletter".
>
> As-tu testé ma solution, ou les tests dont tu parles datent de l'époque de la genèse du code ?
>
> 3/ j'avais soulevé un souci de sécurité il y a un bon moment à propos de données divulguées sans autorisation, mais sans réelle réponse de ta part. Le comportement inapproprié du plugin est toujours d'actualité.
>
> Très bonne journée à toi également.
>
> Pat
>
>
> > Le 3 février 2020 10:11:08 GMT+01:00, Cerdic <[email protected]> a écrit :
> > > Bonjour Patrice,
> > >
> > > je vais revert ce commit car :
> > >
> > > 1/ tu changes unilateralement le wording de toutes les chaines de langue en remplaçant « newsletter » par « liste de diffusion » sans aucune discussion, présumant qu’on a fait n’importe quoi sans réflechir
> > >
> > > 2/ peux-tu décrire dans quel scénario la désinscription de « TOUTES » les newsletters ne marche pas ? car moi j’ai testé et vérifié tous les cas et ça marchait bien jusqu’à preuve du contraire
> > >
> > > Bonne journée à toi,
> > >
> > > --
> > > Cédric
> > > Le 3 févr. 2020 à 03:39 +0100, [email protected], a écrit :
> > > > Author: [email protected]
> > > > Date: 2020-02-03 02:39:03 +0000 (Mon, 03 Feb 2020)
> > > > New Revision: 120910
> > > >
> > > > Modified:
> > > > _plugins_/mailsubscribers/trunk/action/confirm_unsubscribe_mailsubscriber.php
> > > > _plugins_/mailsubscribers/trunk/action/unsubscribe_mailsubscriber.php
> > > > _plugins_/mailsubscribers/trunk/lang/mailsubscriber_fr.php
> > > > _plugins_/mailsubscribers/trunk/lang/newsletter_fr.php
> > > > _plugins_/mailsubscribers/trunk/paquet.xml
> > > > Log:
> > > > - Precisions de langue
> > > > - Bouton "me desinscrire de TOUTES les listes" desormais fonctionnel
> > > >
> > > >
> > > > Details: https://zone.spip.net/trac/spip-zone/changeset/120910
> > > >
> > > > _______________________________________________
> > > > [email protected] - https://listes.rezo.net/mailman/listinfo/spip-zone-commit
>
> --
> Envoyé de mon téléphone Android avec K-9 Mail. Excusez la brièveté.