PluginVulture.py

changeset 69
3c2922b45a9f
parent 68
d7a6b7ea640d
child 74
8bcdbec157d3
diff -r d7a6b7ea640d -r 3c2922b45a9f PluginVulture.py
--- a/PluginVulture.py	Wed Dec 30 11:02:01 2020 +0100
+++ b/PluginVulture.py	Sun Apr 25 16:13:53 2021 +0200
@@ -22,7 +22,7 @@
 author = "Detlev Offenbach <detlev@die-offenbachs.de>"
 autoactivate = True
 deactivateable = True
-version = "3.1.1"
+version = "4.0.0"
 className = "VulturePlugin"
 packageName = "VultureChecker"
 shortDescription = "Plug-in to detect unused code using the vulture library"
@@ -55,7 +55,7 @@
         
         @param ui reference to the user interface object (UI.UserInterface)
         """
-        super(VulturePlugin, self).__init__(ui)
+        super().__init__(ui)
         self.__ui = ui
         self.__initialize()
         
@@ -234,11 +234,10 @@
             self.__projectClosed)
         
         menu = e5App().getObject("Project").getMenu("Checks")
-        if menu:
-            if self.__projectAct is not None:
-                menu.removeAction(self.__projectAct)
-                e5App().getObject("Project").removeE5Actions(
-                    [self.__projectAct])
+        if menu is not None and self.__projectAct is not None:
+            menu.removeAction(self.__projectAct)
+            e5App().getObject("Project").removeE5Actions(
+                [self.__projectAct])
         
         self.__initialize()
     
@@ -272,12 +271,11 @@
         @param menu reference to the menu
         @type QMenu
         """
-        if menuName == "Check":
-            if self.__projectAct is not None:
-                self.__projectAct.setEnabled(
-                    e5App().getObject("Project")
-                    .getProjectLanguage() == "Python3"
-                )
+        if menuName == "Check" and self.__projectAct is not None:
+            self.__projectAct.setEnabled(
+                e5App().getObject("Project")
+                .getProjectLanguage() == "Python3"
+            )
     
     def __projectVultureCheck(self):
         """

eric ide

mercurial