Change writing event to reduce the use of gconf object
[wifihood] / wifiscanner / wifimap / config.py
index 88cde96..78c5974 100644 (file)
@@ -9,38 +9,71 @@ except :
 
 class Configuration :
 
-    def __init__ ( self ) :
-        self.homedir = None
-        self.dbname = None
-        self.mapsdir , self.mapclass = None , None
-        self.lat , self.lon = 0.0 , 0.0
-        self.zoom = 0
+    def __init__ ( self , type ) :
+        self._type = type
         self.read()
 
     def read ( self ) :
-        client = gconf.client_get_default()
-        self.homedir = client.get_string( "/apps/wifihood/basedir" ) or "/home/user/MyDocs"
-        self.dbname = client.get_string( "/apps/wifihood/dbname" ) or "wifiscanner.db"
-        self.mapsdir = client.get_string( "/apps/wifihood/maps" ) or "/home/user/MyDocs/.maps"
-        self.mapclass = client.get_string( "/apps/wifihood/maptype" ) or "OpenStreetMap I"
-        self.lat = client.get_float( "/apps/wifihood/lattitude" ) or client.get_float( "/apps/maemo/maemo-mapper/center_latitude" ) or 40.416
-        self.lon = client.get_float( "/apps/wifihood/longitude" ) or client.get_float( "/apps/maemo/maemo-mapper/center_longitude" ) or -3.683
-        self.zoom = client.get_int( "/apps/wifihood/zoom" ) or client.get_float( "/apps/maemo/maemo-mapper/zoom" ) or 15
-
-    def save ( self ) :
-        client = gconf.client_get_default()
-        client.set_string( "/apps/wifihood/basedir" , self.homedir )
-        client.set_string( "/apps/wifihood/dbname" , self.dbname )
-        client.set_string( "/apps/wifihood/maps" , self.mapsdir )
-        client.set_string( "/apps/wifihood/maptype" , self.mapclass )
-        client.set_float( "/apps/wifihood/lattitude" , self.lat )
-        client.set_float( "/apps/wifihood/longitude" , self.lon )
-        client.set_int( "/apps/wifihood/zoom" , self.zoom )
-
+        self._client = gconf.client_get_default()
+
+        self.homedir = self._client.get_string( "/apps/wifihood/basedir" ) or "/home/user/MyDocs"
+        self.dbname = self._client.get_string( "/apps/wifihood/dbname" ) or "wifiscanner.db"
+        self.mapsdir = self._client.get_string( "/apps/wifihood/maps" ) or "/home/user/MyDocs/.maps"
+        self.mapclass = self._client.get_string( "/apps/wifihood/maptype" ) or "OpenStreetMap I"
+
+        self.scan_period = self._client.get_int( "/apps/wifihood/scan-period" ) or 5000
+
+        self.store_log = self._client.get_bool( "/apps/wifihood/store-logfile" )
+
+        self.use_mapper = self._client.get_bool( "/apps/wifihood/use-mapper" )
+        self.store_gps = self._client.get_bool( "/apps/wifihood/store-gps" )
+        if self.use_mapper :
+            # FIXME : This will reset the stored default
+            self.store_gps = False
+            self.lat = self._client.get_float( "/apps/maemo/maemo-mapper/center_latitude" )
+            self.lon = self._client.get_float( "/apps/maemo/maemo-mapper/center_longitude" )
+            self.zoom = self._client.get_int( "/apps/maemo/maemo-mapper/zoom" )
+        else :
+            self.lat = self._client.get_float( "/apps/wifihood/latitude" )
+            self.lon = self._client.get_float( "/apps/wifihood/longitude" )
+            self.zoom = self._client.get_int( "/apps/wifihood/map-zoom" )
+
+        if self._type == 'scanner' :
+            self.zoom = self._client.get_int( "/apps/wifihood/%s-zoom" % self._type )
+
+        if self.lat == 0.0 and self.lon == 0.0 :
+            self.lat , self.lon = 40.416 , -3.683
+        if self.zoom == 0 :
+            self.zoom = 15
+
+    def set_latlon ( self , ( lat , lon ) ) :
+        if self.store_gps :
+            self._client.set_float( "/apps/wifihood/latitude" , lat )
+            self._client.set_float( "/apps/wifihood/longitude" , lon )
+        self.lat , self.lon = lat , lon
+
+    def set_zoom ( self , zoom ) :
+        if self.store_gps or self._type == 'scanner' :
+            self._client.set_int( "/apps/wifihood/%s-zoom" % self._type , zoom )
+        self.zoom = zoom
+
+    def save ( self , widget , event ) :
+        self._client.set_string( "/apps/wifihood/basedir" , self.homedir )
+        self._client.set_string( "/apps/wifihood/dbname" , self.dbname )
+        self._client.set_string( "/apps/wifihood/maps" , self.mapsdir )
+        self._client.set_string( "/apps/wifihood/maptype" , self.mapclass )
+
+        self._client.set_int( "/apps/wifihood/scan-period" , self.scan_period )
+        self._client.set_bool( "/apps/wifihood/store-logfile" , self.store_log )
+        self._client.set_bool( "/apps/wifihood/use-mapper" , self.use_mapper )
+        self._client.set_bool( "/apps/wifihood/store-gps" , self.store_gps )
+
+        self.set_latlon( ( self.lat , self.lon ) )
+        self.set_zoom( self.zoom )
 
 class AbstractSettingsWindow :
 
-    def __init__ ( self , config , handler=None ) :
+    def __init__ ( self , config , handler ) :
         self.set_title( "Wifihood Settings" )
 
         scrollwin = self.MainArea()
@@ -66,6 +99,7 @@ class AbstractSettingsWindow :
         database.attach(dblabel, 0, 1, 0, 1, gtk.EXPAND|gtk.FILL, 0, 0, 5)
 
         dbvalue = self.Entry()
+        dbvalue.connect( "unrealize" , self.entry_cb , config , "basedir" )
         dbvalue.set_text( config.homedir )
         dbvalue.show()
         database.attach(dbvalue, 1, 2, 0, 1, gtk.EXPAND|gtk.FILL, 0, 0, 5)
@@ -75,6 +109,7 @@ class AbstractSettingsWindow :
         database.attach(dblabel, 0, 1, 1, 2, gtk.EXPAND|gtk.FILL, 0, 0, 5)
 
         dbvalue = self.Entry()
+        dbvalue.connect( "unrealize" , self.entry_cb , config , "dbname" )
         dbvalue.set_text( config.dbname )
         dbvalue.show()
         database.attach(dbvalue, 1, 2, 1, 2, gtk.EXPAND|gtk.FILL, 0, 0, 5)
@@ -111,11 +146,15 @@ class AbstractSettingsWindow :
 
         button = self.CheckButton()
         button.set_label( "Take initial coordinates from maemo-mapper" )
+        button.connect( "toggled" , self.checkbutton_cb , config , "use-mapper" )
+        button.set_active( config.use_mapper )
         button.show()
         gps.attach(button, 0, 2, 0, 1, gtk.EXPAND|gtk.FILL) #, 0, 0, 5)
 
         button = self.CheckButton()
         button.set_label( "Store changes in coordinates" )
+        button.connect( "toggled" , self.checkbutton_cb , config , "store-gps" )
+        button.set_active( config.store_gps )
         button.show()
         gps.attach(button, 0, 2, 1, 2, gtk.EXPAND|gtk.FILL) #, 0, 0, 5)
 
@@ -125,19 +164,45 @@ class AbstractSettingsWindow :
         dataframe.show()
         vbox.pack_start(dataframe, True, True, 0)
 
+        datatable = gtk.Table(2, 2, False)
+        datatable.show()
+        dataframe.add(datatable)
+
+        scanlabel = gtk.Label( "Scanning interval" )
+        scanlabel.show()
+        datatable.attach(scanlabel, 0, 1, 0, 1, gtk.EXPAND|gtk.FILL)
+
+        scanvalue = self.Entry()
+        scanvalue.connect( "unrealize" , self.int_cb , config , "scan-period" , 1000 )
+        scanvalue.set_text( "%s" % ( float(config.scan_period) / 1000 ) )
+        scanvalue.show()
+        datatable.attach(scanvalue, 1, 2, 0, 1, gtk.EXPAND|gtk.FILL)
+
         button = self.CheckButton()
         button.set_label( "Write full logfile" )
+        button.connect( "toggled" , self.checkbutton_cb , config , "store-logfile" )
+        button.set_active( config.store_log )
         button.show()
-        dataframe.add(button)
+        datatable.attach(button, 0, 2, 1, 2, gtk.EXPAND|gtk.FILL)
 
         self.show()
 
+    def entry_cb ( self , entry , config , keyword ) :
+        config._client.set_string( "%s/%s" % ( "/apps/wifihood" , keyword ) , entry.get_text() )
+
+    def int_cb ( self , entry , config , keyword , scale=1 ) :
+        config._client.set_int( "%s/%s" % ( "/apps/wifihood" , keyword ) , int( scale * float( entry.get_text() ) ) )
+
+    def checkbutton_cb ( self , button , config , keyword ) :
+        config._client.set_bool( "%s/%s" % ( "/apps/wifihood" , keyword ) , button.get_active() )
+
 if hildon :
 
   class ZoomDialog ( hildon.TouchSelector ) :
 
-    def __init__ ( self , config , handler=None , extra=None ) :
+    def __init__ ( self , config , handler ) :
         hildon.TouchSelector.__init__( self )
+        self.handler = handler
 
         zooms = gtk.ListStore(str)
 
@@ -154,21 +219,23 @@ if hildon :
         # NOTE : with text=True, we must use 1 instead of 0
         self.set_active( 0 , active )
 
+    def zoomdialog ( self , widget , conf ) :
+        newzoom = int( widget.get_selector().get_current_text() )
+        if self.handler : self.handler( newzoom )
+        conf.set_zoom( newzoom )
+
   class SettingsWindow ( hildon.StackableWindow , AbstractSettingsWindow ) :
 
     def __init__ ( self , config , handler=None ) :
         hildon.StackableWindow.__init__( self )
         AbstractSettingsWindow.__init__( self , config , handler )
 
-    def zoomdialog ( self , widget , conf , extra=None ) :
-        conf.zoom = int( widget.get_selector().get_current_text() )
-
     def ZoomButton ( self , text , config , handler ) :
-        selector = ZoomDialog( config )
+        selector = ZoomDialog( config , handler )
         button = hildon.PickerButton( gtk.HILDON_SIZE_AUTO, hildon.BUTTON_ARRANGEMENT_VERTICAL)
         button.set_title( text )
         button.set_selector( selector )
-        button.connect_object( "value-changed", self.zoomdialog , button , config , handler )
+        button.connect_object( "value-changed", selector.zoomdialog , button , config )
         return button
 
     def MainArea ( self ) :
@@ -187,7 +254,7 @@ else :
 
   class ZoomDialog ( gtk.Dialog ) :
 
-    def __init__ ( self , config , handler=None , extra=None ) :
+    def __init__ ( self , config , handler , labelsetter=None ) :
         gtk.Dialog.__init__( self , "Select zoom level",
                              None,
                              gtk.DIALOG_MODAL,
@@ -211,15 +278,16 @@ else :
 
         self.vbox.pack_start(combo , True, True, 0)
 
-        self.connect_object( "response", self.response , combo , config , handler , extra )
+        self.connect_object( "response", self.response , combo , config , handler , labelsetter )
 
-    def response ( self , combo , response  , config , handler , extra=None ) :
+    def response ( self , combo , response  , config , handler , labelsetter ) :
         if response == gtk.RESPONSE_ACCEPT :
             item = combo.get_active_iter()
             model = combo.get_model()
-            handler( model.get(item,0)[0])
-            if extra : extra( model.get(item,0)[0])
-            config.zoom = model.get(item,0)[0]
+            newzoom = model.get(item,0)[0]
+            if labelsetter : labelsetter( newzoom )
+            if handler : handler( newzoom )
+            config.set_zoom( newzoom )
         self.destroy()
 
   class SettingsWindow ( gtk.Window , AbstractSettingsWindow ) :
@@ -227,10 +295,11 @@ else :
     def __init__ ( self , config , handler=None ) :
         gtk.Window.__init__( self )
         AbstractSettingsWindow.__init__( self , config , handler )
+        self.connect_object("delete_event", config.save , self )
 
     def MainArea ( self ) :
         scrollwin = gtk.ScrolledWindow()
-        scrollwin.set_size_request(-1, 260)
+        scrollwin.set_size_request(-1, 290)
         scrollwin.set_policy(gtk.POLICY_NEVER, gtk.POLICY_AUTOMATIC)
         return scrollwin
 
@@ -240,8 +309,8 @@ else :
     def Button ( self , label=None ) :
         return gtk.Button( label )
 
-    def zoomdialog ( self , widget , config , extra ) :
-        dialog = ZoomDialog( config , widget.set_value , extra )
+    def zoomdialog ( self , widget , config , handler ) :
+        dialog = ZoomDialog( config , handler , widget.set_value )
         dialog.show_all()
 
     def ZoomButton ( self , text , config , handler ) :
@@ -260,9 +329,8 @@ else :
     def CheckButton ( self ) :
         return gtk.CheckButton()
 
-config = Configuration()
-
 if __name__ == "__main__" :
+    config = Configuration( 'scanner' )
     window = SettingsWindow( config )
     window.connect("delete_event", gtk.main_quit, None)
     window.show()