Code quality

Author: PeterSurda

src/bitmessageqt/__init__.py

@@ -1,7 +1,7 @@
 """
 PyQt based UI for bitmessage, the main module
 """
-# pylint: disable=import-error,too-many-lines
+# pylint: disable=import-error,too-many-lines,no-member
 import hashlib
 import locale
 import os
@@ -410,6 +410,7 @@ def init_subscriptions_popup_menu(self, connectSignal=True):
                 'customContextMenuRequested(const QPoint&)'),
                 self.on_context_menuSubscriptions)
 
+    # pylint: disable=unused-argument
     def init_sent_popup_menu(self, connectSignal=True):
         # Actions
         self.actionTrashSentMessage = self.ui.sentContextMenuToolbar.addAction(
@@ -449,7 +450,6 @@ def rerenderTabTreeSubscriptions(self):
         if treeWidget.isSortingEnabled():
             treeWidget.setSortingEnabled(False)
 
-        widgets = {}
         i = 0
         while i < treeWidget.topLevelItemCount():
             widget = treeWidget.topLevelItem(i)
@@ -538,8 +538,6 @@ def rerenderTabTree(self, tab):
                 toAddress, 'enabled')
             isChan = config.safeGetBoolean(
                 toAddress, 'chan')
-            isMaillinglist = config.safeGetBoolean(
-                toAddress, 'mailinglist')
 
             if treeWidget == self.ui.treeWidgetYourIdentities:
                 if isChan:
@@ -577,7 +575,6 @@ def rerenderTabTree(self, tab):
         if treeWidget.isSortingEnabled():
             treeWidget.setSortingEnabled(False)
 
-        widgets = {}
         i = 0
         while i < treeWidget.topLevelItemCount():
             widget = treeWidget.topLevelItem(i)
@@ -652,7 +649,7 @@ def __init__(self, parent=None):
         # Ask the user if we may delete their old version 1 addresses if they
         # have any.
         for addressInKeysFile in config.addresses():
-            status, addressVersionNumber, streamNumber, hash = decodeAddress(
+            status, addressVersionNumber, streamNumber, _ = decodeAddress(
                 addressInKeysFile)
             if addressVersionNumber == 1:
                 displayMsg = _translate(
@@ -3787,7 +3784,7 @@ def on_action_SetAvatar(self, thisTableWidget):
     def setAvatar(self, addressAtCurrentRow):
         if not os.path.exists(state.appdata + 'avatars/'):
             os.makedirs(state.appdata + 'avatars/')
-        hash = hashlib.md5(addBMIfNotPresent(addressAtCurrentRow)).hexdigest()
+        hash_ = hashlib.md5(addBMIfNotPresent(addressAtCurrentRow)).hexdigest()
         extensions = [
             'PNG', 'GIF', 'JPG', 'JPEG', 'SVG', 'BMP', 'MNG', 'PBM',
             'PGM', 'PPM', 'TIFF', 'XBM', 'XPM', 'TGA']
@@ -3813,8 +3810,8 @@ def setAvatar(self, addressAtCurrentRow):
         for ext in extensions:
             filters += [names[ext] + ' (*.' + ext.lower() + ')']
             all_images_filter += ['*.' + ext.lower()]
-            upper = state.appdata + 'avatars/' + hash + '.' + ext.upper()
-            lower = state.appdata + 'avatars/' + hash + '.' + ext.lower()
+            upper = state.appdata + 'avatars/' + hash_ + '.' + ext.upper()
+            lower = state.appdata + 'avatars/' + hash_ + '.' + ext.lower()
             if os.path.isfile(lower):
                 current_files += [lower]
             elif os.path.isfile(upper):
@@ -3826,7 +3823,8 @@ def setAvatar(self, addressAtCurrentRow):
             filter=';;'.join(filters)
         )
         # determine the correct filename (note that avatars don't use the suffix)
-        destination = state.appdata + 'avatars/' + hash + '.' + sourcefile.split('.')[-1]
+        destination = state.appdata + 'avatars/' + hash_ \
+            + '.' + sourcefile.split('.')[-1]
         exists = QtCore.QFile.exists(destination)
         if sourcefile == '':
             # ask for removal of avatar

src/bitmessageqt/bitmessageui.py

@@ -7,7 +7,8 @@
 #
 # WARNING! All changes made in this file will be lost!
 
-from PyQt4 import QtCore, QtGui
+# pylint: disable=attribute-defined-outside-init
+from PyQt4 import QtCore, QtGui  # pylint: disable=import-error
 from bmconfigparser import config
 from foldertree import AddressBookCompleter
 from messageview import MessageView
@@ -56,6 +57,7 @@ def _translate(context, text, disambig,
 
 
 class Ui_MainWindow(object):
+    # pylint: disable=attribute-defined-outside-init
     def setupUi(self, MainWindow):
         MainWindow.setObjectName(_fromUtf8("MainWindow"))
         MainWindow.resize(885, 580)

src/bitmessageqt/messagecompose.py

@@ -2,7 +2,7 @@
 Message editor with a wheel zoom functionality
 """
 
-from PyQt4 import QtCore, QtGui  # pylint disable:import-error
+from PyQt4 import QtCore, QtGui  # pylint disable=import-error
 
 
 class MessageCompose(QtGui.QTextEdit):

src/bitmessageqt/messageview.py

@@ -5,7 +5,7 @@
 
 """
 
-from PyQt4 import QtCore, QtGui
+from PyQt4 import QtCore, QtGui  # pylint: disable=import-error
 
 from safehtmlparser import SafeHTMLParser
 from tr import _translate

src/bitmessageqt/migrationwizard.py

@@ -1,5 +1,5 @@
 #!/usr/bin/env python2.7
-from PyQt4 import QtGui
+from PyQt4 import QtGui  # pylint: disable=import-error
 
 
 class MigrationWizardIntroPage(QtGui.QWizardPage):
@@ -21,7 +21,7 @@ def nextId(self):
 
 
 class MigrationWizardAddressesPage(QtGui.QWizardPage):
-    def __init__(self, addresses):
+    def __init__(self, addresses):  # pylint: disable=unused-argument
         super(QtGui.QWizardPage, self).__init__()
         self.setTitle("Addresses")
 

src/bitmessageqt/networkstatus.py

@@ -4,7 +4,7 @@
 
 import time
 
-from PyQt4 import QtCore, QtGui
+from PyQt4 import QtCore, QtGui  # pylint: disable=import-error
 
 import l10n
 import network.stats

src/bitmessageqt/retranslateui.py

@@ -1,4 +1,4 @@
-from PyQt4 import QtGui
+from PyQt4 import QtGui  # pylint: disable=import-error
 import widgets
 
 

src/bitmessageqt/settings.py

@@ -7,7 +7,7 @@
 import tempfile
 
 import six
-from PyQt4 import QtCore, QtGui
+from PyQt4 import QtCore, QtGui  # pylint: disable=import-error
 
 import debug
 import defaults

src/bitmessageqt/statusbar.py

@@ -2,7 +2,7 @@
 """Status bar Module"""
 
 from time import time
-from PyQt4 import QtGui
+from PyQt4 import QtGui  # pylint: disable=import-error
 
 
 class BMStatusBar(QtGui.QStatusBar):

src/bitmessageqt/support.py

@@ -6,7 +6,7 @@
 import sys
 import time
 
-from PyQt4 import QtCore
+from PyQt4 import QtCore  # pylint: disable=import-error
 
 import account
 import defaults
@@ -133,6 +133,7 @@ def createSupportMessage(myapp):
     architecture = "32" if ctypes.sizeof(ctypes.c_voidp) == 4 else "64"
     pythonversion = sys.version
 
+    # pylint: disable=protected-access
     opensslversion = "%s (Python internal), %s (external for PyElliptic)" % (
         ssl.OPENSSL_VERSION, OpenSSL._version)