Fixes https://pagure.io/pagure/issue/839 -- now with indentation (and rebased)

:thumbsup:
Please note you can update the PR in place by force-pushing to the PR branch, and the web interface will automatically update.

for the record: this was previously PR #870

One sudden thought, do we need repo_admin here? Since it can be true only if g.fas_user is not None, no?

Same question for repo_admin here as below

Good questions on repo_admin, pingou... I didn't know whether pagure allowed some out-of-band users that would make g.fas_user == None. It sounds like that's not the case?

We always check for g.fas_user != None to check if an user is authenticated or not, and there isn't really any special-case users that would be using the UI.

Would g.fas_user be used (ie. different of None) by instance of pagure not using FAS for the user authentication ???

Would g.fas_user be used (ie. different of None) by instance of pagure not using FAS for the user authentication ???

Yes, however unfortunate the variable name is in this case, if you use openid or the local auth in pagure, in both cases the user's info are stored in g.fas_user.

I also removed other unnecessary "if repo_admin" statements in this PR in a separate commit.

We should rework the length of some of these lines but that's for another time.

Thanks for fixing this @pfrields! Merging :)