Commit dbf169af5efeed180c0e010726d47e355a78ec11
Committed by
Paulo Meireles
1 parent
a7daad1b
Exists in
master
and in
22 other branches
[Mezuro] removing and editing range in a metric configuration.
Showing
3 changed files
with
11 additions
and
8 deletions
Show diff stats
plugins/mezuro/controllers/myprofile/mezuro_plugin_range_controller.rb
... | ... | @@ -9,9 +9,11 @@ class MezuroPluginRangeController < MezuroPluginMyprofileController |
9 | 9 | end |
10 | 10 | |
11 | 11 | def edit_range |
12 | - @range = Kalibro::Range.new(params[:range]) | |
13 | 12 | @content_id = params[:id] |
14 | 13 | @metric_configuration_id = params[:metric_configuration_id] |
14 | + ranges = Kalibro::Range.ranges_of params[:metric_configuration_id] | |
15 | + @range = (ranges.select { |range| range.id == params[:range_id].to_i }).first | |
16 | + @reading_labels_and_ids = reading_labels_and_ids | |
15 | 17 | end |
16 | 18 | |
17 | 19 | def create_range |
... | ... | @@ -35,11 +37,12 @@ class MezuroPluginRangeController < MezuroPluginMyprofileController |
35 | 37 | def remove_range |
36 | 38 | configuration_content_id = params[:id] |
37 | 39 | metric_configuration_id = params[:metric_configuration_id] |
38 | - Kalibro::Range.new(params[:range]).destroy | |
39 | - if metric_configuration.metric.compound | |
40 | - redirect_to "/myprofile/#{profile.identifier}/plugin/mezuro/metric_configuration/edit_compound_metric_configuration?id=#{configuration_content_id}&metric_configuration_id=#{metric_configuration_id}" | |
40 | + compound = params[:compound] | |
41 | + Kalibro::Range.new({:id => params[:range_id]}).destroy | |
42 | + if compound | |
43 | + redirect_to "/myprofile/#{profile.identifier}/plugin/mezuro/metric_configuration/edit_compound?id=#{configuration_content_id}&metric_configuration_id=#{metric_configuration_id}" | |
41 | 44 | else |
42 | - redirect_to "/myprofile/#{profile.identifier}/plugin/mezuro/metric_configuration/edit_native_metric_configuration?id=#{configuration_content_id}&metric_configuration_id=#{metric_configuration_id}" | |
45 | + redirect_to "/myprofile/#{profile.identifier}/plugin/mezuro/metric_configuration/edit_native?id=#{configuration_content_id}&metric_configuration_id=#{metric_configuration_id}" | |
43 | 46 | end |
44 | 47 | end |
45 | 48 | ... | ... |
plugins/mezuro/views/mezuro_plugin_metric_configuration/edit_native.html.erb
... | ... | @@ -36,7 +36,7 @@ |
36 | 36 | </tr> |
37 | 37 | <% @ranges.each do |range| %> |
38 | 38 | <%= render :partial => "mezuro_plugin_range/range", :locals => {:range => range, :id => @configuration_content.id, |
39 | - :metric_configuration_id => @metric_configuration.id} %> | |
39 | + :metric_configuration_id => @metric_configuration.id, :reading_group_id => @metric_configuration.reading_group_id, :compound => @metric_configuration.metric.compound} %> | |
40 | 40 | <% end %> |
41 | 41 | </table> |
42 | 42 | ... | ... |
plugins/mezuro/views/mezuro_plugin_range/_range.html.erb
... | ... | @@ -13,9 +13,9 @@ |
13 | 13 | </td> |
14 | 14 | <td bgcolor="#<%= range.color %>"></td> |
15 | 15 | <% if (not user.nil?) && user.id == @profile.id %> |
16 | - <td><%= link_to_remote "Edit", :url => {:action =>"edit_range", :controller => "mezuro_plugin_range", :id => params[:id], :metric_configuration_id => params[:metric_configuration_id], :range_id => range.id} %> | |
16 | + <td><%= link_to_remote "Edit", :url => {:action =>"edit_range", :controller => "mezuro_plugin_range", :id => params[:id], :metric_configuration_id => params[:metric_configuration_id], :range_id => range.id, :reading_group_id => reading_group_id} %> | |
17 | 17 | </td> |
18 | - <td><%= link_to "Remove", :action =>"remove_range", :controller => "mezuro_plugin_range", :id => params[:id], :metric_configuration_id => params[:metric_configuration_id], :range_id => range.id %> | |
18 | + <td><%= link_to "Remove", :action =>"remove_range", :controller => "mezuro_plugin_range", :id => params[:id], :metric_configuration_id => params[:metric_configuration_id], :range_id => range.id, :compound => compound %> | |
19 | 19 | </td> |
20 | 20 | <% else %> |
21 | 21 | <td></td> | ... | ... |