From 0d6ebd25e6328444beecd766c029cddce3b723f5 Mon Sep 17 00:00:00 2001 From: Felix Stupp Date: Sun, 2 Jul 2023 17:35:41 +0200 Subject: [PATCH] UriHolder: disable uri_set setter due to already known reason --- server/entertainment_decider/models/extras/uris.py | 13 +++++++++---- 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/server/entertainment_decider/models/extras/uris.py b/server/entertainment_decider/models/extras/uris.py index 4d49f21..d0124be 100644 --- a/server/entertainment_decider/models/extras/uris.py +++ b/server/entertainment_decider/models/extras/uris.py @@ -97,11 +97,16 @@ class UriHolder: @uri_set.setter def uri_set(self, uri_list: Iterable[Optional[str]]) -> None: - self._clear_uri_set() - self.add_uris(uri_list) + """ + uri_set setter cannot be implemented for now! - # uri_set has no setter due to the problem which uri then becomes primary - # instead, set_as_only_uri & add_uris should be used so the primary becomes obvious + uri_set has no setter due to the problem which uri then becomes primary. + Instead, set_as_only_uri & add_uris should be used so the primary becomes obvious. + In future, when no primary uri is required, it will be implemented. + """ + raise NotImplementedError( + "UriHolder.uri_set setter cannot be implemented (for now)", + ) def is_primary_uri(self, compare_uri: str) -> bool: """Returns True if the given uri is equal to the current primary uri."""