C'est fait ! Plusieurs années après sa première annonce, la voici enfin qui débarque.
Vendredi 28 février 2014 à 22:02
Par Lexpageus, demeure !
—
pom
Discussions
Lexpage v4 - Commentaires, remarques, bugs...
Cette discussion est en relation avec le billet suivant :
Reprise automatique du message précédent.
Pour mémo personnel.
Lors d'une recherche sur le site avec une chaîne contenant un espace (à priori autorisé autour/entre les opérateurs), cela provoque une erreur 500.
Allez, sale fainéant de webmaster, corrige ça vite !
Lors d'une recherche sur le site avec une chaîne contenant un espace (à priori autorisé autour/entre les opérateurs), cela provoque une erreur 500.
Allez, sale fainéant de webmaster, corrige ça vite !
Je n'aime pas l'idée de faire ça sur le forum, pour une question de philosophie du forum (qui est avant tout public, et où tout le monde est invité à participer, je ne veux pas qu'on "cible" les personnes). Ca risque de plus de "flooder" en notifications (alors que le minichat est plutôt calme ! Personnellement, je ne voudrai pas recevoir ces notifications sachant que je lis de toute façon tout le forum, par ex.). Par ailleurs, ça pose des questions techniques, notamment sur le fait de ne pas tenir compte des @machin dans les quote, etc.
Il me semble qu'on en avait parlé dans la v3 quand on discutait des features de la v4, et ça n'avait pas été une proposition retenue, mais je n'en suis plus trop sûr
Il me semble qu'on en avait parlé dans la v3 quand on discutait des features de la v4, et ça n'avait pas été une proposition retenue, mais je n'en suis plus trop sûr
Juste une chasse fixe, puis basta. J'avais regardé pour intégrer le support du langage, mais je ne sais plus pour quelle raison je ne l'ai pas fait (technique, à cause d'OVH, fainéantise, ...). Vu qu'on est passé sur VPS (et la liberté qui va avec), je peux toujours me renseigner à nouveau, mais ce n'est pas (du tout) prioritaire
Voilà, le minichat supporte maintenant les @quelquechose. Comme dit dans un message précédent :
- Je suis très large avec le @machin, j'autorise des caractères avant et après, donc ça va matcher aussi les adresses e-mail. Quoiqu'il en soit, un filtre est appliqué après pour ne restreindre qu'aux utilisateurs actifs, donc ça ne sera pas du tout problématique (mais en gros, vous pouvez faire @truc@machin>bidule ça marchera)
- Une fois la liste des destinataires "calculées", une notification est envoyée à chaque destinataire (une seule par message). La notification informe qu'il y a un message qui attend sur le minichat, avec le nom de l'auteur, et le lien de la notification ouvre la page des archives du minichat avec une ancre sur l'heure du message (j'ai pas encore testé
).
- Pour l'auteur du message, une "notification" (mais sous la forme d'un petit cadre vert en haut de l'écran de la page suivante) indiquera les utilisateurs qui ont été prévenus (permettant de vérifier qu'aucune erreur n'a été faite dans les @machin si nécessaire). A noter que ça ne s'affiche donc que sur la page suivante (que ça soit dans la minute qui suit ou 10 jours après). Je n'ai pas encore regardé pour que ce soit le minichat qui signale ça "directement" après le post.
- L'autocompletion est fonctionnelle. J'ai du pas mal chipoter pour réutiliser la "liste des utilisateurs" déjà proposées en completion à certains endroits du site, pour que ça matche facilement les @machin. Y a un "bug" connu : la mise en cache des résultats... Si vous faites "@guy" la première fois, il va proposer dans la liste "Guybrush". Si vous refaites "@guy" sur la même page (sans refresh donc), il va proposer "@Guybrush". Dans les deux cas, c'est bien "@Guybrush" qui sera ajouté. C'est donc juste un retour visuel un peu particulier (que je vais essayer de corriger dans les 30 min à venir)
[Edité : voilà, c'est bon Normalement, ça aurait pu être très simple, si le script Autocomplete (de Devbridge) n'était pas aussi mal documenté (et ne comportait pas un bug sur sa gestion du délimiteur, où il fait un "trim" après sur le résultat, ce qui fait qu'un autre délimiteur que l'espace rend les choses un peu problématique... J'ai donc contourné un peu tout ça ^^]
- Je suis très large avec le @machin, j'autorise des caractères avant et après, donc ça va matcher aussi les adresses e-mail. Quoiqu'il en soit, un filtre est appliqué après pour ne restreindre qu'aux utilisateurs actifs, donc ça ne sera pas du tout problématique (mais en gros, vous pouvez faire @truc@machin>bidule ça marchera
)- Une fois la liste des destinataires "calculées", une notification est envoyée à chaque destinataire (une seule par message). La notification informe qu'il y a un message qui attend sur le minichat, avec le nom de l'auteur, et le lien de la notification ouvre la page des archives du minichat avec une ancre sur l'heure du message (j'ai pas encore testé
). - Pour l'auteur du message, une "notification" (mais sous la forme d'un petit cadre vert en haut de l'écran de la page suivante) indiquera les utilisateurs qui ont été prévenus (permettant de vérifier qu'aucune erreur n'a été faite dans les @machin si nécessaire). A noter que ça ne s'affiche donc que sur la page suivante (que ça soit dans la minute qui suit ou 10 jours après). Je n'ai pas encore regardé pour que ce soit le minichat qui signale ça "directement" après le post.
- L'autocompletion est fonctionnelle. J'ai du pas mal chipoter pour réutiliser la "liste des utilisateurs" déjà proposées en completion à certains endroits du site, pour que ça matche facilement les @machin. Y a un "bug" connu : la mise en cache des résultats... Si vous faites "@guy" la première fois, il va proposer dans la liste "Guybrush". Si vous refaites "@guy" sur la même page (sans refresh donc), il va proposer "@Guybrush". Dans les deux cas, c'est bien "@Guybrush" qui sera ajouté. C'est donc juste un retour visuel un peu particulier (que je vais essayer de corriger dans les 30 min à venir
)[Edité : voilà, c'est bon
Normalement, ça aurait pu être très simple, si le script Autocomplete (de Devbridge) n'était pas aussi mal documenté (et ne comportait pas un bug sur sa gestion du délimiteur, où il fait un "trim" après sur le résultat, ce qui fait qu'un autre délimiteur que l'espace rend les choses un peu problématique... J'ai donc contourné un peu tout ça ^^]
Ce message a été modifié 1 fois.
Dernière modification : 11 octobre 2014
à 14:07 par
Guybrush.
Je pense que je vais inclure le texte du message "notifié" dans la notification, afin de ne pas avoir à aller lire le minichat si le texte semble "auto-suffisant". Enfin, bref, je vais faire ça, quoi
Quelques modifications rapides :
- L'icône de notification liée au minichat devrait s'afficher correctement (j'ai mis font-awesome à jour, mais je n'avais pas forcé le refresh sur vos petits PC
)
- Le texte de notification du message reprend maintenant le message (avec les smiley, les liens, etc.). J'aurai du le faire plus tôt, j'aurai pu le faire plus tôt, mais j'ai eu un gros doute quant au coté "safe" (par ex, un message contenant du code HTML) vu que la notification ne fait aucun traitement (en réalité, le seul traitement qu'elle fait, c'est justement d'indiquer que le texte est supposé être safe !). Il faut que je regarde attentivement la doc de Django, parce qu'il semble que les données issues de formulaire soient d'office "unsafe" dans la base de données (et donc automatiquement échappées), ce qui m'arrange si c'est le cas (et ça correspond à ce que je vois en pratique), mais j'aimerai en être sûr
- Le nombre minimum de caractères pour débuter l'auto-complétion a été porté à 3 (cela inclut le "@") au lieu de 2. Par cohérence avec la recherche de nom de lexpagiens quand on utilise la messagerie interne, et pour limiter les trop nombreuses requêtes. J'ai renforcé ça aussi au niveau du code, pour qu'une erreur soit générée s'il n'y a pas assez de caractères (pour éviter qu'on ne tente de mettre à genoux le serveur par ce biais)
- L'icône de notification liée au minichat devrait s'afficher correctement (j'ai mis font-awesome à jour, mais je n'avais pas forcé le refresh sur vos petits PC
)- Le texte de notification du message reprend maintenant le message (avec les smiley, les liens, etc.). J'aurai du le faire plus tôt, j'aurai pu le faire plus tôt, mais j'ai eu un gros doute quant au coté "safe" (par ex, un message contenant du code HTML) vu que la notification ne fait aucun traitement (en réalité, le seul traitement qu'elle fait, c'est justement d'indiquer que le texte est supposé être safe !). Il faut que je regarde attentivement la doc de Django, parce qu'il semble que les données issues de formulaire soient d'office "unsafe" dans la base de données (et donc automatiquement échappées), ce qui m'arrange si c'est le cas (et ça correspond à ce que je vois en pratique), mais j'aimerai en être sûr
- Le nombre minimum de caractères pour débuter l'auto-complétion a été porté à 3 (cela inclut le "@") au lieu de 2. Par cohérence avec la recherche de nom de lexpagiens quand on utilise la messagerie interne, et pour limiter les trop nombreuses requêtes. J'ai renforcé ça aussi au niveau du code, pour qu'une erreur soit générée s'il n'y a pas assez de caractères (pour éviter qu'on ne tente de mettre à genoux le serveur par ce biais)