From a72964ab3f04fe8b1d92f6e9ac81cb7c35d103a7 Mon Sep 17 00:00:00 2001
From: Tuna Celik <tuna@jakpark.com>
Date: Sat, 2 Mar 2024 19:44:10 +0100
Subject: [PATCH] learn to make tox happy

---
 radicale/app/base.py      | 4 ++--
 radicale/app/delete.py    | 4 ++--
 radicale/app/proppatch.py | 6 ++----
 radicale/hook/__init__.py | 3 ++-
 4 files changed, 8 insertions(+), 9 deletions(-)

diff --git a/radicale/app/base.py b/radicale/app/base.py
index 4faf52d1..f4a37025 100644
--- a/radicale/app/base.py
+++ b/radicale/app/base.py
@@ -21,8 +21,8 @@ import sys
 import xml.etree.ElementTree as ET
 from typing import Optional
 
-from radicale import (auth, hook, config, httputils, pathutils, rights, storage,
-                      types, web, xmlutils)
+from radicale import (auth, config, hook, httputils, pathutils, rights,
+                      storage, types, web, xmlutils)
 from radicale.log import logger
 
 # HACK: https://github.com/tiran/defusedxml/issues/54
diff --git a/radicale/app/delete.py b/radicale/app/delete.py
index 8335ee11..3d687065 100644
--- a/radicale/app/delete.py
+++ b/radicale/app/delete.py
@@ -91,7 +91,7 @@ class ApplicationPartDelete(ApplicationBase):
                 )
                 xml_answer = xml_delete(
                     base_prefix, path, item.collection, item.href)
-            for i in hook_notification_item_list:
-                self._hook.notify(i)
+            for notification_item in hook_notification_item_list:
+                self._hook.notify(notification_item)
             headers = {"Content-Type": "text/xml; charset=%s" % self._encoding}
             return client.OK, headers, self._xml_response(xml_answer)
diff --git a/radicale/app/proppatch.py b/radicale/app/proppatch.py
index f81e6fba..c15fddfe 100644
--- a/radicale/app/proppatch.py
+++ b/radicale/app/proppatch.py
@@ -22,16 +22,14 @@ import xml.etree.ElementTree as ET
 from http import client
 from typing import Dict, Optional, cast
 
+import defusedxml.ElementTree as DefusedET
+
 import radicale.item as radicale_item
 from radicale import httputils, storage, types, xmlutils
 from radicale.app.base import Access, ApplicationBase
 from radicale.hook import HookNotificationItem, HookNotificationItemTypes
 from radicale.log import logger
 
-# HACK: https://github.com/tiran/defusedxml/issues/54
-import defusedxml.ElementTree as DefusedET  # isort:skip
-sys.modules["xml.etree"].ElementTree = ET  # type:ignore[attr-defined]
-
 
 def xml_proppatch(base_prefix: str, path: str,
                   xml_request: Optional[ET.Element],
diff --git a/radicale/hook/__init__.py b/radicale/hook/__init__.py
index c856bdf5..dc6b74c5 100644
--- a/radicale/hook/__init__.py
+++ b/radicale/hook/__init__.py
@@ -1,9 +1,10 @@
 import json
 from enum import Enum
+from typing import Sequence
 
 from radicale import pathutils, utils
 
-INTERNAL_TYPES = ("none", "rabbitmq")
+INTERNAL_TYPES: Sequence[str] = ("none", "rabbitmq")
 
 
 def load(configuration):