From 4fc124adb0f159754a4eed3e40d5e9dab7f1c5b7 Mon Sep 17 00:00:00 2001 From: Eli Schwartz Date: Mon, 22 Apr 2019 21:36:57 -0400 Subject: [PATCH] py3: also make plugin installer work We check if versions compare favorably to the installed version, and if it None, then we report that it can be installed. But this check always returns None < available_version on python2, and on python3 it is a comparison error -- so lift out the None handling to return as early as possible. --- src/calibre/gui2/dialogs/plugin_updater.py | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/src/calibre/gui2/dialogs/plugin_updater.py b/src/calibre/gui2/dialogs/plugin_updater.py index 36c024a4e9..9c7afcb975 100644 --- a/src/calibre/gui2/dialogs/plugin_updater.py +++ b/src/calibre/gui2/dialogs/plugin_updater.py @@ -412,13 +412,12 @@ class DisplayPluginModel(QAbstractTableModel): return (_('You must upgrade to at least calibre %s before installing this plugin') % self._get_display_version(display_plugin.calibre_required_version)+'\n\n'+ _('Right-click to see more options')) + if display_plugin.installed_version is None: + return (_('You can install this plugin')+'\n\n'+ + _('Right-click to see more options')) if display_plugin.installed_version < display_plugin.available_version: - if display_plugin.installed_version is None: - return (_('You can install this plugin')+'\n\n'+ - _('Right-click to see more options')) - else: - return (_('A new version of this plugin is available')+'\n\n'+ - _('Right-click to see more options')) + return (_('A new version of this plugin is available')+'\n\n'+ + _('Right-click to see more options')) return (_('This plugin is installed and up-to-date')+'\n\n'+ _('Right-click to see more options'))