Properties should be at the end of classes

shana · shana · commit bf72ab6a6d97 · 2017-09-08T19:50:55.000+02:00
diff --git a/src/UnityExtension/Assets/Editor/GitHub.Unity/UI/AuthenticationView.cs b/src/UnityExtension/Assets/Editor/GitHub.Unity/UI/AuthenticationView.cs
@@ -30,33 +30,6 @@ class AuthenticationView : Subview
         [NonSerialized] private bool enterPressed;
         [NonSerialized] private string password = "";
 
-        [NonSerialized] private AuthenticationService authenticationService;
-        private AuthenticationService AuthenticationService
-        {
-            get
-            {
-                if (authenticationService == null)
-                {
-                    UriString host;
-                    if (Repository != null && Repository.CloneUrl != null && Repository.CloneUrl.IsValidUri)
-                    {
-                        host = new UriString(Repository.CloneUrl.ToRepositoryUri()
-                            .GetComponents(UriComponents.SchemeAndServer, UriFormat.SafeUnescaped));
-                    }
-                    else
-                    {
-                        host = UriString.ToUriString(HostAddress.GitHubDotComHostAddress.WebUri);
-                    }
-
-                    AuthenticationService = new AuthenticationService(host, Platform.Keychain);
-                }
-                return authenticationService;
-            }
-            set
-            {
-                authenticationService = value;
-            }
-        }
 
         public override void InitializeView(IView parent)
         {
@@ -124,11 +97,6 @@ public override void OnGUI()
             GUILayout.EndScrollView();
         }
 
-        public override bool IsBusy
-        {
-            get { return isBusy; }
-        }
-
         private void HandleEnterPressed()
         {
             if (Event.current.type != EventType.KeyDown)
@@ -255,5 +223,37 @@ private void ShowErrorMessage()
                 GUILayout.Label(errorMessage, Styles.ErrorLabel);
             }
         }
+
+        private AuthenticationService AuthenticationService
+        {
+            get
+            {
+                if (authenticationService == null)
+                {
+                    UriString host;
+                    if (Repository != null && Repository.CloneUrl != null && Repository.CloneUrl.IsValidUri)
+                    {
+                        host = new UriString(Repository.CloneUrl.ToRepositoryUri()
+                            .GetComponents(UriComponents.SchemeAndServer, UriFormat.SafeUnescaped));
+                    }
+                    else
+                    {
+                        host = UriString.ToUriString(HostAddress.GitHubDotComHostAddress.WebUri);
+                    }
+
+                    AuthenticationService = new AuthenticationService(host, Platform.Keychain);
+                }
+                return authenticationService;
+            }
+            set
+            {
+                authenticationService = value;
+            }
+        }
+
+        public override bool IsBusy
+        {
+            get { return isBusy; }
+        }
     }
 }
diff --git a/src/UnityExtension/Assets/Editor/GitHub.Unity/UI/BranchesView.cs b/src/UnityExtension/Assets/Editor/GitHub.Unity/UI/BranchesView.cs
@@ -128,11 +128,6 @@ public override void OnGUI()
             OnEmbeddedGUI();
         }
 
-        public override bool IsBusy
-        {
-            get { return false; }
-        }
-
         public void OnEmbeddedGUI()
         {
             scroll = GUILayout.BeginScrollView(scroll);
@@ -749,6 +744,11 @@ private void OnTreeNodeChildrenGUI(BranchTreeNode node)
             }
         }
 
+        public override bool IsBusy
+        {
+            get { return false; }
+        }
+
         private enum NodeType
         {
             Folder,
diff --git a/src/UnityExtension/Assets/Editor/GitHub.Unity/UI/ChangesView.cs b/src/UnityExtension/Assets/Editor/GitHub.Unity/UI/ChangesView.cs
@@ -52,11 +52,6 @@ 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))
@@ -217,5 +212,10 @@ private void Commit()
                         isBusy = false;
                     }).Start();
         }
+
+        public override bool IsBusy
+        {
+            get { return isBusy; }
+        }
     }
 }
diff --git a/src/UnityExtension/Assets/Editor/GitHub.Unity/UI/ChangesetTreeView.cs b/src/UnityExtension/Assets/Editor/GitHub.Unity/UI/ChangesetTreeView.cs
@@ -86,17 +86,6 @@ public override void OnGUI()
             GUILayout.EndVertical();
         }
 
-        public override bool IsBusy
-        {
-            get { return false; }
-        }
-
-        private void OnCommitTreeChange()
-        {
-            Height = 0f;
-            Redraw();
-        }
-
         public void UpdateEntries(IList<GitStatusEntry> newEntries)
         {
             // Handle the empty list scenario
@@ -117,6 +106,12 @@ public void UpdateEntries(IList<GitStatusEntry> newEntries)
             OnCommitTreeChange();
         }
 
+        private void OnCommitTreeChange()
+        {
+            Height = 0f;
+            Redraw();
+        }
+
         private void TreeNode(FileTreeNode node)
         {
             GUILayout.Space(Styles.TreeVerticalSpacing);
@@ -280,6 +275,11 @@ private void TreeNode(FileTreeNode node)
             GUILayout.EndHorizontal();
         }
 
+        public override bool IsBusy
+        {
+            get { return false; }
+        }
+
         public float Height { get; protected set; }
 
         public bool Readonly { get; set; }
diff --git a/src/UnityExtension/Assets/Editor/GitHub.Unity/UI/HistoryView.cs b/src/UnityExtension/Assets/Editor/GitHub.Unity/UI/HistoryView.cs
@@ -125,10 +125,6 @@ public override void OnGUI()
             OnEmbeddedGUI();
         }
 
-        public override bool IsBusy
-        {
-            get { return isBusy; }
-        }
         private void AttachHandlers(IRepository repository)
         {
             if (repository == null)
@@ -750,6 +746,11 @@ private void DrawTimelineRectAroundIconRect(Rect parentRect, Rect iconRect)
             EditorGUI.DrawRect(bottomTimelineRect, timelineBarColor);
         }
 
+        public override bool IsBusy
+        {
+            get { return isBusy; }
+        }
+
         private float EntryHeight
         {
             get { return Styles.HistoryEntryHeight + Styles.HistoryEntryPadding; }
diff --git a/src/UnityExtension/Assets/Editor/GitHub.Unity/UI/PopupWindow.cs b/src/UnityExtension/Assets/Editor/GitHub.Unity/UI/PopupWindow.cs
@@ -93,11 +93,6 @@ public override void OnSelectionChange()
             ActiveView.OnSelectionChange();
         }
 
-        public override bool IsBusy
-        {
-            get { return ActiveView.IsBusy; }
-        }
-
         public override void Finish(bool result)
         {
             OnClose.SafeInvoke(result);
@@ -113,6 +108,11 @@ public override void OnDestroy()
             OnClose = null;
         }
 
+        public override bool IsBusy
+        {
+            get { return ActiveView.IsBusy; }
+        }
+
         private Subview ActiveView
         {
             get
diff --git a/src/UnityExtension/Assets/Editor/GitHub.Unity/UI/SettingsView.cs b/src/UnityExtension/Assets/Editor/GitHub.Unity/UI/SettingsView.cs
@@ -85,11 +85,6 @@ public override void OnRepositoryChanged(IRepository oldRepository)
             Refresh();
         }
 
-        public override bool IsBusy
-        {
-            get { return isBusy; }
-        }
-
         public override void Refresh()
         {
             base.Refresh();
@@ -522,5 +517,10 @@ private void OnLoggingSettingsGui()
             }
             EditorGUI.EndDisabledGroup();
         }
+
+        public override bool IsBusy
+        {
+            get { return isBusy; }
+        }
     }
 }
diff --git a/src/UnityExtension/Assets/Editor/GitHub.Unity/UI/Window.cs b/src/UnityExtension/Assets/Editor/GitHub.Unity/UI/Window.cs
@@ -139,11 +139,6 @@ public override void OnSelectionChange()
                 ActiveView.OnSelectionChange();
         }
 
-        public override bool IsBusy
-        {
-            get { return false; }
-        }
-
         public override void Refresh()
         {
             base.Refresh();
@@ -401,6 +396,11 @@ public SettingsView SettingsView
             get { return settingsView; }
         }
 
+        public override bool IsBusy
+        {
+            get { return false; }
+        }
+
         private Subview ActiveView
         {
             get

Original file line number	Diff line number	Diff line change
`@@ -128,11 +128,6 @@ public override void OnGUI()`
`128`	`128`	`OnEmbeddedGUI();`
`129`	`129`	`}`
`130`	`130`
`131`		`- public override bool IsBusy`
`132`		`- {`
`133`		`- get { return false; }`
`134`		`- }`
`135`		`-`
`136`	`131`	`public void OnEmbeddedGUI()`
`137`	`132`	`{`
`138`	`133`	`scroll = GUILayout.BeginScrollView(scroll);`
`@@ -749,6 +744,11 @@ private void OnTreeNodeChildrenGUI(BranchTreeNode node)`
`749`	`744`	`}`
`750`	`745`	`}`
`751`	`746`
	`747`	`+ public override bool IsBusy`
	`748`	`+ {`
	`749`	`+ get { return false; }`
	`750`	`+ }`
	`751`	`+`
`752`	`752`	`private enum NodeType`
`753`	`753`	`{`
`754`	`754`	`Folder,`
Original file line number	Diff line number	Diff line change
`@@ -52,11 +52,6 @@ public override void OnDisable()`
`52`	`52`	`Repository.OnStatusUpdated -= RunStatusUpdateOnMainThread;`
`53`	`53`	`}`
`54`	`54`
`55`		`- public override bool IsBusy`
`56`		`- {`
`57`		`- get { return isBusy; }`
`58`		`- }`
`59`		`-`
`60`	`55`	`private void RunStatusUpdateOnMainThread(GitStatus status)`
`61`	`56`	`{`
`62`	`57`	`new ActionTask(TaskManager.Token, _ => OnStatusUpdate(status))`
`@@ -217,5 +212,10 @@ private void Commit()`
`217`	`212`	`isBusy = false;`
`218`	`213`	`}).Start();`
`219`	`214`	`}`
	`215`	`+`
	`216`	`+ public override bool IsBusy`
	`217`	`+ {`
	`218`	`+ get { return isBusy; }`
	`219`	`+ }`
`220`	`220`	`}`
`221`	`221`	`}`
Original file line number	Diff line number	Diff line change
`@@ -125,10 +125,6 @@ public override void OnGUI()`
`125`	`125`	`OnEmbeddedGUI();`
`126`	`126`	`}`
`127`	`127`
`128`		`- public override bool IsBusy`
`129`		`- {`
`130`		`- get { return isBusy; }`
`131`		`- }`
`132`	`128`	`private void AttachHandlers(IRepository repository)`
`133`	`129`	`{`
`134`	`130`	`if (repository == null)`
`@@ -750,6 +746,11 @@ private void DrawTimelineRectAroundIconRect(Rect parentRect, Rect iconRect)`
`750`	`746`	`EditorGUI.DrawRect(bottomTimelineRect, timelineBarColor);`
`751`	`747`	`}`
`752`	`748`
	`749`	`+ public override bool IsBusy`
	`750`	`+ {`
	`751`	`+ get { return isBusy; }`
	`752`	`+ }`
	`753`	`+`
`753`	`754`	`private float EntryHeight`
`754`	`755`	`{`
`755`	`756`	`get { return Styles.HistoryEntryHeight + Styles.HistoryEntryPadding; }`
Original file line number	Diff line number	Diff line change
`@@ -93,11 +93,6 @@ public override void OnSelectionChange()`
`93`	`93`	`ActiveView.OnSelectionChange();`
`94`	`94`	`}`
`95`	`95`
`96`		`- public override bool IsBusy`
`97`		`- {`
`98`		`- get { return ActiveView.IsBusy; }`
`99`		`- }`
`100`		`-`
`101`	`96`	`public override void Finish(bool result)`
`102`	`97`	`{`
`103`	`98`	`OnClose.SafeInvoke(result);`
`@@ -113,6 +108,11 @@ public override void OnDestroy()`
`113`	`108`	`OnClose = null;`
`114`	`109`	`}`
`115`	`110`
	`111`	`+ public override bool IsBusy`
	`112`	`+ {`
	`113`	`+ get { return ActiveView.IsBusy; }`
	`114`	`+ }`
	`115`	`+`
`116`	`116`	`private Subview ActiveView`
`117`	`117`	`{`
`118`	`118`	`get`
Original file line number	Diff line number	Diff line change
`@@ -85,11 +85,6 @@ public override void OnRepositoryChanged(IRepository oldRepository)`
`85`	`85`	`Refresh();`
`86`	`86`	`}`
`87`	`87`
`88`		`- public override bool IsBusy`
`89`		`- {`
`90`		`- get { return isBusy; }`
`91`		`- }`
`92`		`-`
`93`	`88`	`public override void Refresh()`
`94`	`89`	`{`
`95`	`90`	`base.Refresh();`
`@@ -522,5 +517,10 @@ private void OnLoggingSettingsGui()`
`522`	`517`	`}`
`523`	`518`	`EditorGUI.EndDisabledGroup();`
`524`	`519`	`}`
	`520`	`+`
	`521`	`+ public override bool IsBusy`
	`522`	`+ {`
	`523`	`+ get { return isBusy; }`
	`524`	`+ }`
`525`	`525`	`}`
`526`	`526`	`}`
Original file line number	Diff line number	Diff line change
`@@ -139,11 +139,6 @@ public override void OnSelectionChange()`
`139`	`139`	`ActiveView.OnSelectionChange();`
`140`	`140`	`}`
`141`	`141`
`142`		`- public override bool IsBusy`
`143`		`- {`
`144`		`- get { return false; }`
`145`		`- }`
`146`		`-`
`147`	`142`	`public override void Refresh()`
`148`	`143`	`{`
`149`	`144`	`base.Refresh();`
`@@ -401,6 +396,11 @@ public SettingsView SettingsView`
`401`	`396`	`get { return settingsView; }`
`402`	`397`	`}`
`403`	`398`
	`399`	`+ public override bool IsBusy`
	`400`	`+ {`
	`401`	`+ get { return false; }`
	`402`	`+ }`
	`403`	`+`
`404`	`404`	`private Subview ActiveView`
`405`	`405`	`{`
`406`	`406`	`get`