Plugins/Wormhole problems related to non-mapping of messages IDs

[Aeris1One]

Getting way more critical than we were thinking.

As we do not map messages through a database and only check similarities between messages when we edit them, there are false-positives and some problems may arise:

• when deleting a message, all similar messages in all linked channels get deleted (for example if you delete a message "Hello" all other "Hello" messages in the channel will get deleted)
• when editing a message, all similar messages sent by a webhook in all linked channels will get edited

As an example, if someone links a meme channel (were people only posts attachments without text) and you delete a message with only an attachment and an empty text, all similar messages will get deleted.
I haven't tested as of now, but only messages sent by real people within 14 days would be deleted while all messages sent by a webhook would be if there's no restriction on webhook deleting its own messages.

We should consider mapping messages in database. Maybe we could purge mappings after 14 days because they would mostly be useless after this time.

[VForiel]

Ca peut être résolu sans avoir besoin de mapper les messages

Le problème de la suppression est là depuis longtemps et à été évoqué à plusieurs reprises mais ne s'avère pas réellement être un problème puisque il n'y a en pratique aucune raison de supprimer un message particulier sans supprimer des messages qui sont identiques (ou presque)

Cela dit pour l'édition c'est en effet plus embêtant

Une solution possible serait de faire un correspondance fixe là où actuellement le systèle de propagation d'édition/supression utilise une correspondance à 80% du contenu du message

On pourrait également y ajouter une correspondance au niveau des attachements et du nom du webhook

Encore plus simplement, on pourrait retirer la propagation d'un événement d'édition/supression provenant d'un message de webhook

[ascpial]

Tous les messages du salon cible ne seraient pas supprimés, seulement les quelques messages se situant dans le contexte du message qu'il faut modifier (à la même date).

Cependant, il est possible d'ajouter de nouvelles vérifications pour vérifier si les attachments correspondent (ce qui éviterait que des messages sans contenu texte mais avec des images soient supprimés en masse).

Les évènements de modification et de suppression de la part d'un message du webhook ne sont pas censé être propagé actuellement.
Si c'est le cas, c'est un bug.

[ascpial]

Je vais retravailler un peu le plugin de wormhole pour essayer de régler les problèmes évoqués ici.