Mini refactor en carga de contactos

martinvega · May 15, 2012 · 129e22a · 129e22a
1 parent 58763d8
commit 129e22a
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 11 deletions.
diff --git a/app/controllers/contacts_controller.rb b/app/controllers/contacts_controller.rb
@@ -123,7 +123,14 @@ def update
   end
 
   def load_contacts
-
+    @months = []
+    @years = []
+    MONTHS.each_with_index do |month, i| 
+      @months << [month, i+1] 
+    end 
+    YEARS.each do |year| 
+      @years << year 
+    end 
     unless params[:campaign_month].nil? || params[:campaign_year].nil?
       @selected_year = params[:campaign_year].to_i
       @selected_month = params[:campaign_month].to_i

diff --git a/app/views/contacts/load_contacts.html.erb b/app/views/contacts/load_contacts.html.erb
@@ -1,11 +1,3 @@
-<% months = [] %>
-<% years = [] %>
-<% MONTHS.each_with_index do |month, i| %>
-  <% months << [month, i+1] %>
-<% end %>
-<% YEARS.each do |year| %>
-  <% years << year %>
-<% end %>
 <div id="select_campaign">
   <% unless @contact.nil?  %>
     <%= render :partial => 'contact' %>
@@ -15,8 +7,8 @@
 
     <div class="field">
       <%= label_tag 'Seleccione la campaña' %><br />
-      <%= select_tag :campaign_month, options_for_select(months, @selected_month) %>
-      <%= select_tag :campaign_year, options_for_select(years, @selected_year) %>
+      <%= select_tag :campaign_month, options_for_select(@months, @selected_month) %>
+      <%= select_tag :campaign_year, options_for_select(@years, @selected_year) %>
     </div>
     <div id="search_contact">
       <%= submit_tag 'Buscar Contacto' %>