Change ordering within ActorSelection to fix issue where /foo paths w…

…ere not working within mono on linux.
akkadotnet · Mar 3, 2015 · 0e1c2cd · 0e1c2cd · Petabridge-CI · Mar 3, 2015
1 parent 66e7498
commit 0e1c2cd
Showing 1 changed file with 12 additions and 10 deletions.
diff --git a/src/core/Akka/Actor/ActorRefFactoryShared.cs b/src/core/Akka/Actor/ActorRefFactoryShared.cs
@@ -35,28 +35,30 @@ public static ActorSelection ActorSelection(ActorPath actorPath, ActorSystem sys
         public static ActorSelection ActorSelection(string path, ActorSystem system, ActorRef lookupRoot)
         {
             var provider = ((ActorSystemImpl)system).Provider;
-            if(Uri.IsWellFormedUriString(path, UriKind.Absolute))
-            {
-                ActorPath actorPath;
-                if(!ActorPath.TryParse(path, out actorPath))
-                    return new ActorSelection(provider.DeadLetters, "");
 
-                var actorRef = provider.RootGuardianAt(actorPath.Address);
-                return new ActorSelection(actorRef, actorPath.Elements);
-            }
             //no path given
-            if(string.IsNullOrEmpty(path))
+            if (string.IsNullOrEmpty(path))
             {
                 return new ActorSelection(system.DeadLetters, "");
             }
 
             //absolute path
             var elements = path.Split('/');
-            if(elements[0] == "")
+            if (elements[0] == "")
             {
                 return new ActorSelection(provider.RootGuardian, elements.Skip(1));
             }
 
+            if(Uri.IsWellFormedUriString(path, UriKind.Absolute))
+            {
+                ActorPath actorPath;
+                if(!ActorPath.TryParse(path, out actorPath))
+                    return new ActorSelection(provider.DeadLetters, "");
+
+                var actorRef = provider.RootGuardianAt(actorPath.Address);
+                return new ActorSelection(actorRef, actorPath.Elements);
+            }
+
             return new ActorSelection(lookupRoot, path);
         }
     }