Fixes an issue where computer and group target precedence may be not be processed correctly

Author: ryannewington

src/Lithnet.Laps.Web/Lithnet.Laps.Web/Controllers/LapController.cs

@@ -140,7 +140,7 @@ private TargetElement GetMatchingTargetOrNull(ComputerPrincipal computer)
         {
             List<TargetElement> matchingTargets = new List<TargetElement>();
 
-            foreach (TargetElement target in LapsConfigSection.Configuration.Targets.OfType<TargetElement>().OrderBy(t => t.Type == TargetType.Computer).ThenBy(t => t.Type == TargetType.Group))
+            foreach (TargetElement target in LapsConfigSection.Configuration.Targets.OfType<TargetElement>())
             {
                 if (target.Type == TargetType.Container)
                 {
@@ -186,7 +186,10 @@ private TargetElement GetMatchingTargetOrNull(ComputerPrincipal computer)
                 }
             }
 
-            return matchingTargets.OrderBy(t => t.Type == TargetType.Computer).ThenBy(t => t.Type == TargetType.Group).FirstOrDefault();
+            return 
+                matchingTargets.FirstOrDefault(t => t.Type == TargetType.Computer) ??
+                matchingTargets.FirstOrDefault(t => t.Type == TargetType.Group) ??
+                matchingTargets.FirstOrDefault();
         }
 
         private bool IsReaderAuthorized(ReaderElement reader, UserPrincipal currentUser)