From 6eec4688db5a6469b4dc9a6e780141ba3b92b187 Mon Sep 17 00:00:00 2001
From: Michael An <2331806369@qq.com>
Date: Tue, 19 Dec 2023 12:13:49 +0800
Subject: [PATCH] fix toggle tags popover (#5835)

---
 .../src/components/common/seahub-popover.js   |  4 +--
 .../src/components/cur-dir-path/dir-tool.js   | 31 ++++++++++++++++---
 .../dialog/org-admin-invite-user-dialog.js    |  8 ++---
 ...org-admin-invite-user-via-weixin-dialog.js |  6 ++--
 .../repo-seatable-integration-dialog.js       |  2 +-
 frontend/src/components/search/ai-search.js   | 14 ++++-----
 .../add-seatable-account-setting.js           |  1 -
 frontend/src/pages/markdown-editor/index.js   |  2 +-
 .../src/pages/org-admin/org-users-users.js    |  6 ++--
 9 files changed, 48 insertions(+), 26 deletions(-)

diff --git a/frontend/src/components/common/seahub-popover.js b/frontend/src/components/common/seahub-popover.js
index 68c9f60a33..754a503d59 100644
--- a/frontend/src/components/common/seahub-popover.js
+++ b/frontend/src/components/common/seahub-popover.js
@@ -23,12 +23,12 @@ class SeahubPopover extends React.Component {
   isSelectOpen = false;
 
   componentDidMount() {
-    document.addEventListener('mousedown', this.onMouseDown, true);
+    document.addEventListener('mousedown', this.onMouseDown);
     document.addEventListener('keydown', this.onKeyDown);
   }
 
   componentWillUnmount() {
-    document.removeEventListener('mousedown', this.onMouseDown, true);
+    document.removeEventListener('mousedown', this.onMouseDown);
     document.removeEventListener('keydown', this.onKeyDown);
   }
 
diff --git a/frontend/src/components/cur-dir-path/dir-tool.js b/frontend/src/components/cur-dir-path/dir-tool.js
index 3b02df4286..3f71a3854c 100644
--- a/frontend/src/components/cur-dir-path/dir-tool.js
+++ b/frontend/src/components/cur-dir-path/dir-tool.js
@@ -24,8 +24,13 @@ class DirTool extends React.Component {
     this.tagsIconID = `tags-icon-${uuidv4()}`;
   }
 
-  onShowListRepoTag = (e) => {
-    this.setState({ isListRepoTagShow: true });
+  onMouseDown = (e) => {
+    e.stopPropagation();
+  };
+
+  toggleRepoTag = (e) => {
+    e.stopPropagation();
+    this.setState({ isListRepoTagShow: !this.state.isListRepoTagShow });
   };
 
   hidePopover = (e) => {
@@ -61,7 +66,16 @@ class DirTool extends React.Component {
       toolbarDom = (
         <ul className="path-toolbar">
           <li className="toolbar-item">
-            <a className="op-link sf2-icon-tag" href="#" id={this.tagsIconID} role="button" onClick={this.onShowListRepoTag} title={gettext('Tags')} aria-label={gettext('Tags')}></a>
+            <a
+              className="op-link sf2-icon-tag"
+              href="#"
+              id={this.tagsIconID}
+              role="button"
+              onClick={this.toggleRepoTag}
+              onMouseDown={this.onMouseDown}
+              title={gettext('Tags')}
+              aria-label={gettext('Tags')}
+            ></a>
           </li>
           <li className="toolbar-item">
             <a className="op-link sf2-icon-recycle" href={trashUrl} title={gettext('Trash')} aria-label={gettext('Trash')}></a>
@@ -74,7 +88,16 @@ class DirTool extends React.Component {
       toolbarDom = (
         <ul className="path-toolbar">
           <li className="toolbar-item">
-            <a className="op-link sf2-icon-tag" href="#" id={this.tagsIconID} role="button" onClick={this.onShowListRepoTag} title={gettext('Tags')} aria-label={gettext('Tags')}></a>
+            <a
+              className="op-link sf2-icon-tag"
+              href="#"
+              id={this.tagsIconID}
+              role="button"
+              onClick={this.toggleRepoTag}
+              onMouseDown={this.onMouseDown}
+              title={gettext('Tags')}
+              aria-label={gettext('Tags')}
+            ></a>
           </li>
           <li className="toolbar-item">
             <a className="op-link sf2-icon-recycle" href={trashUrl} title={gettext('Trash')} aria-label={gettext('Trash')}></a>
diff --git a/frontend/src/components/dialog/org-admin-invite-user-dialog.js b/frontend/src/components/dialog/org-admin-invite-user-dialog.js
index 9eee7d3008..2499266072 100644
--- a/frontend/src/components/dialog/org-admin-invite-user-dialog.js
+++ b/frontend/src/components/dialog/org-admin-invite-user-dialog.js
@@ -1,6 +1,6 @@
 import React from 'react';
 import PropTypes from 'prop-types';
-import { Button, Modal, Input, ModalHeader, ModalBody, ModalFooter, Label, Form, InputGroup, InputGroupAddon, FormGroup } from 'reactstrap';
+import { Button, Modal, Input, ModalHeader, ModalBody, ModalFooter, Label, Form, FormGroup } from 'reactstrap';
 import { gettext } from '../../utils/constants';
 
 const propTypes = {
@@ -26,7 +26,7 @@ class OrgAdminInviteUserDialog extends React.Component {
       this.setState({isAddingUser: true});
       this.props.handleSubmit(email.trim());
     }
-  }
+  };
 
   handleKeyPress = (e) => {
     e.preventDefault();
@@ -38,11 +38,11 @@ class OrgAdminInviteUserDialog extends React.Component {
   inputEmail = (e) => {
     let email = e.target.value.trim();
     this.setState({email: email});
-  }
+  };
 
   toggle = () => {
     this.props.toggle();
-  }
+  };
 
   validateInputParams() {
     let errMessage;
diff --git a/frontend/src/components/dialog/org-admin-invite-user-via-weixin-dialog.js b/frontend/src/components/dialog/org-admin-invite-user-via-weixin-dialog.js
index a3c7936741..8b7cd11c9a 100644
--- a/frontend/src/components/dialog/org-admin-invite-user-via-weixin-dialog.js
+++ b/frontend/src/components/dialog/org-admin-invite-user-via-weixin-dialog.js
@@ -20,10 +20,10 @@ class OrgAdminInviteUserViaWeiXinDialog extends React.Component {
     copy(this.props.invitationLink);
     this.props.toggle();
     const message = gettext('Internal link has been copied to clipboard');
-    toaster.success(message), {
+    toaster.success(message, {
       duration: 2
-    };
-  }
+    });
+  };
 
   render() {
     return (
diff --git a/frontend/src/components/dialog/repo-seatable-integration-dialog.js b/frontend/src/components/dialog/repo-seatable-integration-dialog.js
index 4b5b81c555..eb03223888 100644
--- a/frontend/src/components/dialog/repo-seatable-integration-dialog.js
+++ b/frontend/src/components/dialog/repo-seatable-integration-dialog.js
@@ -1,4 +1,4 @@
-import React, { Fragment } from 'react';
+import React from 'react';
 import PropTypes from 'prop-types';
 import { Modal, ModalHeader, ModalBody } from 'reactstrap';
 import SeatableAccountSettingList from '../seatable-integration-account-setting-widgets/seatable-account-setting-list.js';
diff --git a/frontend/src/components/search/ai-search.js b/frontend/src/components/search/ai-search.js
index 0cf654551f..2f11fcf124 100644
--- a/frontend/src/components/search/ai-search.js
+++ b/frontend/src/components/search/ai-search.js
@@ -286,7 +286,7 @@ export default class AISearch extends Component {
     this.updateSearchPageURL(queryData);
     queryData['per_page'] = PER_PAGE;
     queryData['page'] = page;
-     this.onAiSearch(queryData, cancelToken, page);
+    this.onAiSearch(queryData, cancelToken, page);
   };
 
   onAiSearch = (queryData, cancelToken, page) => {
@@ -294,12 +294,12 @@ export default class AISearch extends Component {
     seafileAPI.aiSearchFiles(queryData, cancelToken).then(res => {
       results = [...results, ...this.formatResultItems(res.data.results)];
       this.setState({
-          resultItems: results,
-          isResultGetted: true,
-          isLoading: false,
-          page: page + 1,
-          hasMore: false,
-        });
+        resultItems: results,
+        isResultGetted: true,
+        isLoading: false,
+        page: page + 1,
+        hasMore: false,
+      });
     }).catch(error => {
       /* eslint-disable */
       console.log(error);
diff --git a/frontend/src/components/seatable-integration-account-setting-widgets/add-seatable-account-setting.js b/frontend/src/components/seatable-integration-account-setting-widgets/add-seatable-account-setting.js
index 7ae9c7a700..7915b3f083 100644
--- a/frontend/src/components/seatable-integration-account-setting-widgets/add-seatable-account-setting.js
+++ b/frontend/src/components/seatable-integration-account-setting-widgets/add-seatable-account-setting.js
@@ -66,7 +66,6 @@ class AddSeatableAccountSetting extends Component {
   };
 
   addSeatableAccountSetting = () => {
-    const { t } = this.props;
     let { base_name, seatable_url, seatable_api_token } = this.state;
     base_name = base_name.trim();
     seatable_url = seatable_url.trim();
diff --git a/frontend/src/pages/markdown-editor/index.js b/frontend/src/pages/markdown-editor/index.js
index 238868008f..ef4acb8d0b 100644
--- a/frontend/src/pages/markdown-editor/index.js
+++ b/frontend/src/pages/markdown-editor/index.js
@@ -16,7 +16,7 @@ import '../../css/markdown-viewer/markdown-editor.css';
 const CryptoJS = require('crypto-js');
 const URL = require('url-parse');
 
-const { repoID, filePath, fileName, draftID, isDraft, hasDraft, isLocked, lockedByMe } = window.app.pageOptions;
+const { repoID, filePath, fileName, isDraft, hasDraft, isLocked, lockedByMe } = window.app.pageOptions;
 const { siteRoot, serviceUrl, seafileCollabServer } = window.app.config;
 const userInfo = window.app.userInfo;
 const IMAGE_SUFFIXES = ['png', 'PNG', 'jpg', 'JPG', 'jpeg', 'JPEG', 'gif', 'GIF'];
diff --git a/frontend/src/pages/org-admin/org-users-users.js b/frontend/src/pages/org-admin/org-users-users.js
index c4e4c38c35..53bd2ec3a4 100644
--- a/frontend/src/pages/org-admin/org-users-users.js
+++ b/frontend/src/pages/org-admin/org-users-users.js
@@ -134,7 +134,7 @@ class OrgUsers extends Component {
 
   toggleInviteUserViaWeiXinDialog = () => {
     this.setState({isInviteUserViaWeiXinDialogOpen: !this.state.isInviteUserViaWeiXinDialogOpen});
-  }
+  };
 
   initOrgUsersData = (page) => {
     const { sortBy, sortOrder } = this.state;
@@ -218,13 +218,13 @@ class OrgUsers extends Component {
         orgUsers: users.concat(this.state.orgUsers)
       });
 
-      res.data.success.map(item => {
+      res.data.success.forEach(item => {
         let msg = gettext('successfully sent email to %s.');
         msg = msg.replace('%s', item.email);
         toaster.success(msg);
       });
 
-      res.data.failed.map(item => {
+      res.data.failed.forEach(item => {
         const msg = `${item.email}: ${item.error_msg}`;
         toaster.danger(msg);
       });