From 6bb8bf5e1b56094a10cfa0059152c1f1695e6dcb Mon Sep 17 00:00:00 2001
From: Andrey Sobolev <haiodo@users.noreply.github.com>
Date: Fri, 3 Jun 2022 11:24:38 +0700
Subject: [PATCH] Fix #1947 (#1993)

Signed-off-by: Andrey Sobolev <haiodo@gmail.com>
---
 .gitignore                                    |  3 +-
 models/recruit/src/index.ts                   | 40 ++++++------
 models/recruit/src/plugin.ts                  |  3 +-
 .../presentation/src/components/Card.svelte   |  9 ++-
 plugins/recruit-assets/lang/en.json           |  2 +-
 .../src/components/CreateApplication.svelte   | 19 ++++++
 plugins/view-resources/src/actions.ts         |  9 ++-
 plugins/view/src/index.ts                     |  4 ++
 tests/sanity/tests/actions.spec.ts            | 64 +++++++++++++++++++
 tests/sanity/tests/recruit.spec.ts            |  2 +-
 10 files changed, 131 insertions(+), 24 deletions(-)
 create mode 100644 tests/sanity/tests/actions.spec.ts

diff --git a/.gitignore b/.gitignore
index fb78ca92f0..18efe70935 100644
--- a/.gitignore
+++ b/.gitignore
@@ -71,4 +71,5 @@ common/autoinstallers/*/.npmrc
 bundle.js
 dist
 tsconfig.tsbuildinfo
-ingest-attachment-*.zip
\ No newline at end of file
+ingest-attachment-*.zip
+tsdoc-metadata.json
\ No newline at end of file
diff --git a/models/recruit/src/index.ts b/models/recruit/src/index.ts
index 919de6078e..223503aa1a 100644
--- a/models/recruit/src/index.ts
+++ b/models/recruit/src/index.ts
@@ -389,9 +389,9 @@ export function createModel (builder: Builder): void {
     input: 'focus',
     category: recruit.category.Recruit,
     target: contact.class.Person,
-    context: { mode: ['context', 'browser'] }
+    context: { mode: ['context', 'browser'] },
+    override: [recruit.action.CreateGlobalApplication]
   })
-
   createAction(builder, {
     action: view.actionImpl.ShowPopup,
     actionProps: {
@@ -428,23 +428,27 @@ export function createModel (builder: Builder): void {
     }
   })
 
-  createAction(builder, {
-    action: view.actionImpl.ShowPopup,
-    actionProps: {
-      component: recruit.component.CreateApplication,
-      element: 'top'
+  createAction(
+    builder,
+    {
+      action: view.actionImpl.ShowPopup,
+      actionProps: {
+        component: recruit.component.CreateApplication,
+        element: 'top'
+      },
+      label: recruit.string.CreateApplication,
+      icon: recruit.icon.Create,
+      keyBinding: [],
+      input: 'none',
+      category: recruit.category.Recruit,
+      target: core.class.Doc,
+      context: {
+        mode: ['workbench', 'browser'],
+        application: recruit.app.Recruit
+      }
     },
-    label: recruit.string.CreateApplication,
-    icon: recruit.icon.Create,
-    keyBinding: [],
-    input: 'none',
-    category: recruit.category.Recruit,
-    target: core.class.Doc,
-    context: {
-      mode: ['workbench', 'browser'],
-      application: recruit.app.Recruit
-    }
-  })
+    recruit.action.CreateGlobalApplication
+  )
 
   builder.createDoc(
     task.class.KanbanTemplateSpace,
diff --git a/models/recruit/src/plugin.ts b/models/recruit/src/plugin.ts
index 2cc91b231e..0519cbae3d 100644
--- a/models/recruit/src/plugin.ts
+++ b/models/recruit/src/plugin.ts
@@ -25,7 +25,8 @@ import type { Action, ActionCategory, ViewAction } from '@anticrm/view'
 
 export default mergeIds(recruitId, recruit, {
   action: {
-    CreateOpinion: '' as Ref<Action>
+    CreateOpinion: '' as Ref<Action>,
+    CreateGlobalApplication: '' as Ref<Action>
   },
   actionImpl: {
     CreateOpinion: '' as ViewAction
diff --git a/packages/presentation/src/components/Card.svelte b/packages/presentation/src/components/Card.svelte
index 588fafa079..aaf39e26d1 100644
--- a/packages/presentation/src/components/Card.svelte
+++ b/packages/presentation/src/components/Card.svelte
@@ -36,10 +36,17 @@
         <slot name="header" />
         <span class="antiCard-header__divider">›</span>
       {/if}
-      <span class="antiCard-header__title"><Label {label} params={labelProps ?? {}} /></span>
+      <span class="antiCard-header__title">
+        {#if $$slots.title}
+          <slot name="title" />
+        {:else}
+          <Label {label} params={labelProps ?? {}} />
+        {/if}
+      </span>
     </div>
     <div class="buttons-group small-gap">
       <Button
+        id={'card-close'}
         focusIndex={10002}
         icon={IconClose}
         kind={'transparent'}
diff --git a/plugins/recruit-assets/lang/en.json b/plugins/recruit-assets/lang/en.json
index 370048aada..0687b8a3cb 100644
--- a/plugins/recruit-assets/lang/en.json
+++ b/plugins/recruit-assets/lang/en.json
@@ -14,7 +14,7 @@
     "MakePrivateDescription": "Only members can see it",
     "CreateAnApplication": "New Application",
     "NoApplicationsForCandidate": "There are no applications for this candidate.",
-    "CreateApplication": "Create an Application",
+    "CreateApplication": "New Application",
     "ApplicationCreateLabel": "Application",
     "SelectVacancy": "Select vacancy",
     "Candidate": "Candidate",
diff --git a/plugins/recruit-resources/src/components/CreateApplication.svelte b/plugins/recruit-resources/src/components/CreateApplication.svelte
index a6625a06d5..2887678518 100644
--- a/plugins/recruit-resources/src/components/CreateApplication.svelte
+++ b/plugins/recruit-resources/src/components/CreateApplication.svelte
@@ -27,6 +27,7 @@
     eventToHTMLElement,
     FocusHandler,
     getPlatformColor,
+    Label,
     showPopup,
     Status as StatusControl
   } from '@anticrm/ui'
@@ -223,6 +224,24 @@
       bind:space={doc.space}
     />
   </svelte:fragment>
+  <svelte:fragment slot="title">
+    <div class="flex-row-center gap-2">
+      {#if preserveCandidate}
+        <UserBox
+          readonly
+          _class={contact.class.Person}
+          options={{ sort: { modifiedOn: -1 } }}
+          excluded={existingApplicants}
+          label={recruit.string.Candidate}
+          placeholder={recruit.string.Candidates}
+          bind:value={doc.attachedTo}
+          kind={'no-border'}
+          size={'small'}
+        />
+      {/if}
+      <Label label={recruit.string.CreateApplication} />
+    </div>
+  </svelte:fragment>
   <StatusControl slot="error" {status} />
   <svelte:fragment slot="pool">
     {#key doc}
diff --git a/plugins/view-resources/src/actions.ts b/plugins/view-resources/src/actions.ts
index 10709df689..12e9b35710 100644
--- a/plugins/view-resources/src/actions.ts
+++ b/plugins/view-resources/src/actions.ts
@@ -116,15 +116,19 @@ export function filterActions (
   actions: Array<WithLookup<Action>>,
   derived: Ref<Class<Doc>> = core.class.Doc
 ): Array<WithLookup<Action>> {
-  const result: Array<WithLookup<Action>> = []
+  let result: Array<WithLookup<Action>> = []
   const hierarchy = client.getHierarchy()
   const clazz = hierarchy.getClass(doc._class)
   const ignoreActions = hierarchy.as(clazz, view.mixin.IgnoreActions)
   const ignore = ignoreActions?.actions ?? []
+  const overrideRemove: Array<Ref<Action>> = []
   for (const action of actions) {
     if (ignore.includes(action._id)) {
       continue
     }
+    if (action.override !== undefined) {
+      overrideRemove.push(...action.override)
+    }
     if (action.query !== undefined) {
       const r = matchQuery([doc], action.query, doc._class, hierarchy)
       if (r.length === 0) {
@@ -138,5 +142,8 @@ export function filterActions (
       result.push(action)
     }
   }
+  if (overrideRemove.length > 0) {
+    result = result.filter((it) => !overrideRemove.includes(it._id))
+  }
   return result
 }
diff --git a/plugins/view/src/index.ts b/plugins/view/src/index.ts
index 186d9b2721..66870ae535 100644
--- a/plugins/view/src/index.ts
+++ b/plugins/view/src/index.ts
@@ -235,6 +235,10 @@ export interface Action<T extends Doc = Doc, P = Record<string, any>> extends Do
 
   // Context action is defined for
   context: ViewContext
+
+  // A list of actions replaced by this one.
+  // For example it could be global action and action for focus class, second one fill override first one.
+  override?: Ref<Action>[]
 }
 
 /**
diff --git a/tests/sanity/tests/actions.spec.ts b/tests/sanity/tests/actions.spec.ts
new file mode 100644
index 0000000000..013f94ca2a
--- /dev/null
+++ b/tests/sanity/tests/actions.spec.ts
@@ -0,0 +1,64 @@
+import { expect, test } from '@playwright/test'
+import { PlatformSetting, PlatformURI } from './utils'
+
+test.use({
+  storageState: PlatformSetting
+})
+
+test.describe('actions tests', () => {
+  test.beforeEach(async ({ page }) => {
+    // Create user and workspace
+    await page.goto(`${PlatformURI}/workbench%3Acomponent%3AWorkbenchApp`)
+  })
+  test('action-new-candidate', async ({ page }) => {
+    await page.click('[id="app-recruit\\:string\\:RecruitApplication"]')
+
+    await page.click('text=Candidates')
+    await expect(page).toHaveURL(
+      `${PlatformURI}/workbench%3Acomponent%3AWorkbenchApp/recruit%3Aapp%3ARecruit/candidates`
+    )
+
+    await page.click('td:has-text("Frontend Engineer")')
+
+    await page.press('body', 'Meta+k')
+    await page.fill('[placeholder="type\\ to\\ filter\\.\\.\\."]', 'Candidate')
+    expect(await page.locator('div.selectPopup :text("New Candidate")').count()).toBe(1)
+    await page.click('div.selectPopup :text("New Candidate")')
+    await page.click('button#card-close')
+  })
+
+  test('action-switch-vacancies', async ({ page }) => {
+    await page.click('[id="app-recruit\\:string\\:RecruitApplication"]')
+
+    await page.click('text=Candidates')
+    await expect(page).toHaveURL(
+      `${PlatformURI}/workbench%3Acomponent%3AWorkbenchApp/recruit%3Aapp%3ARecruit/candidates`
+    )
+
+    await page.press('body', 'Meta+k')
+    await page.fill('[placeholder="type\\ to\\ filter\\.\\.\\."]', 'go to')
+    expect(await page.locator('div.selectPopup :text("Go To Vacancies")').count()).toBe(1)
+    await page.click('div.selectPopup :text("Go To Vacancies")')
+
+    await expect(page).toHaveURL(
+      `${PlatformURI}/workbench%3Acomponent%3AWorkbenchApp/recruit%3Aapp%3ARecruit/vacancies`
+    )
+  })
+  test('action-switch-applications', async ({ page }) => {
+    await page.click('[id="app-recruit\\:string\\:RecruitApplication"]')
+
+    await page.click('text=Candidates')
+    await expect(page).toHaveURL(
+      `${PlatformURI}/workbench%3Acomponent%3AWorkbenchApp/recruit%3Aapp%3ARecruit/candidates`
+    )
+
+    await page.press('body', 'Meta+k')
+    await page.fill('[placeholder="type\\ to\\ filter\\.\\.\\."]', 'go to')
+    expect(await page.locator('div.selectPopup :text("Go To Applications")').count()).toBe(1)
+    await page.click('div.selectPopup :text("Go To Applications")')
+
+    await expect(page).toHaveURL(
+      `${PlatformURI}/workbench%3Acomponent%3AWorkbenchApp/recruit%3Aapp%3ARecruit/applicants`
+    )
+  })
+})
diff --git a/tests/sanity/tests/recruit.spec.ts b/tests/sanity/tests/recruit.spec.ts
index 38ce7d7b5d..57e9a81a81 100644
--- a/tests/sanity/tests/recruit.spec.ts
+++ b/tests/sanity/tests/recruit.spec.ts
@@ -20,7 +20,7 @@ test.describe('recruit tests', () => {
     const first = 'Elton-' + generateId(4)
     const last = 'John-' + generateId(4)
     const loc = 'Cupertino'
-    const email = 'ej@test.com'
+    const email = `ej-${generateId(4)}@test.com`
 
     const firstName = page.locator('[placeholder="John"]')
     await firstName.click()