[openstreetmap/openstreetmap-website] Complete @user to current_user transition (#1597)

Tom Hughes notifications at github.com
Thu Jul 27 18:49:41 UTC 2017


This (hopefully) completes the `@user` to `current_user` transition by fixing a few remaining issues and then removing the `@user` variable altogether.
You can view, comment on, or merge this pull request online at:

  https://github.com/openstreetmap/openstreetmap-website/pull/1597

-- Commit Summary --

  * Convert @user to current_user in helpers
  * Change user forms to use current user instead of @user
  * Replace @user with @current_user

-- File Changes --

    M app/controllers/application_controller.rb (11)
    M app/controllers/user_controller.rb (2)
    M app/helpers/application_helper.rb (20)
    M app/helpers/user_roles_helper.rb (2)
    M app/views/user/account.html.erb (6)
    M app/views/user/new.html.erb (28)
    M test/helpers/application_helper_test.rb (22)
    M test/helpers/user_roles_helper_test.rb (16)

-- Patch Links --

https://github.com/openstreetmap/openstreetmap-website/pull/1597.patch
https://github.com/openstreetmap/openstreetmap-website/pull/1597.diff

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/1597
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20170727/cd46ff4b/attachment.html>


More information about the rails-dev mailing list