Use new isOwner var in tests

With the var there now, this makes the code a bit more readable.
5 years ago · 151e996387
parent 55dc1917fe
commit 151e996387
1 changed files with 2 additions and 2 deletions
--- a/collections.go
+++ b/collections.go
@ -770,7 +770,7 @@ func handleViewCollection(app *App, w http.ResponseWriter, r *http.Request) erro
 		}
 	}
 	isOwner := owner != nil
-	if owner == nil {
+	if !isOwner {
 		// Current user doesn't own collection; retrieve owner information
 		owner, err = app.db.GetUserByID(coll.OwnerID)
 		if err != nil {
@ -868,7 +868,7 @@ func handleViewCollectionTag(app *App, w http.ResponseWriter, r *http.Request) e
 		}
 	}
 	isOwner := owner != nil
-	if owner == nil {
+	if !isOwner {
 		// Current user doesn't own collection; retrieve owner information
 		owner, err = app.db.GetUserByID(coll.OwnerID)
 		if err != nil {