variable naming after lonvia PR feedback

diff --git a/lib/template/details-html.php b/lib/template/details-html.php
index 00d6ba6385b2d1b7bd26813e82dde58afe6d6192..89c1c5430086bbcc8b1fb31d62fffdc45a8ba10f 100644 (file)
--- a/lib/template/details-html.php
+++ b/lib/template/details-html.php
@@ -184,17 +184,14 @@
 
 
 
 
 
 

-    if ($aPlaceSearchNameKeywords)
+    if ($bIncludeKeywords)

     {
         headline('Name Keywords');
         foreach($aPlaceSearchNameKeywords as $aRow)
         {
             _one_keyword_row($aRow['word_token'], $aRow['word_id']);
         }
     {
         headline('Name Keywords');
         foreach($aPlaceSearchNameKeywords as $aRow)
         {
             _one_keyword_row($aRow['word_token'], $aRow['word_id']);
         }

-    }

 
 

-    if ($aPlaceSearchAddressKeywords)
-    {

         headline('Address Keywords');
         foreach($aPlaceSearchAddressKeywords as $aRow)
         {
         headline('Address Keywords');
         foreach($aPlaceSearchAddressKeywords as $aRow)
         {

diff --git a/lib/template/details-json.php b/lib/template/details-json.php
index 2464bc34d2a2aef0c1ab1ceccb1098fd09b123dc..44b6bff53245ca7058eca447a5986ed116fca7fc 100644 (file)
--- a/lib/template/details-json.php
+++ b/lib/template/details-json.php
@@ -1,28 +1,88 @@
 <?php
 
 <?php
 

-$aPlaceDetails = $aPointDetails;
+$aPlaceDetails = array();

 
 

-$aPlaceDetails['geojson'] = json_decode($aPointDetails['asgeojson']);
-unset($aPlaceDetails['asgeojson']);
+$aPlaceDetails['place_id'] = (int) $aPointDetails['place_id'];
+$aPlaceDetails['parent_place_id'] = (int) $aPointDetails['parent_place_id'];
+
+$aPlaceDetails['osm_type'] = $aPointDetails['osm_type'];
+$aPlaceDetails['osm_id'] = (int) $aPointDetails['osm_id'];
+
+$aPlaceDetails['category'] = $aPointDetails['class'];
+$aPlaceDetails['type'] = $aPointDetails['type'];
+$aPlaceDetails['admin_level'] = $aPointDetails['admin_level'];
+
+$aPlaceDetails['names'] = $aPointDetails['aNames'];
+
+$aPlaceDetails['addresstags'] = $aPointDetails['aAddressTags'];
+$aPlaceDetails['housenumber'] = $aPointDetails['housenumber'];
+$aPlaceDetails['postcode'] = $aPointDetails['postcode']; // computed
+$aPlaceDetails['country_code'] = $aPointDetails['country_code'];
+
+$aPlaceDetails['indexed_date'] = $aPointDetails['indexed_date'];
+$aPlaceDetails['importance'] = (float) $aPointDetails['importance'];
+$aPlaceDetails['calculated_importance'] = (float) $aPointDetails['calculated_importance'];
+
+$aPlaceDetails['extratags'] = $aPointDetails['aExtraTags'];
+$aPlaceDetails['calculated_wikipedia'] = $aPointDetails['wikipedia'];
+$aPlaceDetails['icon'] = $aPointDetails['icon'];
+
+$aPlaceDetails['rank_address'] = (int) $aPointDetails['rank_address'];
+$aPlaceDetails['rank_search'] = (int) $aPointDetails['rank_search'];
+$aPlaceDetails['rank_search_label'] = $aPointDetails['rank_search_label'];
+
+$aPlaceDetails['isarea'] = ($aPointDetails['isarea'] == 't');
+$aPlaceDetails['lat'] = (float) $aPointDetails['lat'];
+$aPlaceDetails['lon'] = (float) $aPointDetails['lon'];
+
+
+$aPlaceDetails['geometry'] = json_decode($aPointDetails['asgeojson']);
+
+$funcMapAddressLines = function($aFull) {
+    $aMapped = [
+        'localname' => $aFull['localname'],
+        'place_id' => (int) $aFull['place_id'],
+        'osm_id' => (int) $aFull['osm_id'],
+        'osm_type' => formatOSMType($aFull['osm_type']),
+        'class' => $aFull['class'],
+        'type' => $aFull['type'],
+        'admin_level' => (int) $aFull['admin_level'],
+        'rank_address' => (int) $aFull['rank_address'],
+        'distance' => (float) $aFull['distance']
+    ];
+    return $aMapped;
+};
+
+$funcMapKeywords = function($aFull) {
+    $aMapped = [
+        'id' => (int) $aFull['word_id'],
+        'token' => $aFull['word_token']
+    ];
+    return $aMapped;
+};

 
 if ($aAddressLines) {
 
 if ($aAddressLines) {

-    $aPlaceDetails['address_lines'] = $aAddressLines;
+    $aPlaceDetails['address'] = array_map($funcLines, $aAddressLines);

 }
 
 if ($aLinkedLines) {
 }
 
 if ($aLinkedLines) {

-    $aPlaceDetails['linked_lines'] = $aLinkedLines;
+    $aPlaceDetails['linked_places'] = array_map($funcLines, $aLinkedLines);

 }
 
 }
 

-if ($aPlaceSearchNameKeywords) {
-    $aPlaceDetails['place_search_name_keywords'] = $aPlaceSearchNameKeywords;
-}
+if ($bIncludeKeywords) {
+    $aPlaceDetails['keywords'] = array();
+
+    if ($aPlaceSearchNameKeywords) {
+        $aPlaceDetails['keywords']['name'] = array_map($funcKeywords, $aPlaceSearchNameKeywords);
+    }

 
 

-if ($aPlaceSearchAddressKeywords) {
-    $aPlaceDetails['place_search_address_keywords'] = $aPlaceSearchAddressKeywords;
+    if ($aPlaceSearchAddressKeywords) {
+        $aPlaceDetails['keywords']['address'] = array_map($funcKeywords, $aPlaceSearchAddressKeywords);
+    }

 }
 
 }
 

-if ($aParentOfLines) {
-    $aPlaceDetails['parentof_lines'] = $aParentOfLines;
+if ($bIncludeChildPlaces) {
+    $aPlaceDetails['parentof'] =  array_map($funcLines, $aParentOfLines);

 
     if ($bGroupParents) {
         $aGroupedAddressLines = [];
 
     if ($bGroupParents) {
         $aGroupedAddressLines = [];


@@ -34,7 +94,7 @@ if ($aParentOfLines) {
                 $aGroupedAddressLines[$sType] = [];
             $aGroupedAddressLines[$sType][] = $aAddressLine;
         }
                 $aGroupedAddressLines[$sType] = [];
             $aGroupedAddressLines[$sType][] = $aAddressLine;
         }

-        $aPlaceDetails['parentof_lines'] = $aGroupedAddressLines;
+        $aPlaceDetails['parentof'] = $aGroupedAddressLines;

     }
 }
 
     }
 }
 

diff --git a/website/details.php b/website/details.php
index 6c7fa8d3546aa1fc46cd42ab0ac8c0d6b55c96dd..7f26c6ee5b80edc5f58ee6a3873f0ae30a65f941 100755 (executable)
--- a/website/details.php
+++ b/website/details.php
@@ -19,6 +19,7 @@ $sOsmType = $oParams->getSet('osmtype', array('N', 'W', 'R'));
 $iOsmId = $oParams->getInt('osmid', -1);
 $sClass = $oParams->getString('class');
 
 $iOsmId = $oParams->getInt('osmid', -1);
 $sClass = $oParams->getString('class');
 

+$bIncludeKeywords = $oParams->getBool('keywords');

 $bIncludeAddressDetails = $oParams->getBool('addressdetails', $sOutputFormat == 'html');
 $bIncludeLinkedPlaces = $oParams->getBool('linkedplaces', $sOutputFormat == 'html');
 $bIncludeChildPlaces = $oParams->getBool('childplaces', $sOutputFormat == 'html');
 $bIncludeAddressDetails = $oParams->getBool('addressdetails', $sOutputFormat == 'html');
 $bIncludeLinkedPlaces = $oParams->getBool('linkedplaces', $sOutputFormat == 'html');
 $bIncludeChildPlaces = $oParams->getBool('childplaces', $sOutputFormat == 'html');


@@ -239,7 +240,7 @@ if ($bIncludeChildPlaces) {
 
 $aPlaceSearchNameKeywords = false;
 $aPlaceSearchAddressKeywords = false;
 
 $aPlaceSearchNameKeywords = false;
 $aPlaceSearchAddressKeywords = false;

-if ($oParams->getBool('keywords')) {
+if ($bIncludeKeywords) {

     $sSQL = "SELECT * FROM search_name WHERE place_id = $iPlaceID";
     $aPlaceSearchName = $oDB->getRow($sSQL); // can be null
     if (!$aPlaceSearchName || PEAR::isError($aPlaceSearchName)) { // possible timeout
     $sSQL = "SELECT * FROM search_name WHERE place_id = $iPlaceID";
     $aPlaceSearchName = $oDB->getRow($sSQL); // can be null
     if (!$aPlaceSearchName || PEAR::isError($aPlaceSearchName)) { // possible timeout