I named it this because this is the name of the variable in the base class
as well. I felt that it made sense to name them the same thing so that it
was easier to trace what was happening through the whole system.

If you disagree, I can try to come up with a more descriptive name.

On Sep 12, 2013, at 11:04 AM, Ben Orenstein <notifications@github.com>
wrote:

In plugins/learn_auth/plugin.rb:

> @@ -0,0 +1,101 @@
> +# name: discourse-learn_auth
> +# about: learn.thoughtbot.com login support for Discourse
> +# version: 0.1
> +# authors: Chad Pytel, thoughtbot
> +
> +require 'auth/oauth2_authenticator'
> +
> +class LearnAuthenticator < ::Auth::OAuth2Authenticator
> +  def after_authenticate(auth_token)
> +    result = super

I'm not clear what result is. Is there anything more descriptive we can
name it?

—
Reply to this email directly or view it on
GitHub<https://github.com/thoughtbot/discourse/pull/1/files#r6323990>
.