diff --git a/frontend/src/models/shared-folder-info.js b/frontend/src/models/shared-folder-info.js index 9847eb0bab..70e4922a49 100644 --- a/frontend/src/models/shared-folder-info.js +++ b/frontend/src/models/shared-folder-info.js @@ -11,7 +11,6 @@ class SharedFolderInfo { this.group_id = object.group_id; this.group_name = object.group_name; } else { - this.share_type = 'user'; // change personal to user; this.user_name = object.user_name; this.user_email = object.user_email; this.contact_email = object.contact_email; diff --git a/frontend/src/pages/share-admin/folders.js b/frontend/src/pages/share-admin/folders.js index 88be201bfa..939c34c2c0 100644 --- a/frontend/src/pages/share-admin/folders.js +++ b/frontend/src/pages/share-admin/folders.js @@ -1,12 +1,15 @@ -import React, { Component } from 'react'; +import React, { Fragment, Component } from 'react'; import { Link } from '@reach/router'; +import { Dropdown, DropdownToggle, DropdownItem } from 'reactstrap'; import { seafileAPI } from '../../utils/seafile-api'; import { Utils } from '../../utils/utils'; import { gettext, siteRoot, loginUrl, isPro } from '../../utils/constants'; +import Loading from '../../components/loading'; import EmptyTip from '../../components/empty-tip'; +import toaster from '../../components/toast'; import SharePermissionEditor from '../../components/select-editor/share-permission-editor'; import SharedFolderInfo from '../../models/shared-folder-info'; -import toaster from '../../components/toast'; +import PermSelect from '../../components/dialog/perm-select'; class Content extends Component { @@ -21,7 +24,7 @@ class Content extends Component { const { loading, errorMsg, items, sortBy, sortOrder } = this.props; if (loading) { - return ; + return ; } else if (errorMsg) { return

{errorMsg}

; } else { @@ -36,20 +39,29 @@ class Content extends Component { const sortByName = sortBy == 'name'; const sortIcon = sortOrder == 'asc' ? : ; + const isDesktop = Utils.isDesktop(); const table = ( - +
- - - - - - - + {isDesktop ? ( + + + + + + + + ) : ( + + + + + + )} {items.map((item, index) => { - return (); + return (); })}
{/*icon*/}{gettext('Name')} {sortByName && sortIcon}{gettext('Share To')}{gettext('Permission')}
{/*icon*/}{gettext('Name')} {sortByName && sortIcon}{gettext('Share To')}{gettext('Permission')}
@@ -66,57 +78,106 @@ class Item extends Component { super(props); this.state = { share_permission: this.props.item.share_permission, - showOpIcon: false, + isOpIconShown: false, + isOpMenuOpen: false, // for mobile + isPermSelectDialogOpen: false, // for mobile unshared: false }; this.permissions = ['rw', 'r']; if (isPro) { - this.permissions = ['rw', 'r', 'cloud-edit', 'preview']; + this.permissions.push('cloud-edit', 'preview'); } } + toggleOpMenu = () => { + this.setState({ + isOpMenuOpen: !this.state.isOpMenuOpen + }); + } + + togglePermSelectDialog = () => { + this.setState({ + isPermSelectDialogOpen: !this.state.isPermSelectDialogOpen + }); + } + onMouseEnter = () => { - this.setState({showOpIcon: true}); + this.setState({isOpIconShown: true}); } onMouseLeave = () => { - this.setState({showOpIcon: false}); + this.setState({isOpIconShown: false}); } unshare = (e) => { e.preventDefault(); + const item = this.props.item; - this.props.unshareFolder(item); + let options = { + 'p': item.path + }; + if (item.share_type == 'personal') { + Object.assign(options, { + 'share_type': 'user', + 'username': item.user_email + }); + } else { + Object.assign(options, { + 'share_type': item.share_type, // 'group' + 'group_id': item.group_id + }); + } + + seafileAPI.unshareFolder(item.repo_id, options).then((res) => { + this.setState({ + unshared: true + }); + let message = gettext('Successfully unshared {name}').replace('{name}', item.folder_name); + toaster.success(message); + }).catch(error => { + let errMessage = Utils.getErrorMsg(error); + toaster(errMessage); + }); } changePerm = (permission) => { - const perm = permission; - const postData = { - 'permission': perm - }; const item = this.props.item; - let options = { - 'p': item.path, - 'share_type': item.share_type + const postData = { + 'permission': permission }; - if (item.share_type == 'user') { - options.username = item.user_email; + let options = { + 'p': item.path + }; + if (item.share_type == 'personal') { + Object.assign(options, { + 'share_type': 'user', + 'username': item.user_email + }); } else { - options.group_id = item.group_id; + Object.assign(options, { + 'share_type': item.share_type, // 'group' + 'group_id': item.group_id + }); } seafileAPI.updateFolderSharePerm(item.repo_id, postData, options).then((res) => { - this.setState({share_permission: perm}); + this.setState({share_permission: permission}); + toaster.success(gettext('Successfully modified permission.')); }).catch((error) => { let errMessage = Utils.getErrorMsg(error); toaster.danger(errMessage); }); } - getFolderParams = () => { - let item = this.props.item; - let share_permission = this.state.share_permission; + render() { + if (this.state.unshared) { + return null; + } + + const item = this.props.item; + let { share_permission, isOpIconShown, isPermSelectDialogOpen } = this.state; + let is_readonly = false; if (share_permission == 'r' || share_permission == 'preview') { is_readonly = true; @@ -125,43 +186,70 @@ class Item extends Component { let iconTitle = Utils.getFolderIconTitle({ 'permission': share_permission }); - let folderUrl = `${siteRoot}library/${item.repo_id}/${item.repo_name}${Utils.encodePath(item.path)}`; + let folderUrl = `${siteRoot}library/${item.repo_id}/${encodeURIComponent(item.repo_name)}${Utils.encodePath(item.path)}`; - return { iconUrl, iconTitle, folderUrl }; - } - - render() { - const item = this.props.item; - let { iconUrl, iconTitle, folderUrl } = this.getFolderParams(); - - let shareTo; - const shareType = item.share_type; - if (shareType == 'user') { - shareTo = {item.user_name}; - } else if (shareType == 'group') { - shareTo = {item.group_name}; - } - - let iconVisibility = this.state.showOpIcon ? '' : ' invisible'; - let unshareIconClassName = 'unshare action-icon sf2-icon-x3' + iconVisibility; - - return ( + const desktopItem = ( {iconTitle} {item.folder_name} - {shareTo} + + {item.share_type == 'personal' ? + {item.user_name} : item.group_name} + - + ); + + const mobileItem = ( + + + {iconTitle} + + {item.folder_name} + {Utils.sharePerms(share_permission)} +
+ {`${gettext('Share To:')} ${item.share_type == 'personal' ? item.user_name : item.group_name}`} + + + + +
+
+
+ {gettext('Permission')} + {gettext('Unshare')} +
+
+
+ + + {isPermSelectDialogOpen && + + } +
+ ); + + return this.props.isDesktop ? desktopItem : mobileItem; } } @@ -210,7 +298,6 @@ class ShareAdminFolders extends Component { componentDidMount() { seafileAPI.listSharedFolders().then((res) => { - // res: {data: Array(2), status: 200, statusText: "OK", headers: {…}, config: {…}, …} let items = res.data.map(item => { return new SharedFolderInfo(item); }); @@ -232,7 +319,6 @@ class ShareAdminFolders extends Component { errorMsg: gettext('Error') }); } - } else { this.setState({ loading: false, @@ -242,36 +328,6 @@ class ShareAdminFolders extends Component { }); } - unshareFolder = (item) => { - let options = {}; - options['p'] = item.path; - options.share_type = item.share_type; - if (item.share_type == 'user') { // or group - options.username = item.user_email; - } else { - options.group_id = item.group_id; - } - - seafileAPI.unshareFolder(item.repo_id, options).then((res) => { - let items = this.state.items.filter(folderItem => { - if (item.share_type === 'user') { - return folderItem.user_email !== item.user_email; - } else { - return folderItem.group_id !== item.group_id; - } - }); - this.setState({items: items}); - let message = gettext('Successfully unshared {name}').replace('{name}', item.folder_name); - toaster.success(message); - }).catch(error => { - let errMessage = Utils.getErrorMsg(error); - if (errMessage === gettext('Error')) { - errMessage = gettext('Failed to unshare {name}').replace('{name}', item.folder_name); - } - toaster(errMessage); - }); - } - render() { return (
@@ -287,7 +343,6 @@ class ShareAdminFolders extends Component { sortBy={this.state.sortBy} sortOrder={this.state.sortOrder} sortItems={this.sortItems} - unshareFolder={this.unshareFolder} />
diff --git a/frontend/src/pages/share-admin/libraries.js b/frontend/src/pages/share-admin/libraries.js index 4819f319f3..0a534481fd 100644 --- a/frontend/src/pages/share-admin/libraries.js +++ b/frontend/src/pages/share-admin/libraries.js @@ -64,7 +64,6 @@ class Content extends Component { key={index} isDesktop={isDesktop} item={item} - unshareItem={this.props.unshareItem} />); })} @@ -148,17 +147,46 @@ class Item extends Component { unshare = (e) => { e.preventDefault(); - this.props.unshareItem(this.props.item); + + const item = this.props.item; + const share_type = item.share_type; + let options = { + 'share_type': share_type + }; + if (share_type == 'personal') { + options.user = item.user_email; + } else if (share_type == 'group') { + options.group_id = item.group_id; + } + + seafileAPI.unshareRepo(item.repo_id, options).then((res) => { + this.setState({ + unshared: true + }); + let message = gettext('Successfully unshared {name}').replace('{name}', item.repo_name); + toaster.success(message); + }).catch(error => { + let errMessage = Utils.getErrorMsg(error); + toaster(errMessage); + }); } render() { + if (this.state.unshared) { + return null; + } + let { share_permission, is_admin, isOpIconShown, isPermSelectDialogOpen } = this.state; let item = this.props.item; + Object.assign(item, { + share_permission: share_permission, + is_admin: is_admin + }); + let iconUrl = Utils.getLibIconUrl(item); let iconTitle = Utils.getLibIconTitle(item); let repoUrl = `${siteRoot}library/${item.repo_id}/${encodeURIComponent(item.repo_name)}/`; - let { share_permission, is_admin, isOpIconShown, isPermSelectDialogOpen } = this.state; let shareTo; const shareType = item.share_type; @@ -284,44 +312,6 @@ class ShareAdminLibraries extends Component { }); } - unshareItem = (item) => { - const share_type = item.share_type; - let options = { - 'share_type': share_type - }; - if (share_type == 'personal') { - options.user = item.user_email; - } else if (share_type == 'group') { - options.group_id = item.group_id; - } - - seafileAPI.unshareRepo(item.repo_id, options).then((res) => { - let items = []; - if (item.share_type === 'personal') { - items = this.state.items.filter(repoItem => { - return !(repoItem.user_email === item.user_email && repoItem.repo_id === item.repo_id); - }); - } else if (item.share_type === 'group') { - items = this.state.items.filter(repoItem => { - return !(repoItem.group_id === item.group_id && repoItem.repo_id === item.repo_id); - }); - } else if (item.share_type === 'public') { - items = this.state.items.filter(repoItem => { - return !(repoItem.share_type === 'public' && repoItem.repo_id === item.repo_id); - }); - } - this.setState({items: items}); - let message = gettext('Successfully unshared {name}').replace('{name}', item.repo_name); - toaster.success(message); - }).catch(error => { - let errMessage = Utils.getErrorMsg(error); - if (errMessage === gettext('Error')) { - errMessage = gettext('Failed to unshare {name}').replace('{name}', item.repo_name); - } - toaster(errMessage); - }); - } - sortItems = (sortBy, sortOrder) => { this.setState({ sortBy: sortBy, @@ -345,7 +335,6 @@ class ShareAdminLibraries extends Component { sortBy={this.state.sortBy} sortOrder={this.state.sortOrder} sortItems={this.sortItems} - unshareItem={this.unshareItem} />