* Refactored out Create() methods to ensure proper transformation between RegionProfileData and RegionInfo

* Created ToRegionInfo method, still not using it pending peer review. 
* This is a preparatory commit for a subsequent login service refactoring.

2 files changed, 36 insertions, 15 deletions

diff --git a/OpenSim/Data/RegionProfileData.cs b/OpenSim/Data/RegionProfileData.cs
index 0b757f3..5476233 100644
--- a/OpenSim/Data/RegionProfileData.cs
+++ b/OpenSim/Data/RegionProfileData.cs
@@ -279,6 +279,30 @@ namespace OpenSim.Data
             get { return originUUID; }
             set { originUUID = value; }
         }
-        
+
+        public RegionInfo ToRegionInfo( )
+        {
+            return RegionInfo.Create(UUID, regionName, regionLocX, regionLocY, serverIP, httpPort, serverPort, remotingPort);
+        }
+
+        public static RegionProfileData Create(UUID regionID, string regionName, uint locX, uint locY, string externalHostName, uint regionPort, uint httpPort, uint remotingPort, string serverUri)
+        {
+            RegionProfileData regionProfile;
+            regionProfile = new RegionProfileData();
+            regionProfile.regionLocX = locX;
+            regionProfile.regionLocY = locY;
+            regionProfile.regionHandle =
+                Utils.UIntsToLong((regionProfile.regionLocX * Constants.RegionSize),
+                                  (regionProfile.regionLocY*Constants.RegionSize));
+            regionProfile.serverIP = externalHostName;
+            regionProfile.serverPort = regionPort;
+            regionProfile.httpPort = httpPort;
+            regionProfile.remotingPort = remotingPort;
+            regionProfile.serverURI = serverUri;
+            regionProfile.httpServerURI = "http://" + externalHostName + ":" + httpPort + "/";
+            regionProfile.UUID = regionID;
+            regionProfile.regionName = regionName;
+            return regionProfile;
+        }
     }
 }
diff --git a/OpenSim/Data/RegionProfileServiceProxy.cs b/OpenSim/Data/RegionProfileServiceProxy.cs
index 126e363..da71936 100644
--- a/OpenSim/Data/RegionProfileServiceProxy.cs
+++ b/OpenSim/Data/RegionProfileServiceProxy.cs
@@ -56,20 +56,17 @@ namespace OpenSim.Data
 
             if (!responseData.ContainsKey("error"))
             {
-                simData = new RegionProfileData();
-                simData.regionLocX = Convert.ToUInt32((string) responseData["region_locx"]);
-                simData.regionLocY = Convert.ToUInt32((string) responseData["region_locy"]);
-                simData.regionHandle =
-                    Utils.UIntsToLong((simData.regionLocX * Constants.RegionSize),
-                                      (simData.regionLocY*Constants.RegionSize));
-                simData.serverIP = (string) responseData["sim_ip"];
-                simData.serverPort = Convert.ToUInt32((string) responseData["sim_port"]);
-                simData.httpPort = Convert.ToUInt32((string) responseData["http_port"]);
-                simData.remotingPort = Convert.ToUInt32((string) responseData["remoting_port"]);
-                simData.serverURI = (string) responseData["server_uri"];
-                simData.httpServerURI = "http://" + (string)responseData["sim_ip"] + ":" + simData.httpPort.ToString() + "/";
-                simData.UUID = new UUID((string) responseData["region_UUID"]);
-                simData.regionName = (string) responseData["region_name"];
+                uint locX = Convert.ToUInt32((string)responseData["region_locx"]);
+                uint locY = Convert.ToUInt32((string)responseData["region_locy"]);
+                string externalHostName = (string)responseData["sim_ip"];
+                uint simPort = Convert.ToUInt32((string)responseData["sim_port"]);
+                uint httpPort = Convert.ToUInt32((string)responseData["http_port"]);
+                uint remotingPort = Convert.ToUInt32((string)responseData["remoting_port"]);
+                string serverUri = (string)responseData["server_uri"];
+                UUID regionID = new UUID((string)responseData["region_UUID"]);
+                string regionName = (string)responseData["region_name"];
+
+                simData = RegionProfileData.Create(regionID, regionName, locX, locY, externalHostName, simPort, httpPort, remotingPort, serverUri);
             }
 
             return simData;