From 7f5fe8c877e8d767d8936023214f7f7be905cf4a Mon Sep 17 00:00:00 2001
From: Marc Tobias Metten <mtmail@gmx.net>
Date: Sun, 28 Feb 2016 17:06:52 +0100
Subject: [PATCH] moved bIncludePolygonAsPoints related logic into lib.php,
 added tests

---
 lib/Geocode.php | 52 +++++++++++++++++++++++--------------------------
 lib/lib.php     | 26 +++++++++++++++++++++++++
 2 files changed, 50 insertions(+), 28 deletions(-)

diff --git a/lib/Geocode.php b/lib/Geocode.php
index ba1842c3..a6ff279c 100644
--- a/lib/Geocode.php
+++ b/lib/Geocode.php
@@ -862,9 +862,23 @@
 			{
 				// Start with a blank search
 				$aSearches = array(
-					array('iSearchRank' => 0, 'iNamePhrase' => -1, 'sCountryCode' => false, 'aName'=>array(), 'aAddress'=>array(), 'aFullNameAddress'=>array(),
-								'aNameNonSearch'=>array(), 'aAddressNonSearch'=>array(),
-								'sOperator'=>'', 'aFeatureName' => array(), 'sClass'=>'', 'sType'=>'', 'sHouseNumber'=>'', 'fLat'=>'', 'fLon'=>'', 'fRadius'=>'')
+					array('iSearchRank' => 0, 
+								'iNamePhrase' => -1, 
+								'sCountryCode' => false, 
+								'aName' => array(), 
+								'aAddress' => array(), 
+								'aFullNameAddress' => array(),
+								'aNameNonSearch' => array(), 
+								'aAddressNonSearch' => array(),
+								'sOperator' => '', 
+								'aFeatureName' => array(), 
+								'sClass' => '', 
+								'sType' => '', 
+								'sHouseNumber' => '', 
+								'fLat' => '', 
+								'fLon' => '', 
+								'fRadius' => ''
+							)
 				);
 
 				// Do we have a radius search?
@@ -1676,37 +1690,19 @@
 
 						if ($this->bIncludePolygonAsPoints)
 						{
-							// Translate geometry string to point array
-							if (preg_match('#POLYGON\\(\\(([- 0-9.,]+)#',$aPointPolygon['astext'],$aMatch))
-							{
-								preg_match_all('/(-?[0-9.]+) (-?[0-9.]+)/',$aMatch[1],$aPolyPoints,PREG_SET_ORDER);
-							}
-							elseif (preg_match('#MULTIPOLYGON\\(\\(\\(([- 0-9.,]+)#',$aPointPolygon['astext'],$aMatch))
-							{
-								preg_match_all('/(-?[0-9.]+) (-?[0-9.]+)/',$aMatch[1],$aPolyPoints,PREG_SET_ORDER);
-							}
-							elseif (preg_match('#POINT\\((-?[0-9.]+) (-?[0-9.]+)\\)#',$aPointPolygon['astext'],$aMatch))
+							$aPolyPoints[] = geometryText2Points($aPointPolygon['astext'],$fRadius);
+
+							// Output data suitable for display (points and a bounding box)
+							if (isset($aPolyPoints))
 							{
-								$iSteps = max(8, min(100, ($fRadius * 40000)^2));
-								$fStepSize = (2*pi())/$iSteps;
-								$aPolyPoints = array();
-								for($f = 0; $f < 2*pi(); $f += $fStepSize)
+								$aResult['aPolyPoints'] = array();
+								foreach($aPolyPoints as $aPoint)
 								{
-									$aPolyPoints[] = array('',$aMatch[1]+($fRadius*sin($f)),$aMatch[2]+($fRadius*cos($f)));
+									$aResult['aPolyPoints'][] = array($aPoint[1], $aPoint[2]);
 								}
 							}
 						}
 
-						// Output data suitable for display (points and a bounding box)
-						if ($this->bIncludePolygonAsPoints && isset($aPolyPoints))
-						{
-							$aResult['aPolyPoints'] = array();
-							foreach($aPolyPoints as $aPoint)
-							{
-								$aResult['aPolyPoints'][] = array($aPoint[1], $aPoint[2]);
-							}
-						}
-
 						if (abs($aPointPolygon['minlat'] - $aPointPolygon['maxlat']) < 0.0000001)
 						{
 							$aPointPolygon['minlat'] = $aPointPolygon['minlat'] - $fRadius;
diff --git a/lib/lib.php b/lib/lib.php
index 2134e1a4..e81d85be 100644
--- a/lib/lib.php
+++ b/lib/lib.php
@@ -1022,3 +1022,29 @@
 
 		return array('lat' => $fQueryLat, 'lon' => $fQueryLon, 'query' => $sQuery);
 	}
+
+
+
+	function geometryText2Points($geometry_as_text,$fRadius)
+	{
+		$aPolyPoints = NULL;
+		if (preg_match('#POLYGON\\(\\(([- 0-9.,]+)#',$geometry_as_text,$aMatch))
+		{
+			preg_match_all('/(-?[0-9.]+) (-?[0-9.]+)/',$aMatch[1],$aPolyPoints,PREG_SET_ORDER);
+		}
+		elseif (preg_match('#MULTIPOLYGON\\(\\(\\(([- 0-9.,]+)#',$geometry_as_text,$aMatch))
+		{
+			preg_match_all('/(-?[0-9.]+) (-?[0-9.]+)/',$aMatch[1],$aPolyPoints,PREG_SET_ORDER);
+		}
+		elseif (preg_match('#POINT\\((-?[0-9.]+) (-?[0-9.]+)\\)#',$geometry_as_text,$aMatch))
+		{
+			$iSteps = max(8, min(100, ($fRadius * 40000)^2));
+			$fStepSize = (2*pi())/$iSteps;
+			$aPolyPoints = array();
+			for($f = 0; $f < 2*pi(); $f += $fStepSize)
+			{
+				$aPolyPoints[] = array('',$aMatch[1]+($fRadius*sin($f)),$aMatch[2]+($fRadius*cos($f)));
+			}
+		}
+		return $aPolyPoints;
+	}
-- 
2.39.5