From bb0a3bc467ae4d35353e3f1fa37409b5da4c7795 Mon Sep 17 00:00:00 2001
From: Ryan Wang <i@ryanc.cc>
Date: Thu, 27 Jun 2024 18:10:55 +0800
Subject: [PATCH] feat: add support for setting an owner for posts (#6178)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

#### What type of PR is this?

/area ui
/kind feature
/milestone 2.17.x

#### What this PR does / why we need it:

支持手动为文章设置作者。

<img width="734" alt="image" src="https://github.com/halo-dev/halo/assets/21301288/b91b1754-4f50-4333-8478-6735d5846847">


#### Which issue(s) this PR fixes:

Fixes https://github.com/halo-dev/halo/issues/5720

#### Special notes for your reviewer:

需要测试：

1. 打开任意文章的设置，在高级中设置作者并保存，观察是否成功。
2. 选择一批文章，点击批量设置，设置一个作者，观察是否设置成功。

#### Does this PR introduce a user-facing change?

```release-note
支持手动为文章设置作者。
```
---
 .../contents/pages/DeletedSinglePageList.vue  |  7 +++--
 .../pages/components/SinglePageListItem.vue   |  7 +++--
 .../contents/posts/DeletedPostList.vue        |  7 +++--
 .../components/PostBatchSettingModal.vue      | 31 +++++++++++++++++++
 .../posts/components/PostSettingModal.vue     |  5 +++
 .../entity-fields/ContributorsField.vue       |  7 +++--
 ui/env.d.ts                                   |  7 ++++-
 .../components/user/PostContributorList.vue}  | 30 +++++++++++++++---
 ui/src/formkit/formkit.config.ts              |  2 ++
 ui/src/formkit/inputs/user-select.ts          | 26 ++++++++++++++++
 ui/src/locales/en.yaml                        |  4 +++
 ui/src/locales/zh-CN.yaml                     |  4 +++
 ui/src/locales/zh-TW.yaml                     |  4 +++
 .../posts/components/PostListItem.vue         | 17 ++++------
 14 files changed, 134 insertions(+), 24 deletions(-)
 rename ui/{console-src/modules/contents/_components/ContributorList.vue => src/components/user/PostContributorList.vue} (56%)
 create mode 100644 ui/src/formkit/inputs/user-select.ts

diff --git a/ui/console-src/modules/contents/pages/DeletedSinglePageList.vue b/ui/console-src/modules/contents/pages/DeletedSinglePageList.vue
index 294b6ff19..25fa625e0 100644
--- a/ui/console-src/modules/contents/pages/DeletedSinglePageList.vue
+++ b/ui/console-src/modules/contents/pages/DeletedSinglePageList.vue
@@ -1,4 +1,5 @@
 <script lang="ts" setup>
+import PostContributorList from "@/components/user/PostContributorList.vue";
 import { formatDatetime } from "@/utils/date";
 import { usePermission } from "@/utils/permission";
 import type { ListedSinglePage, SinglePage } from "@halo-dev/api-client";
@@ -25,7 +26,6 @@ import { useQuery } from "@tanstack/vue-query";
 import { cloneDeep } from "lodash-es";
 import { ref, watch } from "vue";
 import { useI18n } from "vue-i18n";
-import ContributorList from "../_components/ContributorList.vue";
 
 const { currentUserHasPermission } = usePermission();
 const { t } = useI18n();
@@ -327,7 +327,10 @@ watch(
               <template #end>
                 <VEntityField>
                   <template #description>
-                    <ContributorList :contributors="singlePage.contributors" />
+                    <PostContributorList
+                      :owner="singlePage.owner"
+                      :contributors="singlePage.contributors"
+                    />
                   </template>
                 </VEntityField>
                 <VEntityField v-if="!singlePage?.page?.spec.deleted">
diff --git a/ui/console-src/modules/contents/pages/components/SinglePageListItem.vue b/ui/console-src/modules/contents/pages/components/SinglePageListItem.vue
index 75dac1926..a4de2d71d 100644
--- a/ui/console-src/modules/contents/pages/components/SinglePageListItem.vue
+++ b/ui/console-src/modules/contents/pages/components/SinglePageListItem.vue
@@ -1,4 +1,5 @@
 <script lang="ts" setup>
+import PostContributorList from "@/components/user/PostContributorList.vue";
 import { singlePageLabels } from "@/constants/labels";
 import { formatDatetime } from "@/utils/date";
 import { usePermission } from "@/utils/permission";
@@ -23,7 +24,6 @@ import type { Ref } from "vue";
 import { computed, inject, ref } from "vue";
 import { useI18n } from "vue-i18n";
 import { RouterLink } from "vue-router";
-import ContributorList from "../../_components/ContributorList.vue";
 
 const { currentUserHasPermission } = usePermission();
 const { t } = useI18n();
@@ -184,7 +184,10 @@ const handleDelete = async () => {
     <template #end>
       <VEntityField>
         <template #description>
-          <ContributorList :contributors="singlePage.contributors" />
+          <PostContributorList
+            :owner="singlePage.owner"
+            :contributors="singlePage.contributors"
+          />
         </template>
       </VEntityField>
       <VEntityField :description="publishStatus">
diff --git a/ui/console-src/modules/contents/posts/DeletedPostList.vue b/ui/console-src/modules/contents/posts/DeletedPostList.vue
index 94ddc472f..9f56ca817 100644
--- a/ui/console-src/modules/contents/posts/DeletedPostList.vue
+++ b/ui/console-src/modules/contents/posts/DeletedPostList.vue
@@ -1,4 +1,5 @@
 <script lang="ts" setup>
+import PostContributorList from "@/components/user/PostContributorList.vue";
 import { formatDatetime } from "@/utils/date";
 import { usePermission } from "@/utils/permission";
 import type { ListedPost, Post } from "@halo-dev/api-client";
@@ -25,7 +26,6 @@ import { useQuery } from "@tanstack/vue-query";
 import { cloneDeep } from "lodash-es";
 import { ref, watch } from "vue";
 import { useI18n } from "vue-i18n";
-import ContributorList from "../_components/ContributorList.vue";
 import PostTag from "./tags/components/PostTag.vue";
 
 const { currentUserHasPermission } = usePermission();
@@ -351,7 +351,10 @@ watch(
               <template #end>
                 <VEntityField>
                   <template #description>
-                    <ContributorList :contributors="post.contributors" />
+                    <PostContributorList
+                      :owner="post.owner"
+                      :contributors="post.contributors"
+                    />
                   </template>
                 </VEntityField>
                 <VEntityField v-if="!post?.post?.spec.deleted">
diff --git a/ui/console-src/modules/contents/posts/components/PostBatchSettingModal.vue b/ui/console-src/modules/contents/posts/components/PostBatchSettingModal.vue
index a4c5eb9c6..1b68e249c 100644
--- a/ui/console-src/modules/contents/posts/components/PostBatchSettingModal.vue
+++ b/ui/console-src/modules/contents/posts/components/PostBatchSettingModal.vue
@@ -22,6 +22,10 @@ interface FormData {
     names?: string[];
     op: ArrayPatchOp;
   };
+  owner: {
+    enabled: boolean;
+    value: string;
+  };
   visible: {
     enabled: boolean;
     value: "PUBLIC" | "PRIVATE";
@@ -73,6 +77,14 @@ const { mutate, isLoading } = useMutation({
         });
       }
 
+      if (data.owner.enabled) {
+        jsonPatchInner.push({
+          op: "add",
+          path: "/spec/owner",
+          value: data.owner.value,
+        });
+      }
+
       if (data.visible.enabled) {
         jsonPatchInner.push({
           op: "add",
@@ -235,6 +247,25 @@ function onSubmit(data: FormData) {
           validation="required"
         ></FormKit>
       </FormKit>
+      <FormKit
+        v-slot="{ value }"
+        type="group"
+        name="owner"
+        :label="$t('core.post.batch_setting_modal.fields.owner_group')"
+      >
+        <FormKit
+          :value="false"
+          :label="$t('core.post.batch_setting_modal.fields.common.enabled')"
+          type="checkbox"
+          name="enabled"
+        ></FormKit>
+        <FormKit
+          v-if="value?.enabled"
+          :label="$t('core.post.batch_setting_modal.fields.owner_value')"
+          name="value"
+          type="userSelect"
+        ></FormKit>
+      </FormKit>
       <FormKit
         v-slot="{ value }"
         type="group"
diff --git a/ui/console-src/modules/contents/posts/components/PostSettingModal.vue b/ui/console-src/modules/contents/posts/components/PostSettingModal.vue
index 09dde2e39..99bc24657 100644
--- a/ui/console-src/modules/contents/posts/components/PostSettingModal.vue
+++ b/ui/console-src/modules/contents/posts/components/PostSettingModal.vue
@@ -391,6 +391,11 @@ const showCancelPublishButton = computed(() => {
             </div>
           </div>
           <div class="mt-5 divide-y divide-gray-100 md:col-span-3 md:mt-0">
+            <FormKit
+              v-model="formState.spec.owner"
+              :label="$t('core.post.settings.fields.owner.label')"
+              type="userSelect"
+            ></FormKit>
             <FormKit
               v-model="formState.spec.allowComment"
               :options="[
diff --git a/ui/console-src/modules/contents/posts/components/entity-fields/ContributorsField.vue b/ui/console-src/modules/contents/posts/components/entity-fields/ContributorsField.vue
index d2987096c..c479415a0 100644
--- a/ui/console-src/modules/contents/posts/components/entity-fields/ContributorsField.vue
+++ b/ui/console-src/modules/contents/posts/components/entity-fields/ContributorsField.vue
@@ -1,5 +1,5 @@
 <script lang="ts" setup>
-import ContributorList from "@console/modules/contents/_components/ContributorList.vue";
+import PostContributorList from "@/components/user/PostContributorList.vue";
 import type { ListedPost } from "@halo-dev/api-client";
 import { VEntityField } from "@halo-dev/components";
 
@@ -14,7 +14,10 @@ withDefaults(
 <template>
   <VEntityField>
     <template #description>
-      <ContributorList :contributors="post.contributors" />
+      <PostContributorList
+        :owner="post.owner"
+        :contributors="post.contributors"
+      />
     </template>
   </VEntityField>
 </template>
diff --git a/ui/env.d.ts b/ui/env.d.ts
index e505ad3ad..58ad046f7 100644
--- a/ui/env.d.ts
+++ b/ui/env.d.ts
@@ -3,8 +3,8 @@
 
 export {};
 
-import type { CoreMenuGroupId } from "@halo-dev/console-shared";
 import type { FormKitInputs } from "@formkit/inputs";
+import type { CoreMenuGroupId } from "@halo-dev/console-shared";
 
 import "vue-router";
 
@@ -124,5 +124,10 @@ declare module "@formkit/inputs" {
       type: "code";
       value?: string;
     };
+
+    userSelect: {
+      type: "userSelect";
+      value?: string;
+    };
   }
 }
diff --git a/ui/console-src/modules/contents/_components/ContributorList.vue b/ui/src/components/user/PostContributorList.vue
similarity index 56%
rename from ui/console-src/modules/contents/_components/ContributorList.vue
rename to ui/src/components/user/PostContributorList.vue
index 4f72ea02b..3091f455d 100644
--- a/ui/console-src/modules/contents/_components/ContributorList.vue
+++ b/ui/src/components/user/PostContributorList.vue
@@ -2,20 +2,35 @@
 import { usePermission } from "@/utils/permission";
 import type { Contributor } from "@halo-dev/api-client";
 import { VAvatar, VAvatarGroup } from "@halo-dev/components";
+import { computed } from "vue";
 import { useRouter } from "vue-router";
 
-withDefaults(
+const props = withDefaults(
   defineProps<{
+    allowViewUserDetail?: boolean;
+    owner?: Contributor;
     contributors: Contributor[];
   }>(),
-  {}
+  {
+    allowViewUserDetail: true,
+    owner: undefined,
+  }
 );
 
 const router = useRouter();
 const { currentUserHasPermission } = usePermission();
 
+const contributorsWithoutOwner = computed(() =>
+  props.contributors.filter(
+    (contributor) => contributor.name !== props.owner?.name
+  )
+);
+
 function handleRouteToUserDetail(contributor: Contributor) {
-  if (!currentUserHasPermission(["system:users:view"])) {
+  if (
+    !currentUserHasPermission(["system:users:view"]) ||
+    !props.allowViewUserDetail
+  ) {
     return;
   }
   router.push({
@@ -28,7 +43,14 @@ function handleRouteToUserDetail(contributor: Contributor) {
 <template>
   <VAvatarGroup size="xs" circle>
     <VAvatar
-      v-for="contributor in contributors"
+      v-if="owner"
+      v-tooltip="owner?.displayName"
+      :src="owner.avatar"
+      :alt="owner.displayName"
+      @click="handleRouteToUserDetail(owner)"
+    />
+    <VAvatar
+      v-for="contributor in contributorsWithoutOwner"
       :key="contributor.name"
       v-tooltip="contributor.displayName"
       :src="contributor.avatar"
diff --git a/ui/src/formkit/formkit.config.ts b/ui/src/formkit/formkit.config.ts
index 63ed00790..868b760a2 100644
--- a/ui/src/formkit/formkit.config.ts
+++ b/ui/src/formkit/formkit.config.ts
@@ -25,6 +25,7 @@ import { tagSelect } from "./inputs/tag-select";
 import { verificationForm } from "./inputs/verify-form";
 import theme from "./theme";
 
+import { userSelect } from "./inputs/user-select";
 import autoScrollToErrors from "./plugins/auto-scroll-to-errors";
 import passwordPreventAutocomplete from "./plugins/password-prevent-autocomplete";
 import radioAlt from "./plugins/radio-alt";
@@ -65,6 +66,7 @@ const config: DefaultConfigOptions = {
     tagCheckbox,
     tagSelect,
     verificationForm,
+    userSelect,
   },
   locales: { zh, en },
   locale: "zh",
diff --git a/ui/src/formkit/inputs/user-select.ts b/ui/src/formkit/inputs/user-select.ts
new file mode 100644
index 000000000..8d264766a
--- /dev/null
+++ b/ui/src/formkit/inputs/user-select.ts
@@ -0,0 +1,26 @@
+// TODO: This is a temporary approach.
+// We will provide searchable user selection components in the future.
+
+import type { FormKitNode, FormKitTypeDefinition } from "@formkit/core";
+import { defaultIcon, select, selects } from "@formkit/inputs";
+import { coreApiClient } from "@halo-dev/api-client";
+
+function optionsHandler(node: FormKitNode) {
+  node.on("created", async () => {
+    const { data } = await coreApiClient.user.listUser();
+
+    node.props.options = data.items.map((user) => {
+      return {
+        value: user.metadata.name,
+        label: user.spec.displayName,
+      };
+    });
+  });
+}
+
+export const userSelect: FormKitTypeDefinition = {
+  ...select,
+  props: ["placeholder"],
+  forceTypeProp: "select",
+  features: [optionsHandler, selects, defaultIcon("select", "select")],
+};
diff --git a/ui/src/locales/en.yaml b/ui/src/locales/en.yaml
index 80ce5f9fd..88ae41ca2 100644
--- a/ui/src/locales/en.yaml
+++ b/ui/src/locales/en.yaml
@@ -279,6 +279,8 @@ core:
           label: Template
         cover:
           label: Cover
+        owner:
+          label: Owner
     tag:
       filters:
         sort:
@@ -306,6 +308,8 @@ core:
         visible_value: "Select visible option "
         allow_comment_group: " Allow comment"
         allow_comment_value: Choose whether to allow comments
+        owner_group: Owner
+        owner_value: Select owner
   deleted_post:
     title: Deleted Posts
     empty:
diff --git a/ui/src/locales/zh-CN.yaml b/ui/src/locales/zh-CN.yaml
index 36e788081..beb04aa47 100644
--- a/ui/src/locales/zh-CN.yaml
+++ b/ui/src/locales/zh-CN.yaml
@@ -267,6 +267,8 @@ core:
           label: 自定义模板
         cover:
           label: 封面图
+        owner:
+          label: 作者
     tag:
       filters:
         sort:
@@ -294,6 +296,8 @@ core:
         visible_value: 选择可见性
         allow_comment_group: 允许评论
         allow_comment_value: 选择是否允许评论
+        owner_group: 作者
+        owner_value: 设置作者
   deleted_post:
     title: 文章回收站
     empty:
diff --git a/ui/src/locales/zh-TW.yaml b/ui/src/locales/zh-TW.yaml
index fcb007b95..86396220c 100644
--- a/ui/src/locales/zh-TW.yaml
+++ b/ui/src/locales/zh-TW.yaml
@@ -267,6 +267,8 @@ core:
           label: 自定義模板
         cover:
           label: 封面圖
+        owner:
+          label: 作者
     batch_setting_modal:
       title: 文章批量設置
       fields:
@@ -286,6 +288,8 @@ core:
         visible_value: 選擇可見性
         allow_comment_group: 允許評論
         allow_comment_value: 選擇是否允許評論
+        owner_group: 作者
+        owner_value: 設置作者
   deleted_post:
     title: 文章回收站
     empty:
diff --git a/ui/uc-src/modules/contents/posts/components/PostListItem.vue b/ui/uc-src/modules/contents/posts/components/PostListItem.vue
index 4e47ec7e5..a3229e7da 100644
--- a/ui/uc-src/modules/contents/posts/components/PostListItem.vue
+++ b/ui/uc-src/modules/contents/posts/components/PostListItem.vue
@@ -1,6 +1,7 @@
 <script lang="ts" setup>
 import StatusDotField from "@/components/entity-fields/StatusDotField.vue";
 import HasPermission from "@/components/permission/HasPermission.vue";
+import PostContributorList from "@/components/user/PostContributorList.vue";
 import { postLabels } from "@/constants/labels";
 import { formatDatetime } from "@/utils/date";
 import PostTag from "@console/modules/contents/posts/tags/components/PostTag.vue";
@@ -13,8 +14,6 @@ import {
   IconEyeOff,
   IconTimerLine,
   Toast,
-  VAvatar,
-  VAvatarGroup,
   VDropdownDivider,
   VDropdownItem,
   VEntity,
@@ -180,15 +179,11 @@ function handleUnpublish() {
     <template #end>
       <VEntityField>
         <template #description>
-          <VAvatarGroup size="xs" circle>
-            <VAvatar
-              v-for="{ name, avatar, displayName } in post.contributors"
-              :key="name"
-              v-tooltip="displayName"
-              :src="avatar"
-              :alt="displayName"
-            ></VAvatar>
-          </VAvatarGroup>
+          <PostContributorList
+            :owner="post.owner"
+            :contributors="post.contributors"
+            :allow-view-user-detail="false"
+          />
         </template>
       </VEntityField>
       <VEntityField :description="publishStatus">