Fixes formatting.

Freeseer · Nov 18, 2014 · fa33a7c · fa33a7c
1 parent c1772a4
commit fa33a7c
Show file tree

Hide file tree

Showing 3 changed files with 20 additions and 24 deletions.
diff --git a/src/freeseer/framework/config/core.py b/src/freeseer/framework/config/core.py
@@ -58,12 +58,10 @@ def presentation(self, value):
 
     def schema(self):
         """Returns the json schema for an Option."""
-        return {
-            'type': self.schema_type
-        } if self.default == self.NotSpecified else {
-            'type': self.schema_type,
-            'default': self.default,
-        }
+        schema = {'type': self.schema_type}
+        if self.default != self.NotSpecified:
+            schema['default'] = self.default
+        return schema
 
     # Override these!
 

diff --git a/src/freeseer/framework/config/options.py b/src/freeseer/framework/config/options.py
@@ -141,9 +141,7 @@ def decode(self, value):
             raise InvalidDecodeValueError(value)
 
     def schema(self):
-        return {
-            'enum': self.choices
-        } if self.default == Option.NotSpecified else {
-            'enum': self.choices,
-            'default': self.default,
-        }
+        schema = {'enum': self.choices}
+        if self.default != Option.NotSpecified:
+            schema['default'] = self.default
+        return schema
diff --git a/src/freeseer/tests/framework/test_config.py b/src/freeseer/tests/framework/test_config.py
@@ -74,51 +74,51 @@ def test_schema(self):
                 },
                 'auto_hide': {
                     'default': False,
-                    'type': 'boolean'
+                    'type': 'boolean',
                 },
                 'enable_audio_recording': {
                     'default': True,
-                    'type': 'boolean'
+                    'type': 'boolean',
                 },
                 'enable_video_recording': {
                     'default': True,
-                    'type': 'boolean'
+                    'type': 'boolean',
                 },
                 'videomixer': {
                     'default': 'Video Passthrough',
-                    'type': 'string'
+                    'type': 'string',
                 },
                 'audiomixer': {
                     'default': 'Audio Passthrough',
-                    'type': 'string'
+                    'type': 'string',
                 },
                 'record_to_file': {
                     'default': True,
-                    'type': 'boolean'
+                    'type': 'boolean',
                 },
                 'record_to_file_plugin': {
                     'default': 'Ogg Output',
-                    'type': 'string'
+                    'type': 'string',
                 },
                 'record_to_stream': {
                     'default': False,
-                    'type': 'boolean'
+                    'type': 'boolean',
                 },
                 'record_to_stream_plugin': {
                     'default': 'RTMP Streaming',
-                    'type': 'string'
+                    'type': 'string',
                 },
                 'audio_feedback': {
                     'default': False,
-                    'type': 'boolean'
+                    'type': 'boolean',
                 },
                 'video_preview': {
                     'default': True,
-                    'type': 'boolean'
+                    'type': 'boolean',
                 },
                 'default_language': {
                     'default': 'tr_en_US.qm',
-                    'type': 'string'
+                    'type': 'string',
                 },
             },
         }