Adding IsBusy to Subview

Author: StanleyGoldman

src/UnityExtension/Assets/Editor/GitHub.Unity/UI/AuthenticationView.cs

@@ -119,6 +119,11 @@ public override void OnGUI()
             GUILayout.EndScrollView();
         }
 
+        public override bool IsBusy
+        {
+            get { return isBusy; }
+        }
+
         private void HandleEnterPressed()
         {
             if (Event.current.type != EventType.KeyDown)

src/UnityExtension/Assets/Editor/GitHub.Unity/UI/BranchesView.cs

@@ -150,6 +150,11 @@ public override void OnGUI()
             }
         }
 
+        public override bool IsBusy
+        {
+            get { return false; }
+        }
+
         public void OnEmbeddedGUI()
         {
             scroll = GUILayout.BeginScrollView(scroll);

src/UnityExtension/Assets/Editor/GitHub.Unity/UI/ChangesView.cs

@@ -19,9 +19,9 @@ class ChangesView : Subview
         private const string OneChangedFileLabel = "1 changed file";
         private const string NoChangedFilesLabel = "No changed files";
 
-        [NonSerialized] private bool busy = true;
-        [SerializeField] private string commitBody = "";
+        [NonSerialized] private bool isBusy = true;
 
+        [SerializeField] private string commitBody = "";
         [SerializeField] private string commitMessage = "";
         [SerializeField] private string currentBranch = "[unknown]";
         [SerializeField] private Vector2 horizontalScroll;
@@ -52,6 +52,11 @@ public override void OnDisable()
             Repository.OnStatusUpdated -= RunStatusUpdateOnMainThread;
         }
 
+        public override bool IsBusy
+        {
+            get { return isBusy; }
+        }
+
         private void RunStatusUpdateOnMainThread(GitStatus status)
         {
             new ActionTask(TaskManager.Token, _ => OnStatusUpdate(status))
@@ -72,10 +77,9 @@ private void OnStatusUpdate(GitStatus update)
             // (Re)build tree
             tree.UpdateEntries(update.Entries.Where(x => x.Status != GitFileStatus.Ignored).ToList());
 
-            busy = false;
+            isBusy = false;
         }
 
-
         public override void OnGUI()
         {
             GUILayout.BeginHorizontal();
@@ -144,7 +148,7 @@ private void OnCommitDetailsAreaGUI()
                     GUILayout.Space(Styles.CommitAreaPadding);
 
                     // Disable committing when already committing or if we don't have all the data needed
-                    EditorGUI.BeginDisabledGroup(busy || string.IsNullOrEmpty(commitMessage) || !tree.CommitTargets.Any(t => t.Any));
+                    EditorGUI.BeginDisabledGroup(isBusy || string.IsNullOrEmpty(commitMessage) || !tree.CommitTargets.Any(t => t.Any));
                     {
                         GUILayout.BeginHorizontal();
                         {
@@ -187,7 +191,7 @@ private void SelectNone()
         private void Commit()
         {
             // Do not allow new commits before we have received one successful update
-            busy = true;
+            isBusy = true;
 
             var files = Enumerable.Range(0, tree.Entries.Count)
                 .Where(i => tree.CommitTargets[i].All)
@@ -212,7 +216,7 @@ private void Commit()
                     {
                         commitMessage = "";
                         commitBody = "";
-                        busy = false;
+                        isBusy = false;
                     }).Start();
         }
     }

src/UnityExtension/Assets/Editor/GitHub.Unity/UI/ChangesetTreeView.cs

@@ -86,6 +86,11 @@ public override void OnGUI()
             GUILayout.EndVertical();
         }
 
+        public override bool IsBusy
+        {
+            get { return false; }
+        }
+
         private void OnCommitTreeChange()
         {
             Height = 0f;
@@ -276,6 +281,7 @@ private void TreeNode(FileTreeNode node)
         }
 
         public float Height { get; protected set; }
+
         public bool Readonly { get; set; }
 
         public IList<GitStatusEntry> Entries

src/UnityExtension/Assets/Editor/GitHub.Unity/UI/HistoryView.cs

@@ -47,6 +47,7 @@ class HistoryView : Subview
         [NonSerialized] private int selectionIndex;
         [NonSerialized] private bool updated = true;
         [NonSerialized] private bool useScrollTime;
+        [NonSerialized] private bool isBusy;
 
 #if ENABLE_BROADMODE
         [SerializeField] private bool broadMode;
@@ -60,7 +61,6 @@ class HistoryView : Subview
 
         [SerializeField] private ChangesetTreeView changesetTree = new ChangesetTreeView();
         [SerializeField] private List<GitLogEntry> history = new List<GitLogEntry>();
-        [SerializeField] private bool isBusy;
         [SerializeField] private string currentRemote;
         [SerializeField] private bool isPublished;
 
@@ -146,6 +146,11 @@ public override void OnGUI()
 #endif
         }
 
+        public override bool IsBusy
+        {
+            get { return isBusy; }
+        }
+
 #if ENABLE_BROADMODE
         public void OnBroadGUI()
         {

src/UnityExtension/Assets/Editor/GitHub.Unity/UI/PublishView.cs

@@ -232,6 +232,11 @@ public override void OnGUI()
             EditorGUI.EndDisabledGroup();
         }
 
+        public override bool IsBusy
+        {
+            get { return isBusy; }
+        }
+
         private bool IsFormValid
         {
             get { return !string.IsNullOrEmpty(repoName) && !isBusy && selectedOwner != 0; }

src/UnityExtension/Assets/Editor/GitHub.Unity/UI/SettingsView.cs

@@ -67,6 +67,7 @@ class SettingsView : Subview
         private const string DefaultRepositoryRemoteName = "origin";
 
         [NonSerialized] private int newGitIgnoreRulesSelection = -1;
+        [NonSerialized] private bool isBusy;
 
         [SerializeField] private string gitName;
         [SerializeField] private string gitEmail;
@@ -78,7 +79,6 @@ class SettingsView : Subview
         [SerializeField] private string repositoryRemoteName;
         [SerializeField] private string repositoryRemoteUrl;
         [SerializeField] private Vector2 scroll;
-        [SerializeField] private bool isBusy;
         [SerializeField] private int lockedFileSelection = -1;
         [SerializeField] private bool hasRemote;
         [SerializeField] private bool remoteHasChanged;
@@ -121,6 +121,11 @@ public override void OnRepositoryChanged(IRepository oldRepository)
             Refresh();
         }
 
+        public override bool IsBusy
+        {
+            get { return isBusy; }
+        }
+
         public override void Refresh()
         {
             base.Refresh();

src/UnityExtension/Assets/Editor/GitHub.Unity/UI/Subview.cs

@@ -57,7 +57,7 @@ public virtual void OnRepositoryChanged(IRepository oldRepository)
         public IApplicationManager Manager { get { return Parent.Manager; } }
         public IRepository Repository { get { return Parent.Repository; } }
         public bool HasRepository { get { return Parent.HasRepository; } }
-
+        public abstract bool IsBusy { get; }
         protected ITaskManager TaskManager { get { return Manager.TaskManager; } }
         protected IGitClient GitClient { get { return Manager.GitClient; } }
         protected IEnvironment Environment { get { return Manager.Environment; } }