From 09f01e6ea46c35dc80754f5216f13247d789ee57 Mon Sep 17 00:00:00 2001
From: crupest <crupest@outlook.com>
Date: Tue, 19 Jan 2021 16:46:15 +0800
Subject: ...

---
 .../views/timeline-common/TimelineCardTemplate.tsx | 67 ++++++++++++++++------
 .../views/timeline-common/TimelinePageTemplate.tsx | 37 ------------
 .../timeline-common/TimelinePageTemplateUI.tsx     |  2 -
 3 files changed, 49 insertions(+), 57 deletions(-)

(limited to 'FrontEnd/src')

diff --git a/FrontEnd/src/app/views/timeline-common/TimelineCardTemplate.tsx b/FrontEnd/src/app/views/timeline-common/TimelineCardTemplate.tsx
index e62f76fa..b9f296c5 100644
--- a/FrontEnd/src/app/views/timeline-common/TimelineCardTemplate.tsx
+++ b/FrontEnd/src/app/views/timeline-common/TimelineCardTemplate.tsx
@@ -3,18 +3,19 @@ import clsx from "clsx";
 import { useTranslation } from "react-i18next";
 import { Dropdown, Button } from "react-bootstrap";
 
-import { timelineVisibilityTooltipTranslationMap } from "@/services/timeline";
+import {
+  timelineService,
+  timelineVisibilityTooltipTranslationMap,
+} from "@/services/timeline";
 
 import { TimelineCardComponentProps } from "../timeline-common/TimelinePageTemplateUI";
 import SyncStatusBadge from "../timeline-common/SyncStatusBadge";
 import CollapseButton from "../timeline-common/CollapseButton";
+import { useUser } from "@/services/user";
+import { pushAlert } from "@/services/alert";
 
 export interface TimelineCardTemplateProps
   extends Omit<TimelineCardComponentProps<"">, "operations"> {
-  operations: Pick<
-    TimelineCardComponentProps<"">["operations"],
-    "onHighlight" | "onBookmark"
-  >;
   infoArea: React.ReactElement;
   manageArea:
     | { type: "member"; onMember: () => void }
@@ -37,13 +38,13 @@ function TimelineCardTemplate({
   collapse,
   infoArea,
   manageArea,
-  operations,
   toggleCollapse,
   syncStatus,
   className,
 }: TimelineCardTemplateProps): React.ReactElement | null {
   const { t } = useTranslation();
-  const { onBookmark, onHighlight } = operations;
+
+  const user = useUser();
 
   return (
     <div className={clsx("cru-card p-2 clearfix", className)}>
@@ -58,22 +59,52 @@ function TimelineCardTemplate({
           {t(timelineVisibilityTooltipTranslationMap[timeline.visibility])}
         </small>
         <div className="text-right mt-2">
-          {onHighlight != null ? (
-            <i
-              className={clsx(
-                timeline.isHighlight ? "bi-star-fill" : "bi-star",
-                "icon-button text-yellow mr-3"
-              )}
-              onClick={onHighlight}
-            />
-          ) : null}
-          {onBookmark != null ? (
+          <i
+            className={clsx(
+              timeline.isHighlight ? "bi-star-fill" : "bi-star",
+              "icon-button text-yellow mr-3"
+            )}
+            onClick={
+              user != null && user.hasHighlightTimelineAdministrationPermission
+                ? () => {
+                    timelineService
+                      .setHighlight(timeline.name, !timeline.isHighlight)
+                      .catch(() => {
+                        pushAlert({
+                          message: {
+                            type: "i18n",
+                            key: timeline.isHighlight
+                              ? "timeline.removeHighlightFail"
+                              : "timeline.addHighlightFail",
+                          },
+                          type: "danger",
+                        });
+                      });
+                  }
+                : undefined
+            }
+          />
+          {user != null ? (
             <i
               className={clsx(
                 timeline.isBookmark ? "bi-bookmark-fill" : "bi-bookmark",
                 "icon-button text-yellow mr-3"
               )}
-              onClick={onBookmark}
+              onClick={() => {
+                timelineService
+                  .setBookmark(timeline.name, !timeline.isBookmark)
+                  .catch(() => {
+                    pushAlert({
+                      message: {
+                        type: "i18n",
+                        key: timeline.isBookmark
+                          ? "timeline.removeBookmarkFail"
+                          : "timeline.addBookmarkFail",
+                      },
+                      type: "danger",
+                    });
+                  });
+              }}
             />
           ) : null}
           {manageArea.type === "manage" ? (
diff --git a/FrontEnd/src/app/views/timeline-common/TimelinePageTemplate.tsx b/FrontEnd/src/app/views/timeline-common/TimelinePageTemplate.tsx
index bff4547e..9b76635e 100644
--- a/FrontEnd/src/app/views/timeline-common/TimelinePageTemplate.tsx
+++ b/FrontEnd/src/app/views/timeline-common/TimelinePageTemplate.tsx
@@ -1,7 +1,6 @@
 import React from "react";
 
 import { UiLogicError } from "@/common";
-import { pushAlert } from "@/services/alert";
 import { useUser } from "@/services/user";
 import {
   TimelinePostInfo,
@@ -98,42 +97,6 @@ export default function TimelinePageTemplate<TManageItem>(
             }
           : undefined,
         onMember: () => setDialog("member"),
-        onBookmark:
-          user != null
-            ? () => {
-                service
-                  .setBookmark(timeline.name, !timeline.isBookmark)
-                  .catch(() => {
-                    pushAlert({
-                      message: {
-                        type: "i18n",
-                        key: timeline.isBookmark
-                          ? "timeline.removeBookmarkFail"
-                          : "timeline.addBookmarkFail",
-                      },
-                      type: "danger",
-                    });
-                  });
-              }
-            : undefined,
-        onHighlight:
-          user != null && user.hasHighlightTimelineAdministrationPermission
-            ? () => {
-                service
-                  .setHighlight(timeline.name, !timeline.isHighlight)
-                  .catch(() => {
-                    pushAlert({
-                      message: {
-                        type: "i18n",
-                        key: timeline.isHighlight
-                          ? "timeline.removeHighlightFail"
-                          : "timeline.addHighlightFail",
-                      },
-                      type: "danger",
-                    });
-                  });
-              }
-            : undefined,
       };
 
       const posts = ((): TimelinePostInfo[] | "forbid" | undefined => {
diff --git a/FrontEnd/src/app/views/timeline-common/TimelinePageTemplateUI.tsx b/FrontEnd/src/app/views/timeline-common/TimelinePageTemplateUI.tsx
index dbb47387..ed21d6b5 100644
--- a/FrontEnd/src/app/views/timeline-common/TimelinePageTemplateUI.tsx
+++ b/FrontEnd/src/app/views/timeline-common/TimelinePageTemplateUI.tsx
@@ -15,8 +15,6 @@ export interface TimelineCardComponentProps<TManageItems> {
   operations: {
     onManage?: (item: TManageItems | "property") => void;
     onMember: () => void;
-    onBookmark?: () => void;
-    onHighlight?: () => void;
   };
   collapse: boolean;
   toggleCollapse: () => void;
-- 
cgit v1.2.3