fixes #386: removed overly agressive new/swtich branch disabling

Author: telamonian

src/components/BranchHeader.tsx

@@ -50,15 +50,23 @@ export class BranchHeader extends React.Component<
 
   /** Switch current working branch */
   async switchBranch(branchName: string) {
-    await this.props.model.checkout({ branchname: branchName });
+    const result = await this.props.model.checkout({ branchname: branchName });
+    if (result.code !== 0) {
+      showErrorMessage('Error switching branch', result.message);
+    }
+
     this.toggleSelect();
   }
 
   createNewBranch = async (branchName: string) => {
-    await this.props.model.checkout({
+    const result = await this.props.model.checkout({
       newBranch: true,
       branchname: branchName
     });
+    if (result.code !== 0) {
+      showErrorMessage('Error creating new branch', result.message);
+    }
+
     this.toggleNewBranchBox();
   };
 

src/components/GitPanel.tsx

@@ -26,7 +26,6 @@ export interface IGitSessionNodeState {
   stagedFiles: Git.IStatusFileResult[];
   unstagedFiles: Git.IStatusFileResult[];
   untrackedFiles: Git.IStatusFileResult[];
-  hasChangedFiles: boolean;
 
   isHistoryVisible: boolean;
 }
@@ -50,7 +49,6 @@ export class GitPanel extends React.Component<
       branches: [],
       currentBranch: '',
       upstreamBranch: '',
-      hasChangedFiles: false,
       pastCommits: [],
       stagedFiles: [],
       unstagedFiles: [],
@@ -127,14 +125,9 @@ export class GitPanel extends React.Component<
       let stagedFiles = new Array<Git.IStatusFileResult>();
       let unstagedFiles = new Array<Git.IStatusFileResult>();
       let untrackedFiles = new Array<Git.IStatusFileResult>();
-      let changedFiles = 0;
       let statusFiles = this.props.model.status;
       if (statusFiles.length > 0) {
         for (let i = 0; i < statusFiles.length; i++) {
-          // If file has been changed
-          if (statusFiles[i].x !== '?' && statusFiles[i].x !== '!') {
-            changedFiles++;
-          }
           // If file is untracked
           if (statusFiles[i].x === '?' && statusFiles[i].y === '?') {
             untrackedFiles.push(statusFiles[i]);
@@ -154,8 +147,7 @@ export class GitPanel extends React.Component<
       this.setState({
         stagedFiles: stagedFiles,
         unstagedFiles: unstagedFiles,
-        untrackedFiles: untrackedFiles,
-        hasChangedFiles: changedFiles > 0
+        untrackedFiles: untrackedFiles
       });
     }
   };
@@ -218,11 +210,7 @@ export class GitPanel extends React.Component<
             upstreamBranch={this.state.upstreamBranch}
             stagedFiles={this.state.stagedFiles}
             data={this.state.branches}
-            // No uncommitted changed files, allow switching branches
-            // No committed files ever, disable switching branches
-            disabled={
-              this.state.hasChangedFiles || this.state.pastCommits.length === 0
-            }
+            disabled={this.state.pastCommits.length === 0}
             toggleSidebar={this.toggleSidebar}
             sideBarExpanded={this.state.isHistoryVisible}
           />

src/model.ts

@@ -539,19 +539,15 @@ export class GitExtension implements IGitExtension, IDisposable {
    * If a filename is provided, check the file out
    * If nothing is provided, check all files out
    */
-  async checkout(options?: Git.ICheckoutOptions): Promise<Response> {
+  async checkout(options?: Git.ICheckoutOptions): Promise<Git.ICheckoutResult> {
     await this.ready;
     const path = this.pathRepository;
 
     if (path === null) {
-      return Promise.resolve(
-        new Response(
-          JSON.stringify({
-            code: -1,
-            message: 'Not in a git repository.'
-          })
-        )
-      );
+      return Promise.resolve({
+        code: -1,
+        message: 'Not in a git repository.'
+      });
     }
 
     const body = {
@@ -587,7 +583,7 @@ export class GitExtension implements IGitExtension, IDisposable {
       } else {
         this.refreshStatus();
       }
-      return response;
+      return response.json();
     } catch (err) {
       throw new ServerConnection.NetworkError(err);
     }

src/tokens.ts

@@ -132,7 +132,7 @@ export interface IGitExtension {
    *
    * @param options Checkout options
    */
-  checkout(options?: Git.ICheckoutOptions): Promise<Response>;
+  checkout(options?: Git.ICheckoutOptions): Promise<Git.ICheckoutResult>;
 
   /**
    * Make request to commit all staged files in repository
@@ -270,9 +270,8 @@ export namespace Git {
     filename?: string;
   }
 
-  /** Interface for GitShowPrefix request result,
-   * has the prefix path of a directory in a repository,
-   * with respect to the root directory.
+  /** Interface for GitCheckout request result.
+   * For reporting errors in checkout
    */
   export interface ICheckoutResult {
     code: number;
@@ -292,7 +291,7 @@ export namespace Git {
   }
 
   /** Interface for GitBranch request result,
-   * has the result of changing the current working branch
+   * has the result of fetching info on all branches
    */
   export interface IBranchResult {
     code: number;