[Mezuro] Finished configuration views refactoring

Diego Camarinha · Paulo Meireles
1 parent 2a69d623
Showing 3 changed files with 38 additions and 35 deletions Show diff stats
plugins/mezuro/controllers/mezuro_plugin_myprofile_controller.rb
plugins/mezuro/test/functional/mezuro_plugin_myprofile_controller_test.rb
plugins/mezuro/views/mezuro_plugin_myprofile/edit_metric_configuration.html.erb
@@ -57,8 +57,8 @@ class MezuroPluginMyprofileController &lt; ProfileController
   def update_metric_configuration
     @configuration_name = params[:configuration_name]
+    metric_name = params[:metric_configuration][:metric][:name]
     metric_configuration_client = Kalibro::Client::MetricConfigurationClient.new
-    metric_name = params[:metric][:name]
     metric_configuration = metric_configuration_client.metric_configuration(@configuration_name, metric_name)  
     metric_configuration = assign_metric_configuration_instance(metric_configuration)
     metric_configuration_client.save(metric_configuration, @configuration_name)
@@ -152,19 +152,19 @@ class MezuroPluginMyprofileController &lt; ProfileController
     assign_compound_metric_configuration_instance (metric_configuration)
   end
-  def assign_metric_configuration_instance (metric_configuration)
-    metric_configuration.metric.name = params[:metric][:name]
-    metric_configuration.metric.description = params[:description]
-    metric_configuration.metric.origin = params[:metric][:origin]
-    metric_configuration.metric.scope = params[:scope]
-    metric_configuration.metric.language = params[:language]
+  def assign_metric_configuration_instance(metric_configuration)
+    metric_configuration.metric.name = params[:metric_configuration][:metric][:name]
+    metric_configuration.metric.description = params[:metric_configuration][:metric][:description]
+    metric_configuration.metric.origin = params[:metric_configuration][:metric][:origin]
+    metric_configuration.metric.scope = params[:metric_configuration][:metric][:scope]
+    metric_configuration.metric.language = params[:metric_configuration][:metric][:language]
     metric_configuration.code = params[:metric_configuration][:code]
     metric_configuration.weight = params[:metric_configuration][:weight]
     metric_configuration.aggregation_form = params[:metric_configuration][:aggregation_form]
     metric_configuration
   end
-  def assign_compound_metric_configuration_instance (metric_configuration)
+  def assign_compound_metric_configuration_instance(metric_configuration)
     metric_configuration.metric.name = params[:metric_configuration][:metric][:name]
     metric_configuration.metric.description = params[:metric_configuration][:metric][:description]
     metric_configuration.metric.scope = params[:metric_configuration][:metric][:scope]
@@ -76,9 +76,9 @@ class MezuroPluginMyprofileControllerTest &lt; ActionController::TestCase
   should 'test metric creation' do
     Kalibro::Client::MetricConfigurationClient.expects(:new).returns(@metric_configuration_client)
     @metric_configuration_client.expects(:save)
-    get :create_metric_configuration, :profile => @profile.identifier, :configuration_name => "test name", :description => @metric.description,
-    :scope => @metric.scope, :language => @metric.language, :metric => { :name => @metric.name, :origin => @metric.origin},
-    :metric_configuration => { :code => @metric_configuration.code, :weight => @metric_configuration.code, :aggregation => @metric_configuration.aggregation_form }
+    get :create_metric_configuration, :profile => @profile.identifier, :configuration_name => "test name", 
+    :metric_configuration => { :code => @metric_configuration.code, :weight => @metric_configuration.code, :aggregation => @metric_configuration.aggregation_form, 
+    :metric => { :name => @metric.name, :origin => @metric.origin, :description => @metric.description, :scope => @metric.scope, :language => @metric.language }}
     assert_equal assigns(:configuration_name), "test name"
     assert_response 302
   end
@@ -112,9 +112,9 @@ class MezuroPluginMyprofileControllerTest &lt; ActionController::TestCase
     Kalibro::Client::MetricConfigurationClient.expects(:new).returns(@metric_configuration_client)
     @metric_configuration_client.expects(:metric_configuration).with(@configuration.name, @metric_configuration.metric.name).returns(@metric_configuration)
     @metric_configuration_client.expects(:save)
-    get :update_metric_configuration, :profile => @profile.identifier, :configuration_name => @configuration.name, :description => @metric.description,
-    :scope => @metric.scope, :language => @metric.language, :metric => { :name => @metric.name, :origin => @metric.origin},
-    :metric_configuration => { :code => @metric_configuration.code, :weight => @metric_configuration.code, :aggregation => @metric_configuration.aggregation_form }
+    get :update_metric_configuration, :profile => @profile.identifier, :configuration_name => @configuration.name, 
+    :metric_configuration => { :code => @metric_configuration.code, :weight => @metric_configuration.weight, :aggregation => @metric_configuration.aggregation_form, 
+    :metric => { :name => @metric.name, :origin => @metric.origin, :description => @metric.description, :scope => @metric.scope, :language => @metric.language }}
     assert_response 302
   end
@@ -4,31 +4,34 @@
 <% form_for :metric_configuration, :url => {:action =>"update_metric_configuration", :controller => "mezuro_plugin_myprofile"}, :method => :get do |f| %>
   <%= hidden_field_tag :configuration_name, @configuration_name %>
-  <%= hidden_field_tag :scope, @metric.scope %>
-  <% @metric.language.each do |language| %>
-    <%= hidden_field_tag "language[]", language %>
+  <% f.fields_for :metric do |m| %>
+  
+    <% @metric.language.each do |language| %>
+      <%= m.hidden_field :language, :multiple => true, :value => language %>
+    <% end %>
+    
+    <%= m.hidden_field "scope", :value => @metric.scope %>
+    <p>
+      <%= m.label :origin, "Collector Name:" %>
+      <%= @metric.origin %>
+      <%= m.hidden_field "origin", :value => @metric.origin %>
+    </p>
+    <p>
+      <%= m.label :metric_name, "Metric Name:" %>
+      <%= @metric.name %>
+      <%= m.hidden_field "name", :value => @metric.name %>
+    </p>
+    <p>    
+      <%= m.label :description, "Description:" %>
+      <%= @metric.description %>
+      <%= m.hidden_field "description", :value => @metric.description %>
+    </p>
   <% end %>
-
-  <p>
-    <%= f.label :origin, "Collector Name:" %>
-    <%= @metric.origin %>
-    <%= hidden_field_tag "metric[origin]", @metric.origin %>
-  </p>
-  <p>
-    <%= f.label :metric_name, "Metric Name:" %>
-    <%= @metric.name %>
-    <%= hidden_field_tag "metric[name]", @metric.name %>
-  </p>
-  <p>
-    <%= f.label :description, "Description:" %>
-    <%= @metric.description %>
-    <%= text_field_tag "metric[description]", @metric.description %>
-  </p>
   <p>
     <%= f.label :code, "Code:" %>
     <%= @metric_configuration.code %>
-    <%= f.hidden_field :code, :value => @metric_configuration.code %>
+    <%= f.hidden_field "code", :value => @metric_configuration.code %>
   </p>
   <p>
     <%= f.label :aggregation_form, "Aggregation Form:" %>
@@ -37,7 +40,7 @@
   </p>
   <p>
     <%= f.label :weight, "Weight:" %>
-    <%= f.text_field :weight %>
+    <%= f.text_field "weight", :value => @metric_configuration.weight %>
   </p>
   <p>