From d8d2fe1174b5bb0a1ea9c200676af4b4dfa6dfca Mon Sep 17 00:00:00 2001
From: Etienne Trimaille <gustrimaille@yahoo.fr>
Date: Mon, 6 Jan 2025 17:06:49 +0100
Subject: [PATCH] Use correct online help and Git repository according to the
 correct QGIS Server wrapper used

---
 lizmap/app/system/mainconfig.ini.php          |  2 +-
 .../locales/en_US/admin.UTF-8.properties      |  2 +-
 .../admin/templates/server_information.tpl    | 42 +++++++++++--------
 3 files changed, 27 insertions(+), 19 deletions(-)

diff --git a/lizmap/app/system/mainconfig.ini.php b/lizmap/app/system/mainconfig.ini.php
index 6baab44f63..84c573bfcf 100644
--- a/lizmap/app/system/mainconfig.ini.php
+++ b/lizmap/app/system/mainconfig.ini.php
@@ -35,7 +35,7 @@
 ; QGIS server required minimum version
 qgisServer="3.28"
 ; Lizmap server QGIS plugin required minimum version
-lizmapServerPlugin="2.11.2"
+lizmapServerPlugin="2.12.0"
 ; Lizmap QGIS desktop plugin required/recommended minimum version for newly or updated project only
 ; This version MUST match at least on https://plugins.qgis.org/plugins/lizmap/#plugin-versions
 ; with the minimum QGIS server version supported above.
diff --git a/lizmap/modules/admin/locales/en_US/admin.UTF-8.properties b/lizmap/modules/admin/locales/en_US/admin.UTF-8.properties
index 5a0c5a7216..e974c5abe1 100644
--- a/lizmap/modules/admin/locales/en_US/admin.UTF-8.properties
+++ b/lizmap/modules/admin/locales/en_US/admin.UTF-8.properties
@@ -272,7 +272,6 @@ server.information.qgis.label=QGIS Server
 server.information.qgis.metadata=Version
 server.information.qgis.version=Number
 server.information.qgis.name=Name
-server.information.qgis.commit_id=Commit ID
 server.information.qgis.action=Action
 
 server.information.qgis.plugins=Plugins
@@ -280,6 +279,7 @@ server.information.qgis.plugins.version=Version number
 server.information.qgis.plugin=Plugin
 server.information.qgis.plugin.version=Version
 server.information.qgis.plugin.help=Help
+server.information.not.installed=Not installed
 
 server.information.qgis.test.ok=QGIS Server is correctly installed and returns the expected response for OGC requests.
 server.information.qgis.test.error=QGIS Server is not correctly installed in your server or the URL for OGC requests given in Lizmap configuration is not correct.
diff --git a/lizmap/modules/admin/templates/server_information.tpl b/lizmap/modules/admin/templates/server_information.tpl
index 542fd7ba2d..7d1856949a 100644
--- a/lizmap/modules/admin/templates/server_information.tpl
+++ b/lizmap/modules/admin/templates/server_information.tpl
@@ -115,6 +115,8 @@
             <td>
                 <a href="https://github.com/qgis/QGIS/releases/tag/{$data['qgis_server_info']['metadata']['tag']}" target="_blank">
                     {$data['qgis_server_info']['metadata']['version']}
+                </a> - <a href="https://github.com/qgis/QGIS/commit/{$data['qgis_server_info']['metadata']['commit_id']}" target="_blank">
+                    {$data['qgis_server_info']['metadata']['commit_id']}
                 </a>
             </td>
         </tr>
@@ -123,32 +125,38 @@
             <td>{$data['qgis_server_info']['metadata']['name']}</td>
         </tr>
         <tr>
-            <th>{@admin.server.information.qgis.commit_id@}</th>
-            <td><a href="https://github.com/qgis/QGIS/commit/{$data['qgis_server_info']['metadata']['commit_id']}" target="_blank">{$data['qgis_server_info']['metadata']['commit_id']}</a></td>
-        </tr>
-        <tr>
-            <th>Py-QGIS-Server <a href="https://docs.3liz.org/py-qgis-server" target="_blank"><span class='badge rounded-pill bg-secondary'>{@admin.server.information.qgis.plugin.help@}</span></a></th>
-            <td>
-                {if $data['qgis_server_info']['py_qgis_server']['found']}
+            {if $data['qgis_server_info']['py_qgis_server']['found']}
+                <th>
+                    {$data['qgis_server_info']['py_qgis_server']['name']}
+                    {if $data['qgis_server_info']['py_qgis_server']['documentation_url']}
+                        <a href="{$data['qgis_server_info']['py_qgis_server']['documentation_url']}" target="_blank">
+                            <span class='badge rounded-pill bg-secondary'>{@admin.server.information.qgis.plugin.help@}</span>
+                        </a>
+                    {/if}
+                </th>
+                <td>
                     {if $data['qgis_server_info']['py_qgis_server']['stable']}
                         {if $data['qgis_server_info']['py_qgis_server']['version'] == 'n/a'}
                             {* If the value is n/a, Py-QGIS-Server failed to fetch the version *}
                             {* https://github.com/3liz/py-qgis-server/blob/b11bba45495d32e348457c0802fe08f2bf952b8b/pyqgisserver/version.py#L17 *}
                             {$data['qgis_server_info']['py_qgis_server']['version']}
                         {else}
-                            <a href="https://github.com/3liz/py-qgis-server/releases/tag/{$data['qgis_server_info']['py_qgis_server']['version']}" target="_blank">
-                                {$data['qgis_server_info']['py_qgis_server']['version']}
-                            </a>
+                            {if $data['qgis_server_info']['py_qgis_server']['git_repository_url']}
+                                <a href="{$data['qgis_server_info']['py_qgis_server']['git_repository_url']}/releases/tag/{$data['qgis_server_info']['py_qgis_server']['version']}" target="_blank">
+                                    {$data['qgis_server_info']['py_qgis_server']['version']}
+                                </a>
+                            {/if}
                         {/if}
                     {else}
-                    <a href="https://github.com/3liz/py-qgis-server/commit/{$data['qgis_server_info']['py_qgis_server']['commit_id']}" target="_blank">
-                        {$data['qgis_server_info']['py_qgis_server']['version']} - {$data['qgis_server_info']['py_qgis_server']['commit_id']}
-                    </a>
+                        <a href="{$data['qgis_server_info']['py_qgis_server']['git_repository_url']}/commit/{$data['qgis_server_info']['py_qgis_server']['commit_id']}" target="_blank">
+                            {$data['qgis_server_info']['py_qgis_server']['version']} - {$data['qgis_server_info']['py_qgis_server']['commit_id']}
+                        </a>
                     {/if}
-                {else}
-                    {$data['qgis_server_info']['py_qgis_server']['version']}
-                {/if}
-            </td>
+                </td>
+            {else}
+                {* Generic name for Py-QGIS-Server and QJazz ? *}
+                <th>QGIS Server wrapper</th><td>{@admin.server.information.not.installed@}</td>
+            {/if}
         </tr>
         {if $qgisServerNeedsUpdate }
         <tr>