From 26d75d99fdad813ced9b72c724824de01a37f1c6 Mon Sep 17 00:00:00 2001
From: lakshyacis <lakshya.p@cisinlabs.com>
Date: Mon, 20 Jan 2020 19:57:58 +0530
Subject: [PATCH 1/3] Logging and modules issues fixes

---
 src/bitmessagemain.py | 2 +-
 src/network/udp.py    | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/bitmessagemain.py b/src/bitmessagemain.py
index 65afbe7f..b99f071a 100755
--- a/src/bitmessagemain.py
+++ b/src/bitmessagemain.py
@@ -371,7 +371,7 @@ class Main(object):
 
             elif state.kivy:
                 config.remove_option('bitmessagesettings', 'dontconnect')
-                # pylint: disable=no-member, import-error
+                # pylint: disable=no-member, import-error, no-name-in-module
                 from bitmessagekivy.mpybit import NavigateApp
                 state.kivyapp = NavigateApp()
                 state.kivyapp.run()
diff --git a/src/network/udp.py b/src/network/udp.py
index b6228fa4..ddd27fe3 100644
--- a/src/network/udp.py
+++ b/src/network/udp.py
@@ -91,8 +91,8 @@ class UDPSocket(BMProto):  # pylint: disable=too-many-instance-attributes
         if remoteport is False:
             return True
         logger.debug(
-            "received peer discovery from {}:{} (port {}):",
-            self.destination.host, self.destination.port, remoteport)
+            "received peer discovery from {}:{} (port {}):".format(
+                self.destination.host, self.destination.port, remoteport))
         if self.local:
             state.discoveredPeers[Peer(self.destination.host, remoteport)] = \
                 time.time()

From 1decaad15691ba88a1e5ed47fb682052dc73849d Mon Sep 17 00:00:00 2001
From: lakshyacis <lakshya.p@cisinlabs.com>
Date: Tue, 21 Jan 2020 13:39:56 +0530
Subject: [PATCH 2/3] invalid syntax pylint issue resolve

---
 src/bitmessagekivy/mpybit.py | 17 +++++++++--------
 src/bitmessagemain.py        |  2 +-
 src/network/udp.py           |  1 +
 3 files changed, 11 insertions(+), 9 deletions(-)

diff --git a/src/bitmessagekivy/mpybit.py b/src/bitmessagekivy/mpybit.py
index 0a483621..641da408 100644
--- a/src/bitmessagekivy/mpybit.py
+++ b/src/bitmessagekivy/mpybit.py
@@ -1,8 +1,8 @@
 """
 Bitmessage android(mobile) interface
 """
-# pylint: disable=import-error,no-name-in-module,unused-argument,too-few-public-methods,too-many-arguments
-# pylint: disable=too-many-ancestors,too-many-locals,useless-super-delegation,attribute-defined-outside-init
+# pylint: disable=too-many-lines,import-error,no-name-in-module,unused-argument
+# pylint: disable=too-many-ancestors,too-many-locals,useless-super-delegation
 import os
 import time
 from bitmessagekivy import identiconGeneration
@@ -62,7 +62,7 @@ from semaphores import kivyuisignaler
 
 import state
 from addresses import decodeAddress
-
+# pylint: disable=too-few-public-methods,too-many-arguments,attribute-defined-outside-init
 
 KVFILES = [
     'settings', 'popup', 'allmails', 'draft',
@@ -509,7 +509,8 @@ class AddressBook(Screen):
     def delete_address(self, address, instance, *args):
         """Delete inbox mail from inbox listing"""
         self.ids.ml.remove_widget(instance.parent.parent)
-        if len(self.ids.ml.children) == 0:
+        # if len(self.ids.ml.children) == 0:
+        if self.ids.ml.children is not None:
             self.ids.identi_tag.children[0].text = ''
         sqlExecute(
             "DELETE FROM  addressbook WHERE address = '{}';".format(address))
@@ -1263,7 +1264,7 @@ class Setting(Screen):
 
 class NavigateApp(MDApp):
     """Navigation Layout of class"""
-    # pylint: disable=too-many-public-methods
+    # pylint: disable=too-many-public-methods,inconsistent-return-statements
 
     # theme_cls = ThemeManager()
     previous_date = ObjectProperty()
@@ -1280,8 +1281,7 @@ class NavigateApp(MDApp):
     def build(self):
         """Method builds the widget"""
         for kv_file in KVFILES:
-            Builder.load_file(
-                os.path.join(os.path.dirname(__file__), f"kv/{kv_file}.kv"))
+            Builder.load_file(os.path.join(os.path.dirname(__file__), "kv/{}.kv").format(kv_file))
         self.obj_1 = AddressBook()
         kivysignalthread = UIkivySignaler()
         kivysignalthread.daemon = True
@@ -1298,7 +1298,6 @@ class NavigateApp(MDApp):
     @staticmethod
     def showmeaddresses(name="text"):
         """Show the addresses in spinner to make as dropdown"""
-        # pylint: disable=inconsistent-return-statements
         if name == "text":
             if BMConfigParser().addresses():
                 return BMConfigParser().addresses()[0][:16] + '..'
@@ -1590,6 +1589,7 @@ class NavigateApp(MDApp):
             msg_counter_objs.allmail_cnt.badge_text = state.all_count
 
     def on_start(self):
+        """Setting message count"""
         self.set_message_count()
 
     # @staticmethod
@@ -2585,6 +2585,7 @@ class NavigationDrawerDivider(OneLineListItem):
     _txt_bot_pad = NumericProperty(dp(8))
 
     def __init__(self, **kwargs):
+        # pylint: disable=bad-super-call
         super(OneLineListItem, self).__init__(**kwargs)
         self.height = dp(16)
 
diff --git a/src/bitmessagemain.py b/src/bitmessagemain.py
index b99f071a..4a62a8fe 100755
--- a/src/bitmessagemain.py
+++ b/src/bitmessagemain.py
@@ -371,7 +371,7 @@ class Main(object):
 
             elif state.kivy:
                 config.remove_option('bitmessagesettings', 'dontconnect')
-                # pylint: disable=no-member, import-error, no-name-in-module
+                # pylint: disable=no-member,import-error,no-name-in-module,relative-import
                 from bitmessagekivy.mpybit import NavigateApp
                 state.kivyapp = NavigateApp()
                 state.kivyapp.run()
diff --git a/src/network/udp.py b/src/network/udp.py
index ddd27fe3..aac32554 100644
--- a/src/network/udp.py
+++ b/src/network/udp.py
@@ -14,6 +14,7 @@ import state
 from queues import receiveDataQueue
 
 logger = logging.getLogger('default')
+# pylint: disable=logging-format-interpolation
 
 
 class UDPSocket(BMProto):  # pylint: disable=too-many-instance-attributes

From fe3a98c2b61ea0ad4656b73d7d9c51686709bb61 Mon Sep 17 00:00:00 2001
From: lakshyacis <lakshya.p@cisinlabs.com>
Date: Tue, 21 Jan 2020 15:48:21 +0530
Subject: [PATCH 3/3] unidiomatic-typecheck and logging-format-interpolation
 issue suprressed

---
 pybitmessage                     | 1 -
 src/bmconfigparser.py            | 1 +
 src/multiqueue.py                | 2 +-
 src/network/bmproto.py           | 5 +++--
 src/network/connectionchooser.py | 2 +-
 src/network/connectionpool.py    | 2 +-
 src/network/tcp.py               | 2 +-
 src/protocol.py                  | 3 ++-
 src/shared.py                    | 2 +-
 src/tests/test_networkgroup.py   | 1 +
 10 files changed, 12 insertions(+), 9 deletions(-)
 delete mode 120000 pybitmessage

diff --git a/pybitmessage b/pybitmessage
deleted file mode 120000
index e8310385..00000000
--- a/pybitmessage
+++ /dev/null
@@ -1 +0,0 @@
-src
\ No newline at end of file
diff --git a/src/bmconfigparser.py b/src/bmconfigparser.py
index a5574156..d96d994d 100644
--- a/src/bmconfigparser.py
+++ b/src/bmconfigparser.py
@@ -117,6 +117,7 @@ class BMConfigParser(configparser.ConfigParser):
             return default
 
     def items(self, section, raw=False, variables=None):
+        # pylint: disable=signature-differs
         """Return section variables as parent,
         but override the "raw" argument to always True"""
         return configparser.ConfigParser.items(self, section, True, variables)
diff --git a/src/multiqueue.py b/src/multiqueue.py
index 4230d050..792427a0 100644
--- a/src/multiqueue.py
+++ b/src/multiqueue.py
@@ -22,7 +22,7 @@ class MultiQueue(Queue.Queue):
         Queue.Queue.__init__(self, maxsize)
 
     # Initialize the queue representation
-    def _init(self, maxsize):
+    def _init(self, _):
         self.iter = 0
         self.queues = []
         for _ in range(self.queueCount):
diff --git a/src/network/bmproto.py b/src/network/bmproto.py
index 5c57748a..adca21f5 100644
--- a/src/network/bmproto.py
+++ b/src/network/bmproto.py
@@ -1,7 +1,7 @@
 """
 Bitmessage Protocol
 """
-# pylint: disable=attribute-defined-outside-init, too-few-public-methods
+# pylint: disable=attribute-defined-outside-init, too-few-public-methods, logging-format-interpolation
 import base64
 import hashlib
 import logging
@@ -123,7 +123,8 @@ class BMProto(AdvancedDispatcher, ObjectTracker):
             length=protocol.Header.size, expectBytes=self.payloadLength)
         return True
 
-    def state_bm_command(self):  # pylint: disable=too-many-branches, too-many-statements
+    def state_bm_command(self):
+        # pylint: disable=too-many-branches, too-many-statements
         """Process incoming command"""
         self.payload = self.read_buf[:self.payloadLength]
         if self.checksum != hashlib.sha512(self.payload).digest()[0:4]:
diff --git a/src/network/connectionchooser.py b/src/network/connectionchooser.py
index 87fea685..4ea36a02 100644
--- a/src/network/connectionchooser.py
+++ b/src/network/connectionchooser.py
@@ -1,7 +1,7 @@
 """
 Select which node to connect to
 """
-# pylint: disable=too-many-branches
+# pylint: disable=too-many-branches, logging-format-interpolation, unidiomatic-typecheck
 import logging
 import random  # nosec
 
diff --git a/src/network/connectionpool.py b/src/network/connectionpool.py
index 5e59d99d..36f8c2b3 100644
--- a/src/network/connectionpool.py
+++ b/src/network/connectionpool.py
@@ -190,7 +190,7 @@ class BMConnectionPool(object):
     def startListening(self, bind=None):
         """Open a listening socket and start accepting connections on it"""
         if bind is None:
-            "this return blank host"
+            """this return blank host"""
             bind = self.getListeningIP()
         port = BMConfigParser().safeGetInt("bitmessagesettings", "port")
         # correct port even if it changed
diff --git a/src/network/tcp.py b/src/network/tcp.py
index 33ac4df5..47553751 100644
--- a/src/network/tcp.py
+++ b/src/network/tcp.py
@@ -30,7 +30,7 @@ from network.socks5 import Socks5Connection
 from network.tls import TLSDispatcher
 from .node import Peer
 from queues import UISignalQueue, invQueue, receiveDataQueue
-
+# pylint: disable=logging-format-interpolation
 
 logger = logging.getLogger('default')
 
diff --git a/src/protocol.py b/src/protocol.py
index 8b6a71d8..6f84236e 100644
--- a/src/protocol.py
+++ b/src/protocol.py
@@ -2,7 +2,7 @@
 Low-level protocol-related functions.
 """
 # pylint: disable=too-many-boolean-expressions,too-many-return-statements
-# pylint: disable=too-many-locals,too-many-statements
+# pylint: disable=too-many-locals,too-many-statements,logging-format-interpolation
 
 import base64
 import hashlib
@@ -65,6 +65,7 @@ Header = Struct('!L12sL4s')
 VersionPacket = Struct('>LqQ20s4s36sH')
 
 # Bitfield
+# pylint: disable=unidiomatic-typecheck
 
 
 def getBitfield(address):
diff --git a/src/shared.py b/src/shared.py
index b1a19c08..0f2a816c 100644
--- a/src/shared.py
+++ b/src/shared.py
@@ -24,7 +24,7 @@ from addresses import decodeAddress, encodeVarint
 from bmconfigparser import BMConfigParser
 from debug import logger
 from helper_sql import sqlQuery
-
+# pylint: disable=logging-format-interpolation
 
 verbose = 1
 # This is obsolete with the change to protocol v3
diff --git a/src/tests/test_networkgroup.py b/src/tests/test_networkgroup.py
index 76cfb033..14e61a8e 100644
--- a/src/tests/test_networkgroup.py
+++ b/src/tests/test_networkgroup.py
@@ -10,6 +10,7 @@ class TestNetworkGroup(unittest.TestCase):
     """
     def test_network_group(self):
         """Test various types of network groups"""
+        # pylint: disable=import-error
         from pybitmessage.protocol import network_group
 
         test_ip = '1.2.3.4'