refactor

scm-ui/src/config/containers/GlobalPermissionRoleForm.js

@@ -1,7 +1,9 @@
 // @flow
 import React from "react";
-import { InputField, SubmitButton } from "@scm-manager/ui-components";
+import { connect } from "react-redux";
 import { translate } from "react-i18next";
+import type { Role } from "@scm-manager/ui-types";
+import { InputField, SubmitButton } from "@scm-manager/ui-components";
 import PermissionCheckbox from "../../repos/permissions/components/PermissionCheckbox";
 import {
   fetchAvailableVerbs,
@@ -10,15 +12,16 @@ import {
   isFetchVerbsPending
 } from "../modules/roles";
 import { getRepositoryVerbsLink } from "../../modules/indexResource";
-import { connect } from "react-redux";
 
 type Props = {
   submitForm: CustomRoleRequest => void,
-  transmittedName?: string,
+  role?: Role,
   loading?: boolean,
   availableVerbs: string[],
   selectedVerbs: string[],
   verbsLink: string,
+
+  // context objects
   t: string => string,
 
   // dispatch functions
@@ -26,7 +29,7 @@ type Props = {
 };
 
 type State = {
-  name?: string,
+  role: Role,
   verbs: string[]
 };
 
@@ -35,56 +38,56 @@ class GlobalPermissionRoleForm extends React.Component<Props, State> {
     super(props);
 
     this.state = {
-      name: props.transmittedName ? props.transmittedName : "",
+      role: {
+        name: "",
+        verbs: [],
+        system: false,
+        _links: {}
+      },
       verbs: props.availableVerbs
     };
   }
 
-  componentWillMount() {
-    const { fetchAvailableVerbs, verbsLink } = this.props;
-    fetchAvailableVerbs(verbsLink);
-  }
-
   componentDidMount() {
-    const {
+    const { fetchAvailableVerbs, verbsLink, role } = this.props;
-      fetchAvailableVerbs,
-      verbsLink,
-    } = this.props;
     fetchAvailableVerbs(verbsLink);
 
+    if (role) {
+      this.setState({ role: { ...role } });
+    }
   }
-
-  isValid = () => {
-    const { name, verbs } = this.state;
-    return !(this.isFalsy(name) || this.isFalsy(verbs) || verbs.isEmpty());
-  };
 
   isFalsy(value) {
     return !value;
   }
 
+  isValid = () => {
+    const { role } = this.state;
+    return !(
+      this.isFalsy(role) ||
+      this.isFalsy(role.name) ||
+      this.isFalsy(role.verbs)
+    );
+  };
+
   handleNameChange = (name: string) => {
     this.setState({
-      ...this.state,
+      role: {
+        ...this.state.role,
         name
+      }
     });
   };
 
   handleVerbChange = (value: boolean, name: string) => {
-    const { selectedVerbs } = this.state;
+    const { selectedVerbs } = this.props;
     const newVerbs = { ...selectedVerbs, [name]: value };
-    this.setState({ selectedVerbs: newVerbs });
+    this.setState({ verbs: newVerbs });
   };
 
   render() {
-    const {
+    const { loading, availableVerbs, t } = this.props;
-      t,
+    const { role, verbs } = this.state;
-      transmittedName,
-      loading,
-      disabled,
-      availableVerbs
-    } = this.props;
-    const { verbs } = this.state;
 
     const verbSelectBoxes =
       !!availableVerbs &&
@@ -99,7 +102,6 @@ class GlobalPermissionRoleForm extends React.Component<Props, State> {
       ));
 
     return (
-      <div>
       <form onSubmit={this.submit}>
         <div className="columns">
           <div className="column">
@@ -107,8 +109,8 @@ class GlobalPermissionRoleForm extends React.Component<Props, State> {
               name="name"
               label={t("roles.create.name")}
               onChange={this.handleNameChange}
-                value={name ? name : ""}
+              value={role.name ? role.name : ""}
-                disabled={!!transmittedName || disabled}
+              disabled={!!role.name} // || disabled
             />
           </div>
         </div>
@@ -116,14 +118,13 @@ class GlobalPermissionRoleForm extends React.Component<Props, State> {
         <div className="columns">
           <div className="column">
             <SubmitButton
-                disabled={disabled || !this.isValid()}
               loading={loading}
               label={t("roles.create.submit")}
+              //disabled={disabled || !this.isValid()}
             />
           </div>
         </div>
       </form>
-      </div>
     );
   }
 }

scm-ui/src/config/containers/GlobalPermissionRoles.js

@@ -5,14 +5,6 @@ import {withRouter} from "react-router-dom";
 import { translate } from "react-i18next";
 import type { History } from "history";
 import type { Role, PagedCollection } from "@scm-manager/ui-types";
-import {
-  fetchRolesByPage,
-  getRolesFromState,
-  selectListAsCollection,
-  isPermittedToCreateRoles,
-  isFetchRolesPending,
-  getFetchRolesFailure
-} from "../modules/roles";
 import {
   Title,
   Loading,
@@ -21,6 +13,14 @@ import {
   urls,
   CreateButton
 } from "@scm-manager/ui-components";
+import {
+  fetchRolesByPage,
+  getRolesFromState,
+  selectListAsCollection,
+  isPermittedToCreateRoles,
+  isFetchRolesPending,
+  getFetchRolesFailure
+} from "../modules/roles";
 import PermissionRoleTable from "../components/table/PermissionRoleTable";
 import { getRolesLink } from "../../modules/indexResource";
 type Props = {

scm-ui/src/config/modules/roles.js

@@ -338,7 +338,7 @@ export function getRolesFromState(state: Object) {
 }
 
 export function getVerbsFromState(state: Object) {
-  return state.roles.verbs.verbs
+  return state.roles.verbs.verbs;
 }
 
 export function isFetchRolesPending(state: Object) {