From a63abab58e287f3cdadf6a114941b12b44ba47b1 Mon Sep 17 00:00:00 2001 From: Charles Haley Date: Fri, 16 Jun 2023 21:11:39 +0100 Subject: [PATCH] Require a name when saving a multi-sort. --- src/calibre/gui2/dialogs/multisort.py | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/src/calibre/gui2/dialogs/multisort.py b/src/calibre/gui2/dialogs/multisort.py index 7a8a7479a7..c9cbbb3cb7 100644 --- a/src/calibre/gui2/dialogs/multisort.py +++ b/src/calibre/gui2/dialogs/multisort.py @@ -150,9 +150,13 @@ class ChooseMultiSort(Dialog): d.setLabelText(_('Choose a name for these settings')) if d.exec(): name = d.textValue() - q = self.saved_specs - q[name] = spec - self.saved_specs = q + if name: + q = self.saved_specs + q[name] = spec + self.saved_specs = q + else: + error_dialog(self, _('No name provided'), _( + 'You must provide a name for the settings'), show=True) def populate_load_menu(self): m = self.load_menu