Commit fbc6eb3729b3396f4bded43e67e923280b01cc7b
Committed by
Paulo Meireles
1 parent
e43b3875
Exists in
master
and in
29 other branches
[Mezuro] Added (not fully functional) ranges edit
Showing
5 changed files
with
90 additions
and
1 deletions
Show diff stats
plugins/mezuro/controllers/mezuro_plugin_profile_controller.rb
| @@ -90,7 +90,36 @@ class MezuroPluginProfileController < ProfileController | @@ -90,7 +90,36 @@ class MezuroPluginProfileController < ProfileController | ||
| 90 | metric_configuration.add_range(@range) | 90 | metric_configuration.add_range(@range) |
| 91 | metric_configuration_client.save(metric_configuration, configuration_name) | 91 | metric_configuration_client.save(metric_configuration, configuration_name) |
| 92 | end | 92 | end |
| 93 | + | ||
| 94 | + def edit_range | ||
| 95 | + @configuration_name = params[:configuration_name] | ||
| 96 | + @metric_name = params[:metric_name] | ||
| 97 | + @range_beginning = params[:range_beginning] | ||
| 98 | + | ||
| 99 | + metric_configuration_client = Kalibro::Client::MetricConfigurationClient.new | ||
| 100 | + metric_configuration = metric_configuration_client.metric_configuration(@configuration_name, @metric_name) | ||
| 93 | 101 | ||
| 102 | + metric_configuration.ranges.each do |r| | ||
| 103 | + @range = r if r.beginning == @range_beginning.to_f | ||
| 104 | + end | ||
| 105 | + end | ||
| 106 | + | ||
| 107 | + def update_range | ||
| 108 | + @configuration_name = params[:configuration_name] | ||
| 109 | + metric_name = params[:metric_name] | ||
| 110 | + range_beginning = params[:beginning_id] | ||
| 111 | + metric_configuration_client = Kalibro::Client::MetricConfigurationClient.new | ||
| 112 | + metric_configuration = metric_configuration_client.metric_configuration(@configuration_name, metric_name) | ||
| 113 | + index = 0 | ||
| 114 | + metric_configuration.ranges.each do |r| | ||
| 115 | + break if r.beginning == range_beginning.to_f | ||
| 116 | + index = index + 1 | ||
| 117 | + end | ||
| 118 | + metric_configuration.ranges[index] = new_range_instance | ||
| 119 | + Kalibro::Client::MetricConfigurationClient.new.save(metric_configuration, @configuration_name) | ||
| 120 | + redirect_to "/#{profile.identifier}/#{@configuration_name.downcase.gsub(/\s/, '-')}" | ||
| 121 | + end | ||
| 122 | + | ||
| 94 | def remove_metric_configuration | 123 | def remove_metric_configuration |
| 95 | configuration_name = params[:configuration_name] | 124 | configuration_name = params[:configuration_name] |
| 96 | metric_name = params[:metric_name] | 125 | metric_name = params[:metric_name] |
plugins/mezuro/views/mezuro_plugin_profile/_range.html.erb
| @@ -12,4 +12,6 @@ | @@ -12,4 +12,6 @@ | ||
| 12 | <%=range.grade%> | 12 | <%=range.grade%> |
| 13 | </td> | 13 | </td> |
| 14 | <td bgcolor="#<%= range.color[2..-1] %>"></td> | 14 | <td bgcolor="#<%= range.color[2..-1] %>"></td> |
| 15 | + <td><%= link_to "Edit", :controller => "mezuro_plugin_profile", :action => "edit_range", :params => | ||
| 16 | + {:configuration_name => params[:configuration_name], :metric_name => params[:metric_name], :range_beginning => range.beginning} %></td> | ||
| 15 | </tr> | 17 | </tr> |
plugins/mezuro/views/mezuro_plugin_profile/edit_metric_configuration.html.erb
| @@ -66,7 +66,8 @@ | @@ -66,7 +66,8 @@ | ||
| 66 | </tr> | 66 | </tr> |
| 67 | <% if (@metric_configuration.ranges!=nil) | 67 | <% if (@metric_configuration.ranges!=nil) |
| 68 | @metric_configuration.ranges.each do |range| %> | 68 | @metric_configuration.ranges.each do |range| %> |
| 69 | - <%= render :partial => "range", :locals => {:range => range} %> | 69 | + <%= render :partial => "range", :locals => {:range => range, :configuration_name => @configuration_name, |
| 70 | + :metric_name => @metric_configuration.metric.name} %> | ||
| 70 | <% end | 71 | <% end |
| 71 | end %> | 72 | end %> |
| 72 | </table> | 73 | </table> |
plugins/mezuro/views/mezuro_plugin_profile/edit_range.html.erb
0 → 100644
| @@ -0,0 +1,56 @@ | @@ -0,0 +1,56 @@ | ||
| 1 | +<% remote_form_for :range, :url => {:action =>"update_range", :controller => "mezuro_plugin_profile"}, :method => :get do |f| %> | ||
| 2 | + <%= hidden_field_tag :configuration_name, @configuration_name %> | ||
| 3 | + <%= hidden_field_tag :metric_name, @metric_name %> | ||
| 4 | + <%= hidden_field_tag :beginning_id, @range_beginning %> | ||
| 5 | + <table> | ||
| 6 | + <tr> | ||
| 7 | + <td> | ||
| 8 | + <%= f.label :label, "Label:" %> | ||
| 9 | + </td> | ||
| 10 | + <td> | ||
| 11 | + <%= f.text_field :label %> | ||
| 12 | + </td> | ||
| 13 | + </tr> | ||
| 14 | + <tr> | ||
| 15 | + <td> | ||
| 16 | + <%= f.label :beginning, "Beginning:" %> | ||
| 17 | + </td> | ||
| 18 | + <td> | ||
| 19 | + <%= f.text_field :beginning %> | ||
| 20 | + </td> | ||
| 21 | + </tr> | ||
| 22 | + <tr> | ||
| 23 | + <td> | ||
| 24 | + <%= f.label :end, "End:" %> | ||
| 25 | + </td> | ||
| 26 | + <td> | ||
| 27 | + <%= f.text_field :end %> | ||
| 28 | + </td> | ||
| 29 | + </tr> | ||
| 30 | + <tr> | ||
| 31 | + <td> | ||
| 32 | + <%= f.label :grade, "Grade:" %> | ||
| 33 | + </td> | ||
| 34 | + <td> | ||
| 35 | + <%= f.text_field :grade %> | ||
| 36 | + </td> | ||
| 37 | + </tr> | ||
| 38 | + <tr> | ||
| 39 | + <td> | ||
| 40 | + <%= f.label :color, "Color:" %> | ||
| 41 | + </td> | ||
| 42 | + <td> | ||
| 43 | + <%= f.text_field :color %> | ||
| 44 | + </td> | ||
| 45 | + </tr> | ||
| 46 | + <tr> | ||
| 47 | + <td> | ||
| 48 | + <%= f.label :comments, "Comments:" %> | ||
| 49 | + </td> | ||
| 50 | + <td> | ||
| 51 | + <%= f.text_field :comments %> | ||
| 52 | + </td> | ||
| 53 | + </tr> | ||
| 54 | + </table> | ||
| 55 | + <%= f.submit "Save Range" %> | ||
| 56 | +<% end %> |
plugins/mezuro/views/mezuro_plugin_profile/update_range.html.erb
0 → 100644
| @@ -0,0 +1 @@ | @@ -0,0 +1 @@ | ||
| 1 | +<h3>Update Range Action</h3> |