Merge pull request #1727 from appwrite/fix-relationship-message

Fix: Message on relationship deletion

Author: stnguyen90

src/lib/components/trim.svelte

@@ -3,8 +3,8 @@
     import { throttle } from '$lib/helpers/functions';
     import { onMount } from 'svelte';
 
-    export let alternativeTrim = false;
     let showTooltip = false;
+    export let alternativeTrim = false;
     let container: HTMLSpanElement | null;
 
     onMount(onResize);
@@ -21,6 +21,7 @@
     bind:this={container}
     use:tooltip={{
         disabled: !showTooltip,
+        appendTo: 'parent',
         content: container?.innerText ?? undefined,
         maxWidth: '30rem'
     }}>

src/routes/(console)/project-[project]/databases/database-[database]/collection-[collection]/document-[document]/delete.svelte

@@ -2,9 +2,9 @@
     import { goto } from '$app/navigation';
     import { base } from '$app/paths';
     import { page } from '$app/stores';
-    import { Submit, trackEvent, trackError } from '$lib/actions/analytics';
     import { Alert, Modal, Trim } from '$lib/components';
     import { Button, InputChoice } from '$lib/elements/forms';
+    import { Submit, trackError, trackEvent } from '$lib/actions/analytics';
     import {
         TableBody,
         TableCell,
@@ -17,7 +17,7 @@
     import { addNotification } from '$lib/stores/notifications';
     import { sdk } from '$lib/stores/sdk';
     import { attributes, collection } from '../store';
-    import { isRelationship } from './attributes/store';
+    import { isRelationship, isRelationshipToMany } from './attributes/store';
     import type { Models } from '@appwrite.io/console';
 
     export let showDelete = false;
@@ -52,11 +52,20 @@
     enum Deletion {
         'setNull' = 'Set document ID as NULL in all related documents',
         'cascade' = 'All related documents will be deleted',
-        'restrict' = 'Document can not be deleted'
+        'restrict' = 'Document cannot be deleted'
     }
 
-    $: relAttributes = $attributes?.filter((attribute) =>
-        isRelationship(attribute)
+    $: relAttributes = $attributes?.filter(
+        (attribute) =>
+            isRelationship(attribute) &&
+            // One-to-One are always included
+            (attribute.relationType === 'oneToOne' ||
+                // One-to-Many: Only if parent is deleted
+                (attribute.relationType === 'oneToMany' && attribute.side === 'parent') ||
+                // Many-to-One: Only include if child is deleted
+                (attribute.relationType === 'manyToOne' && attribute.side === 'child') ||
+                // Many-to-Many: Only include if the parent is being deleted
+                (isRelationshipToMany(attribute) && attribute.side === 'parent'))
     ) as Models.AttributeRelationship[];
 </script>
 

src/routes/(console)/project-[project]/databases/database-[database]/collection-[collection]/table.svelte

@@ -152,8 +152,17 @@
         'restrict' = 'Document can not be deleted'
     }
 
-    $: relAttributes = $attributes?.filter((attribute) =>
-        isRelationship(attribute)
+    $: relAttributes = $attributes?.filter(
+        (attribute) =>
+            isRelationship(attribute) &&
+            // One-to-One are always included
+            (attribute.relationType === 'oneToOne' ||
+                // One-to-Many: Only if parent is deleted
+                (attribute.relationType === 'oneToMany' && attribute.side === 'parent') ||
+                // Many-to-One: Only include if child is deleted
+                (attribute.relationType === 'manyToOne' && attribute.side === 'child') ||
+                // Many-to-Many: Only include if the parent is being deleted
+                (isRelationshipToMany(attribute) && attribute.side === 'parent'))
     ) as Models.AttributeRelationship[];
 
     let checked = false;