diff options
author | Jean-Philippe Lang <jp_lang@yahoo.fr> | 2011-03-10 19:44:32 +0000 |
---|---|---|
committer | Jean-Philippe Lang <jp_lang@yahoo.fr> | 2011-03-10 19:44:32 +0000 |
commit | 28feb58754ff9be93d5cfe17ecb532103429715e (patch) | |
tree | 1e4c6bdbb4f77e35f7f73db91d0fcf47c5bb80cf | |
parent | ef189164bf8ad4ee08c729f7c8f23f951ff2a391 (diff) | |
download | redmine-28feb58754ff9be93d5cfe17ecb532103429715e.tar.gz redmine-28feb58754ff9be93d5cfe17ecb532103429715e.zip |
Speeds up WikiHelper#wiki_page_options_for_select.
git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@5080 e93f8b46-1217-0410-a6f0-8f06a7374b81
-rw-r--r-- | app/helpers/wiki_helper.rb | 17 |
1 files changed, 10 insertions, 7 deletions
diff --git a/app/helpers/wiki_helper.rb b/app/helpers/wiki_helper.rb index 32225d83e..48527a9ba 100644 --- a/app/helpers/wiki_helper.rb +++ b/app/helpers/wiki_helper.rb @@ -18,14 +18,17 @@ module WikiHelper def wiki_page_options_for_select(pages, selected = nil, parent = nil, level = 0) + pages = pages.group_by(&:parent) unless pages.is_a?(Hash) s = '' - pages.select {|p| p.parent == parent}.each do |page| - attrs = "value='#{page.id}'" - attrs << " selected='selected'" if selected == page - indent = (level > 0) ? (' ' * level * 2 + '» ') : nil - - s << "<option #{attrs}>#{indent}#{h page.pretty_title}</option>\n" + - wiki_page_options_for_select(pages, selected, page, level + 1) + if pages.has_key?(parent) + pages[parent].each do |page| + attrs = "value='#{page.id}'" + attrs << " selected='selected'" if selected == page + indent = (level > 0) ? (' ' * level * 2 + '» ') : nil + + s << "<option #{attrs}>#{indent}#{h page.pretty_title}</option>\n" + + wiki_page_options_for_select(pages, selected, page, level + 1) + end end s end |