SVN Commit by andreas: r4500 - in trunk/pgadmin3/src: ctl frm main

Author: andreas
Date: 2005-10-05 11:55:28 +0100 (Wed, 05 Oct 2005)
New Revision: 4500

Modified:
trunk/pgadmin3/src/ctl/ctlSQLResult.cpp
trunk/pgadmin3/src/frm/frmExport.cpp
trunk/pgadmin3/src/main/events.cpp
Log:
some minor fixes

Modified: trunk/pgadmin3/src/ctl/ctlSQLResult.cpp
===================================================================
--- trunk/pgadmin3/src/ctl/ctlSQLResult.cpp 2005-10-05 10:41:15 UTC (rev 4499)
+++ trunk/pgadmin3/src/ctl/ctlSQLResult.cpp 2005-10-05 10:55:28 UTC (rev 4500)
@@ -45,7 +45,7 @@
if (rowsRetrieved>0 || (thread && thread->DataSet()->NumRows() > 0))
{
frmExport dlg(this);
- if (dlg.ShowModal() > 0)
+ if (dlg.ShowModal() == wxID_OK)
{
if (rowsRetrieved> 0)
return dlg.Export(this);

Modified: trunk/pgadmin3/src/frm/frmExport.cpp
===================================================================
--- trunk/pgadmin3/src/frm/frmExport.cpp 2005-10-05 10:41:15 UTC (rev 4499)
+++ trunk/pgadmin3/src/frm/frmExport.cpp 2005-10-05 10:55:28 UTC (rev 4500)
@@ -128,7 +128,7 @@


if (IsModal())
- EndModal(1);
+ EndModal(wxID_OK);
else
Destroy();
}
@@ -257,7 +257,7 @@
void frmExport::OnCancel(wxCommandEvent &ev)
{
if (IsModal())
- EndModal(-1);
+ EndModal(wxID_CANCEL);
else
Destroy();
}

Modified: trunk/pgadmin3/src/main/events.cpp
===================================================================
--- trunk/pgadmin3/src/main/events.cpp 2005-10-05 10:41:15 UTC (rev 4499)
+++ trunk/pgadmin3/src/main/events.cpp 2005-10-05 10:55:28 UTC (rev 4500)
@@ -372,9 +372,13 @@
wxMenuItem *item=menuBar->FindItem(id);
if (item)
{
- wxMenu *menu=item->GetMenu();
- size_t position=0;
- do
+ wxMenu *menu=item->GetSubMenu();
+ wxASSERT(menu);
+ if (!menu)
+ return;
+
+ size_t position;
+ for (position = 0 ; position < menu->GetMenuItemCount() ; position++)
{
item = menu->FindItemByPosition(position);
if (item && item->IsEnabled())
@@ -382,9 +386,8 @@
menuBar->Enable(id, true);
return;
}
- position++;
}
- while (item);
+
menuBar->Enable(id, false);
}
}