Hi,

PFA minor patch to add object validation in table node for SQL and
Properties tab .
RM#2395


--
Regards,
Murtuza Zabuawala
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
diff --git 
a/web/pgadmin/browser/server_groups/servers/databases/schemas/tables/__init__.py
 
b/web/pgadmin/browser/server_groups/servers/databases/schemas/tables/__init__.py
index b6bf7ed..aee46c9 100644
--- 
a/web/pgadmin/browser/server_groups/servers/databases/schemas/tables/__init__.py
+++ 
b/web/pgadmin/browser/server_groups/servers/databases/schemas/tables/__init__.py
@@ -1108,6 +1108,10 @@ class TableView(PGChildNodeView, DataTypeReader, 
VacuumSettings):
         status, res = self.conn.execute_dict(SQL)
         if not status:
             return internal_server_error(errormsg=res)
+
+        if len(res['rows']) == 0:
+                return gone(gettext("Could not find the object."))
+
         data = res['rows'][0]
 
         data['vacuum_settings_str'] = ""
@@ -1690,8 +1694,7 @@ class TableView(PGChildNodeView, DataTypeReader, 
VacuumSettings):
             if not status:
                 return internal_server_error(errormsg=res)
             data = res['rows'][0]
-            # TODO://
-            # Find SQL which can enable all or disable all triggers
+
             SQL = render_template("/".join([self.template_path,
                                             'enable_disable_trigger.sql']),
                                   data=data, is_enable_trigger=is_enable)
@@ -2494,6 +2497,9 @@ class TableView(PGChildNodeView, DataTypeReader, 
VacuumSettings):
         if not status:
             return internal_server_error(errormsg=res)
 
+        if len(res['rows']) == 0:
+                return gone(gettext("Could not find the object."))
+
         data = res['rows'][0]
 
         # Table & Schema declaration so that we can use them in child nodes
-- 
Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers

Reply via email to