[glom/glom-1-20] Really prevent changing to developer mode for non-developers.



commit f656671332cd0ad96cac55e51d199db74f8a98b5
Author: Murray Cumming <murrayc murrayc com>
Date:   Thu Feb 2 09:40:37 2012 +0100

    Really prevent changing to developer mode for non-developers.
    
    	* glom/frame_glom.cc: on_menu_developer_developer(): Actually use the
    	result of Privs::get_user_is_in_group().
    	* tests/test_selfhosting_new_from_example_operator.cc: Test this
    	same function here, though this test currently fails anyway.
    	Really prevent changing to developer mode for non-developers.
    	This fixes bug #669043 (alien)
    
    Conflicts:
    
    	tests/test_selfhosting_new_from_example_operator.cc

 ChangeLog          |   10 ++++++++++
 glom/frame_glom.cc |    4 +++-
 2 files changed, 13 insertions(+), 1 deletions(-)
---
diff --git a/ChangeLog b/ChangeLog
index 581c482..8c1a298 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,13 @@
+2012-02-02  Murray Cumming  <murrayc murrayc com>
+
+	Really prevent changing to developer mode for non-developers.
+
+	* glom/frame_glom.cc: on_menu_developer_developer(): Actually use the
+	result of Privs::get_user_is_in_group().
+	* tests/test_selfhosting_new_from_example_operator.cc: Test this
+	same function here, though this test currently fails anyway.
+	This fixes bug #669043 (alien) 
+
 2012-02-01  Murray Cumming  <murrayc murrayc com>
 
 	Allow user and group names to have spaces and other special characters.
diff --git a/glom/frame_glom.cc b/glom/frame_glom.cc
index 25df6f8..8f0b2ec 100644
--- a/glom/frame_glom.cc
+++ b/glom/frame_glom.cc
@@ -464,7 +464,9 @@ void Frame_Glom::on_menu_developer_developer(const Glib::RefPtr<Gtk::RadioAction
       bool test = true;
 
       if(sharedconnection && sharedconnection->get_gda_connection()->supports_feature(Gnome::Gda::CONNECTION_FEATURE_USERS))
-        Privs::get_user_is_in_group(connection_pool->get_user(), GLOM_STANDARD_GROUP_NAME_DEVELOPER);
+      {
+        test = Privs::get_user_is_in_group(connection_pool->get_user(), GLOM_STANDARD_GROUP_NAME_DEVELOPER);
+      }
 
       if(test)
       {



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