From 7a50d2f9964421e932313b15f5d7d3b05ded4c45 Mon Sep 17 00:00:00 2001
From: Sarah Hoffmann <lonvia@denofr.de>
Date: Sun, 12 Jun 2016 12:39:09 +0200
Subject: [PATCH] factor out link formatting function and osm type translation

---
 lib/output.php                      | 43 ++++++++++++++++++++
 lib/template/address-json.php       |  5 +--
 lib/template/address-jsonv2.php     |  3 +-
 lib/template/address-xml.php        |  3 +-
 lib/template/details-error-html.php | 31 ++++++--------
 lib/template/details-html.php       | 33 ++-------------
 lib/template/search-batch-json.php  |  2 +-
 lib/template/search-json.php        |  3 +-
 lib/template/search-jsonv2.php      |  2 +-
 lib/template/search-xml.php         |  3 +-
 website/deletable.php               |  7 ++--
 website/details.php                 |  2 +-
 website/hierarchy.php               | 63 ++++++++++++++++-------------
 website/polygons.php                |  9 ++---
 website/reverse.php                 |  1 +
 website/search.php                  |  1 +
 16 files changed, 111 insertions(+), 100 deletions(-)
 create mode 100644 lib/output.php

diff --git a/lib/output.php b/lib/output.php
new file mode 100644
index 00000000..722d73b1
--- /dev/null
+++ b/lib/output.php
@@ -0,0 +1,43 @@
+<?php
+
+	function formatOSMType($sType, $bIncludeExternal=true)
+	{
+		if ($sType == 'N') return 'node';
+		if ($sType == 'W') return 'way';
+		if ($sType == 'R') return 'relation';
+
+		if (!$bIncludeExternal) return '';
+
+		if ($sType == 'T') return 'tiger';
+		if ($sType == 'I') return 'way';
+
+		return '';
+	}
+
+	function osmLink($aFeature, $sRefText=false)
+	{
+		$sOSMType = formatOSMType($aFeature['osm_type'], false);
+		if ($sOSMType)
+		{
+			return '<a href="//www.openstreetmap.org/'.$sOSMType.'/'.$aFeature['osm_id'].'">'.$sOSMType.' '.($sRefText?$sRefText:$aFeature['osm_id']).'</a>';
+		}
+		return '';
+	}
+
+	function wikipediaLink($aFeature)
+	{
+		if ($aFeature['wikipedia'])
+		{
+			list($sLanguage, $sArticle) = explode(':',$aFeature['wikipedia']);
+			return '<a href="https://'.$sLanguage.'.wikipedia.org/wiki/'.urlencode($sArticle).'" target="_blank">'.$aFeature['wikipedia'].'</a>';
+		}
+		return '';
+	}
+
+	function detailsLink($aFeature, $sTitle=false)
+	{
+		if (!$aFeature['place_id']) return '';
+
+		return '<a href="details.php?place_id='.$aFeature['place_id'].'">'.($sTitle?$sTitle:$aFeature['place_id']).'</a>';
+	}
+
diff --git a/lib/template/address-json.php b/lib/template/address-json.php
index 38db01c8..bf6c700b 100644
--- a/lib/template/address-json.php
+++ b/lib/template/address-json.php
@@ -12,8 +12,7 @@
 	{
 		if (isset($aPlace['place_id'])) $aFilteredPlaces['place_id'] = $aPlace['place_id'];
 		$aFilteredPlaces['licence'] = "Data © OpenStreetMap contributors, ODbL 1.0. http://www.openstreetmap.org/copyright";
-		$sOSMType = ($aPlace['osm_type'] == 'N'?'node':($aPlace['osm_type'] == 'W'?'way':($aPlace['osm_type'] == 'R'?'relation':
-		($aPlace['osm_type'] == 'T'?'tiger':($aPlace['osm_type'] == 'I'?'interpolation':'')))));
+		$sOSMType = formatOSMType($aPlace['osm_type']);
 		if ($sOSMType)
 		{
 			$aFilteredPlaces['osm_type'] = $sOSMType;
@@ -50,8 +49,6 @@
 		{
 			$aFilteredPlaces['geokml'] = $aPlace['askml'];
 		}
-
-
 	}
 
 	javascript_renderData($aFilteredPlaces);
diff --git a/lib/template/address-jsonv2.php b/lib/template/address-jsonv2.php
index 2cef5e31..d6214176 100644
--- a/lib/template/address-jsonv2.php
+++ b/lib/template/address-jsonv2.php
@@ -12,8 +12,7 @@
 	{
 		if ($aPlace['place_id']) $aFilteredPlaces['place_id'] = $aPlace['place_id'];
 		$aFilteredPlaces['licence'] = "Data © OpenStreetMap contributors, ODbL 1.0. http://www.openstreetmap.org/copyright";
-		$sOSMType = ($aPlace['osm_type'] == 'N'?'node':($aPlace['osm_type'] == 'W'?'way':($aPlace['osm_type'] == 'R'?'relation':
-		($aPlace['osm_type'] == 'T'?'tiger':($aPlace['osm_type'] == 'I'?'interpolation':'')))));
+		$sOSMType = formatOSMType($aPlace['osm_type']);
 		if ($sOSMType)
 		{
 			$aFilteredPlaces['osm_type'] = $sOSMType;
diff --git a/lib/template/address-xml.php b/lib/template/address-xml.php
index 5e74846e..2bfcffbf 100644
--- a/lib/template/address-xml.php
+++ b/lib/template/address-xml.php
@@ -22,8 +22,7 @@
 	{
 		echo "<result";
 		if ($aPlace['place_id']) echo ' place_id="'.$aPlace['place_id'].'"';
-		$sOSMType = ($aPlace['osm_type'] == 'N'?'node':($aPlace['osm_type'] == 'W'?'way':($aPlace['osm_type'] == 'R'?'relation':
-		($aPlace['osm_type'] == 'T'?'tiger':($aPlace['osm_type'] == 'I'?'interpolation':'')))));
+		$sOSMType = formatOSMType($aPlace['osm_type']);
 		if ($sOSMType) echo ' osm_type="'.$sOSMType.'"'.' osm_id="'.$aPlace['osm_id'].'"';
 		if ($aPlace['ref']) echo ' ref="'.htmlspecialchars($aPlace['ref']).'"';
 		if (isset($aPlace['lat'])) echo ' lat="'.htmlspecialchars($aPlace['lat']).'"';
diff --git a/lib/template/details-error-html.php b/lib/template/details-error-html.php
index 545dc52b..f93cca4d 100644
--- a/lib/template/details-error-html.php
+++ b/lib/template/details-error-html.php
@@ -9,26 +9,21 @@
 
 <?php
 
-	function osm_link($aFeature)
-	{
-		$sOSMType = ($aFeature['osm_type'] == 'N'?'node':($aFeature['osm_type'] == 'W'?'way':($aFeature['osm_type'] == 'R'?'relation':'')));
-		if ($sOSMType) {
-			return '<a href="http://www.openstreetmap.org/'.$sOSMType.'/'.$aFeature['osm_id'].'">'.$sOSMType.' '.$aFeature['osm_id'].'</a>';
-		}
-		return '';
-	}
-
-	function osm_map_url($aFeature)
+	function osmMapUrl($aFeature)
 	{
 		$sLon = $aFeature['error_x'];
 		$sLat = $aFeature['error_y'];
 
-		if (isset($sLat))
+		if (isset($sFeature['error_x']) && isset($sFeature['error_y']))
 		{
-			$sOSMType = ($aFeature['osm_type'] == 'N'?'node':($aFeature['osm_type'] == 'W'?'way':($aFeature['osm_type'] == 'R'?'relation':'')));
+			$sBaseUrl = '//www.openstreetmap.org/';
+			$sOSMType = formatOSMType($aFeature['osm_type'], false);
 			if ($sOSMType)
 			{
-				return "http://www.openstreetmap.org/?lat=".$sLat."&lon=".$sLon."&zoom=18&layers=M&".$sOSMType."=".$aFeature['osm_id'];
+				$sBaseUrl += $sOSMType.'/'.$aFeature['osm_id'];
+			}
+
+				return '<a href="'.$sBaseUrl.'?mlat='.$sLat.'&mlon='.$sLon.'">view on osm.org</a>';
 			}
 		}
 		return '';
@@ -45,7 +40,7 @@
 			return "http://localhost:8111/load_and_zoom?left=".($sLon-$fWidth)."&right=".($sLon+$fWidth)."&top=".($sLat+$fWidth)."&bottom=".($sLat-$fWidth);
 		}
 
-		$sOSMType = ($aFeature['osm_type'] == 'N'?'node':($aFeature['osm_type'] == 'W'?'way':($aFeature['osm_type'] == 'R'?'relation':'')));
+		$sOSMType = formatOSMType($aFeature['osm_type'], false);
 		if ($sOSMType)
 		{
 			return 'http://localhost:8111/import?url=http://www.openstreetmap.org/api/0.6/'.$sOSMType.'/'.$aFeature['osm_id'].'/full';
@@ -63,7 +58,7 @@
 
 		if (isset($sLat))
 		{
-			return "http://www.openstreetmap.org/edit?editor=potlatch2&bbox=".($sLon-$fWidth).",".($sLat-$fWidth).",".($sLon+$fWidth).",".($sLat+$fWidth);
+			return "//www.openstreetmap.org/edit?editor=potlatch2&bbox=".($sLon-$fWidth).",".($sLat-$fWidth).",".($sLon+$fWidth).",".($sLat+$fWidth);
 		}
 		return '';
 	}
@@ -87,7 +82,7 @@
 					</div>
 
 					<div>
-						OSM: <span class="label"><?php echo osm_link($aPointDetails); ?><span>
+						OSM: <span class="label"><?php echo osmLink($aPointDetails); ?><span>
 					</div>
 
 
@@ -95,9 +90,7 @@
 					<p>
 						<?php echo $aPointDetails['errormessage']?$aPointDetails['errormessage']:'unknown'; ?>
 					</p>
-					<?php if (osm_map_url($aPointDetails)) { ?>
-						<a href="<?php echo osm_map_url($aPointDetails); ?>">view on osm.org</a>
-					<?php } ?>
+					<?php echo osmMapUrl($aPointDetails); ?>
 
 					<h4>Edit</h4>
 					<ul>
diff --git a/lib/template/details-html.php b/lib/template/details-html.php
index 7d9696b8..71fcc7b3 100644
--- a/lib/template/details-html.php
+++ b/lib/template/details-html.php
@@ -7,7 +7,6 @@
 </head>
 
 
-
 <?php
 
 	function headline($sTitle)
@@ -20,30 +19,6 @@
 		echo "<tr class='all-columns'><td colspan='6'><h3>".$sTitle."</h3></td></tr>\n";
 	}
 
-	function osm_link($aFeature)
-	{
-		$sOSMType = ($aFeature['osm_type'] == 'N'?'node':($aFeature['osm_type'] == 'W'?'way':($aFeature['osm_type'] == 'R'?'relation':'')));
-		if ($sOSMType)
-		{
-			return '<a href="http://www.openstreetmap.org/'.$sOSMType.'/'.$aFeature['osm_id'].'">'.$sOSMType.' '.$aFeature['osm_id'].'</a>';
-		}
-		return '';
-	}
-
-	function wikipedia_link($aFeature)
-	{
-		if ($aFeature['wikipedia'])
-		{
-			list($sWikipediaLanguage,$sWikipediaArticle) = explode(':',$aFeature['wikipedia']);
-			return '<a href="https://'.$sWikipediaLanguage.'.wikipedia.org/wiki/'.urlencode($sWikipediaArticle).'" target="_blank">'.$aFeature['wikipedia'].'</a>';
-		}
-		return '';
-	}
-
-	function nominatim_link($aFeature, $sTitle)
-	{
-		return '<a href="details.php?place_id='.$aFeature['place_id'].'">'.$sTitle.'</a>';
-	}
 
 	function format_distance($fDistance)
 	{
@@ -92,10 +67,10 @@
 		echo '<tr class="' . ($bNotUsed?'notused':'') . '">'."\n";
 		echo '  <td class="name">'.(trim($aAddressLine['localname'])?$aAddressLine['localname']:'<span class="noname">No Name</span>')."</td>\n";
 		echo '  <td>' . $aAddressLine['class'].':'.$aAddressLine['type'] . "</td>\n";
-		echo '  <td>' . osm_link($aAddressLine) . "</td>\n";
+		echo '  <td>' . osmLink($aAddressLine) . "</td>\n";
 		echo '  <td>' . (isset($aAddressLine['admin_level']) ? $aAddressLine['admin_level'] : '') . "</td>\n";
 		echo '  <td>' . format_distance($aAddressLine['distance'])."</td>\n";
-		echo '  <td>' . nominatim_link($aAddressLine,'details &gt;') . "</td>\n";
+		echo '  <td>' . detailsLink($aAddressLine,'details &gt;') . "</td>\n";
 		echo "</tr>\n";
 	}
 
@@ -143,10 +118,10 @@
 					}
 					kv('Coverage'        , ($aPointDetails['isarea']=='t'?'Polygon':'Point') );
 					kv('Centre Point'    , $aPointDetails['lat'].','.$aPointDetails['lon'] );
-					kv('OSM'             , osm_link($aPointDetails) );
+					kv('OSM'             , osmLink($aPointDetails) );
 					if ($aPointDetails['wikipedia'])
 					{
-						kv('Wikipedia Calculated' , wikipedia_link($aPointDetails) );
+						kv('Wikipedia Calculated' , wikipediaLink($aPointDetails) );
 					}
 
 					kv('Extra Tags'      , hash_to_subtable($aPointDetails['aExtraTags']) );
diff --git a/lib/template/search-batch-json.php b/lib/template/search-batch-json.php
index edfe3871..60d38041 100644
--- a/lib/template/search-batch-json.php
+++ b/lib/template/search-batch-json.php
@@ -14,7 +14,7 @@
 						'place_id'=>$aPointDetails['place_id'],
 					);
 
-			$sOSMType = ($aPointDetails['osm_type'] == 'N'?'node':($aPointDetails['osm_type'] == 'W'?'way':($aPointDetails['osm_type'] == 'R'?'relation':'')));
+			$sOSMType = formatOSMType($aPointDetails['osm_type']);
 			if ($sOSMType)
 			{
 				$aPlace['osm_type'] = $sOSMType;
diff --git a/lib/template/search-json.php b/lib/template/search-json.php
index 5d5ebfe8..d3dc00f3 100644
--- a/lib/template/search-json.php
+++ b/lib/template/search-json.php
@@ -9,8 +9,7 @@
 				'licence'=>"Data © OpenStreetMap contributors, ODbL 1.0. http://www.openstreetmap.org/copyright",
 			);
         
-		$sOSMType = ($aPointDetails['osm_type'] == 'N'?'node':($aPointDetails['osm_type'] == 'W'?'way':($aPointDetails['osm_type'] == 'R'?
-					'relation':($aPointDetails['osm_type'] == 'T'?'tiger':($aPointDetails['osm_type'] == 'I'?'interpolation':'')))));
+		$sOSMType = formatOSMType($aPointDetails['osm_type']);
 		if ($sOSMType)
 		{
 			$aPlace['osm_type'] = $sOSMType;
diff --git a/lib/template/search-jsonv2.php b/lib/template/search-jsonv2.php
index 251388b5..e907a081 100644
--- a/lib/template/search-jsonv2.php
+++ b/lib/template/search-jsonv2.php
@@ -7,7 +7,7 @@
 				'licence'=>"Data © OpenStreetMap contributors, ODbL 1.0. http://www.openstreetmap.org/copyright",
 			);
 
-		$sOSMType = ($aPointDetails['osm_type'] == 'N'?'node':($aPointDetails['osm_type'] == 'W'?'way':($aPointDetails['osm_type'] == 'R'?'relation':($aPointDetails['osm_type'] == 'T'?'tiger':($aPointDetails['osm_type'] == 'I'?'interpolation':'')))));
+		$sOSMType = formatOSMType($aPointDetails['osm_type']);
 		if ($sOSMType)
 		{
 			$aPlace['osm_type'] = $sOSMType;
diff --git a/lib/template/search-xml.php b/lib/template/search-xml.php
index 7cbcc04b..7a5c997f 100644
--- a/lib/template/search-xml.php
+++ b/lib/template/search-xml.php
@@ -25,8 +25,7 @@
 	foreach($aSearchResults as $iResNum => $aResult)
 	{
 		echo "<place place_id='".$aResult['place_id']."'";
-		$sOSMType = ($aResult['osm_type'] == 'N'?'node':($aResult['osm_type'] == 'W'?'way':($aResult['osm_type'] == 'R'?'relation':
-		($aResult['osm_type'] == 'T'?'tiger':($aResult['osm_type'] == 'I'?'interpolation':'')))));
+		$sOSMType = formatOSMType($aResult['osm_type']);
 		if ($sOSMType)
 		{
 			echo " osm_type='$sOSMType'";
diff --git a/website/deletable.php b/website/deletable.php
index efd20ab3..6521e774 100755
--- a/website/deletable.php
+++ b/website/deletable.php
@@ -2,6 +2,7 @@
 	require_once(dirname(dirname(__FILE__)).'/settings/settings.php');
 	require_once(CONST_BasePath.'/lib/init-website.php');
 	require_once(CONST_BasePath.'/lib/log.php');
+	require_once(CONST_BasePath.'/lib/output.php');
 
 	$sOutputFormat = 'html';
 	ini_set('memory_limit', '200M');
@@ -66,6 +67,7 @@ table td {
 
 <table>
 <?php
+	if (!$aPolygons) exit;
 	echo "<tr>";
 //var_dump($aPolygons[0]);
 	foreach($aPolygons[0] as $sCol => $sVal)
@@ -81,11 +83,10 @@ table td {
 			switch($sCol)
 			{
 				case 'osm_id':
-					$sOSMType = ($aRow['osm_type'] == 'N'?'node':($aRow['osm_type'] == 'W'?'way':($aRow['osm_type'] == 'R'?'relation':'')));
-					echo '<td><a href="http://www.openstreetmap.org/'.$sOSMType.'/'.$sVal.'" target="_new">'.$sVal.'</a></td>';
+					echo '<td>'.osmLink($aRow).'</td>';
 					break;
 				case 'place_id':
-					echo '<td><a href="'.CONST_Website_BaseURL.'details?place_id='.$sVal.'">'.$sVal.'</a></td>';
+					echo '<td>'.detailsLink($aRow).'</td>';
 					break;
 				default:
 					echo "<td>".($sVal?$sVal:'&nbsp;')."</td>";
diff --git a/website/details.php b/website/details.php
index 2891ecfc..b3c9591b 100755
--- a/website/details.php
+++ b/website/details.php
@@ -4,6 +4,7 @@
 	require_once(dirname(dirname(__FILE__)).'/settings/settings.php');
 	require_once(CONST_BasePath.'/lib/init-website.php');
 	require_once(CONST_BasePath.'/lib/log.php');
+	require_once(CONST_BasePath.'/lib/output.php');
 
 	$sOutputFormat = 'html';
 	/*
@@ -173,5 +174,4 @@
 		$sTileAttribution = CONST_Map_Tile_Attribution;
 	}
 
-	
 	include(CONST_BasePath.'/lib/template/details-'.$sOutputFormat.'.php');
diff --git a/website/hierarchy.php b/website/hierarchy.php
index 8c0ae1f5..7740abbc 100755
--- a/website/hierarchy.php
+++ b/website/hierarchy.php
@@ -5,6 +5,7 @@
 	require_once(CONST_BasePath.'/lib/init-website.php');
 	require_once(CONST_BasePath.'/lib/log.php');
 	require_once(CONST_BasePath.'/lib/PlaceLookup.php');
+	require_once(CONST_BasePath.'/lib/output.php');
 	ini_set('memory_limit', '200M');
 
 	$oDB =& getDB();
@@ -66,12 +67,17 @@
 	foreach($aPlaceAddress as $i => $aPlace)
 	{
 		if (!$aPlace['place_id']) continue;
-		$aBreadcrums[] = array('placeId'=>$aPlace['place_id'], 'osmType'=>$aPlace['osm_type'], 'osmId'=>$aPlace['osm_id'], 'localName'=>$aPlace['localname']);
-		$sPlaceUrl = 'hierarchy.php?place_id='.$aPlace['place_id'];
-		$sOSMType = ($aPlace['osm_type'] == 'N'?'node':($aPlace['osm_type'] == 'W'?'way':($aPlace['osm_type'] == 'R'?'relation':'')));
-				$sOSMUrl = 'http://www.openstreetmap.org/'.$sOSMType.'/'.$aPlace['osm_id'];
-		if ($sOutputFormat == 'html') if ($i) echo " > ";
-		if ($sOutputFormat == 'html') echo '<a href="'.$sPlaceUrl.'">'.$aPlace['localname'].'</a> (<a href="'.$sOSMUrl.'">osm</a>)';
+		$aBreadcrums[] = array('placeId'   => $aPlace['place_id'],
+		                       'osmType'   => $aPlace['osm_type'],
+		                       'osmId'     => $aPlace['osm_id'],
+		                       'localName' => $aPlace['localname']);
+
+		if ($sOutputFormat == 'html')
+		{
+			$sPlaceUrl = 'hierarchy.php?place_id='.$aPlace['place_id'];
+			if ($i) echo " &gt; ";
+			echo '<a href="'.$sPlaceUrl.'">'.$aPlace['localname'].'</a> ('.osmLink($aPlace).')';
+		}
 	}
 
 
@@ -114,28 +120,29 @@
 
 			if (!isset($aGroupedAddressLines[$aAddressLine['label']])) $aGroupedAddressLines[$aAddressLine['label']] = array();
 				$aGroupedAddressLines[$aAddressLine['label']][] = $aAddressLine;
-			}
-			foreach($aGroupedAddressLines as $sGroupHeading => $aParentOfLines)
+		}
+
+		foreach($aGroupedAddressLines as $sGroupHeading => $aParentOfLines)
+		{
+			echo "<h3>$sGroupHeading</h3>";
+			foreach($aParentOfLines as $aAddressLine)
 			{
-				echo "<h3>$sGroupHeading</h3>";
-				foreach($aParentOfLines as $aAddressLine)
-				{
-					$aAddressLine['localname'] = $aAddressLine['localname']?$aAddressLine['localname']:$aAddressLine['housenumber'];
-					$sOSMType = ($aAddressLine['osm_type'] == 'N'?'node':($aAddressLine['osm_type'] == 'W'?'way':($aAddressLine['osm_type'] == 'R'?'relation':'')));
-
-					echo '<div class="line">';
-					echo '<span class="name">'.(trim($aAddressLine['localname'])?$aAddressLine['localname']:'<span class="noname">No Name</span>').'</span>';
-					echo ' (';
-					echo '<span class="area">'.($aAddressLine['isarea']=='t'?'Polygon':'Point').'</span>';
-					if ($sOSMType) echo ', <span class="osm"><span class="label"></span>'.$sOSMType.' <a href="http://www.openstreetmap.org/'.$sOSMType.'/'.$aAddressLine['osm_id'].'">'.$aAddressLine['osm_id'].'</a></span>';
-					echo ', <a href="hierarchy.php?place_id='.$aAddressLine['place_id'].'">GOTO</a>';
-					echo ', '.$aAddressLine['area'];
-					echo ')';
-					echo '</div>';
-				}
+				$aAddressLine['localname'] = $aAddressLine['localname']?$aAddressLine['localname']:$aAddressLine['housenumber'];
+				$sOSMType = formatOSMType($aAddressLine['osm_type'], false);
+
+				echo '<div class="line">';
+				echo '<span class="name">'.(trim($aAddressLine['localname'])?$aAddressLine['localname']:'<span class="noname">No Name</span>').'</span>';
+				echo ' (';
+				echo '<span class="area">'.($aAddressLine['isarea']=='t'?'Polygon':'Point').'</span>';
+				if ($sOSMType) echo ', <span class="osm"><span class="label"></span>'.$sOSMType.' '.osmLink($aAddressLine).'</span>';
+				echo ', <a href="hierarchy.php?place_id='.$aAddressLine['place_id'].'">GOTO</a>';
+				echo ', '.$aAddressLine['area'];
+				echo ')';
+				echo '</div>';
 			}
-			if (sizeof($aParentOfLines) >= 500) {
-				echo '<p>There are more child objects which are not shown.</p>';
-			}
-			echo '</div>';
 		}
+		if (sizeof($aParentOfLines) >= 500) {
+			echo '<p>There are more child objects which are not shown.</p>';
+		}
+		echo '</div>';
+	}
diff --git a/website/polygons.php b/website/polygons.php
index b6146d95..bb350e6c 100755
--- a/website/polygons.php
+++ b/website/polygons.php
@@ -2,6 +2,7 @@
 	require_once(dirname(dirname(__FILE__)).'/settings/settings.php');
 	require_once(CONST_BasePath.'/lib/init-website.php');
 	require_once(CONST_BasePath.'/lib/log.php');
+	require_once(CONST_BasePath.'/lib/output.php');
 	ini_set('memory_limit', '200M');
 
 	$oDB =& getDB();
@@ -77,6 +78,7 @@ table td {
 <?php
 
 	echo "<p>Total number of broken polygons: $iTotalBroken</p>";
+	if (!$aPolygons) exit;
 	echo "<table>";
 	echo "<tr>";
 //var_dump($aPolygons[0]);
@@ -110,8 +112,7 @@ table td {
 				}
 				break;
 			case 'id':
-				$sOSMType = ($aRow['type'] == 'N'?'node':($aRow['type'] == 'W'?'way':($aRow['type'] == 'R'?'relation':'')));
-				echo '<td><a href="http://www.openstreetmap.org/'.$sOSMType.'/'.$aRow['id'].'" target="_new">'.$aRow['id'].'</a></td>';
+				echo '<td>'.osmLink($aRow).'</td>';
 				break;
 			default:
 				echo "<td>".($sVal?$sVal:'&nbsp;')."</td>";
@@ -130,10 +131,6 @@ table td {
 		echo "</tr>";
 	}
 	echo "</table>";
-
-
-
-//	include(CONST_BasePath.'/lib/template/details-'.$sOutputFormat.'.php');
 ?>
 </body>
 </html>
diff --git a/website/reverse.php b/website/reverse.php
index 430d9e39..c6c7305f 100755
--- a/website/reverse.php
+++ b/website/reverse.php
@@ -6,6 +6,7 @@
 	require_once(CONST_BasePath.'/lib/log.php');
 	require_once(CONST_BasePath.'/lib/PlaceLookup.php');
 	require_once(CONST_BasePath.'/lib/ReverseGeocode.php');
+	require_once(CONST_BasePath.'/lib/output.php');
 
 	if (strpos(CONST_BulkUserIPs, ','.$_SERVER["REMOTE_ADDR"].',') !== false)
 	{
diff --git a/website/search.php b/website/search.php
index 776cfa78..3c85c8f7 100755
--- a/website/search.php
+++ b/website/search.php
@@ -5,6 +5,7 @@
 	require_once(CONST_BasePath.'/lib/init-website.php');
 	require_once(CONST_BasePath.'/lib/log.php');
 	require_once(CONST_BasePath.'/lib/Geocode.php');
+	require_once(CONST_BasePath.'/lib/output.php');
 
 	ini_set('memory_limit', '200M');
 
-- 
2.39.5