From 1e68060a982626d30bd770fe27ace5ddf2a8aaee Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Maren=20S=C3=BCwer?= <maren.suewer@cloudogu.com>
Date: Mon, 3 Sep 2018 14:19:44 +0200
Subject: [PATCH] remove if else to show permission add option also in repos
 without any permissions

---
 .../src/permissions/containers/Permissions.js | 59 +++++++++----------
 1 file changed, 27 insertions(+), 32 deletions(-)

diff --git a/scm-ui/src/permissions/containers/Permissions.js b/scm-ui/src/permissions/containers/Permissions.js
index 03d34414eb..67bac16e8b 100644
--- a/scm-ui/src/permissions/containers/Permissions.js
+++ b/scm-ui/src/permissions/containers/Permissions.js
@@ -15,7 +15,6 @@ import {
 import Loading from "../../components/Loading";
 import ErrorPage from "../../components/ErrorPage";
 import type {
-  Permission,
   PermissionCollection,
   PermissionEntry
 } from "../types/Permissions";
@@ -70,6 +69,7 @@ class Permissions extends React.Component<Props> {
       loadingCreatePermission,
       hasPermissionToCreate
     } = this.props;
+    console.log(permissions);
 
     if (error) {
       return (
@@ -98,36 +98,32 @@ class Permissions extends React.Component<Props> {
         loading={loadingCreatePermission}
       />
     ) : null;
-
-    if (permissions.length > 0)
-      return (
-        <div>
-          <table className="table is-hoverable is-fullwidth">
-            <thead>
-              <tr>
-                <th>{t("permission.name")}</th>
-                <th className="is-hidden-mobile">{t("permission.type")}</th>
-                <th>{t("permission.group-permission")}</th>
-              </tr>
-            </thead>
-            <tbody>
-              {permissions.map((permission, index) => {
-                return (
-                  <SinglePermission
-                    key={index}
-                    namespace={namespace}
-                    repoName={repoName}
-                    permission={permission}
-                  />
-                );
-              })}
-            </tbody>
-          </table>
-          {createPermissionForm}
-        </div>
-      );
-
-    return <div />;
+    return (
+      <div>
+        <table className="table is-hoverable is-fullwidth">
+          <thead>
+            <tr>
+              <th>{t("permission.name")}</th>
+              <th className="is-hidden-mobile">{t("permission.type")}</th>
+              <th>{t("permission.group-permission")}</th>
+            </tr>
+          </thead>
+          <tbody>
+            {permissions.map(permission => {
+              return (
+                <SinglePermission
+                  key={permission.name}
+                  namespace={namespace}
+                  repoName={repoName}
+                  permission={permission}
+                />
+              );
+            })}
+          </tbody>
+        </table>
+        {createPermissionForm}
+      </div>
+    );
   }
 }
 
@@ -142,7 +138,6 @@ const mapStateToProps = (state, ownProps) => {
     namespace,
     repoName
   );
-  console.log(permissions);
   const hasPermissionToCreate = hasCreatePermission(state, namespace, repoName);
   return {
     namespace,