Merge pull request #887 from BlueGreenMagick/deck-rename-checkpoint

don't set checkpoint when deck rename is cancelled
This commit is contained in:
Damien Elmes 2021-01-04 13:44:28 +10:00 committed by Damien Elmes
parent 5841fa8a83
commit da50e40249

View file

@ -90,13 +90,13 @@ class NewSidebarTreeView(SidebarTreeViewBase):
m.exec_(QCursor.pos()) m.exec_(QCursor.pos())
def rename_deck(self, item: "aqt.browser.SidebarItem") -> None: 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) deck = self.mw.col.decks.get(item.id)
old_name = deck["name"] old_name = deck["name"]
new_name = getOnlyText(tr(TR.DECKS_NEW_DECK_NAME), default=old_name) new_name = getOnlyText(tr(TR.DECKS_NEW_DECK_NAME), default=old_name)
new_name = new_name.replace('"', "") new_name = new_name.replace('"', "")
if not new_name or new_name == old_name: if not new_name or new_name == old_name:
return return
self.mw.checkpoint(tr(TR.ACTIONS_RENAME_DECK))
try: try:
self.mw.col.decks.rename(deck, new_name) self.mw.col.decks.rename(deck, new_name)
self.browser.maybeRefreshSidebar() self.browser.maybeRefreshSidebar()