Cards fixes (#8186)

hcengineering · Mar 11, 2025 · a89f75a · a89f75a
1 parent 92273f5
commit a89f75a
Show file tree

Hide file tree

Showing 20 changed files with 152 additions and 75 deletions.
diff --git a/models/card/src/index.ts b/models/card/src/index.ts
@@ -16,13 +16,12 @@ import {
   CardEvents,
   cardId,
   DOMAIN_CARD,
-  type ParentInfo,
   type Card,
   type MasterTag,
+  type ParentInfo,
   type Tag
 } from '@hcengineering/card'
 import chunter from '@hcengineering/chunter'
-import contact from '@hcengineering/contact'
 import core, {
   AccountRole,
   DOMAIN_MODEL,
@@ -368,8 +367,7 @@ export function createModel (builder: Builder): void {
       name: 'tagrelation',
       label: card.string.TagRelations,
       icon: setting.icon.Relations,
-      props: { _classes: [card.class.Card, contact.class.Contact], exclude: [] },
-      component: setting.component.RelationSetting,
+      component: card.component.RelationSetting,
       group: 'settings-editor',
       role: AccountRole.Maintainer,
       order: 4501
@@ -381,7 +379,7 @@ export function createModel (builder: Builder): void {
     setting.class.SettingsCategory,
     core.space.Model,
     {
-      name: 'masterTags',
+      name: 'types',
       label: card.string.MasterTags,
       icon: card.icon.Card,
       component: card.component.ManageMasterTagsContent,

diff --git a/models/server-card/src/index.ts b/models/server-card/src/index.ts
@@ -77,12 +77,11 @@ export function createModel (builder: Builder): void {
   })
 
   builder.createDoc(serverCore.class.Trigger, core.space.Model, {
-    trigger: serverCard.trigger.OnCardParentChange,
+    trigger: serverCard.trigger.OnCardUpdate,
     isAsync: true,
     txMatch: {
       _class: core.class.TxUpdateDoc,
-      objectClass: card.class.Card,
-      operations: { parent: { $exists: true } }
+      objectClass: card.class.Card
     }
   })
 

diff --git a/plugins/card-assets/lang/de.json b/plugins/card-assets/lang/de.json
@@ -5,10 +5,10 @@
     "Cards": "Karten",
     "Content": "Inhalt",
     "CreateCard": "Karte erstellen",
-    "CreateMasterTag": "Master-Tag erstellen",
+    "CreateMasterTag": "Typ erstellen",
     "CreateTag": "Tag erstellen",
-    "MasterTag": "Master-Tag",
-    "MasterTags": "Master-Tags",
+    "MasterTag": "Typ",
+    "MasterTags": "Typen",
     "Parent": "Eltern",
     "Tags": "Tags",
     "Tag": "Tag",
@@ -18,11 +18,11 @@
     "TagRelations": "Tag-Beziehungen",
     "DeleteTag": "Tag löschen",
     "DeleteTagConfirm": "Möchten Sie diesen Tag wirklich löschen? Alle zugehörigen Eigenschaften werden gelöscht",
-    "DeleteMasterTag": "Master-Tag löschen",
-    "DeleteMasterTagConfirm": "Möchten Sie diesen Master-Tag wirklich löschen? Alle mit diesem Master-Tag verbundenen Objekte werden gelöscht",
+    "DeleteMasterTag": "Typ löschen",
+    "DeleteMasterTagConfirm": "Möchten Sie diesen Typ wirklich löschen? Alle mit diesem Typ verbundenen Objekte werden gelöscht",
     "UnsetParent": "Elternteil entfernen",
     "SetParent": "Elternteil setzen",
     "CreateChild": "Kind erstellen",
     "Children": "Kinder"
   }
-}
+}
diff --git a/plugins/card-resources/src/components/CardPresenter.svelte b/plugins/card-resources/src/components/CardPresenter.svelte
@@ -40,6 +40,9 @@
 {:else if value}
   {#if type === 'link'}
     <div class="flex-row-center">
+      {#if showParent}
+        <ParentNamesPresenter {value} />
+      {/if}
       <DocNavLink
         object={value}
         {onClick}
@@ -58,9 +61,6 @@
           {/if}
           <span class="overflow-label" class:select-text={!noSelect} title={value?.title}>
             {value.title}
-            {#if showParent}
-              <ParentNamesPresenter {value} />
-            {/if}
             <slot name="details" />
           </span>
         </span>

diff --git a/plugins/card-resources/src/components/MasterTagAttributes.svelte b/plugins/card-resources/src/components/MasterTagAttributes.svelte
@@ -74,7 +74,7 @@
         ev.stopPropagation()
         const loc = getCurrentResolvedLocation()
         loc.path[2] = settingId
-        loc.path[3] = 'masterTags'
+        loc.path[3] = 'types'
         loc.path[4] = value._class
         loc.path.length = 5
         loc.fragment = undefined

diff --git a/plugins/card-resources/src/components/ParentNamesPresenter.svelte b/plugins/card-resources/src/components/ParentNamesPresenter.svelte
@@ -72,7 +72,7 @@
       &:active {
         color: var(--theme-content-color);
       }
-      &::before {
+      &::after {
         content: '›';
         padding: 0 0.25rem;
       }

diff --git a/plugins/card-resources/src/components/SetParentActionPopup.svelte b/plugins/card-resources/src/components/SetParentActionPopup.svelte
@@ -14,7 +14,7 @@
 -->
 <script lang="ts">
   import { Card } from '@hcengineering/card'
-  import { FindOptions, SortingOrder } from '@hcengineering/core'
+  import { Doc, FindOptions, SortingOrder } from '@hcengineering/core'
   import { ObjectPopup, getClient } from '@hcengineering/presentation'
   import { createEventDispatcher } from 'svelte'
   import card from '../plugin'
@@ -48,6 +48,13 @@
 
   $: selected = !Array.isArray(value) ? value.parent ?? undefined : undefined
   $: ignoreObjects = !Array.isArray(value) ? [value._id] : undefined
+
+  $: cards = new Set(Array.isArray(value) ? value.map((p) => p._id) : [value._id])
+
+  const filter = (it: Doc): boolean => {
+    const card = it as Card
+    return !card.parentInfo.some((p) => cards.has(p._id))
+  }
 </script>
 
 <ObjectPopup
@@ -59,6 +66,7 @@
   allowDeselect={true}
   placeholder={card.string.SetParent}
   create={undefined}
+  {filter}
   {ignoreObjects}
   shadows={true}
   {width}

diff --git a/plugins/card-resources/src/components/TagAttributes.svelte b/plugins/card-resources/src/components/TagAttributes.svelte
@@ -81,7 +81,7 @@
         ev.stopPropagation()
         const loc = getCurrentResolvedLocation()
         loc.path[2] = settingId
-        loc.path[3] = 'masterTags'
+        loc.path[3] = 'types'
         loc.path[4] = tag._id
         loc.path.length = 5
         loc.fragment = undefined

diff --git a/plugins/card-resources/src/components/TagHierarchy.svelte b/plugins/card-resources/src/components/TagHierarchy.svelte
@@ -17,7 +17,7 @@
   import { Class, Doc, Ref } from '@hcengineering/core'
   import { getClient } from '@hcengineering/presentation'
   import { NavItem } from '@hcengineering/ui'
-  import { NavLink, showMenu } from '@hcengineering/view-resources'
+  import { NavLink } from '@hcengineering/view-resources'
   import { createEventDispatcher } from 'svelte'
   import card from '../plugin'
 
@@ -67,9 +67,6 @@
       on:click={() => {
         dispatch('select', clazz._id)
       }}
-      on:contextmenu={(evt) => {
-        showMenu(evt, { object: clazz })
-      }}
     />
   </NavLink>
   {#if (descendants.get(clazz._id)?.length ?? 0) > 0}

diff --git a/plugins/card-resources/src/components/settings/CreateRelation.svelte b/plugins/card-resources/src/components/settings/CreateRelation.svelte
@@ -0,0 +1,17 @@
+<script lang="ts">
+  import contact from '@hcengineering/contact'
+  import { Class, Doc, Ref } from '@hcengineering/core'
+  import { getClient } from '@hcengineering/presentation'
+  import { CreateRelation } from '@hcengineering/setting-resources'
+  import card from '../../plugin'
+
+  export let aClass: Ref<Class<Doc>> | undefined = undefined
+
+  const client = getClient()
+  const hierarchy = client.getHierarchy()
+  const _classes = [...hierarchy.getDescendants(card.class.Card), contact.class.Contact].filter(
+    (c) => c !== card.class.Card
+  )
+</script>
+
+<CreateRelation {aClass} {_classes} exclude={[]} on:close />
diff --git a/plugins/card-resources/src/components/settings/ManageMasterTagsContent.svelte b/plugins/card-resources/src/components/settings/ManageMasterTagsContent.svelte
@@ -35,7 +35,7 @@
   onDestroy(resolvedLocationStore.subscribe(handleLocationChanged))
 
   function handleLocationChanged ({ path }: Location): void {
-    if (path[3] !== 'masterTags' || path[4] === undefined) {
+    if (path[3] !== 'types' || path[4] === undefined) {
       selectedTagId = undefined
     } else {
       selectedTagId = path[4] as Ref<MasterTag>

diff --git a/plugins/card-resources/src/components/settings/RelationSetting.svelte b/plugins/card-resources/src/components/settings/RelationSetting.svelte
@@ -0,0 +1,14 @@
+<script lang="ts">
+  import { getClient } from '@hcengineering/presentation'
+  import { RelationSetting } from '@hcengineering/setting-resources'
+  import contact from '@hcengineering/contact'
+  import card from '../../plugin'
+
+  const client = getClient()
+  const hierarchy = client.getHierarchy()
+  const _classes = [...hierarchy.getDescendants(card.class.Card), contact.class.Contact].filter(
+    (c) => c !== card.class.Card
+  )
+</script>
+
+<RelationSetting {_classes} exclude={[]} />
diff --git a/plugins/card-resources/src/components/settings/RelationsSection.svelte b/plugins/card-resources/src/components/settings/RelationsSection.svelte
@@ -13,15 +13,15 @@
 // limitations under the License.
 -->
 <script lang="ts">
-  import card, { MasterTag } from '@hcengineering/card'
-  import contact from '@hcengineering/contact'
+  import { MasterTag } from '@hcengineering/card'
   import core, { Association, Class, Doc, Ref } from '@hcengineering/core'
   import { IntlString } from '@hcengineering/platform'
   import { createQuery, getClient } from '@hcengineering/presentation'
   import setting from '@hcengineering/setting'
   import { clearSettingsStore, settingsStore } from '@hcengineering/setting-resources'
   import { ButtonIcon, Icon, IconAdd, Label, showPopup } from '@hcengineering/ui'
   import { onDestroy } from 'svelte'
+  import CreateRelation from './CreateRelation.svelte'
 
   export let masterTag: MasterTag
 
@@ -49,10 +49,8 @@
   }
 
   function addRelation (): void {
-    showPopup(setting.component.CreateRelation, {
-      aClass: masterTag._id,
-      exclude: [],
-      _classes: [card.class.Card, contact.class.Contact]
+    showPopup(CreateRelation, {
+      aClass: masterTag._id
     })
   }
 

diff --git a/plugins/card-resources/src/index.ts b/plugins/card-resources/src/index.ts
@@ -35,6 +35,7 @@ import TagsSection from './components/settings/TagsSection.svelte'
 import RelationsSection from './components/settings/RelationsSection.svelte'
 import ChildsSection from './components/settings/ChildsSection.svelte'
 import SetParentActionPopup from './components/SetParentActionPopup.svelte'
+import RelationSetting from './components/settings/RelationSetting.svelte'
 
 export default async (): Promise<Resources> => ({
   component: {
@@ -51,7 +52,8 @@ export default async (): Promise<Resources> => ({
     TagsSection,
     RelationsSection,
     ChildsSection,
-    SetParentActionPopup
+    SetParentActionPopup,
+    RelationSetting
   },
   completion: {
     CardQuery: queryCard

diff --git a/plugins/card-resources/src/plugin.ts b/plugins/card-resources/src/plugin.ts
@@ -35,7 +35,8 @@ export default mergeIds(cardId, card, {
     TagsSection: '' as AnyComponent,
     ChildsSection: '' as AnyComponent,
     RelationsSection: '' as AnyComponent,
-    SetParentActionPopup: '' as AnyComponent
+    SetParentActionPopup: '' as AnyComponent,
+    RelationSetting: '' as AnyComponent
   },
   completion: {
     CardQuery: '' as Resource<ObjectSearchFactory>,

diff --git a/plugins/setting-resources/src/components/CreateRelation.svelte b/plugins/setting-resources/src/components/CreateRelation.svelte
@@ -82,7 +82,9 @@
         const clazz = hierarchy.getClass(key)
         result.push([
           { id: key, label: clazz.label, icon: clazz.icon },
-          value.map((it) => ({ id: it._id, label: it.label, icon: it.icon }))
+          value
+            .map((it) => ({ id: it._id, label: it.label, icon: it.icon }))
+            .sort((a, b) => a.label.localeCompare(b.label))
         ])
       } catch {}
     }

diff --git a/plugins/setting-resources/src/index.ts b/plugins/setting-resources/src/index.ts
@@ -75,7 +75,9 @@ export {
   ClassSetting,
   filterDescendants,
   SpaceTypeGeneralSectionEditor,
-  ClassHierarchy
+  ClassHierarchy,
+  RelationSetting,
+  CreateRelation
 }
 
 async function DeleteMixin (object: Mixin<Class<Doc>>): Promise<void> {

diff --git a/plugins/view-resources/src/components/RelationEditor.svelte b/plugins/view-resources/src/components/RelationEditor.svelte
@@ -48,7 +48,7 @@
   $: query.query(
     view.class.Viewlet,
     {
-      attachTo: _class
+      attachTo: client.getHierarchy().getBaseClass(_class)
     },
     (res) => {
       viewlet = res[0]