Fixes bugs created by code quality fixes

Author: PeterSurda

src/bitmessageqt/__init__.py

@@ -4296,16 +4296,15 @@ def __init__(self, *argv):
             # for whenever a second instance tries to run focus the application.
             self.server = QLocalServer()
             self.server.listen(self.UUID)
-            self.server.newConnection.connect(MyForm.on_new_connection)
+            self.server.newConnection.connect(self.on_new_connection)
 
         self.setStyleSheet("QStatusBar::item { border: 0px solid black }")
 
     def __del__(self):
         if self.server:
             self.server.close()
 
-    @staticmethod
-    def on_new_connection():
+    def on_new_connection(self):  # pylint: disable=no-self-use
         if myapp:
             myapp.appIndicatorShow()
 

src/inventory.py

@@ -40,6 +40,14 @@ def __getattr__(self, attr):
         else:
             return realRet
 
+    def __contains__(self, key):
+        """
+        Look up inventory item by hash.
+        This method is needed due to how new-style classes work.
+        """
+        self.numberOfInventoryLookupsPerformed += 1
+        return key in self._realInventory
+
     def __getitem__(self, key):
         """hint for pylint, this is dictionary like object"""
         return self._realInventory[key]