Updates to endpoint arguments, and some WIP implementation.

Author: FranciscoCanas

Diff for: src/freeseer/frontend/controller/configuration.py

@@ -24,7 +24,7 @@
 
 import signal
 
-from flask import Blueprint
+from flask import Blueprint, request
 from freeseer import settings, logging
 from freeseer.framework.plugin import PluginManager
 from freeseer.frontend.controller.server import http_response, ServerError
@@ -49,12 +49,15 @@ def configure_configuration():
     Runs on first call to server.
     """
     signal.signal(signal.SIGINT, teardown_configuration)
-    configuration.profile = settings.profile_manager.get()
-    configuration.config = configuration.profile.get_config('freeseer.conf',
-                                                            settings.FreeseerConfig,
-                                                            storage_args=['Global'],
-                                                            read_only=True)
-    configuration.plugin_manager = PluginManager(configuration.profile)
+    #configuration.plugin_manager = PluginManager(configuration.profile)
+
+
+def load_configuration(profile):
+    """
+    Returns the configuration for a given profile.
+    """
+    profile = settings.profile_manager.get(profile)
+    return profile.get_config('freeseer.conf', settings.FreeseerConfig, storage_args=['Global'], read_only=False)
 
 
 def teardown_configuration(signum, frame):
@@ -73,45 +76,58 @@ def teardown_configuration(signum, frame):
 def list_profiles():
     """
     List available configuration profiles.
+
+    TODO: add a method to the ProfileManager class to list existing profiles.
     """
     raise ServerError('unimplemented')
 
 
-@configuration.route('/profiles/:profile', methods=['GET'])
+@configuration.route('/profiles/<string:profile>', methods=['GET'])
 @http_response(200)
 def view_profile(profile):
     """
     View the configuration profile specified by :profile.
     """
-    raise ServerError('unimplemented')
+    configuration = load_configuration(profile)
+    return {'profile_configuration': configuration.values}
 
 
 @configuration.route('/profiles/', methods=['POST'])
 @http_response(200)
 def create_profile():
     """
     Create new profile under 'name' specified in request arg.
+
+    TODO: add a method to ProfileManager to explicitly create new profiles.
     """
+    profile_name = request.form['name']
     raise ServerError('unimplemented')
 
 
-@configuration.route('/profiles/:profile', methods=['DELETE'])
+@configuration.route('/profiles/<string:profile>', methods=['DELETE'])
 @http_response(200)
 def delete_profile(profile):
     """
     Delete the profile specified by :profile.
+
+    TODO: implement delete profile in ProfileManager.
     """
     raise ServerError('unimplemented')
 
 
-@configuration.route('/profiles/:profile', methods=['PATCH'])
+@configuration.route('/profiles/<string:profile>', methods=['PATCH'])
 @http_response(200)
 def modify_profile(profile):
     """
     Modify the profile specified by :profile.
     """
+    configuration = load_configuration(profile)
+    changes = request.form
+
+    configuration.save()
     raise ServerError('unimplemented')
 
+
 #
 # End Profile Endpoints
 #
@@ -121,7 +137,7 @@ def modify_profile(profile):
 #
 
 
-@configuration.route('/profiles/:profile/general', methods=['GET'])
+@configuration.route('/profiles/<string:profile>/general', methods=['GET'])
 @http_response(200)
 def view_general_configuration(profile):
     """
@@ -132,14 +148,15 @@ def view_general_configuration(profile):
             'auto_hide': configuration.config.auto_hide, }
 
 
-@configuration.route('/profiles/:profile/general', methods=['PATCH'])
+@configuration.route('/profiles/<string:profile>/general', methods=['PATCH'])
 @http_response(200)
 def modify_general_configuration(profile):
     """
     Modifies the general configuration for the given :profile.
     """
     raise ServerError('unimplemented')
 
+
 #
 # General Configuration Endpoints
 #
@@ -149,7 +166,7 @@ def modify_general_configuration(profile):
 #
 
 
-@configuration.route('/profiles/:profile/recording', methods=['GET'])
+@configuration.route('/profiles/<string:profile>/recording', methods=['GET'])
 @http_response(200)
 def view_recording_configuration():
     """
@@ -164,13 +181,15 @@ def view_recording_configuration():
             'videomixer': configuration.config.videomixer, }
 
 
-@configuration.route('/profiles/:profile/recording', methods=['PATCH'])
+@configuration.route('/profiles/<string:profile>/recording', methods=['PATCH'])
 @http_response(200)
 def put_recording_configuration():
     """
     Modifies the recording configuration for the given :profile.
     """
     raise ServerError('unimplemented')
+
+
 #
 # End recording configuration endpoints.
 #
@@ -179,7 +198,7 @@ def put_recording_configuration():
 #
 # Plugin Category endpoints.
 #
-@configuration.route('/profiles/:profile/recording/<string:category>', methods=['GET'])
+@configuration.route('/profiles/<string:profile>/recording/<string:category>', methods=['GET'])
 @http_response(200)
 def list_plugin_category(profile, category):
     """
@@ -195,7 +214,7 @@ def list_plugin_category(profile, category):
 #
 # Plugin endpoints.
 #
-@configuration.route('/profiles/:profile/recording/<string:category>/<string:plugin>', methods=['GET'])
+@configuration.route('/profiles/<string:profile>/recording/<string:category>/<string:plugin>', methods=['GET'])
 @http_response(200)
 def list_plugin_instances(profile, category, plugin):
     """
@@ -204,7 +223,7 @@ def list_plugin_instances(profile, category, plugin):
     raise ServerError('unimplemented')
 
 
-@configuration.route('/profiles/:profile/recording/<string:category>/<string:plugin>/:id', methods=['GET'])
+@configuration.route('/profiles/<string:profile>/recording/<string:category>/<string:plugin>/:id', methods=['GET'])
 @http_response(200)
 def view_plugin_instance(profile, category, plugin, id):
     """
@@ -213,7 +232,7 @@ def view_plugin_instance(profile, category, plugin, id):
     raise ServerError('unimplemented')
 
 
-@configuration.route('/profiles/:profile/recording/<string:category>/<string:plugin>', methods=['POST'])
+@configuration.route('/profiles/<string:profile>/recording/<string:category>/<string:plugin>', methods=['POST'])
 @http_response(200)
 def create_plugin_instance(profile, category, plugin):
     """
@@ -222,7 +241,7 @@ def create_plugin_instance(profile, category, plugin):
     raise ServerError('unimplemented')
 
 
-@configuration.route('/profiles/:profile/recording/<string:category>/<string:plugin/:id>', methods=['PATCH'])
+@configuration.route('/profiles/<string:profile>/recording/<string:category>/<string:plugin>/:id', methods=['PATCH'])
 @http_response(200)
 def modify_plugin_instance(profile, category, plugin, id):
     """