From: Laurence Maultsby <laurence.maultsby@riftio.com>
Date: Thu, 4 May 2017 16:47:06 +0000 (-0400)
Subject: User Enable/Disable
X-Git-Url: https://osm.etsi.org/gitweb/?a=commitdiff_plain;h=04997f0db765e39bd3a1fa76d40043eda9cdd326;p=osm%2FUI.git

User Enable/Disable

Signed-off-by: Laurence Maultsby <laurence.maultsby@riftio.com>
---

diff --git a/skyquake/framework/widgets/form_controls/input.jsx b/skyquake/framework/widgets/form_controls/input.jsx
index 1563f2912..d6056149e 100644
--- a/skyquake/framework/widgets/form_controls/input.jsx
+++ b/skyquake/framework/widgets/form_controls/input.jsx
@@ -97,10 +97,12 @@ function buildRadioButtons(props) {
        <div className={className}>
             {
                 props.options.map((o,i) => {
+                    let label = o.label || o;
+                    let value = o.value || o;
                     return (
                         <label key={i}>
-                            {o.label}
-                            <input type="radio" checked={props.value == o.value} value={o.value} onChange={props.onChange} />
+                            {label}
+                            <input type="radio" checked={props.value == value} value={value} onChange={props.onChange} />
                         </label>
                     )
                 })
diff --git a/skyquake/plugins/user_management/src/dashboard/dashboard.jsx b/skyquake/plugins/user_management/src/dashboard/dashboard.jsx
index cb6b9a017..4432881ab 100644
--- a/skyquake/plugins/user_management/src/dashboard/dashboard.jsx
+++ b/skyquake/plugins/user_management/src/dashboard/dashboard.jsx
@@ -19,6 +19,7 @@ import SelectOption from 'widgets/form_controls/selectOption.jsx';
 import 'widgets/form_controls/formControls.scss';
 import imgAdd from '../../node_modules/open-iconic/svg/plus.svg'
 import imgRemove from '../../node_modules/open-iconic/svg/trash.svg';
+import {merge} from 'lodash';
 
 import ROLES from 'utils/roleConstants.js';
 const PLATFORM = ROLES.PLATFORM;
@@ -107,6 +108,12 @@ class UserManagementDashboard extends React.Component {
         if(this.state['new-password'] != this.state['confirm-password']) {
             this.props.actions.showNotification('Passwords do not match')
         } else {
+            let isDisabled = {};
+            if (this.state.disabled == "TRUE") {
+                let isDisabled = {
+                    disabled: [null]
+                }
+            }
             this.Store.createUser({
                 'user-name': this.state['user-name'],
                 'user-domain': this.state['user-domain'],
@@ -121,22 +128,35 @@ class UserManagementDashboard extends React.Component {
         e.stopPropagation();
         let validatedPasswords = validatePasswordFields(this.state);
         if(validatedPasswords) {
+            let isDisabled = {};
+            let password = {};
+            if (self.state.disabled == "TRUE") {
+                isDisabled = {
+                    disabled: [null]
+                }
+            }
+            if (this.state['new-password'] != '') {
+                password = {'password': this.state['new-password']}
+            } else {
+                password = {
+                    'password': this.state.currentPassword
+                }
+            }
             this.Store.updateUser(_.merge({
                             'user-name': this.state['user-name'],
-                            'user-domain': this.state['user-domain'],
-                            'password': this.state['new-password']
-                        }));
+                            'user-domain': this.state['user-domain']
+                        }, _.merge(isDisabled, password)));
         }
         function validatePasswordFields(state) {
             let oldOne = state['old-password'];
             let newOne = state['new-password'];
             let confirmOne = state['confirm-password'];
             if(true) {
-                if(!oldOne || !newOne) {
-                     self.props.actions.showNotification('Please fill in all fields.');
-                    return false;
-                }
-                if(oldOne == newOne) {
+                // if(!oldOne || !newOne) {
+                //      self.props.actions.showNotification('Please fill in all fields.');
+                //     return false;
+                // }
+                if((oldOne || newOne) && (oldOne == newOne)) {
                     self.props.actions.showNotification('Your new password must not match your old one');
                     return false;
                 }
@@ -172,12 +192,7 @@ class UserManagementDashboard extends React.Component {
     disableChange = (e) => {
         let value = e.target.value;
         value = value.toUpperCase();
-        if (value=="TRUE") {
-            value = true;
-        } else {
-            value = false;
-        }
-        console.log(value)
+        this.actions.handleDisabledChange(value);
     }
     render() {
         let self = this;
@@ -280,6 +295,13 @@ class UserManagementDashboard extends React.Component {
                                         : <Input  readonly={state.isReadOnly}  label="Username" value={state['user-name']} onChange={this.updateInput.bind(null, 'user-name')} />
                                 }
                                 <Input readonly={true} label="Domain" value={state['user-domain']}  onChange={this.updateInput.bind(null, 'user-domain')}></Input>
+                                <Input
+                                    type="radiogroup"
+                                    onChange={this.disableChange}
+                                    label="Disabled"
+                                    value={this.state.disabled}
+                                    options={["TRUE","FALSE"]}
+                                    readonly={state.isReadOnly} />
                             </FormSection>
                             <FormSection title="PLATFORM ROLES" style={{display:'none'}}>
                                 <Input label="Super Admin" onChange={this.platformChange.bind(null, 'super_admin')} checked={state.platformRoles.super_admin} type="checkbox" />
diff --git a/skyquake/plugins/user_management/src/dashboard/userMgmtActions.js b/skyquake/plugins/user_management/src/dashboard/userMgmtActions.js
index a0a34d9b3..7277ce115 100644
--- a/skyquake/plugins/user_management/src/dashboard/userMgmtActions.js
+++ b/skyquake/plugins/user_management/src/dashboard/userMgmtActions.js
@@ -20,6 +20,7 @@ module.exports = function(Alt) {
                                        'handleUpdateUser',
                                        'updateUserSuccess',
                                        'createUserSuccess',
-                                       'deleteUserSuccess'
+                                       'deleteUserSuccess',
+                                       'handleDisabledChange'
                                        );
 }
diff --git a/skyquake/plugins/user_management/src/dashboard/userMgmtStore.js b/skyquake/plugins/user_management/src/dashboard/userMgmtStore.js
index 19952fba6..ada70677e 100644
--- a/skyquake/plugins/user_management/src/dashboard/userMgmtStore.js
+++ b/skyquake/plugins/user_management/src/dashboard/userMgmtStore.js
@@ -12,7 +12,7 @@ export default class UserManagementStore {
         this.users = [];
         this['user-name'] = '';
         this['user-domain'] = 'system';
-        this.disabled = false;
+        this.disabled = "FALSE";
         this.platformRoles = {
             super_admin: false,
             platform_admin: false,
@@ -70,8 +70,9 @@ export default class UserManagementStore {
             'user-name': user['user-name'],
             'user-domain': user['user-domain'],
             platformRoles: user.platformRoles || this.platformRoles,
-            disabled: user.disabled || this.disabled,
-            projectRoles: user.projectRoles || this.projectRoles
+            disabled: user.hasOwnProperty('disabled').toString().toUpperCase(),
+            projectRoles: user.projectRoles || this.projectRoles,
+            currentPassword: user.password
         }
         let state = _.merge({
             activeIndex: userIndex,
@@ -168,6 +169,9 @@ export default class UserManagementStore {
     }
     handleCreateUser() {
 
+    }
+    handleDisabledChange(value) {
+        this.disabled = value;
     }
     handleUpdateUser() {