From 4e41429165fa49259533d83b413e1450c4965c46 Mon Sep 17 00:00:00 2001
From: NSBum <duncan.alan@me.com>
Date: Wed, 14 Jul 2021 21:22:10 -0400
Subject: [PATCH] Fixed deck config id actions (#273)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

The follow deck configuration actions were updated to conform to Anki’s current deck configuration access. Specifically, `decks.conf` is no longer available.

setDeckConfigId no longer check for existence of configuration id, because the access to the list of config id’s appears to be only through decks; and the config may theoretically be orphaned.
---
 plugin/__init__.py | 25 ++++++++++++++-----------
 1 file changed, 14 insertions(+), 11 deletions(-)

diff --git a/plugin/__init__.py b/plugin/__init__.py
index 36ce781..d18dff4 100644
--- a/plugin/__init__.py
+++ b/plugin/__init__.py
@@ -562,37 +562,40 @@ class AnkiConnect:
 
     @util.api()
     def setDeckConfigId(self, decks, configId):
-        configId = str(configId)
+        configId = int(configId)
         for deck in decks:
             if not deck in self.deckNames():
                 return False
 
         collection = self.collection()
-        if configId not in collection.decks.dconf:
-            return False
 
         for deck in decks:
-            did = str(collection.decks.id(deck))
-            aqt.mw.col.decks.decks[did]['conf'] = configId
+            try:
+                did = str(collection.decks.id(deck))
+                deck_dict = aqt.mw.col.decks.decks[did]
+                deck_dict['conf'] = configId
+                collection.decks.save(deck_dict)
+            except:
+                return False
 
         return True
 
 
     @util.api()
     def cloneDeckConfigId(self, name, cloneFrom='1'):
-        configId = str(cloneFrom)
-        if configId not in self.collection().decks.dconf:
+        configId = int(cloneFrom)
+        collection = self.collection()
+        if configId not in [c['id'] for c in collection.decks.all_config()]:
             return False
 
-        config = self.collection().decks.getConf(configId)
-        return self.collection().decks.confId(name, config)
+        config = collection.decks.getConf(configId)
+        return collection.decks.confId(name, config)
 
 
     @util.api()
     def removeDeckConfigId(self, configId):
-        configId = str(configId)
         collection = self.collection()
-        if configId not in collection.decks.dconf:
+        if int(configId) not in [c['id'] for c in collection.decks.all_config()]:
             return False
 
         collection.decks.remConf(configId)