Merge pull request #986 from mtmail/php-replace-sizeof

[nominatim.git] / website / polygons.php

diff --git a/website/polygons.php b/website/polygons.php
index 989c8f66d7c656c88351730cdf8d50ea47c053ed..0f49183f50b649a6cf1f3d9e34639afb065c715b 100755 (executable)
--- a/website/polygons.php
+++ b/website/polygons.php
@@ -9,7 +9,7 @@ ini_set('memory_limit', '200M');
 $oParams = new Nominatim\ParameterParser();
 
 $sOutputFormat = 'html';
-$iDays = $oParams->getInt('days', 1);
+$iDays = $oParams->getInt('days', false);
 $bReduced = $oParams->getBool('reduced', false);
 $sClass = $oParams->getString('class', false);
 
@@ -18,15 +18,24 @@ $oDB =& getDB();
 $iTotalBroken = (int) chksql($oDB->getOne('select count(*) from import_polygon_error'));
 
 $aPolygons = array();
-while ($iTotalBroken && !sizeof($aPolygons)) {
+while ($iTotalBroken && empty($aPolygons)) {
     $sSQL = 'select osm_type as "type",osm_id as "id",class as "key",type as "value",name->\'name\' as "name",';
     $sSQL .= 'country_code as "country",errormessage as "error message",updated';
     $sSQL .= ' from import_polygon_error';
-    $sSQL .= " where updated > 'now'::timestamp - '".$iDays." day'::interval";
-    $iDays++;
 
-    if ($bReduced) $sSQL .= " and errormessage like 'Area reduced%'";
-    if ($sClass) $sSQL .= " and class = '".pg_escape_string($sClass)."'";
+    $aWhere = array();
+    if ($iDays) {
+        $aWhere[] = "updated > 'now'::timestamp - '".$iDays." day'::interval";
+        $iDays++;
+    }
+
+    if ($bReduced) $aWhere[] = "errormessage like 'Area reduced%'";
+    if ($sClass) $sWhere[] = "class = '".pg_escape_string($sClass)."'";
+
+    if (!empty($aWhere)) {
+        $sSQL .= ' where '.join(' and ', $aWhere);
+    }
+
     $sSQL .= ' order by updated desc limit 1000';
     $aPolygons = chksql($oDB->getAll($sSQL));
 }
@@ -93,7 +102,6 @@ foreach ($aPolygons[0] as $sCol => $sVal) {
     echo '<th>'.$sCol.'</th>';
 }
 echo '<th>&nbsp;</th>';
-echo '<th>&nbsp;</th>';
 echo '</tr>';
 $aSeen = array();
 foreach ($aPolygons as $aRow) {
@@ -106,7 +114,7 @@ foreach ($aPolygons as $aRow) {
                 if (preg_match('/Self-intersection\\[([0-9.\\-]+) ([0-9.\\-]+)\\]/', $sVal, $aMatch)) {
                     $aRow['lat'] = $aMatch[2];
                     $aRow['lon'] = $aMatch[1];
-                    echo '<td><a href="http://www.openstreetmap.org/?lat='.$aMatch[2].'&lon='.$aMatch[1].'&zoom=18&layers=M&'.$sOSMType.'='.$aRow['id'].'">'.($sVal?$sVal:'&nbsp;').'</a></td>';
+                    echo '<td><a href="https://www.openstreetmap.org/?lat='.$aMatch[2].'&lon='.$aMatch[1].'&zoom=18&layers=M&'.$sOSMType.'='.$aRow['id'].'">'.($sVal?$sVal:'&nbsp;').'</a></td>';
                 } else {
                     echo '<td>'.($sVal?$sVal:'&nbsp;').'</td>';
                 }
@@ -119,12 +127,7 @@ foreach ($aPolygons as $aRow) {
                 break;
         }
     }
-    echo '<td><a href="http://localhost:8111/import?url=http://www.openstreetmap.org/api/0.6/'.$sOSMType.'/'.$aRow['id'].'/full" target="josm">josm</a></td>';
-    if (isset($aRow['lat'])) {
-        echo '<td><a href="http://open.mapquestapi.com/dataedit/index_flash.html?lat='.$aRow['lat'].'&lon='.$aRow['lon'].'&zoom=18" target="potlatch2">P2</a></td>';
-    } else {
-        echo '<td>&nbsp;</td>';
-    }
+    echo '<td><a href="http://localhost:8111/import?url=https://www.openstreetmap.org/api/0.6/'.$sOSMType.'/'.$aRow['id'].'/full" target="josm">josm</a></td>';
     echo '</tr>';
 }
 echo '</table>';