[glom] Remove unneded alert dialogs / TODO items.



commit 2605185b8848dec74b711f80076234318b2301e7
Author: Daniel Borgmann <danielb openismus com>
Date:   Wed Mar 31 10:21:56 2010 +0100

    Remove unneded alert dialogs / TODO items.
    
    * glom/frame_glom.cc: No table warnings are not needed anymore
    for menu items, so remove them.

 ChangeLog          |    7 +++++++
 glom/frame_glom.cc |   40 +++++++++++-----------------------------
 2 files changed, 18 insertions(+), 29 deletions(-)
---
diff --git a/ChangeLog b/ChangeLog
index aa924fd..d6c31bb 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,12 @@
 2010-03-31  Daniel Borgmann  <danielb openismus com>
 
+	Remove unneded alert dialogs / TODO items.
+
+	* glom/frame_glom.cc: No table warnings are not needed anymore
+	for menu items, so remove them.
+
+2010-03-31  Daniel Borgmann  <danielb openismus com>
+
 	Set window icon for OK dialogs.
 
 	* glom/utils_ui.cc: show_ok_dialog: Set window icon.
diff --git a/glom/frame_glom.cc b/glom/frame_glom.cc
index f26c1f9..08fa4f5 100644
--- a/glom/frame_glom.cc
+++ b/glom/frame_glom.cc
@@ -1765,13 +1765,9 @@ void Frame_Glom::do_menu_developer_fields(Gtk::Window& parent)
 {
   //Check that there is a table to show:
   if(m_table_name.empty())
-  {
-    alert_no_table(); //TODO: Disable the menu item instead.
-  }
-  else
-  {
-    do_menu_developer_fields(parent, m_table_name);
-  }
+    return;
+
+  do_menu_developer_fields(parent, m_table_name);
 }
 
 
@@ -1818,13 +1814,9 @@ void Frame_Glom::on_menu_developer_relationships()
 {
   //Check that there is a table to show:
   if(m_table_name.empty())
-  {
-    alert_no_table(); //TODO: Disable the menu item instead.
-  }
-  else
-  {
-    do_menu_developer_relationships(*get_app_window(), m_table_name);
-  }
+    return;
+
+  do_menu_developer_relationships(*get_app_window(), m_table_name);
 }
 
 void Frame_Glom::on_menu_developer_users()
@@ -1860,25 +1852,18 @@ void Frame_Glom::on_menu_developer_layout()
 {
   //Check that there is a table to show:
   if(m_table_name.empty())
-  {
-    alert_no_table(); //TODO: Disable the menu item instead.
-  }
-  else
-  {
-    Notebook_Glom* notebook_current = dynamic_cast<Notebook_Glom*>(m_pBox_Mode->get_child());
-    if(notebook_current)
-      notebook_current->do_menu_developer_layout();
-  }
+    return;
+
+  Notebook_Glom* notebook_current = dynamic_cast<Notebook_Glom*>(m_pBox_Mode->get_child());
+  if(notebook_current)
+    notebook_current->do_menu_developer_layout();
 }
 
 void Frame_Glom::on_menu_developer_reports()
 {
   //Check that there is a table to show:
   if(m_table_name.empty())
-  {
-    alert_no_table(); //TODO: Disable the menu item instead.
     return;
-  }
 
   //Create the widget if necessary:
   if(!m_pBox_Reports)
@@ -1909,10 +1894,7 @@ void Frame_Glom::on_menu_developer_print_layouts()
 {
   //Check that there is a table to show:
   if(m_table_name.empty())
-  {
-    alert_no_table(); //TODO: Disable the menu item instead.
     return;
-  }
 
   //Create the widget if necessary:
   if(!m_pBox_PrintLayouts)



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]