Backend: Remove redundant condition of `ROLE_VIEWER` (#19211)

a41a7ea6 · Masudur Rahman · Torkel Ödegaard · 14f1cf29 · a41a7ea6
Commit a41a7ea6 authored Sep 19, 2019 by Masudur Rahman Committed by Torkel Ödegaard Sep 19, 2019
Show whitespace changes
Inline Side-by-side

Showing with 1 additions and 5 deletions

pkg/models/org_user.go
+1 -5

No files found.
--- a/pkg/models/org_user.go
+++ b/pkg/models/org_user.go
@@ -36,11 +36,7 @@ func (r RoleType) Includes(other RoleType) bool {
 		return other != ROLE_ADMIN
 	}

-	if r == ROLE_VIEWER {
-		return other == ROLE_VIEWER
-	}
-
-	return false
+	return r == other
 }

 func (r *RoleType) UnmarshalJSON(data []byte) error {