Skip to content

Commit 92ab6cc

Browse files
made serializer code dry
1 parent 4b3436d commit 92ab6cc

File tree

1 file changed

+10
-13
lines changed

1 file changed

+10
-13
lines changed

sde_collections/serializers.py

Lines changed: 10 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -235,10 +235,12 @@ class Meta:
235235
)
236236

237237
def get_tdamm_tag(self, obj):
238+
empty_categories = {"messengers": [], "objects": [], "signals": []}
238239
if not obj.tdamm_tag or obj.tdamm_tag == ["NOT_TDAMM"]:
239-
return {"messengers": [], "objects": [], "signals": []}
240+
return empty_categories
240241

241-
categorized_tags = {"messengers": [], "objects": [], "signals": []}
242+
categories = empty_categories.copy()
243+
prefix_mapping = {"MMA_M_": "messengers", "MMA_O_": "objects", "MMA_S_": "signals"}
242244

243245
for tag in obj.tdamm_tag:
244246
if tag == "NOT_TDAMM":
@@ -248,17 +250,12 @@ def get_tdamm_tag(self, obj):
248250
if not tag_text:
249251
continue
250252

251-
if tag.startswith("MMA_M_"):
252-
transformed_tag = tag_text.replace(" - ", "/")
253-
categorized_tags["messengers"].append(transformed_tag)
254-
elif tag.startswith("MMA_O_"):
255-
transformed_tag = tag_text.replace(" - ", "/")
256-
categorized_tags["objects"].append(transformed_tag)
257-
elif tag.startswith("MMA_S_"):
258-
transformed_tag = tag_text.replace(" - ", "/")
259-
categorized_tags["signals"].append(transformed_tag)
260-
261-
return categorized_tags
253+
for prefix, category in prefix_mapping.items():
254+
if tag.startswith(prefix):
255+
categories[category].append(tag_text.replace(" - ", "/"))
256+
break
257+
258+
return categories
262259

263260
def get_document_type(self, obj):
264261
if obj.document_type is not None:

0 commit comments

Comments
 (0)