[openstreetmap-website] Site cleanup (#150)

Tom Hughes notifications at github.com
Sun Jan 6 16:09:17 GMT 2013


> @@ -44,6 +44,11 @@ def list
>        @title = t 'trace.list.public_traces_from', :user => target_user.display_name
>      end
>  
> +    # set description
> +
> +    @description = t 'trace.list.description'

Is there a particular reason for looking this up here and storing it in a member variable rather than just looking it up when you use it in the view? The only reason `@title` is setup here is because it can vary based on the selection criteria.

---
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/150/files#r2560194
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20130106/0307685c/attachment.html>


More information about the rails-dev mailing list