commit b4e3becf258be824863beea6656f377306fa738f
author Mitja Nikolaus <mitja@fairphone.com> Thu Aug 30 17:16:21 2018 +0200
committer Karsten Tausche <karsten@fairphone.com> Thu Sep 13 05:02:16 2018 +0000
tree 1e16519e9ba18f642ebac6bd286c97ce88ed70ad
parent e1389bdb6f8990b447e786b29fcdef232ea6aa04

Remove unneccesary else statements after return

Issue: HIC-170
Change-Id: I88365341e93114e0421922292198fce91c8d48aa

crashreports/permissions.py

diff --git a/crashreports/permissions.py b/crashreports/permissions.py
index 0ff8ae8..3d953eb 100644
--- a/crashreports/permissions.py
+++ b/crashreports/permissions.py
@@ -40,23 +40,22 @@
     """
     if user.groups.filter(name="FairphoneSoftwareTeam").exists():
         return True
-    else:
-        return user.has_perms(
-            [
-                # Crashreports
-                "crashreports.add_crashreport",
-                "crashreports.change_crashreport",
-                "crashreports.del_crashreport",
-                # Heartbeats
-                "heartbeat.add_crashreport",
-                "heartbeat.change_crashreport",
-                "heartbeat.del_crashreport",
-                # Logfiles
-                "heartbeat.add_logfile",
-                "heartbeat.change_logfile",
-                "heartbeat.del_logfile",
-            ]
-        )
+    return user.has_perms(
+        [
+            # Crashreports
+            "crashreports.add_crashreport",
+            "crashreports.change_crashreport",
+            "crashreports.del_crashreport",
+            # Heartbeats
+            "heartbeat.add_crashreport",
+            "heartbeat.change_crashreport",
+            "heartbeat.del_crashreport",
+            # Logfiles
+            "heartbeat.add_logfile",
+            "heartbeat.change_logfile",
+            "heartbeat.del_logfile",
+        ]
+    )
 
 
 class HasStatsAccess(BasePermission):