From c46788c45114b335fa592e2f42fba2514c555f56 Mon Sep 17 00:00:00 2001
From: Ryan Wang <i@ryanc.cc>
Date: Fri, 30 Sep 2022 19:03:38 +0800
Subject: [PATCH] fix: ui permissions (halo-dev/console#633)

---
 src/main.ts                                                | 7 +++----
 src/modules/contents/attachments/AttachmentList.vue        | 4 ++--
 .../contents/comments/components/CommentListItem.vue       | 4 ++--
 src/modules/contents/pages/SinglePageList.vue              | 4 ++--
 src/modules/contents/posts/PostList.vue                    | 4 ++--
 src/modules/contents/posts/tags/components/PostTag.vue     | 7 ++-----
 .../interface/menus/components/MenuItemListItem.vue        | 2 +-
 src/modules/interface/menus/components/MenuList.vue        | 4 ++--
 src/modules/interface/themes/components/ThemeListModal.vue | 2 +-
 src/modules/system/plugins/components/PluginListItem.vue   | 2 +-
 src/modules/system/roles/RoleList.vue                      | 2 +-
 src/modules/system/users/UserList.vue                      | 4 ++--
 12 files changed, 21 insertions(+), 25 deletions(-)

diff --git a/src/main.ts b/src/main.ts
index 20ab48413..9539c0cbf 100644
--- a/src/main.ts
+++ b/src/main.ts
@@ -173,10 +173,9 @@ async function loadCurrentUser() {
   app.directive(
     "permission",
     (el: HTMLElement, binding: DirectiveBinding<string[]>) => {
-      // const uiPermissions = Array.from<string>(
-      //   currentPermissions.uiPermissions
-      // );
-      const uiPermissions = Array.from<string>(["system:attachments:view"]);
+      const uiPermissions = Array.from<string>(
+        currentPermissions.uiPermissions
+      );
       const { value } = binding;
       const { any, enable } = binding.modifiers;
 
diff --git a/src/modules/contents/attachments/AttachmentList.vue b/src/modules/contents/attachments/AttachmentList.vue
index bb3530185..b4d72db1c 100644
--- a/src/modules/contents/attachments/AttachmentList.vue
+++ b/src/modules/contents/attachments/AttachmentList.vue
@@ -609,7 +609,7 @@ onMounted(() => {
                 <VEntity :is-selected="isChecked(attachment)">
                   <template
                     v-if="
-                      !currentUserHasPermission(['system:attachments:manage'])
+                      currentUserHasPermission(['system:attachments:manage'])
                     "
                     #checkbox
                   >
@@ -695,7 +695,7 @@ onMounted(() => {
                   </template>
                   <template
                     v-if="
-                      !currentUserHasPermission(['system:attachments:manage'])
+                      currentUserHasPermission(['system:attachments:manage'])
                     "
                     #dropdownItems
                   >
diff --git a/src/modules/contents/comments/components/CommentListItem.vue b/src/modules/contents/comments/components/CommentListItem.vue
index 32e482518..5f9e69560 100644
--- a/src/modules/contents/comments/components/CommentListItem.vue
+++ b/src/modules/contents/comments/components/CommentListItem.vue
@@ -251,7 +251,7 @@ const subjectRefResult = computed(() => {
       <div class="absolute inset-x-0 bottom-0 h-[1px] bg-black/50"></div>
     </template>
     <template
-      v-if="!currentUserHasPermission(['system:comments:manage'])"
+      v-if="currentUserHasPermission(['system:comments:manage'])"
       #checkbox
     >
       <slot name="checkbox" />
@@ -343,7 +343,7 @@ const subjectRefResult = computed(() => {
       </VEntityField>
     </template>
     <template
-      v-if="!currentUserHasPermission(['system:comments:manage'])"
+      v-if="currentUserHasPermission(['system:comments:manage'])"
       #dropdownItems
     >
       <VButton
diff --git a/src/modules/contents/pages/SinglePageList.vue b/src/modules/contents/pages/SinglePageList.vue
index 239b588c9..fc3a3cde4 100644
--- a/src/modules/contents/pages/SinglePageList.vue
+++ b/src/modules/contents/pages/SinglePageList.vue
@@ -536,7 +536,7 @@ function handleSortItemChange(sortItem?: SortItem) {
       <li v-for="(singlePage, index) in singlePages.items" :key="index">
         <VEntity :is-selected="checkAll">
           <template
-            v-if="!currentUserHasPermission(['system:singlepages:manage'])"
+            v-if="currentUserHasPermission(['system:singlepages:manage'])"
             #checkbox
           >
             <input
@@ -636,7 +636,7 @@ function handleSortItemChange(sortItem?: SortItem) {
             </VEntityField>
           </template>
           <template
-            v-if="!currentUserHasPermission(['system:singlepages:manage'])"
+            v-if="currentUserHasPermission(['system:singlepages:manage'])"
             #dropdownItems
           >
             <VButton
diff --git a/src/modules/contents/posts/PostList.vue b/src/modules/contents/posts/PostList.vue
index 5d60b9b86..7c3bc0b94 100644
--- a/src/modules/contents/posts/PostList.vue
+++ b/src/modules/contents/posts/PostList.vue
@@ -800,7 +800,7 @@ function handleContributorChange(user?: User) {
         <li v-for="(post, index) in posts.items" :key="index">
           <VEntity :is-selected="checkSelection(post.post)">
             <template
-              v-if="!currentUserHasPermission(['system:posts:manage'])"
+              v-if="currentUserHasPermission(['system:posts:manage'])"
               #checkbox
             >
               <input
@@ -922,7 +922,7 @@ function handleContributorChange(user?: User) {
               </VEntityField>
             </template>
             <template
-              v-if="!currentUserHasPermission(['system:posts:manage'])"
+              v-if="currentUserHasPermission(['system:posts:manage'])"
               #dropdownItems
             >
               <VButton
diff --git a/src/modules/contents/posts/tags/components/PostTag.vue b/src/modules/contents/posts/tags/components/PostTag.vue
index 1e863c076..ba76104ff 100644
--- a/src/modules/contents/posts/tags/components/PostTag.vue
+++ b/src/modules/contents/posts/tags/components/PostTag.vue
@@ -2,8 +2,6 @@
 import { VTag } from "@halo-dev/components";
 import type { Tag } from "@halo-dev/api-client";
 import { computed } from "vue";
-// @ts-ignore
-import Color from "colorjs.io";
 import { useRouter } from "vue-router";
 
 const props = withDefaults(
@@ -21,9 +19,8 @@ const labelColor = computed(() => {
   if (!color) {
     return "inherit";
   }
-  const onWhite = Math.abs(Color.contrast(color, "white", "APCA"));
-  const onBlack = Math.abs(Color.contrast(color, "black", "APCA"));
-  return onWhite > onBlack ? "white" : "#333";
+  // TODO computed label color
+  return "#333";
 });
 
 const router = useRouter();
diff --git a/src/modules/interface/menus/components/MenuItemListItem.vue b/src/modules/interface/menus/components/MenuItemListItem.vue
index 16fa1f135..8461946b6 100644
--- a/src/modules/interface/menus/components/MenuItemListItem.vue
+++ b/src/modules/interface/menus/components/MenuItemListItem.vue
@@ -103,7 +103,7 @@ function getMenuItemRefDisplayName(menuItem: MenuTreeItem) {
             </VEntityField>
           </template>
           <template
-            v-if="!currentUserHasPermission(['system:menus:manage'])"
+            v-if="currentUserHasPermission(['system:menus:manage'])"
             #dropdownItems
           >
             <VButton
diff --git a/src/modules/interface/menus/components/MenuList.vue b/src/modules/interface/menus/components/MenuList.vue
index 8d9c89555..8d3bf33db 100644
--- a/src/modules/interface/menus/components/MenuList.vue
+++ b/src/modules/interface/menus/components/MenuList.vue
@@ -165,7 +165,7 @@ defineExpose({
             </VEntityField>
           </template>
           <template
-            v-if="!currentUserHasPermission(['system:menus:manage'])"
+            v-if="currentUserHasPermission(['system:menus:manage'])"
             #dropdownItems
           >
             <VButton
@@ -188,7 +188,7 @@ defineExpose({
         </VEntity>
       </li>
     </ul>
-    <template v-if="!currentUserHasPermission(['system:menus:manage'])" #footer>
+    <template v-if="currentUserHasPermission(['system:menus:manage'])" #footer>
       <VButton block type="secondary" @click="handleOpenEditingModal()">
         新增
       </VButton>
diff --git a/src/modules/interface/themes/components/ThemeListModal.vue b/src/modules/interface/themes/components/ThemeListModal.vue
index f48806599..dd0159ed4 100644
--- a/src/modules/interface/themes/components/ThemeListModal.vue
+++ b/src/modules/interface/themes/components/ThemeListModal.vue
@@ -246,7 +246,7 @@ defineExpose({
             </VEntityField>
           </template>
           <template
-            v-if="!currentUserHasPermission(['system:themes:manage'])"
+            v-if="currentUserHasPermission(['system:themes:manage'])"
             #dropdownItems
           >
             <VButton
diff --git a/src/modules/system/plugins/components/PluginListItem.vue b/src/modules/system/plugins/components/PluginListItem.vue
index 60087ffae..8ce89ed72 100644
--- a/src/modules/system/plugins/components/PluginListItem.vue
+++ b/src/modules/system/plugins/components/PluginListItem.vue
@@ -101,7 +101,7 @@ const { isStarted, changeStatus, uninstall } = usePluginLifeCycle(plugin);
       </VEntityField>
     </template>
     <template
-      v-if="!currentUserHasPermission(['system:plugins:manage'])"
+      v-if="currentUserHasPermission(['system:plugins:manage'])"
       #dropdownItems
     >
       <VButton v-close-popper block type="danger" @click="uninstall">
diff --git a/src/modules/system/roles/RoleList.vue b/src/modules/system/roles/RoleList.vue
index f12cb482c..f5aa46141 100644
--- a/src/modules/system/roles/RoleList.vue
+++ b/src/modules/system/roles/RoleList.vue
@@ -276,7 +276,7 @@ const handleDelete = async (role: Role) => {
               </VEntityField>
             </template>
             <template
-              v-if="!currentUserHasPermission(['system:roles:manage'])"
+              v-if="currentUserHasPermission(['system:roles:manage'])"
               #dropdownItems
             >
               <VButton
diff --git a/src/modules/system/users/UserList.vue b/src/modules/system/users/UserList.vue
index 6b9a81d84..9878c0fa9 100644
--- a/src/modules/system/users/UserList.vue
+++ b/src/modules/system/users/UserList.vue
@@ -388,7 +388,7 @@ onMounted(() => {
         <li v-for="(user, index) in searchResults" :key="index">
           <VEntity :is-selected="checkSelection(user)">
             <template
-              v-if="!currentUserHasPermission(['system:users:manage'])"
+              v-if="currentUserHasPermission(['system:users:manage'])"
               #checkbox
             >
               <input
@@ -446,7 +446,7 @@ onMounted(() => {
               </VEntityField>
             </template>
             <template
-              v-if="!currentUserHasPermission(['system:users:manage'])"
+              v-if="currentUserHasPermission(['system:users:manage'])"
               #dropdownItems
             >
               <VButton