SVN Commit by dpage: r4380 - in trunk/pgadmin3: . src/dlg

Author: dpage
Date: 2005-07-21 09:26:58 +0100 (Thu, 21 Jul 2005)
New Revision: 4380

Modified:
trunk/pgadmin3/CHANGELOG.txt
trunk/pgadmin3/src/dlg/dlgTable.cpp
Log:
Prompt the user before removing columns, constraints or inherited tables in the table dialogue, per gripe from Anoop Kumar.

Modified: trunk/pgadmin3/CHANGELOG.txt
===================================================================
--- trunk/pgadmin3/CHANGELOG.txt 2005-07-06 10:26:21 UTC (rev 4379)
+++ trunk/pgadmin3/CHANGELOG.txt 2005-07-21 08:26:58 UTC (rev 4380)
@@ -17,6 +17,7 @@
</ul>
<br>
<ul>
+ <li>2005-07-21 DP Prompt the user before removing columns, constraints or inherited tables in the table dialogue.
<li>2005-07-06 DP Prompt the user to cancel running queries before closing the Query tool.
<li>2005-07-04 DP Check for and display asynchronous notifications in the query tool.
<li>2005-06-21 AP typo fixes, rename "initial db" to "maintenance db"

Modified: trunk/pgadmin3/src/dlg/dlgTable.cpp
===================================================================
--- trunk/pgadmin3/src/dlg/dlgTable.cpp 2005-07-06 10:26:21 UTC (rev 4379)
+++ trunk/pgadmin3/src/dlg/dlgTable.cpp 2005-07-21 08:26:58 UTC (rev 4380)
@@ -596,6 +596,9 @@

void dlgTable::OnRemoveTable(wxCommandEvent &ev)
{
+ if (wxMessageBox(_("Are you sure you wish to remove the selected table?"), _("Remove table?"), wxYES_NO | wxNO_DEFAULT | wxICON_QUESTION) == wxNO)
+ return;
+
int sel=lbTables->GetSelection();
if (sel >= 0)
{
@@ -665,6 +668,9 @@

void dlgTable::OnRemoveCol(wxCommandEvent &ev)
{
+ if (wxMessageBox(_("Are you sure you wish to remove the selected column?"), _("Remove column?"), wxYES_NO | wxNO_DEFAULT | wxICON_QUESTION) == wxNO)
+ return;
+
lstColumns->DeleteCurrentItem();

btnRemoveCol->Disable();
@@ -742,6 +748,9 @@

void dlgTable::OnRemoveConstr(wxCommandEvent &ev)
{
+ if (wxMessageBox(_("Are you sure you wish to remove the selected constraint?"), _("Remove constraint?"), wxYES_NO | wxNO_DEFAULT | wxICON_QUESTION) == wxNO)
+ return;
+
int pos=lstConstraints->GetSelection();
if (pos < 0)
return;