Hi,

While working on the dialogs' review, I was wondering why we show the SQL tab for the Server's properties dialog. I think we shouldn't display this tab on this dialog. The patch enclosed does this.

I'm not sure this qualifies as a bug. But it shouldn't be hard to apply the patch on the 1.8 branch.

Comments?


--
Guillaume.
 http://www.postgresqlfr.org
 http://dalibo.com
Index: pgadmin/dlg/dlgProperty.cpp
===================================================================
--- pgadmin/dlg/dlgProperty.cpp	(revision 7393)
+++ pgadmin/dlg/dlgProperty.cpp	(working copy)
@@ -311,8 +311,11 @@
 
 void dlgProperty::CreateAdditionalPages()
 {
-    sqlPane = new ctlSQLBox(nbNotebook, CTL_PROPSQL, wxDefaultPosition, wxDefaultSize, wxTE_MULTILINE | wxSUNKEN_BORDER | wxTE_READONLY | wxTE_RICH2);
-    nbNotebook->AddPage(sqlPane, wxT("SQL"));
+    if (wxString(factory->GetTypeName()).Cmp(wxT("Server")))
+    {
+      sqlPane = new ctlSQLBox(nbNotebook, CTL_PROPSQL, wxDefaultPosition, wxDefaultSize, wxTE_MULTILINE | wxSUNKEN_BORDER | wxTE_READONLY | wxTE_RICH2);
+      nbNotebook->AddPage(sqlPane, wxT("SQL"));
+    }
 }
 
 
-- 
Sent via pgadmin-hackers mailing list ([email protected])
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers

Reply via email to