From adb8be92a0302c9ed635b7e40d24ee6caa5c7e32 Mon Sep 17 00:00:00 2001 From: BlueGreenMagick <50060875+BlueGreenMagick@users.noreply.github.com> Date: Tue, 5 Jan 2021 19:50:54 +0900 Subject: [PATCH] format ask_delete_deck method more readable --- qt/aqt/deckbrowser.py | 21 ++++++++++----------- 1 file changed, 10 insertions(+), 11 deletions(-) diff --git a/qt/aqt/deckbrowser.py b/qt/aqt/deckbrowser.py index 6830f0375..87b1cec85 100644 --- a/qt/aqt/deckbrowser.py +++ b/qt/aqt/deckbrowser.py @@ -283,17 +283,16 @@ class DeckBrowser: def ask_delete_deck(self, did: int) -> bool: deck = self.mw.col.decks.get(did) - extra = None - if not deck["dyn"]: - count = self.mw.col.decks.card_count(did, include_subdecks=True) - if count: - extra = tr(TR.DECKS_IT_HAS_CARD, count=count) - if ( - deck["dyn"] - or not extra - or askUser( - (tr(TR.DECKS_ARE_YOU_SURE_YOU_WISH_TO, val=deck["name"])) + " " + extra - ) + if deck["dyn"]: + return True + + count = self.mw.col.decks.card_count(did, include_subdecks=True) + if not count: + return True + + extra = tr(TR.DECKS_IT_HAS_CARD, count=count) + if askUser( + tr(TR.DECKS_ARE_YOU_SURE_YOU_WISH_TO, val=deck["name"]) + " " + extra ): return True return False