Code quality

Author: PeterSurda

setup.cfg

@@ -24,7 +24,9 @@ ignore = bitmessagekivy
 [pylint.messages_control]
 disable =
     invalid-name,bare-except,broad-except,relative-import,
-    superfluous-parens,bad-option-value,fixme
+    superfluous-parens,bad-option-value,fixme,missing-docstring,
+    too-many-locals,too-few-public-methods,too-many-statements,
+    too-many-instance-attributes,too-many-public-methods
 # invalid-name: needs fixing during a large, project-wide refactor
 # bare-except,broad-except: Need fixing once thorough testing is easier
 # bad-option-value is for backward compatibility between python 2 and 3
@@ -42,7 +44,9 @@ ignore = bitmessagekivy
 [MESSAGES CONTROL]
 disable =
     invalid-name,bare-except,broad-except,relative-import,
-    superfluous-parens,bad-option-value,fixme
+    superfluous-parens,bad-option-value,fixme,missing-docstring,
+    too-many-locals,too-few-public-methods,too-many-statements,
+    too-many-instance-attributes,too-many-public-methods
 
 [DESIGN]
 max-args = 8

src/bitmessagecli.py

@@ -188,7 +188,6 @@ def apiInit(apiEnabled):
 def apiData():
     """TBC"""
 
-    global keysName
     global keysPath
     global usrPrompt
 

src/bitmessageqt/__init__.py

@@ -2,6 +2,7 @@
 PyQt based UI for bitmessage, the main module
 """
 # pylint: disable=import-error,too-many-lines,no-member
+# pylint: disable=too-many-branches
 import hashlib
 import locale
 import os
@@ -760,6 +761,11 @@ def __init__(self, parent=None):
 
         self.unreadCount = 0
 
+        self.currentTrayIconFileName = ""
+        self.actionQuiet = None
+        self.actionShow = None
+        self.tray = None
+
         # Set the icon sizes for the identicons
         identicon_size = 3 * 7
         self.ui.tableWidgetInbox.setIconSize(QtCore.QSize(identicon_size, identicon_size))
@@ -843,7 +849,7 @@ def __init__(self, parent=None):
 
         self.initSettings()
         self.resetNamecoinConnection()
-        self.sqlInit()
+        MyForm.sqlInit()
         self.indicatorInit()
         self.notifierInit()
         self.updateStartOnLogon()
@@ -1107,7 +1113,8 @@ def propagateUnreadCount(self, folder=None, widget=None):
                     if newCount != folderItem.unreadCount:
                         folderItem.setUnreadCount(newCount)
 
-    def addMessageListItem(self, tableWidget, items):
+    @staticmethod
+    def addMessageListItem(tableWidget, items):
         sortingEnabled = tableWidget.isSortingEnabled()
         if sortingEnabled:
             tableWidget.setSortingEnabled(False)
@@ -1117,8 +1124,9 @@ def addMessageListItem(self, tableWidget, items):
         if sortingEnabled:
             tableWidget.setSortingEnabled(True)
 
+    @staticmethod
     def addMessageListItemSent(
-        self, tableWidget, toAddress, fromAddress, subject,
+        tableWidget, toAddress, fromAddress, subject,
         status, ackdata, lastactiontime
     ):
         acct = accountClass(fromAddress) or BMAccount(fromAddress)
@@ -1191,12 +1199,13 @@ def addMessageListItemSent(
                 str(subject), text_type(acct.subject, 'utf-8', 'replace')),
             MessageList_TimeWidget(
                 statusText, False, lastactiontime, ackdata)]
-        self.addMessageListItem(tableWidget, items)
+        MyForm.addMessageListItem(tableWidget, items)
 
         return acct
 
+    @staticmethod
     def addMessageListItemInbox(
-        self, tableWidget, toAddress, fromAddress, subject,
+        tableWidget, toAddress, fromAddress, subject,
         msgid, received, read
     ):
         if toAddress == str_broadcast_subscribers:
@@ -1218,12 +1227,12 @@ def addMessageListItemInbox(
             MessageList_TimeWidget(
                 l10n.formatTimestamp(received), not read, received, msgid)
         ]
-        self.addMessageListItem(tableWidget, items)
+        MyForm.addMessageListItem(tableWidget, items)
 
         return acct
 
-    # Load Sent items from database
     def loadSent(self, tableWidget, account, where="", what=""):
+        """Load Sent items from database"""
         if tableWidget == self.ui.tableWidgetInboxSubscriptions:
             tableWidget.setColumnHidden(0, True)
             tableWidget.setColumnHidden(1, False)
@@ -1241,7 +1250,7 @@ def loadSent(self, tableWidget, account, where="", what=""):
             xAddress, account, "sent", where, what, False)
 
         for row in queryreturn:
-            self.addMessageListItemSent(tableWidget, *row)
+            MyForm.addMessageListItemSent(tableWidget, *row)
 
         tableWidget.horizontalHeader().setSortIndicator(
             3, QtCore.Qt.DescendingOrder)
@@ -1250,11 +1259,11 @@ def loadSent(self, tableWidget, account, where="", what=""):
             _translate("MainWindow", "Sent"))
         tableWidget.setUpdatesEnabled(True)
 
-    # Load messages from database file
     def loadMessagelist(
         self, tableWidget, account, folder="inbox", where="", what="",
         unreadOnly=False
     ):
+        """Load messages from database file"""
         tableWidget.setUpdatesEnabled(False)
         tableWidget.setSortingEnabled(False)
         tableWidget.setRowCount(0)
@@ -1282,7 +1291,7 @@ def loadMessagelist(
 
         for row in queryreturn:
             toAddress, fromAddress, subject, _, msgid, received, read = row
-            self.addMessageListItemInbox(
+            MyForm.addMessageListItemInbox(
                 tableWidget, toAddress, fromAddress, subject,
                 msgid, received, read)
 
@@ -1359,8 +1368,9 @@ def appIndicatorInit(self, _app):
         self.tray.setContextMenu(m)
         self.tray.show()
 
-    # returns the number of unread messages and subscriptions
-    def getUnread(self):
+    @staticmethod
+    def getUnread():
+        """returns the number of unread messages and subscriptions"""
         counters = [0, 0]
 
         queryreturn = sqlQuery('''
@@ -1443,8 +1453,9 @@ def _choose_ext(basename):
 
         self._player(soundFilename)
 
-    # Adapters and converters for QT <-> sqlite
-    def sqlInit(self):
+    @staticmethod
+    def sqlInit():
+        """Adapters and converters for QT <-> sqlite"""
         register_adapter(QtCore.QByteArray, str)
 
     def indicatorInit(self):
@@ -1813,15 +1824,16 @@ def setStatusIcon(self, color):
     def initTrayIcon(self, iconFileName, _app):
         self.currentTrayIconFileName = iconFileName
         self.tray = QtGui.QSystemTrayIcon(
-            self.calcTrayIcon(iconFileName,
-                              self.findInboxUnreadCount()),
+            MyForm.calcTrayIcon(iconFileName,
+                                self.findInboxUnreadCount()),
             _app)
 
     def setTrayIconFile(self, iconFileName):
         self.currentTrayIconFileName = iconFileName
         self.drawTrayIcon(iconFileName, self.findInboxUnreadCount())
 
-    def calcTrayIcon(self, iconFileName, inboxUnreadCount):
+    @staticmethod
+    def calcTrayIcon(iconFileName, inboxUnreadCount):
         pixmap = QtGui.QPixmap(":/newPrefix/images/" + iconFileName)
         if inboxUnreadCount > 0:
             # choose font and calculate font parameters
@@ -1854,7 +1866,8 @@ def calcTrayIcon(self, iconFileName, inboxUnreadCount):
         return QtGui.QIcon(pixmap)
 
     def drawTrayIcon(self, iconFileName, inboxUnreadCount):
-        self.tray.setIcon(self.calcTrayIcon(iconFileName, inboxUnreadCount))
+        self.tray.setIcon(MyForm.calcTrayIcon(iconFileName,
+                                              inboxUnreadCount))
 
     def changedInboxUnread(self, row=None):
         self.drawTrayIcon(
@@ -3978,12 +3991,12 @@ def on_context_menuInbox(self, point):
         popMenuInbox.addAction(self.actionReply)
         popMenuInbox.addAction(self.actionAddSenderToAddressBook)
         # pylint: disable=no-member
-        self.actionClipboardMessagelist = self.ui.inboxContextMenuToolbar.addAction(
+        actionClipboardMessagelist = self.ui.inboxContextMenuToolbar.addAction(
             _translate("MainWindow", "Copy subject to clipboard")
             if tableWidget.currentColumn() == 2 else
             _translate("MainWindow", "Copy address to clipboard"),
             self.on_action_ClipboardMessagelist)
-        popMenuInbox.addAction(self.actionClipboardMessagelist)
+        popMenuInbox.addAction(actionClipboardMessagelist)
         # pylint: disable=no-member
         self._contact_selected = tableWidget.item(currentRow, 1)
         # preloaded gui.menu plugins with prefix 'address'
@@ -4229,7 +4242,7 @@ class BitmessageQtApplication(QtGui.QApplication):
     """
 
     # Unique identifier for this application
-    uuid = '6ec0149b-96e1-4be1-93ab-1465fb3ebf7c'
+    UUID = '6ec0149b-96e1-4be1-93ab-1465fb3ebf7c'
 
     @staticmethod
     def get_windowstyle():
@@ -4241,7 +4254,6 @@ def get_windowstyle():
 
     def __init__(self, *argv):
         super(BitmessageQtApplication, self).__init__(*argv)
-        id = BitmessageQtApplication.uuid
 
         QtCore.QCoreApplication.setOrganizationName("PyBitmessage")
         QtCore.QCoreApplication.setOrganizationDomain("bitmessage.org")
@@ -4259,14 +4271,14 @@ def __init__(self, *argv):
         self.is_running = False
 
         socket = QLocalSocket()
-        socket.connectToServer(id)
+        socket.connectToServer(self.UUID)
         self.is_running = socket.waitForConnected()
 
         # Cleanup past crashed servers
         if not self.is_running:
             if socket.error() == QLocalSocket.ConnectionRefusedError:
                 socket.disconnectFromServer()
-                QLocalServer.removeServer(id)
+                QLocalServer.removeServer(self.UUID)
 
         socket.abort()
 
@@ -4278,7 +4290,7 @@ def __init__(self, *argv):
             # Nope, create a local server with this id and assign on_new_connection
             # for whenever a second instance tries to run focus the application.
             self.server = QLocalServer()
-            self.server.listen(id)
+            self.server.listen(self.UUID)
             self.server.newConnection.connect(self.on_new_connection)
 
         self.setStyleSheet("QStatusBar::item { border: 0px solid black }")
@@ -4293,20 +4305,20 @@ def on_new_connection(self):
 
 
 def init():
-    global app
+    global app  # pylint: disable=global-statement
     if not app:
         app = BitmessageQtApplication(sys.argv)
     return app
 
 
 def run():
-    global myapp
-    app = init()
-    myapp = MyForm()
+    global myapp  # pylint: disable=global-statement
+    _app = init()
+    myapp = MyForm()  # pylint: disable=redefined-outer-name
 
     myapp.appIndicatorInit(app)
 
-    if myapp._firstrun:
+    if myapp._firstrun:  # pylint: disable=protected-access
         myapp.showConnectDialog()  # ask the user if we may connect
 
     # only show after wizards and connect dialogs have completed
@@ -4315,4 +4327,4 @@ def run():
         QtCore.QTimer.singleShot(
             30000, lambda: myapp.setStatusIcon(state.statusIconColor))
 
-    app.exec_()
+    _app.exec_()

src/bitmessageqt/account.py

@@ -68,6 +68,7 @@ def accountClass(address):
             if subscription.type != AccountMixin.BROADCAST:
                 return None
         else:
+            # pylint: disable=redefined-variable-type
             subscription = SubscriptionAccount(address)
             if subscription.type != AccountMixin.SUBSCRIPTION:
                 # e.g. deleted chan

src/bitmessageqt/bitmessage_icons_rc.py

@@ -7,6 +7,7 @@
 #
 # WARNING! All changes made in this file will be lost!
 
+# pylint: disable=too-many-lines
 from PyQt4 import QtCore  # pylint: disable=import-error
 
 qt_resource_data = "\

src/bitmessageqt/foldertree.py

@@ -410,6 +410,7 @@ def setLabel(self, label=None):
                 AccountMixin.NORMAL,
                 AccountMixin.CHAN, AccountMixin.MAILINGLIST):
             try:
+                # pylint: disable=redefined-variable-type
                 newLabel = unicode(
                     config.get(self.address, 'label'),
                     'utf-8', 'ignore')

src/bitmessageqt/migrationwizard.py

@@ -66,6 +66,7 @@ def __init__(self):
 
 
 class Ui_MigrationWizard(QtGui.QWizard):
+    # pylint: disable=redefined-variable-type
     def __init__(self, addresses):
         super(QtGui.QWizard, self).__init__()
 

src/bitmessageqt/uisignaler.py

@@ -17,6 +17,7 @@ def get(cls):
             cls._instance = UISignaler()
         return cls._instance
 
+    # pylint: disable=too-many-branches
     def run(self):
         while True:
             command, data = queues.UISignalQueue.get()

src/class_singleWorker.py

@@ -1399,7 +1399,7 @@ def requestPubKey(self, toAddress):
         TTL = 2.5 * 24 * 60 * 60
         TTL *= 2 ** retryNumber
         if TTL > 28 * 24 * 60 * 60:
-            TTL = 28 * 24 * 60 * 60
+            TTL = float(28 * 24 * 60 * 60)
         # add some randomness to the TTL
         TTL = TTL + helper_random.randomrandrange(-300, 300)
         embeddedTime = int(time.time() + TTL)