fix editing permissions

This commit is contained in:
or-else
2021-08-13 20:35:39 -07:00
parent d741f6ec92
commit d6ca3f9f3c
20 changed files with 106 additions and 18 deletions
+2 -2
View File
@@ -653,8 +653,8 @@ table.permission-editor td.checkbox {
} }
.permission-editor .material-icons { .permission-editor .material-icons {
font-size: 1.1rem; font-size: 1.3rem;
line-height: 1.1rem; line-height: 1.3rem;
} }
.material-icons.clickable { .material-icons.clickable {
+1 -1
View File
File diff suppressed because one or more lines are too long
+7
View File
@@ -1340,5 +1340,12 @@
"description": "Alert title when deleting the topic.", "description": "Alert title when deleting the topic.",
"missing": false, "missing": false,
"obsolete": false "obsolete": false
},
"permissions_user": {
"translation": "",
"defaultMessage": "User's Permissions",
"description": "Title for editing user's permissions",
"missing": true,
"obsolete": false
} }
} }
+7
View File
@@ -1361,5 +1361,12 @@
"description": "Alert title when deleting the topic.", "description": "Alert title when deleting the topic.",
"missing": false, "missing": false,
"obsolete": false "obsolete": false
},
"permissions_user": {
"translation": "User's Permissions",
"defaultMessage": "User's Permissions",
"description": "Title for editing user's permissions",
"missing": false,
"obsolete": false
} }
} }
+7
View File
@@ -1361,5 +1361,12 @@
"description": "Alert title when deleting the topic.", "description": "Alert title when deleting the topic.",
"missing": false, "missing": false,
"obsolete": false "obsolete": false
},
"permissions_user": {
"translation": "",
"defaultMessage": "User's Permissions",
"description": "Title for editing user's permissions",
"missing": true,
"obsolete": false
} }
} }
+4
View File
@@ -569,6 +569,10 @@
"defaultMessage": "Authenticated", "defaultMessage": "Authenticated",
"description": "Title for editing default authenticated permissions" "description": "Title for editing default authenticated permissions"
}, },
"permissions_user": {
"defaultMessage": "User's Permissions",
"description": "Title for editing user's permissions"
},
"phone_dative": { "phone_dative": {
"defaultMessage": "phone", "defaultMessage": "phone",
"description": "Dative case of 'phone', i.e. 'phone' in 'by phone'" "description": "Dative case of 'phone', i.e. 'phone' in 'by phone'"
+7
View File
@@ -1371,5 +1371,12 @@
"description": "Alert title when deleting the topic.", "description": "Alert title when deleting the topic.",
"missing": false, "missing": false,
"obsolete": false "obsolete": false
},
"permissions_user": {
"translation": "",
"defaultMessage": "User's Permissions",
"description": "Title for editing user's permissions",
"missing": true,
"obsolete": false
} }
} }
+7
View File
@@ -1361,5 +1361,12 @@
"description": "Alert title when deleting the topic.", "description": "Alert title when deleting the topic.",
"missing": false, "missing": false,
"obsolete": false "obsolete": false
},
"permissions_user": {
"translation": "",
"defaultMessage": "User's Permissions",
"description": "Title for editing user's permissions",
"missing": true,
"obsolete": false
} }
} }
+7
View File
@@ -1361,5 +1361,12 @@
"description": "Alert title when deleting the topic.", "description": "Alert title when deleting the topic.",
"missing": false, "missing": false,
"obsolete": false "obsolete": false
},
"permissions_user": {
"translation": "",
"defaultMessage": "User's Permissions",
"description": "Title for editing user's permissions",
"missing": true,
"obsolete": false
} }
} }
+7
View File
@@ -1360,5 +1360,12 @@
"description": "Alert title when deleting the topic.", "description": "Alert title when deleting the topic.",
"missing": false, "missing": false,
"obsolete": false "obsolete": false
},
"permissions_user": {
"translation": "",
"defaultMessage": "User's Permissions",
"description": "Title for editing user's permissions",
"missing": true,
"obsolete": false
} }
} }
+7
View File
@@ -1361,5 +1361,12 @@
"description": "Alert title when deleting the topic.", "description": "Alert title when deleting the topic.",
"missing": false, "missing": false,
"obsolete": false "obsolete": false
},
"permissions_user": {
"translation": "",
"defaultMessage": "User's Permissions",
"description": "Title for editing user's permissions",
"missing": true,
"obsolete": false
} }
} }
+1 -1
View File
File diff suppressed because one or more lines are too long
+8 -1
View File
@@ -66,6 +66,11 @@ const messages = defineMessages({
defaultMessage: 'Anonymous', defaultMessage: 'Anonymous',
description: 'Title for editing default anonymous permissions' description: 'Title for editing default anonymous permissions'
}, },
perm_user: {
id: 'permissions_user',
defaultMessage: 'User\'s Permissions',
description: 'Title for editing user\'s permissions'
},
edit_permissions: { edit_permissions: {
id: 'menu_item_edit_permissions', id: 'menu_item_edit_permissions',
defaultMessage: 'Edit permissions', defaultMessage: 'Edit permissions',
@@ -252,6 +257,7 @@ class InfoView extends React.Component {
} }
handlePermissionsChanged(which, perm) { handlePermissionsChanged(which, perm) {
console.log("handlePermissionsChanged", which, perm);
switch (which) { switch (which) {
case 'auth': case 'auth':
this.props.onTopicDescUpdate(this.props.topic, null, null, {auth: perm}); this.props.onTopicDescUpdate(this.props.topic, null, null, {auth: perm});
@@ -407,6 +413,7 @@ class InfoView extends React.Component {
render() { render() {
const args = (this.props.panel || 'info').split('/'); const args = (this.props.panel || 'info').split('/');
console.log("args:", args);
const view = args[0]; const view = args[0];
args.shift(); args.shift();
@@ -449,7 +456,7 @@ class InfoView extends React.Component {
userTitle={this.state.userPermissionsTitle} userTitle={this.state.userPermissionsTitle}
item={this.state.userPermissionsEdited} item={this.state.userPermissionsEdited}
userAvatar={this.state.userPermissionsAvatar} userAvatar={this.state.userPermissionsAvatar}
onSubmit={this.handlePermissionsChanged.bind(...args)} onSubmit={(mode) => this.handlePermissionsChanged(args[0], mode)}
onCancel={this.handleBackNavigate} /> onCancel={this.handleBackNavigate} />
: :
view == 'general' ? view == 'general' ?
+3 -3
View File
@@ -67,7 +67,7 @@ class PermissionsEditor extends React.Component {
handleChange(val) { handleChange(val) {
let mode = this.state.mode; let mode = this.state.mode;
let idx = mode.indexOf(val); const idx = mode.indexOf(val);
if (idx == -1) { if (idx == -1) {
mode += val; mode += val;
} else { } else {
@@ -78,8 +78,8 @@ class PermissionsEditor extends React.Component {
handleSubmit() { handleSubmit() {
// Normalize string, otherwise cannot check if mode has changed. // Normalize string, otherwise cannot check if mode has changed.
var mode = (this.state.mode || 'N').split('').sort().join(''); const mode = (this.state.mode || 'N').split('').sort().join('');
var before = (this.props.mode || 'N').split('').sort().join('') const before = (this.props.mode || 'N').split('').sort().join('')
if (mode !== before) { if (mode !== before) {
this.props.onSubmit(mode); this.props.onSubmit(mode);
} else { } else {
+16 -5
View File
File diff suppressed because one or more lines are too long
+1 -1
View File
File diff suppressed because one or more lines are too long
+1 -1
View File
File diff suppressed because one or more lines are too long
File diff suppressed because one or more lines are too long
+11 -1
View File
File diff suppressed because one or more lines are too long
+1 -1
View File
File diff suppressed because one or more lines are too long