From 9a9cd511e4972869b5525a950f15a00a3a4ae0b3 Mon Sep 17 00:00:00 2001
From: drfho <dr.frank.hoffmann@web.de>
Date: Mon, 24 Feb 2025 15:01:52 +0800
Subject: [PATCH] ZMI: consider ZMSSqlDb-case 'connection closed

---
 .../com.zms.foundation/ZMSSqlDb/__init__.py   |  2 +-
 .../ZMSSqlDb/renderShort.zpt                  | 19 +++++++++++++++----
 Products/zms/zmssqldb.py                      |  3 ++-
 3 files changed, 18 insertions(+), 6 deletions(-)

diff --git a/Products/zms/conf/metaobj_manager/com.zms.foundation/ZMSSqlDb/__init__.py b/Products/zms/conf/metaobj_manager/com.zms.foundation/ZMSSqlDb/__init__.py
index b7f3b857c..71366010b 100644
--- a/Products/zms/conf/metaobj_manager/com.zms.foundation/ZMSSqlDb/__init__.py
+++ b/Products/zms/conf/metaobj_manager/com.zms.foundation/ZMSSqlDb/__init__.py
@@ -26,7 +26,7 @@ class ZMSSqlDb:
 	package = "com.zms.foundation"
 
 	# Revision
-	revision = "5.0.0"
+	revision = "5.1.0"
 
 	# Type
 	type = "ZMSObject"
diff --git a/Products/zms/conf/metaobj_manager/com.zms.foundation/ZMSSqlDb/renderShort.zpt b/Products/zms/conf/metaobj_manager/com.zms.foundation/ZMSSqlDb/renderShort.zpt
index 085a96202..56554089b 100644
--- a/Products/zms/conf/metaobj_manager/com.zms.foundation/ZMSSqlDb/renderShort.zpt
+++ b/Products/zms/conf/metaobj_manager/com.zms.foundation/ZMSSqlDb/renderShort.zpt
@@ -1,19 +1,30 @@
 <!-- ZMSSqlDb.renderShort -->
 
-<tal:block tal:define="zmscontext options/zmscontext">
+<tal:block tal:define="
+	zmscontext options/zmscontext;
+	entities python:zmscontext.getEntities()">
 	<p class="my-2" style="color:#008ac7;">
 		<i class="fas fa-database"></i>
 		<strong tal:content="python:zmscontext.connection_id">the connection-id</strong>
 	</p>
-	<ul class="fa-ul">
-		<li tal:repeat="table python:[x for x in zmscontext.getEntities() if x['type']=='table']">
+	<ul class="fa-ul mb-5" tal:condition="entities">
+		<li tal:repeat="table python:[x for x in entities if x['type']=='table']">
 			<a class="font-monospace" tal:attributes="href python:'%s/manage_main?qentity=%s'%(zmscontext.id,table['id'])" >
 				<span class="fa-li"><i class="fas fa-table"></i></span>
 				<span class="table_id" tal:content="table/id">the table</span>
 			</a>
 		</li>
 	</ul>
-	<br/>
+	<div class="alert alert-warning ml-0" tal:condition="not:entities">
+		No tables found in this database. 
+		<a href="#" target="_blank" 
+			tal:attributes="
+				href python:'%s/%s/manage_main'%(zmscontext.getHome().absolute_url(),zmscontext.connection_id);
+				title python:'%s/%s/manage_main'%(zmscontext.getHome().absolute_url(),zmscontext.connection_id);">
+			Database connection
+		</a> 
+		may be closed.
+	</div>
 </tal:block>
 
 <!-- /ZMSSqlDb.renderShort -->
\ No newline at end of file
diff --git a/Products/zms/zmssqldb.py b/Products/zms/zmssqldb.py
index c89dfa131..2ee5bf2cc 100644
--- a/Products/zms/zmssqldb.py
+++ b/Products/zms/zmssqldb.py
@@ -882,7 +882,8 @@ def getEntities(self):
       
       entities = []
       da = self.getDA()
-      if da is None: return entities
+      if da is None or da.connected() is None:
+        return entities
       
       tableBrwsrs = da.tpValues()