From a1ce048ed55f95a11d638c7519d70b29228cec30 Mon Sep 17 00:00:00 2001 From: Damien Elmes Date: Sun, 25 Apr 2021 19:26:55 +1000 Subject: [PATCH] fix error when user cancels out of set due date/reposition --- qt/aqt/browser/browser.py | 10 ++++++---- qt/aqt/reviewer.py | 5 +++-- 2 files changed, 9 insertions(+), 6 deletions(-) diff --git a/qt/aqt/browser/browser.py b/qt/aqt/browser/browser.py index 9e42e70ec..5b654e86c 100644 --- a/qt/aqt/browser/browser.py +++ b/qt/aqt/browser/browser.py @@ -788,17 +788,19 @@ where id in %s""" showInfo(tr.browsing_only_new_cards_can_be_repositioned(), parent=self) return - reposition_new_cards_dialog( + if op := reposition_new_cards_dialog( parent=self, card_ids=self.selected_cards() - ).run_in_background() + ): + op.run_in_background() @ensure_editor_saved_on_trigger def set_due_date(self) -> None: - set_due_date_dialog( + if op := set_due_date_dialog( parent=self, card_ids=self.selected_cards(), config_key=Config.String.SET_DUE_BROWSER, - ).run_in_background() + ): + op.run_in_background() @ensure_editor_saved_on_trigger def forget_cards(self) -> None: diff --git a/qt/aqt/reviewer.py b/qt/aqt/reviewer.py index 62a968599..7d5c36a30 100644 --- a/qt/aqt/reviewer.py +++ b/qt/aqt/reviewer.py @@ -859,11 +859,12 @@ time = %(time)d; if self.mw.state != "review" or not self.card: return - set_due_date_dialog( + if op := set_due_date_dialog( parent=self.mw, card_ids=[self.card.id], config_key=Config.String.SET_DUE_REVIEWER, - ).run_in_background() + ): + op.run_in_background() def suspend_current_note(self) -> None: suspend_note(