UBER-408 Issue as task (#3497)

Signed-off-by: Denis Bykhov <bykhov.denis@gmail.com>

models/task/src/index.ts

@@ -15,6 +15,8 @@
 
 import type { Employee } from '@hcengineering/contact'
 import contact from '@hcengineering/contact'
+import attachment from '@hcengineering/model-attachment'
+import chunter from '@hcengineering/model-chunter'
 import { Arr, Attribute, Class, Doc, Domain, IndexKind, Ref, Space, Status, Timestamp } from '@hcengineering/core'
 import {
   Builder,
@@ -88,8 +90,8 @@ export class TLostState extends TDoneState implements LostState {}
 @Model(task.class.Task, core.class.AttachedDoc, DOMAIN_TASK)
 @UX(task.string.Task, task.icon.Task, task.string.Task)
 export class TTask extends TAttachedDoc implements Task {
-  @Prop(TypeRef(task.class.State), task.string.TaskState, { _id: task.attribute.State })
-    state!: Ref<State>
+  @Prop(TypeRef(core.class.Status), task.string.TaskState, { _id: task.attribute.State })
+    status!: Ref<Status>
 
   @Prop(TypeRef(task.class.DoneState), task.string.TaskStateDone, { _id: task.attribute.DoneState })
     doneState!: Ref<DoneState> | null
@@ -105,13 +107,16 @@ export class TTask extends TAttachedDoc implements Task {
   @Prop(TypeDate(), task.string.DueDate, { editor: task.component.DueDateEditor })
     dueDate!: Timestamp | null
 
-  @Prop(TypeDate(), task.string.StartDate)
-    startDate!: Timestamp | null
-
   declare rank: string
 
   @Prop(Collection(tags.class.TagReference, task.string.TaskLabels), task.string.TaskLabels)
-    labels!: number
+    labels?: number
+
+  @Prop(Collection(chunter.class.Comment), chunter.string.Comments)
+    comments?: number
+
+  @Prop(Collection(attachment.class.Attachment), attachment.string.Attachments, { shortLabel: attachment.string.Files })
+    attachments?: number
 }
 
 @Model(task.class.TodoItem, core.class.AttachedDoc, DOMAIN_TASK)

models/task/src/migration.ts

@@ -13,11 +13,13 @@
 // limitations under the License.
 //
 
-import { Class, Doc, Domain, Ref, Space, TxOperations } from '@hcengineering/core'
+import { Class, DOMAIN_TX, Doc, Domain, Ref, Space, TxOperations } from '@hcengineering/core'
 import { MigrateOperation, MigrationClient, MigrationUpgradeClient, createOrUpdate } from '@hcengineering/model'
 import core from '@hcengineering/model-core'
 import tags from '@hcengineering/model-tags'
 import { DoneStateTemplate, KanbanTemplate, StateTemplate, genRanks } from '@hcengineering/task'
+import view, { Filter } from '@hcengineering/view'
+import { DOMAIN_TASK } from '.'
 import task from './plugin'
 
 /**
@@ -125,8 +127,72 @@ async function createDefaults (tx: TxOperations): Promise<void> {
   await createDefaultSequence(tx)
 }
 
+async function renameState (client: MigrationClient): Promise<void> {
+  const toUpdate = await client.find(DOMAIN_TASK, { state: { $exists: true } })
+  if (toUpdate.length > 0) {
+    for (const doc of toUpdate) {
+      await client.update(
+        DOMAIN_TX,
+        { objectId: doc._id },
+        { $rename: { 'attributes.state': 'attributes.status', 'operations.state': 'operations.status' } }
+      )
+      await client.update(
+        DOMAIN_TX,
+        { 'tx.objectId': doc._id },
+        { $rename: { 'tx.attributes.state': 'tx.attributes.status', 'tx.operations.state': 'tx.operations.status' } }
+      )
+    }
+    await client.update(DOMAIN_TASK, { _id: { $in: toUpdate.map((p) => p._id) } }, { $rename: { state: 'status' } })
+  }
+}
+
+async function renameStatePrefs (client: MigrationUpgradeClient): Promise<void> {
+  const txop = new TxOperations(client, core.account.System)
+  const prefs = await client.findAll(view.class.ViewletPreference, {})
+  for (const pref of prefs) {
+    let update = false
+    const config = pref.config
+    for (let index = 0; index < config.length; index++) {
+      const conf = config[index]
+      if (typeof conf === 'string') {
+        if (conf === 'state') {
+          config[index] = 'status'
+          update = true
+        } else if (conf === '$lookup.state') {
+          config[index] = '$lookup.status'
+          update = true
+        }
+      } else if (conf.key === 'state') {
+        conf.key = 'status'
+        update = true
+      }
+    }
+    if (update) {
+      await txop.update(pref, {
+        config
+      })
+    }
+  }
+  const res = await client.findAll(view.class.FilteredView, { filters: /"key":"state"/ as any })
+  if (res.length > 0) {
+    for (const doc of res) {
+      const filters = JSON.parse(doc.filters) as Filter[]
+      for (const filter of filters) {
+        if (filter.key.key === 'state') {
+          filter.key.key = 'status'
+        }
+      }
+      await txop.update(doc, {
+        filters: JSON.stringify(filters)
+      })
+    }
+  }
+}
+
 export const taskOperation: MigrateOperation = {
-  async migrate (client: MigrationClient): Promise<void> {},
+  async migrate (client: MigrationClient): Promise<void> {
+    await renameState(client)
+  },
   async upgrade (client: MigrationUpgradeClient): Promise<void> {
     const tx = new TxOperations(client, core.account.System)
     await createDefaults(tx)
@@ -144,5 +210,6 @@ export const taskOperation: MigrateOperation = {
       },
       task.category.TaskTag
     )
+    await renameStatePrefs(client)
   }
 }