From 66e1f876d7341da050e1e5c57fc6fd74422de700 Mon Sep 17 00:00:00 2001 From: Arthur Milchior Date: Thu, 23 Jan 2020 21:46:42 -0800 Subject: [PATCH] "unused" variable become optional. Adding this parameter broke one of my add-on, see https://github.com/Arthur-Milchior/anki-keep-model-in-add-cards/issues/1 Since those parameters are not used, setting them to None by default, as was done in some other method, seems acceptable --- qt/aqt/addcards.py | 2 +- qt/aqt/deckchooser.py | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/qt/aqt/addcards.py b/qt/aqt/addcards.py index 91c1fe12e..be6308e30 100644 --- a/qt/aqt/addcards.py +++ b/qt/aqt/addcards.py @@ -89,7 +89,7 @@ class AddCards(QDialog): def setAndFocusNote(self, note: Note) -> None: self.editor.setNote(note, focusTo=0) - def onModelChange(self, unused) -> None: + def onModelChange(self, unused=None) -> None: oldNote = self.editor.note note = self.mw.col.newNote() self.previousNote = None diff --git a/qt/aqt/deckchooser.py b/qt/aqt/deckchooser.py index c62ea1031..a7cf5863e 100644 --- a/qt/aqt/deckchooser.py +++ b/qt/aqt/deckchooser.py @@ -66,7 +66,7 @@ class DeckChooser(QHBoxLayout): def cleanup(self) -> None: gui_hooks.current_note_type_did_change.remove(self.onModelChangeNew) - def onModelChangeNew(self, unused): + def onModelChangeNew(self, unused=None): self.onModelChange() def onModelChange(self):