Simplifying some code

shana · shana · commit ea5312b65557 · 2018-05-31T18:56:46.000+02:00
diff --git a/src/UnityExtension/Assets/Editor/GitHub.Unity/UI/ProjectWindowInterface.cs b/src/UnityExtension/Assets/Editor/GitHub.Unity/UI/ProjectWindowInterface.cs
@@ -162,13 +162,15 @@ private static bool IsObjectUnlocked(Object selected)
             NPath repositoryPath = manager.Environment.GetRepositoryPath(assetPath);
 
             var alreadyLocked = locks.Any(x => repositoryPath == x.Path);
+            if (alreadyLocked)
+                return false;
+
             GitFileStatus status = GitFileStatus.None;
             if (entries != null)
             {
                 status = entries.FirstOrDefault(x => repositoryPath == x.Path.ToNPath()).Status;
             }
-
-            return !alreadyLocked && status != GitFileStatus.Untracked && status != GitFileStatus.Ignored;
+            return status != GitFileStatus.Untracked && status != GitFileStatus.Ignored;
         }
 
         private static bool IsObjectLocked(Object selected)

Original file line number	Diff line number	Diff line change
`@@ -162,13 +162,15 @@ private static bool IsObjectUnlocked(Object selected)`
`162`	`162`	`NPath repositoryPath = manager.Environment.GetRepositoryPath(assetPath);`
`163`	`163`
`164`	`164`	`var alreadyLocked = locks.Any(x => repositoryPath == x.Path);`
	`165`	`+ if (alreadyLocked)`
	`166`	`+ return false;`
	`167`	`+`
`165`	`168`	`GitFileStatus status = GitFileStatus.None;`
`166`	`169`	`if (entries != null)`
`167`	`170`	`{`
`168`	`171`	`status = entries.FirstOrDefault(x => repositoryPath == x.Path.ToNPath()).Status;`
`169`	`172`	`}`
`170`		`-`
`171`		`- return !alreadyLocked && status != GitFileStatus.Untracked && status != GitFileStatus.Ignored;`
	`173`	`+ return status != GitFileStatus.Untracked && status != GitFileStatus.Ignored;`
`172`	`174`	`}`
`173`	`175`
`174`	`176`	`private static bool IsObjectLocked(Object selected)`