fix: Library uuid replaced by name in matrix view (#1198)

intuitem · Dec 20, 2024 · e5b1b38 · e5b1b38
2 parents 5aaa64b + 585d429
commit e5b1b38
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 7 deletions.
diff --git a/backend/core/serializers.py b/backend/core/serializers.py
@@ -119,6 +119,7 @@ class AssessmentReadSerializer(BaseModelSerializer):
 class RiskMatrixReadSerializer(ReferentialSerializer):
     folder = FieldsRelatedField()
     json_definition = serializers.JSONField(source="get_json_translated")
+    library = FieldsRelatedField(["name", "id"])
 
     class Meta:
         model = RiskMatrix

diff --git a/frontend/src/lib/utils/crud.ts b/frontend/src/lib/utils/crud.ts
@@ -178,7 +178,8 @@ export const URL_MODEL_MAP: ModelMap = {
 		verboseName: 'Risk matrix',
 		verboseNamePlural: 'Risk matrices',
 		foreignKeyFields: [
-			{ field: 'folder', urlModel: 'folders', urlParams: 'content_type=DO&content_type=GL' }
+			{ field: 'folder', urlModel: 'folders', urlParams: 'content_type=DO&content_type=GL' },
+			{ field: 'library', urlModel: 'libraries' }
 		]
 	},
 	'risk-assessments': {

diff --git a/frontend/src/routes/(app)/(internal)/risk-matrices/[id=uuid]/+page.svelte b/frontend/src/routes/(app)/(internal)/risk-matrices/[id=uuid]/+page.svelte
@@ -38,12 +38,17 @@
 									{/each}
 								</ul>
 							{:else if value.id}
-								{@const itemHref = `/${
-									URL_MODEL_MAP['risk-matrices']['foreignKeyFields']?.find(
-										(item) => item.field === key
-									)?.urlModel
-								}/${value.id}`}
-								<a href={itemHref} class="anchor">{value.str}</a>
+								{#if key === 'library'}
+									{@const itemHref = `/libraries/${value.id}?loaded`}
+									<a href={itemHref} class="anchor">{value.name}</a>
+								{:else}
+									{@const itemHref = `/${
+										URL_MODEL_MAP['risk-matrices']['foreignKeyFields']?.find(
+											(item) => item.field === key
+										)?.urlModel
+									}/${value.id}`}
+									<a href={itemHref} class="anchor">{value.str}</a>
+								{/if}
 							{:else}
 								{value.str ?? value}
 							{/if}