#1668 Small code cleaning and moving around
Merged by pingou. Opened by pingou.
code_cleaning  into  master

Download 1668.patch

We had a bunch of if repo_admin one after the other, that wasn't
quite needed, one is enough.

We moved updating the status near the top since that's one action that
is allowed for admins and the person who created the ticket. Everything
else will only be allowed for admins.

While this piece of code could still use a nice refactoring, this already
helps a little on readability and logic.

both if I am guessing it right

Just that one comment else it looks good to me

Thanks :)

1 new commit added

  • Fix typo in the doc as pointed out by @farhaan

i guess you missed it :)

1 new commit added

  • Fix typo in the doc as pointed out by @farhaan - take 2

Take 2 looks good to me :)

Thanks :)

Pull-Request has been merged by pingou

Metadata