skyeagle / nested_set

Rails 3 support! An awesome replacement for acts_as_nested_set and better_nested_set.
MIT License
171 stars 24 forks source link

Rdoc typos #33

Open clayheaton opened 13 years ago

clayheaton commented 13 years ago

The nested_set rdoc text for sorted_nested_set_options(class_or_item, sort_proc, mover = nil, level = 0) has several typos. The "Usage" section reads:

<%= f.select :parent_id, sorted_nested_set_options(Category, lambda(&:name)) {|i, level| "#{'–' * level} #{i.name}" }) %>
OR
sort_method = lambda{|x| x.name.mb_chars.downcase }
<%= f.select :parent_id, nested_set_options(Category, sort_method) {|i, level| "#{'–' * level} #{i.name}" }) %>

Both code examples contain an extra closing parenthesis. The second example also should use the function sorted_nested_set_options() instead of nested_set_options().

clayheaton commented 13 years ago

At the beginning of the same section, it also reads: +class_or_item+ - Class name or top level times It should read "items," not "times"

skyeagle commented 13 years ago

Could you provide a pull request?

clayheaton commented 13 years ago

Sorry - I'm pretty new to this and I've never done that and don't know how...

skyeagle commented 13 years ago

Ok, never mind. I will correct it later.

pripple commented 12 years ago

Commit 94446216d8ced6c3f75feb7100b90060407d44d3 resolved this -- can be closed.