From da50e40249ff381a8afe9ab48e8f959ac770afdd Mon Sep 17 00:00:00 2001 From: Damien Elmes Date: Mon, 4 Jan 2021 13:44:28 +1000 Subject: [PATCH] Merge pull request #887 from BlueGreenMagick/deck-rename-checkpoint don't set checkpoint when deck rename is cancelled --- qt/aqt/sidebar.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/qt/aqt/sidebar.py b/qt/aqt/sidebar.py index 2f5e9a839..683bdf707 100644 --- a/qt/aqt/sidebar.py +++ b/qt/aqt/sidebar.py @@ -90,13 +90,13 @@ class NewSidebarTreeView(SidebarTreeViewBase): m.exec_(QCursor.pos()) def rename_deck(self, item: "aqt.browser.SidebarItem") -> None: - self.mw.checkpoint(tr(TR.ACTIONS_RENAME_DECK)) deck = self.mw.col.decks.get(item.id) old_name = deck["name"] new_name = getOnlyText(tr(TR.DECKS_NEW_DECK_NAME), default=old_name) new_name = new_name.replace('"', "") if not new_name or new_name == old_name: return + self.mw.checkpoint(tr(TR.ACTIONS_RENAME_DECK)) try: self.mw.col.decks.rename(deck, new_name) self.browser.maybeRefreshSidebar()