summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJean-Philippe Lang <jp_lang@yahoo.fr>2011-03-10 19:44:32 +0000
committerJean-Philippe Lang <jp_lang@yahoo.fr>2011-03-10 19:44:32 +0000
commit28feb58754ff9be93d5cfe17ecb532103429715e (patch)
tree1e4c6bdbb4f77e35f7f73db91d0fcf47c5bb80cf
parentef189164bf8ad4ee08c729f7c8f23f951ff2a391 (diff)
downloadredmine-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.rb17
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) ? ('&nbsp;' * level * 2 + '&#187; ') : 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) ? ('&nbsp;' * level * 2 + '&#187; ') : 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