refactor

Author: odeimaiz

services/static-webserver/client/source/class/osparc/share/Collaborators.js

@@ -244,6 +244,8 @@ qx.Class.define("osparc.share.Collaborators", {
         case "template":
         case "tutorial":
         case "hypertool":
+        case "function":
+        case "tag":
           rolesLayout = osparc.data.Roles.createRolesStudyInfo();
           break;
         case "service":
@@ -252,9 +254,6 @@ qx.Class.define("osparc.share.Collaborators", {
         case "workspace":
           rolesLayout = osparc.data.Roles.createRolesWorkspaceInfo();
           break;
-        case "tag":
-          rolesLayout = osparc.data.Roles.createRolesStudyInfo();
-          break;
       }
       return rolesLayout;
     },

services/static-webserver/client/source/class/osparc/ui/list/CollaboratorListItem.js

@@ -75,17 +75,25 @@ qx.Class.define("osparc.ui.list.CollaboratorListItem", {
 
   members: {
     __getRoleInfo: function(id) {
+      let roleInfo = undefined;
       const resource = this.getResourceType();
-      if (["study", "template", "tutorial", "hypertool"].includes(resource)) {
-        return osparc.data.Roles.STUDY[id];
-      } else if (resource === "service") {
-        return osparc.data.Roles.SERVICES[id];
-      } else if (resource === "workspace") {
-        return osparc.data.Roles.WORKSPACE[id];
-      } else if (resource === "tag") {
-        return osparc.data.Roles.STUDY[id];
+      switch (resource) {
+        case "study":
+        case "template":
+        case "tutorial":
+        case "hypertool":
+        case "function":
+        case "tag":
+          roleInfo = osparc.data.Roles.STUDY[id];
+          break;
+        case "service":
+          roleInfo = osparc.data.Roles.SERVICES[id];
+          break;
+        case "workspace":
+          roleInfo = osparc.data.Roles.WORKSPACE[id];
+          break;
       }
-      return undefined;
+      return roleInfo;
     },
 
     _createChildControlImpl: function(id) {