Commit dea59087e9d8c25c1d9147600d9168de6cdb74b4
Committed by
Paulo Meireles
1 parent
b7adb7d6
Exists in
staging
and in
42 other branches
[Mezuro] Fixed ranges with +/- Infinity beginning or end values
Showing
2 changed files
with
11 additions
and
5 deletions
Show diff stats
plugins/mezuro/controllers/mezuro_plugin_myprofile_controller.rb
| ... | ... | @@ -84,16 +84,14 @@ class MezuroPluginMyprofileController < ProfileController |
| 84 | 84 | @configuration_content = profile.articles.find(params[:id]) |
| 85 | 85 | @metric_name = params[:metric_name] |
| 86 | 86 | @beginning_id = params[:beginning_id] |
| 87 | - | |
| 88 | 87 | metric_configuration = Kalibro::MetricConfiguration.find_by_configuration_name_and_metric_name(@configuration_content.name, @metric_name) |
| 89 | - @range = metric_configuration.ranges.find{ |range| range.beginning == @beginning_id.to_f } | |
| 88 | + @range = metric_configuration.ranges.find{|range| range.beginning == @beginning_id.to_f || ( @beginning_id =="-Infinity" && range.beginning == -1.0/0.0) } | |
| 90 | 89 | end |
| 91 | 90 | |
| 92 | 91 | def create_range |
| 93 | 92 | @configuration_content = profile.articles.find(params[:id]) |
| 94 | 93 | @range = Kalibro::Range.new params[:range] |
| 95 | 94 | metric_name = params[:metric_name] |
| 96 | - beginning_id = params[:beginning_id] | |
| 97 | 95 | metric_configuration = Kalibro::MetricConfiguration.find_by_configuration_name_and_metric_name(@configuration_content.name, metric_name) |
| 98 | 96 | metric_configuration.add_range(@range) |
| 99 | 97 | metric_configuration.save | ... | ... |
plugins/mezuro/views/mezuro_plugin_myprofile/_range_form.html.erb
| ... | ... | @@ -14,7 +14,11 @@ |
| 14 | 14 | <%= f.label :beginning, "(*) Beginning:" %> |
| 15 | 15 | </td> |
| 16 | 16 | <td> |
| 17 | - <%= f.text_field :beginning %> | |
| 17 | + <% if @range.beginning.to_s == "-Infinity" %> | |
| 18 | + <%= f.text_field :beginning, :value => "-INF" %> | |
| 19 | + <% else %> | |
| 20 | + <%= f.text_field :beginning %> | |
| 21 | + <% end %> | |
| 18 | 22 | </td> |
| 19 | 23 | </tr> |
| 20 | 24 | <tr> |
| ... | ... | @@ -22,7 +26,11 @@ |
| 22 | 26 | <%= f.label :end, "(*) End:" %> |
| 23 | 27 | </td> |
| 24 | 28 | <td> |
| 25 | - <%= f.text_field :end %> | |
| 29 | + <% if @range.end.to_s == "Infinity" %> | |
| 30 | + <%= f.text_field :end, :value => "INF" %> | |
| 31 | + <% else %> | |
| 32 | + <%= f.text_field :end %> | |
| 33 | + <% end %> | |
| 26 | 34 | </td> |
| 27 | 35 | </tr> |
| 28 | 36 | <tr> | ... | ... |