yea, we generally have stuck to a style where when we have Models in
variables we call them what the name of the model is.


On Thu, Jun 6, 2013 at 8:58 PM, David Thomas <notifications@github.com>wrote:

> In controllers/account_controller.py:
>
> > +            return None
> > +
> > +        path = os.path.join(os.path.dirname(__file__), '../templates/account/register.html')
> > +        self.response.out.write(template.render(path, self.template_values))
> > +
> > +    def post(self):
> > +        self._require_login('/account/register')
> > +        if self.user_bundle.account.registered:
> > +            self.redirect('/account')
> > +            return None
> > +
> > +        # Check to make sure that they aren't trying to edit another user
> > +        real_account_id = self.user_bundle.account.key.id()
> > +        check_account_id = self.request.get('account_id')
> > +        if check_account_id == real_account_id:
> > +            user = Account.get_by_id(self.user_bundle.account.key.id())
>
> No real reason. I'm assuming account is the desired name?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/gregmarra/the-blue-alliance/pull/650/files#r4582767>
> .
>