[release] Bump to version 39.1

- Fix bug in tests for notes saving

PROJECT.org

@@ -480,6 +480,12 @@ whatever time KoReader reports, we need to know, given the date and the user
 profile's historic timezone, how many hours to adjust the KoReader time to get
 to GMT to save it in the database.
 
+* Version 39.1 [1/1]
+** DONE [#A] Fix bug in tests for notes saving :bug:scrobbles:forms:
+:PROPERTIES:
+:ID:       68a011b2-bb6f-3ba8-2312-5947c41db9ac
+:END:
+
 * Version 39.0 [3/3]
 ** DONE [#B] Clean up org-mode tasks metadata :bug:tasks:metadata:
 :PROPERTIES:

pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "vrobbler"
-version = "39.0"
+version = "39.1"
 description = ""
 authors = ["Colin Powell <colin@unbl.ink>"]
 

tests/scrobbles_tests/test_views.py

@@ -607,7 +607,8 @@ def test_scrobble_detail_view_post_updates_log(client):
 
     scrobble.refresh_from_db()
     assert scrobble.log["description"] == "Updated description"
-    assert scrobble.log["notes"] == ["Updated note"]
+    assert isinstance(scrobble.log["notes"], dict)
+    assert list(scrobble.log["notes"].values()) == ["Updated note"]
 
 
 @pytest.mark.skip("Need to refactor")

vrobbler/apps/scrobbles/forms.py

@@ -112,11 +112,14 @@ class NotesDictWidget(forms.Widget):
 
     def value_from_datadict(self, data, files, name):
         timestamps = data.getlist(f"{name}_timestamps")
-        contents = data.getlist(f"{name}_contents")
-        return {
-            "timestamps": timestamps,
-            "contents": contents,
-        }
+        if timestamps:
+            contents = data.getlist(f"{name}_contents")
+            result = {}
+            for i, ts in enumerate(timestamps):
+                if i < len(contents) and ts:
+                    result[ts] = contents[i]
+            return result if result else ""
+        return data.get(name, "")
 
     def get_context(self, name, value, attrs):
         context = super().get_context(name, value, attrs)
@@ -139,10 +142,14 @@ class NotesDictField(forms.Field):
     def clean(self, value):
         if not value:
             return {}
-        result = {}
-        timestamps = value.get("timestamps", [])
-        contents = value.get("contents", [])
-        for i, ts in enumerate(timestamps):
-            if i < len(contents) and ts and contents[i].strip():
-                result[ts] = contents[i].strip()
-        return result if result else {}
+
+        if isinstance(value, str):
+            if value.strip():
+                from datetime import datetime
+                return {datetime.now().isoformat(): value.strip()}
+            return {}
+
+        if isinstance(value, dict):
+            return value
+
+        return {}