Merge pull request #209 from github-for-unity/fixes/history-view-list-locks-exception

shana · web-flow · commit 618540a46252 · 2017-08-22T20:06:45.000+02:00
Fix to checking if remote is set before listing locks
diff --git a/src/UnityExtension/Assets/Editor/GitHub.Unity/UI/SettingsView.cs b/src/UnityExtension/Assets/Editor/GitHub.Unity/UI/SettingsView.cs
@@ -124,8 +124,10 @@ public override void OnRepositoryChanged(IRepository oldRepository)
         public override void Refresh()
         {
             base.Refresh();
-            if (Repository != null)
+            if (Repository != null && Repository.CurrentRemote.HasValue)
+            {
                 Repository.ListLocks().Start();
+            }
         }
 
         private void AttachHandlers(IRepository repository)

Original file line number	Diff line number	Diff line change
`@@ -124,8 +124,10 @@ public override void OnRepositoryChanged(IRepository oldRepository)`
`124`	`124`	`public override void Refresh()`
`125`	`125`	`{`
`126`	`126`	`base.Refresh();`
`127`		`- if (Repository != null)`
	`127`	`+ if (Repository != null && Repository.CurrentRemote.HasValue)`
	`128`	`+ {`
`128`	`129`	`Repository.ListLocks().Start();`
	`130`	`+ }`
`129`	`131`	`}`
`130`	`132`
`131`	`133`	`private void AttachHandlers(IRepository repository)`