From 03e25086a7f5302eb2df296f8c6fba5135fe19c9 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Ren=C3=A9=20Pfeuffer?= <rene.pfeuffer@cloudogu.com>
Date: Thu, 4 Apr 2019 13:03:51 +0200
Subject: [PATCH] Use transmitted branch name

---
 scm-ui/src/repos/branches/components/BranchForm.js   | 6 ++++--
 scm-ui/src/repos/branches/containers/CreateBranch.js | 9 ++++-----
 2 files changed, 8 insertions(+), 7 deletions(-)

diff --git a/scm-ui/src/repos/branches/components/BranchForm.js b/scm-ui/src/repos/branches/components/BranchForm.js
index 67440620cb..2938b06a2a 100644
--- a/scm-ui/src/repos/branches/components/BranchForm.js
+++ b/scm-ui/src/repos/branches/components/BranchForm.js
@@ -30,7 +30,8 @@ class BranchForm extends React.Component<Props, State> {
     super(props);
 
     this.state = {
-      nameValidationError: false
+      nameValidationError: false,
+      name: props.transmittedName
     };
   }
 
@@ -55,7 +56,7 @@ class BranchForm extends React.Component<Props, State> {
   };
 
   render() {
-    const { t, branches, loading } = this.props;
+    const { t, branches, loading, transmittedName } = this.props;
     const { name } = this.state;
     orderBranches(branches);
     const options = branches.map(branch => ({
@@ -82,6 +83,7 @@ class BranchForm extends React.Component<Props, State> {
                 value={name ? name : ""}
                 validationError={this.state.nameValidationError}
                 errorMessage={t("validation.branch.nameInvalid")}
+                disabled={!!transmittedName}
               />
             </div>
           </div>
diff --git a/scm-ui/src/repos/branches/containers/CreateBranch.js b/scm-ui/src/repos/branches/containers/CreateBranch.js
index b801c98f1c..1b0ef76408 100644
--- a/scm-ui/src/repos/branches/containers/CreateBranch.js
+++ b/scm-ui/src/repos/branches/containers/CreateBranch.js
@@ -56,7 +56,7 @@ class CreateBranch extends React.Component<Props> {
     this.props.createBranch(branch, () => this.branchCreated(branch));
   };
 
-  matchesTransmittedName = (url: string) => {
+  transmittedName = (url: string) => {
     const params = queryString.parse(url);
     return params.name;
   };
@@ -68,7 +68,7 @@ class CreateBranch extends React.Component<Props> {
       return <ErrorNotification error={error} />;
     }
 
-    if (!branches) {
+    if (loading || !branches) {
       return <Loading />;
     }
 
@@ -80,7 +80,7 @@ class CreateBranch extends React.Component<Props> {
           loading={loading}
           repository={repository}
           branches={branches}
-          transmittedName={this.matchesTransmittedName(location.search)}
+          transmittedName={this.transmittedName(location.search)}
         />
       </>
     );
@@ -107,8 +107,7 @@ const mapDispatchToProps = dispatch => {
 
 const mapStateToProps = (state, ownProps) => {
   const { repository } = ownProps;
-  const loading =
-    isFetchBranchesPending(state, repository) || isCreateBranchPending(state);
+  const loading = isFetchBranchesPending(state, repository); //|| isCreateBranchPending(state);
   const error =
     getFetchBranchesFailure(state, repository) || getCreateBranchFailure(state);
   const branches = getBranches(state, repository);