# HG changeset patch
# User Goffi <goffi@goffi.org>
# Date 1562408545 -7200
# Node ID b5f8cb26ef6fb957089d21f864ddd6057136ac11
# Parent  8990ed9aad311b0dfe5e0176e7fee0964e59fd07
quick frontend (chat), primitivus(chat): properly clear occupants on resync:

occupants widgets were not cleared in Primitivus on resync while occupants dictionary was, resulting in nicknames sometimes apearing doubled in Primitivus MUC rooms.

A new occupantsClear method has been added in QuickChat, so it can be overriden by frontends to clear widgets or other representations of occupants.

fix 304

diff -r 8990ed9aad31 -r b5f8cb26ef6f sat_frontends/primitivus/chat.py
--- a/sat_frontends/primitivus/chat.py	Fri Jul 05 15:58:15 2019 +0200
+++ b/sat_frontends/primitivus/chat.py	Sat Jul 06 12:22:25 2019 +0200
@@ -245,6 +245,9 @@
             occupant_data = self.parent.occupants[occupant]
             self.occupants_walker.append(OccupantWidget(occupant_data))
 
+    def clear(self):
+        del self.occupants_walker[:]
+
     def updateFooter(self):
         """update footer widget"""
         txt = OCCUPANTS_FOOTER.format(len(self.parent.occupants))
@@ -498,6 +501,10 @@
         if occupant is not None:
             self.occupants_widget.removeUser(occupant)
 
+    def occupantsClear(self):
+        super(Chat, self).occupantsClear()
+        self.occupants_widget.clear()
+
     def _occupantsClicked(self, occupant, clicked_wid):
         assert self.type == C.CHAT_GROUP
         contact_list = self.host.contact_lists[self.profile]
diff -r 8990ed9aad31 -r b5f8cb26ef6f sat_frontends/quick_frontend/quick_chat.py
--- a/sat_frontends/quick_frontend/quick_chat.py	Fri Jul 05 15:58:15 2019 +0200
+++ b/sat_frontends/quick_frontend/quick_chat.py	Sat Jul 06 12:22:25 2019 +0200
@@ -393,6 +393,13 @@
         self.sync = True
         self._resync_lock = False
 
+    def occupantsClear(self):
+        """Remove all occupants
+
+        Must be overridden by frontends to clear their own representations of occupants
+        """
+        self.occupants.clear()
+
     def resync(self):
         if self._resync_lock:
             return
@@ -408,7 +415,7 @@
             self.historyPrint(callback=self._resyncComplete, profile=self.profile)
             return
         if self.type == C.CHAT_GROUP:
-            self.occupants.clear()
+            self.occupantsClear()
             self.host.bridge.mucOccupantsGet(
                 unicode(self.target), self.profile, callback=self.updateOccupants,
                 errback=log.error)