[openstreetmap/openstreetmap-website] Further refactoring to avoid @user (#1595)

Tom Hughes notifications at github.com
Thu Jul 27 09:13:35 UTC 2017


tomhughes requested changes on this pull request.



> @@ -47,7 +49,7 @@ def require_user
   end
 
   def require_oauth
-    @oauth = @user.access_token(OAUTH_KEY) if @user && defined? OAUTH_KEY
+    @oauth = @user.access_token(OAUTH_KEY) if current_user && defined? OAUTH_KEY

You missed one in this line ;-)

As far as I could tell last night that was the only one in any view or controller that had been missed, which is probably pretty good doing.

-- 
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/1595#pullrequestreview-52598643
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20170727/b2643391/attachment.html>


More information about the rails-dev mailing list