- def on_button_video_rebuild_clicked(self, widget):
+ def on_button_media_rebuild_clicked(self, widget):
"""
- Rebuild video cache requested
+ Rebuild media cache requested
@param widget: GTK-Widget
"""
> Can you also remove the @param here and place everything on one line.
> Thanks.
Paul,
Not a lot of things to say except that everything looks good.
I've spent some time toying with it and it was lovely working.
Test are okay.
Lint gives me some issues but as you're actively hacking on future, you have time to fix those.
Thanks for the work,
Samuel-
A few details below :
=== modified file 'cfg/content.conf'
--- cfg/content.conf 2009-05-31 17:11:16 +0000
+++ cfg/content.conf 2009-07-14 01:00:37 +0000
metric_units = True
> Maybe you can remove that "metric_units" it's deprecated.
=== modified file 'entertainerlib /dialog. py' dialog. py 2009-06-30 01:06:01 +0000 dialog. py 2009-07-14 00:56:36 +0000
--- entertainerlib/
+++ entertainerlib/
@@ -50,33 +50,25 @@
def on_url_ dialog_ ok_button_ clicked( self, widget):
self. on_location_ find_button_ clicked( widget)
@@ -555,9 +434,9 @@
"""
- def on_button_ video_rebuild_ clicked( self, widget): media_rebuild_ clicked( self, widget):
+ def on_button_
"""
- Rebuild video cache requested
+ Rebuild media cache requested
@param widget: GTK-Widget
"""
> Can you also remove the @param here and place everything on one line.
> Thanks.
=== modified file 'entertainerlib /tests/ test_configurat ion.py' tests/test_ configuration. py 2009-06-30 01:06:01 +0000 tests/test_ configuration. py 2009-07-14 01:04:31 +0000
self. assertEqual( self.configurat ion.get_ slideshow_ step(), 5)
--- entertainerlib/
+++ entertainerlib/
@@ -108,11 +108,6 @@
'''Test getting the slideshow step'''
- def test_hidden_ files_folders( self): hidden_ files_folders` setting.""" l(self. configuration. display_ hidden_ files_folders( ),
- """Test getting the `display_
- self.assertEqua
- False)
-
> Why did you keep the setting in content-conf and removed the test?