Implement feedback from @ugisozols.

resolve · May 12, 2012 · c8dabae · c8dabae
1 parent e4c0aae
commit c8dabae
Showing 1 changed file with 2 additions and 5 deletions.
diff --git a/pages/app/helpers/refinery/admin/pages_helper.rb b/pages/app/helpers/refinery/admin/pages_helper.rb
@@ -3,15 +3,12 @@ module Admin
     module PagesHelper
       def parent_id_nested_set_options(current_page)
         pages = []
-        options = nested_set_options(::Refinery::Page, current_page) do |page|
-          pages << page
-          page
-        end
+        nested_set_options(::Refinery::Page, current_page) {|page| pages << page}
         # page.title needs the :translations association, doing something like
         # nested_set_options(::Refinery::Page.includes(:translations), page) doesn't work, yet.
         # See https://github.com/collectiveidea/awesome_nested_set/pull/123
         ActiveRecord::Associations::Preloader.new(pages, :translations).run
-        options.map {|page, id| ["#{'-' * page.level} #{page.title}", id]}
+        pages.map {|page| ["#{'-' * page.level} #{page.title}", page.id]}
       end
     end
   end