🐛 bugfix: Copy thumbnail only if exists (ITISFoundation#2548)

* copy thumbnail only if exists
* only ask for checkpoints if can read

Author: odeimaiz

services/web/client/source/class/osparc/data/Permissions.js

@@ -165,6 +165,7 @@ qx.Class.define("osparc.data.Permissions", {
           "user.role.update",
           "user.clusters.create",
           "study.service.update",
+          "study.snapshot.read",
           "study.snapshot.create",
           "study.nodestree.uuid.read",
           "study.filestree.uuid.read",

services/web/client/source/class/osparc/data/model/Study.js

@@ -232,6 +232,10 @@ qx.Class.define("osparc.data.model.Study", {
 
     getSnapshots: function(studyId) {
       return new Promise((resolve, reject) => {
+        if (!osparc.data.Permissions.getInstance().canDo("study.snapshot.read")) {
+          reject();
+          return;
+        }
         const params = {
           url: {
             "studyId": studyId
@@ -254,9 +258,8 @@ qx.Class.define("osparc.data.model.Study", {
           .then(snapshots => {
             resolve(Boolean(snapshots.length));
           })
-          .catch(err => {
-            console.error(err);
-            reject(err);
+          .catch(() => {
+            resolve(false);
           });
       });
     }

services/web/client/source/class/osparc/utils/Study.js

@@ -41,7 +41,9 @@ qx.Class.define("osparc.utils.Study", {
               const newUuid = osparc.utils.Utils.uuidv4();
               const minStudyData = osparc.data.model.Study.createMyNewStudyObject();
               minStudyData["name"] = service["name"];
-              minStudyData["thumbnail"] = service["thumbnail"];
+              if (service["thumbnail"]) {
+                minStudyData["thumbnail"] = service["thumbnail"];
+              }
               minStudyData["workbench"][newUuid] = {
                 "key": service["key"],
                 "version": service["version"],