fix(Metadata): generic AttributeMetadata

wmakeev · Aug 24, 2022 · 06b1641 · 06b1641
1 parent 3fd992f
commit 06b1641
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 18 deletions.
diff --git a/src/model/Metadata.ts b/src/model/Metadata.ts
@@ -21,26 +21,25 @@ export type MetadataMeta<T extends MetadataMetaType = MetadataMetaType> = {
   }
 }
 
-export type AttributeMetadata = Entity<'attributemetadata'> & {
-  /** Наименование пользовательского поля */
-  readonly name: string
+export type AttributeMetadata<T extends AttributeType = AttributeType> =
+  Entity<'attributemetadata'> & {
+    /** Наименование пользовательского поля */
+    readonly name: string
 
-  readonly required: boolean
+    readonly required: boolean
 
-  // TODO Только для документов (нужно разделять ради этого?)
-  readonly show: boolean
-} & (
-    | {
-        /** Тип значения пользовательского поля */
-        readonly type: 'customentity'
-
-        readonly customEntityMeta: Meta<'customentitymetadata'>
-      }
-    | {
-        /** Тип значения пользовательского поля */
-        readonly type: Exclude<AttributeType, 'customentity'>
-      }
-  )
+    // TODO Только для документов (нужно разделять ради этого?)
+    readonly show: boolean
+  } & (T extends 'customentity'
+      ? {
+          /** Тип значения пользовательского поля */
+          readonly type: T
+          readonly customEntityMeta: Meta<'customentitymetadata'>
+        }
+      : {
+          /** Тип значения пользовательского поля */
+          readonly type: T
+        })
 
 export type DocumentMetadata<T extends DocumentMetaType = DocumentMetaType> =
   MetadataMeta<T> & {

diff --git a/test/model/Metadata.test-d.ts b/test/model/Metadata.test-d.ts
@@ -74,3 +74,8 @@ t6.customEntityMeta.type
 if (t6.type === 'customentity') {
   expectType<'customentitymetadata'>(t6.customEntityMeta.type)
 }
+
+const t7 = {} as AttributeMetadata<'customentity'>
+
+expectType<'customentity'>(t7.type)
+expectType<'customentitymetadata'>(t7.customEntityMeta.type)