[Mezuro] edit_repository view fully working

João M. M. da Silva + Alessandro Palmeira · Alessandro Palmeira
1 parent e83773f9
Showing 2 changed files with 10 additions and 11 deletions Show diff stats
plugins/mezuro/lib/mezuro_plugin/helpers/content_viewer_helper.rb
plugins/mezuro/views/mezuro_plugin_repository/edit.html.erb
@@ -6,8 +6,8 @@ class MezuroPlugin::Helpers::ContentViewerHelper
     sprintf("%.2f", grade.to_f)
   end
-  def self.create_periodicity_options
-   [["Not Periodically", 0], ["1 day", 1], ["2 days", 2], ["Weekly", 7], ["Biweeky", 15], ["Monthly", 30]]
+  def self.periodicity_options
+    [["Not Periodically", 0], ["1 day", 1], ["2 days", 2], ["Weekly", 7], ["Biweeky", 15], ["Monthly", 30]]
   end
   def self.create_license_options
@@ -39,11 +39,6 @@ class MezuroPlugin::Helpers::ContentViewerHelper
                 )
   end
-  def self.get_periodicity_option(index)
-    options = [["Not Periodically", 0], ["1 day", 1], ["2 days", 2], ["Weekly", 7], ["Biweeky", 15], ["Monthly", 30]]
-    selected_option = options.find { |option| option.last == index.to_i }
-    selected_option.first
-  end
   def self.format_name(metric_configuration_snapshot)
     metric_configuration_snapshot.metric.name.delete("() ")
@@ -8,19 +8,23 @@
     <%= f.label :name, "Name:" %>
     <%= f.text_field :name %>
   </p>
+
   <p>
     <%= f.label :description, "Description:" %>
     <%= f.text_field :description %>
   </p>
-  <%= required labelled_form_field _('License'),
-  f.select(:license, MezuroPlugin::Helpers::ContentViewerHelper.create_license_options) %><br/>
   <p>
+    <%= required labelled_form_field _('License'),
+    f.select(:license, MezuroPlugin::Helpers::ContentViewerHelper.create_license_options) %><br/>
+  </p>
   <p>
     <%= f.label :process_period, "Process Period:" %>
-    <%= f.select(:process_period, MezuroPlugin::Helpers::ContentViewerHelper.create_periodicity_options) %>
+    <%= f.select(:process_period, MezuroPlugin::Helpers::ContentViewerHelper.periodicity_options, :selected => @repository.process_period.to_i) %>
   </p>
+
+  <p>
     <%= f.label :type, "Type:" %>
     <%= f.select :type, @repository_types%>
   </p>
@@ -30,7 +34,7 @@
     <%= f.text_field :address%>
   </p>
-  </p>
+  <p>
     <%= f.label :configuration_id, "Configuration:" %>
     <%= f.select :configuration_id, @configuration_select, :selected => @repository.configuration_id.to_i %>
   </p>