Skip to content

Fabien

Yakulu requested to merge fabien/ep_delete_empty_pads:fabien into master

J'ai quelque peu simplifié le code et ait fait le choix d'employer directement Pad et PadManager. C'est sans doute légèrement plus risqué qu'API.js (rapport aux changements internes) mais API.js n'a pas de garantie intrinsèque non plus et apporte une surcharge car prévu pour un usage externe.

Je n'ai pas réintégré la vérification de l'existence du pad car je ne vois pas bien l'usage réel :

  • Si l'utilisateur quitte le pad, il est supprimé.
  • En cas de nouvelle installation ou de plantage du serveur, tous les pads sont vérifiés pour être supprimés si non utilisés.

Mais le cas où plusieurs utilisateurs sont sur un pad vide et que l'un d'entre eux quitte le pad et entraine de fait la suppression n'a pas posé de soucis pour moi : etherpad recrée le pad à la prochaine révision.

PS : le plugin ne fonctionne pas correctement avec la base dirty, mais ceci est dû au fonctionnement de cette base et à ueberDb et non au plugin en lui-même.

Merge request reports