Mercurial Repositories > Print Remover Tool / changeset
changeset
Implemented some code simplifications.
Mon, 26 Apr 2021 19:46:00 +0200
- author
- Detlev Offenbach <detlev@die-offenbachs.de>
- date
- Mon, 26 Apr 2021 19:46:00 +0200
- changeset 54
- d061dda35cef
- parent 53
- 359f9805af93
- child 55
- 4c9ea93f36f4
Implemented some code simplifications.
--- a/ChangeLog Wed Dec 30 11:02:14 2020 +0100 +++ b/ChangeLog Mon Apr 26 19:46:00 2021 +0200 @@ -1,5 +1,8 @@ ChangeLog --------- +Version 3.1.0: +- implemented some code simplifications + Version 3.0.0: - removed support for Python2
--- a/PluginPrintRemover.py Wed Dec 30 11:02:14 2020 +0100 +++ b/PluginPrintRemover.py Mon Apr 26 19:46:00 2021 +0200 @@ -7,6 +7,7 @@ Module implementing the Print Remover plug-in. """ +import contextlib import os from PyQt5.QtCore import QObject, QTranslator, QCoreApplication @@ -21,7 +22,7 @@ author = "Detlev Offenbach <detlev@die-offenbachs.de>" autoactivate = True deactivateable = True -version = "3.0.0" +version = "3.1.0" className = "PrintRemoverPlugin" packageName = "PrintRemover" shortDescription = "Remove print() like debug statements." @@ -64,14 +65,12 @@ try: usesDarkPalette = e5App().usesDarkPalette() except AttributeError: + # for eric6 < 20.4 from PyQt5.QtGui import QPalette palette = e5App().palette() lightness = palette.color(QPalette.Window).lightness() usesDarkPalette = lightness <= 128 - if usesDarkPalette: - iconSuffix = "dark" - else: - iconSuffix = "light" + iconSuffix = "dark" if usesDarkPalette else "light" return { "printRemoverPage": [ @@ -102,7 +101,7 @@ @param ui reference to the user interface object (UI.UserInterface) """ - QObject.__init__(self, ui) + super().__init__(ui) self.__ui = ui self.__defaults = { @@ -272,12 +271,10 @@ @param editor reference to the editor (QScintilla.Editor) """ - try: + with contextlib.suppress(KeyError): del self.__editors[editor] if not self.__editors: self.__menu.setEnabled(False) - except KeyError: - pass def __editorShowMenu(self, menuName, menu, editor): """ @@ -288,15 +285,17 @@ @param menu reference to the menu (QMenu) @param editor reference to the editor """ - if menuName == "Tools": - if self.__menu.menuAction() not in menu.actions(): - # Re-add our menu - self.__editors[editor] = [] - if not menu.isEmpty(): - act = menu.addSeparator() - self.__editors[editor].append(act) - act = menu.addMenu(self.__menu) + if ( + menuName == "Tools" and + self.__menu.menuAction() not in menu.actions() + ): + # Re-add our menu + self.__editors[editor] = [] + if not menu.isEmpty(): + act = menu.addSeparator() self.__editors[editor].append(act) + act = menu.addMenu(self.__menu) + self.__editors[editor].append(act) def __showMenu(self): """
--- /dev/null Thu Jan 01 00:00:00 1970 +0000 +++ b/PrintRemover.epj Mon Apr 26 19:46:00 2021 +0200 @@ -0,0 +1,271 @@ +{ + "header": { + "comment": "eric project file for project PrintRemover", + "copyright": "Copyright (C) 2021 Detlev Offenbach, detlev@die-offenbachs.de" + }, + "project": { + "AUTHOR": "Detlev Offenbach", + "CHECKERSPARMS": { + "Pep8Checker": { + "AnnotationsChecker": { + "AllowUntypedDefs": false, + "AllowUntypedNested": false, + "DispatchDecorators": [ + "singledispatch", + "singledispatchmethod" + ], + "MaximumComplexity": 3, + "MaximumLength": 7, + "MinimumCoverage": 75, + "MypyInitReturn": false, + "OverloadDecorators": [ + "overload" + ], + "SuppressDummyArgs": false, + "SuppressNoneReturning": true + }, + "BlankLines": [ + 2, + 1 + ], + "BuiltinsChecker": { + "bytes": [ + "unicode" + ], + "chr": [ + "unichr" + ], + "str": [ + "unicode" + ] + }, + "CommentedCodeChecker": { + "Aggressive": false, + "WhiteList": [ + "pylint", + "pyright", + "noqa", + "type:\\s*ignore", + "fmt:\\s*(on|off)", + "TODO", + "FIXME", + "WARNING", + "NOTE", + "TEST", + "DOCU", + "XXX", + "- " + ] + }, + "CopyrightAuthor": "", + "CopyrightMinFileSize": 0, + "DocstringType": "eric", + "EnabledCheckerCategories": "C, D, E, M, N, S, Y, W", + "ExcludeFiles": "*/Ui_*.py, */*_rc.py", + "ExcludeMessages": "C101,E265,E266,E305,E402,M201,M301,M302,M303,M304,M305,M306,M307,M308,M311,M312,M313,M314,M315,M321,M701,M702,M811,M834,N802,N803,N807,N808,N821,W293,W504,Y119,Y401,Y402", + "FixCodes": "", + "FixIssues": false, + "FutureChecker": "", + "HangClosing": false, + "IncludeMessages": "", + "LineComplexity": 25, + "LineComplexityScore": 10, + "MaxCodeComplexity": 10, + "MaxDocLineLength": 79, + "MaxLineLength": 79, + "NoFixCodes": "E501", + "RepeatMessages": true, + "SecurityChecker": { + "CheckTypedException": false, + "HardcodedTmpDirectories": [ + "/tmp", + "/var/tmp", + "/dev/shm", + "~/tmp" + ], + "InsecureHashes": [ + "md4", + "md5", + "sha", + "sha1" + ], + "InsecureSslProtocolVersions": [ + "PROTOCOL_SSLv2", + "SSLv2_METHOD", + "SSLv23_METHOD", + "PROTOCOL_SSLv3", + "PROTOCOL_TLSv1", + "SSLv3_METHOD", + "TLSv1_METHOD" + ], + "WeakKeySizeDsaHigh": "1024", + "WeakKeySizeDsaMedium": "2048", + "WeakKeySizeEcHigh": "160", + "WeakKeySizeEcMedium": "224", + "WeakKeySizeRsaHigh": "1024", + "WeakKeySizeRsaMedium": "2048" + }, + "ShowIgnored": false, + "ValidEncodings": "latin-1, utf-8" + } + }, + "DESCRIPTION": "Plug-in to remove lines starting with a configurable string. Usually this is used to remove print() kind debug statements.", + "DOCSTRING": "", + "DOCUMENTATIONPARMS": { + "ERIC4DOC": { + "cssFile": "%PYTHON%/eric6/CSSs/default.css", + "ignoreDirectories": [ + ".ropeproject", + ".eric5project", + ".hg", + "_ropeproject", + "_eric5project", + ".eric6project", + "_eric6project" + ], + "ignoreFilePatterns": [ + "Ui_*.py" + ], + "outputDirectory": "PrintRemover/Documentation/source", + "qtHelpEnabled": false, + "useRecursion": true + } + }, + "EMAIL": "detlev@die-offenbachs.de", + "EOL": 1, + "FILETYPES": { + "*.idl": "INTERFACES", + "*.py": "SOURCES", + "*.py3": "SOURCES", + "*.pyw": "SOURCES", + "*.pyw3": "SOURCES", + "*.qm": "TRANSLATIONS", + "*.qrc": "RESOURCES", + "*.ts": "TRANSLATIONS", + "*.ui": "FORMS", + "*.ui.h": "FORMS", + "Ui_*.py": "__IGNORE__" + }, + "FORMS": [ + "PrintRemover/ConfigurationPage/PrintRemoverPage.ui" + ], + "HASH": "d994091d1e81ad5afcdcbc00ecea86f23163c696", + "IDLPARAMS": { + "DefinedNames": [], + "IncludeDirs": [], + "UndefinedNames": [] + }, + "INTERFACES": [], + "LEXERASSOCS": {}, + "MAINSCRIPT": "PluginPrintRemover.py", + "MAKEPARAMS": { + "MakeEnabled": false, + "MakeExecutable": "", + "MakeFile": "", + "MakeParameters": "", + "MakeTarget": "", + "MakeTestOnly": true + }, + "MIXEDLANGUAGE": false, + "OTHERS": [ + ".hgignore", + "ChangeLog", + "PKGLIST", + "PluginPrintRemover.zip", + "PrintRemover.e4p", + "PrintRemover/Documentation/LICENSE.GPL3", + "PrintRemover/Documentation/source", + "PrintRemover/icons/edit-dark.svg", + "PrintRemover/icons/edit-light.svg", + "PrintRemover/icons/printRemover-dark.svg", + "PrintRemover/icons/printRemover-light.svg", + "PrintRemover/icons/separatorAdd-dark.svg", + "PrintRemover/icons/separatorAdd-light.svg", + "PrintRemover.epj" + ], + "OTHERTOOLSPARMS": {}, + "PACKAGERSPARMS": {}, + "PROGLANGUAGE": "Python3", + "PROJECTTYPE": "E6Plugin", + "PROJECTTYPESPECIFICDATA": {}, + "PROTOCOLS": [], + "RCCPARAMS": { + "CompressLevel": 0, + "CompressionDisable": false, + "CompressionThreshold": 70, + "PathPrefix": "" + }, + "RESOURCES": [], + "SOURCES": [ + "PluginPrintRemover.py", + "PrintRemover/ConfigurationPage/PrintRemoverPage.py", + "PrintRemover/ConfigurationPage/__init__.py", + "PrintRemover/__init__.py", + "__init__.py" + ], + "SPELLEXCLUDES": "", + "SPELLLANGUAGE": "en_US", + "SPELLWORDS": "", + "TRANSLATIONEXCEPTIONS": [], + "TRANSLATIONPATTERN": "PrintRemover/i18n/printremover_%language%.ts", + "TRANSLATIONS": [ + "PrintRemover/i18n/printremover_de.qm", + "PrintRemover/i18n/printremover_de.ts", + "PrintRemover/i18n/printremover_en.qm", + "PrintRemover/i18n/printremover_en.ts", + "PrintRemover/i18n/printremover_es.qm", + "PrintRemover/i18n/printremover_es.ts", + "PrintRemover/i18n/printremover_pt.qm", + "PrintRemover/i18n/printremover_pt.ts", + "PrintRemover/i18n/printremover_ru.qm", + "PrintRemover/i18n/printremover_ru.ts" + ], + "TRANSLATIONSBINPATH": "", + "UICPARAMS": { + "Package": "", + "PackagesRoot": "", + "RcSuffix": "" + }, + "VCS": "Mercurial", + "VCSOPTIONS": { + "add": [ + "" + ], + "checkout": [ + "" + ], + "commit": [ + "" + ], + "diff": [ + "" + ], + "export": [ + "" + ], + "global": [ + "" + ], + "history": [ + "" + ], + "log": [ + "" + ], + "remove": [ + "" + ], + "status": [ + "" + ], + "tag": [ + "" + ], + "update": [ + "" + ] + }, + "VCSOTHERDATA": {}, + "VERSION": "2.x" + } +} \ No newline at end of file
--- a/PrintRemover/ConfigurationPage/PrintRemoverPage.py Wed Dec 30 11:02:14 2020 +0100 +++ b/PrintRemover/ConfigurationPage/PrintRemoverPage.py Mon Apr 26 19:46:00 2021 +0200 @@ -32,21 +32,19 @@ @param plugin reference to the plugin object """ - super(PrintRemoverPage, self).__init__() + super().__init__() self.setupUi(self) self.setObjectName("PrintRemoverPage") try: usesDarkPalette = e5App().usesDarkPalette() except AttributeError: + # for eric6 < 20.4 from PyQt5.QtGui import QPalette palette = e5App().palette() lightness = palette.color(QPalette.Window).lightness() usesDarkPalette = lightness <= 128 - if usesDarkPalette: - iconSuffix = "dark" - else: - iconSuffix = "light" + iconSuffix = "dark" if usesDarkPalette else "light" self.editButton.setIcon(UI.PixmapCache.getIcon( os.path.join("PrintRemover", "icons",
--- a/PrintRemover/Documentation/source/Plugin_Tools_Print_Remover.PluginPrintRemover.html Wed Dec 30 11:02:14 2020 +0100 +++ b/PrintRemover/Documentation/source/Plugin_Tools_Print_Remover.PluginPrintRemover.html Mon Apr 26 19:46:00 2021 +0200 @@ -258,7 +258,7 @@ Public method to activate this plugin. </p> <dl> -<dt>Returns:</dt> +<dt>Return:</dt> <dd> tuple of None and activation status (boolean) </dd> @@ -285,7 +285,7 @@ </dd> </dl> <dl> -<dt>Returns:</dt> +<dt>Return:</dt> <dd> the requested setting </dd> @@ -326,7 +326,7 @@ </dd> </dl> <dl> -<dt>Returns:</dt> +<dt>Return:</dt> <dd> reference to the configuration page </dd> @@ -342,7 +342,7 @@ Module function returning data as required by the configuration dialog. </p> <dl> -<dt>Returns:</dt> +<dt>Return:</dt> <dd> dictionary containing the relevant data </dd>
