@AntonKhorev commented on this pull request.
> + @ways = @changeset.old_ways
+ .order(:way_id, :version)
+ .offset(ELEMENTS_PER_PAGE * (@current_way_page - 1))
+ .limit(ELEMENTS_PER_PAGE)
+ end
+
+ def load_relations
+ @relations_count = @changeset.actual_num_changed_relations
+ @current_relation_page = params[:relation_page].to_i.clamp(1,
element_pages_count(@relations_count))
+ @relations = @changeset.old_relations
+ .order(:relation_id, :version)
+ .offset(ELEMENTS_PER_PAGE * (@current_relation_page
- 1))
+ .limit(ELEMENTS_PER_PAGE)
+ end
+
+ helper_method def element_pages_count(elements_count)
`def` returns a symbol. Rails' documentation of `helper_method`
[predates](https://github.com/rails/rails/commit/e976c489e6416cdc4714721df78dd43dd6d13d99)
[that](https://stackoverflow.com/a/26216164).
Do you want an extra line of code for `helper_method` separate from the method
itself?
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6316#discussion_r2271897958
You are receiving this because you are subscribed to this thread.
Message ID:
<openstreetmap/openstreetmap-website/pull/6316/review/3113731...@github.com>
_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev
