From e14e69c883d49670d562055aa71c319830bdaa41 Mon Sep 17 00:00:00 2001
From: Goffi <goffi@goffi.org>
Date: Sun, 30 Jun 2019 15:41:59 +0200
Subject: [PATCH] quick frontends (app, chat): disabled the 2 most verbose
 logs, as they are barely useful and make reading logs difficult.

---
 sat_frontends/quick_frontend/quick_app.py  | 28 ++++++++++++----------
 sat_frontends/quick_frontend/quick_chat.py |  6 +++--
 2 files changed, 19 insertions(+), 15 deletions(-)

diff --git a/sat_frontends/quick_frontend/quick_app.py b/sat_frontends/quick_frontend/quick_app.py
index c40dd420..81f9e518 100644
--- a/sat_frontends/quick_frontend/quick_app.py
+++ b/sat_frontends/quick_frontend/quick_app.py
@@ -826,19 +826,21 @@ class QuickApp(object):
         raise NotImplementedError
 
     def presenceUpdateHandler(self, entity_s, show, priority, statuses, profile):
-        log.debug(
-            _(
-                u"presence update for %(entity)s (show=%(show)s, priority=%(priority)s, "
-                u"statuses=%(statuses)s) [profile:%(profile)s]"
-            )
-            % {
-                "entity": entity_s,
-                C.PRESENCE_SHOW: show,
-                C.PRESENCE_PRIORITY: priority,
-                C.PRESENCE_STATUSES: statuses,
-                "profile": profile,
-            }
-        )
+        # XXX: this log is commented because it's really too verbose even for DEBUG logs
+        #      but it is kept here as it may still be useful for troubleshooting
+        # log.debug(
+        #     _(
+        #         u"presence update for %(entity)s (show=%(show)s, priority=%(priority)s, "
+        #         u"statuses=%(statuses)s) [profile:%(profile)s]"
+        #     )
+        #     % {
+        #         "entity": entity_s,
+        #         C.PRESENCE_SHOW: show,
+        #         C.PRESENCE_PRIORITY: priority,
+        #         C.PRESENCE_STATUSES: statuses,
+        #         "profile": profile,
+        #     }
+        # )
         entity = jid.JID(entity_s)
 
         if entity == self.profiles[profile].whoami:
diff --git a/sat_frontends/quick_frontend/quick_chat.py b/sat_frontends/quick_frontend/quick_chat.py
index c2b50999..79aec100 100644
--- a/sat_frontends/quick_frontend/quick_chat.py
+++ b/sat_frontends/quick_frontend/quick_chat.py
@@ -486,8 +486,10 @@ class QuickChat(quick_widgets.QuickWidget):
         """Set the whole list of occupants"""
         assert len(self.occupants) == 0
         for nick, data in occupants.iteritems():
-            log.debug(u"adding occupant {nick} to {room}".format(
-                nick=nick, room=self.target))
+            # XXX: this log is disabled because it's really too verbose
+            #      but kept commented as it may be useful for debugging
+            # log.debug(u"adding occupant {nick} to {room}".format(
+            #     nick=nick, room=self.target))
             self.occupants[nick] = Occupant(self, data, self.profile)
 
     def updateOccupants(self, occupants):