From 3663bc42a62c4a2dc146ab3880ee3d5aaa954600 Mon Sep 17 00:00:00 2001
From: Pascal Barth <pak.barth@gmail.com>
Date: Tue, 26 Nov 2024 16:31:22 +0100
Subject: [PATCH] PB-1222 : link to object time=None fix

Do not add time information for layer that aren't time enabled.
---
 chsdi/templates/htmlpopup/base.mako | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

diff --git a/chsdi/templates/htmlpopup/base.mako b/chsdi/templates/htmlpopup/base.mako
index 27f6156709..edf0023f44 100644
--- a/chsdi/templates/htmlpopup/base.mako
+++ b/chsdi/templates/htmlpopup/base.mako
@@ -79,8 +79,12 @@
             <td class="cell-left"></td>
             <td>
           % endif
-              <a href="${''.join((c['baseUrl'], '?', c['layerBodId'], '=', str(c['featureId']), '&time={}'.format(timestamp), '&lang=', lang, '&topic=', topic))}" target="new">
-                ${_('Link to object')}
+            % if timestamp is None:
+              <a href="${''.join((c['baseUrl'], '?', c['layerBodId'], '=', str(c['featureId']), '&lang=', lang, '&topic=', topic))}" target="_blank">
+            % else:
+              <a href="${''.join((c['baseUrl'], '?', c['layerBodId'], '=', str(c['featureId']), '&time={}'.format(timestamp), '&lang=', lang, '&topic=', topic))}" target="_blank">
+            % endif
+             ${_('Link to object')}
               </a>
             </td>
           </tr>
@@ -95,7 +99,7 @@
     </a>
     <div class="float-right">
       % if c['stable_id'] is True:
-      <a class="link" href="${''.join((c['baseUrl'], '?', c['layerBodId'], '=', str(c['featureId']), '&lang=', lang, '&topic=', topic))}" target="new">
+      <a class="link" href="${''.join((c['baseUrl'], '?', c['layerBodId'], '=', str(c['featureId']), '&lang=', lang, '&topic=', topic))}" target="_blank">
         ${_('Link to object')}
       </a>
       &nbsp;|&nbsp;