From 57ae3d03a1ccfb464216b58e7e70abaf3a3d8c48 Mon Sep 17 00:00:00 2001
From: Sarah Hoffmann <lonvia@denofr.de>
Date: Sun, 9 Feb 2020 15:45:38 +0100
Subject: [PATCH] return place_id link to details when not an OSM object

Stop-gap solution to find the right object for Tiger and
interpolation objects.
---
 lib/output.php                    |  2 +-
 test/php/Nominatim/OutputTest.php | 57 +++++++++++++++++++++++++++++++
 2 files changed, 58 insertions(+), 1 deletion(-)
 create mode 100644 test/php/Nominatim/OutputTest.php

diff --git a/lib/output.php b/lib/output.php
index 9d4b7502..8715efbc 100644
--- a/lib/output.php
+++ b/lib/output.php
@@ -48,5 +48,5 @@ function detailsPermaLink($aFeature, $sRefText = false)
         $sLabel = $sRefText ? $sRefText : $sOSMType.' '.$aFeature['osm_id'];
         return '<a href="details.php?osmtype='.$aFeature['osm_type'].'&osmid='.$aFeature['osm_id'].'&class='.$aFeature['class'].'">'.$sLabel.'</a>';
     }
-    return '';
+    return detailsLink($aFeature, $sRefText);
 }
diff --git a/test/php/Nominatim/OutputTest.php b/test/php/Nominatim/OutputTest.php
new file mode 100644
index 00000000..ce9bfa35
--- /dev/null
+++ b/test/php/Nominatim/OutputTest.php
@@ -0,0 +1,57 @@
+<?php
+
+namespace Nominatim;
+
+require_once(CONST_BasePath.'/lib/output.php');
+
+class OutputTest extends \PHPUnit\Framework\TestCase
+{
+    public function testDetailsPermaLinkNode()
+    {
+        $aFeature = array('osm_type' => 'N', 'osm_id'=> 38274, 'class' => 'place');
+        $this->assertSame(
+                detailsPermaLink($aFeature),
+                '<a href="details.php?osmtype=N&osmid=38274&class=place">node 38274</a>'
+                );
+    }
+
+    public function testDetailsPermaLinkWay()
+    {
+        $aFeature = array('osm_type' => 'W', 'osm_id'=> 65, 'class' => 'highway');
+        $this->assertSame(
+                detailsPermaLink($aFeature),
+                '<a href="details.php?osmtype=W&osmid=65&class=highway">way 65</a>'
+                );
+    }
+
+    public function testDetailsPermaLinkRelation()
+    {
+        $aFeature = array('osm_type' => 'R', 'osm_id'=> 9908, 'class' => 'waterway');
+        $this->assertSame(
+                detailsPermaLink($aFeature),
+                '<a href="details.php?osmtype=R&osmid=9908&class=waterway">relation 9908</a>'
+                );
+    }
+
+    public function testDetailsPermaLinkTiger()
+    {
+        $aFeature = array('osm_type' => 'T', 'osm_id'=> 2, 'place_id' => 334);
+        $this->assertSame(
+                detailsPermaLink($aFeature, 'foo'),
+                '<a href="details.php?place_id=334">foo</a>'
+                );
+    }
+
+    public function testDetailsPermaLinkInterpolation()
+    {
+        $aFeature = array('osm_type' => 'I', 'osm_id'=> 400, 'place_id' => 3);
+        $this->assertSame(
+                detailsPermaLink($aFeature, 'foo'),
+                '<a href="details.php?place_id=3">foo</a>'
+                );
+    }
+
+
+
+
+}
-- 
2.39.5