[openstreetmap/openstreetmap-website] Use current_user to represent the currently logged in user (#1583)

Andy Allan notifications at github.com
Wed Jul 12 15:34:58 UTC 2017

This is already used by the oauth plugin, and is a general rails convention.

I've left out the users_controller and the parts of the application_controller, since they will need a lot more scrutiny when changed. 

This helps make the controller code feel more familiar to new users, and is preparatory work for moving to a standard login system at some point in the future.
You can view, comment on, or merge this pull request online at:


-- Commit Summary --

  * Use current_user to represent the currently logged in user.

-- File Changes --

    M app/controllers/api_controller.rb (2)
    M app/controllers/browse_controller.rb (4)
    M app/controllers/changeset_controller.rb (42)
    M app/controllers/diary_entry_controller.rb (42)
    M app/controllers/message_controller.rb (36)
    M app/controllers/node_controller.rb (6)
    M app/controllers/notes_controller.rb (10)
    M app/controllers/oauth_clients_controller.rb (14)
    M app/controllers/old_controller.rb (2)
    M app/controllers/redactions_controller.rb (2)
    M app/controllers/relation_controller.rb (6)
    M app/controllers/site_controller.rb (4)
    M app/controllers/trace_controller.rb (54)
    M app/controllers/user_blocks_controller.rb (8)
    M app/controllers/user_preference_controller.rb (14)
    M app/controllers/user_roles_controller.rb (4)
    M app/controllers/way_controller.rb (6)

-- Patch Links --


You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20170712/6cf31d80/attachment.html>

More information about the rails-dev mailing list