Merge pull request #1641 from csordasmarton/fix-gui-no-report

Fix non existing report in the GUI
Ericsson · Jul 10, 2018 · 49b5965 · 49b5965
2 parents 8b6abe5 + b5f45b1
commit 49b5965
Showing 1 changed file with 17 additions and 3 deletions.
diff --git a/www/scripts/codecheckerviewer/ListOfBugs.js b/www/scripts/codecheckerviewer/ListOfBugs.js
@@ -472,8 +472,12 @@ function (declare, dom, style, Deferred, ObjectStore, Store, QueryResults,
         if (sender && sender !== that._grid)
           return;
 
-        if (reportData !== null && !(reportData instanceof CC_OBJECTS.ReportData))
-          reportData = CC_SERVICE.getReport(reportData);
+        if (reportData !== null &&
+            !(reportData instanceof CC_OBJECTS.ReportData)) {
+          try {
+            reportData = CC_SERVICE.getReport(reportData);
+          } catch (ex) { console.warn(ex); }
+        }
 
         var getAndUseReportHash = reportHash && (!reportData ||
           reportData.reportId === null || reportData.bugHash !== reportHash);
@@ -484,7 +488,7 @@ function (declare, dom, style, Deferred, ObjectStore, Store, QueryResults,
           reportFilter.reportHash = [reportHash];
           reportFilter.isUnique = false;
 
-          if (reportData)
+          if (reportData && reportData.checkedFile)
             reportFilter.filepath = ['*' + reportData.checkedFile];
 
           // We set a sort option to select a report which has the shortest
@@ -516,6 +520,16 @@ function (declare, dom, style, Deferred, ObjectStore, Store, QueryResults,
 
         that.reportData = reportData;
 
+        // No report has been found.
+        if (!reportData) {
+          console.warn('No report can be found: ' + reportHash);
+          hashHelper.setStateValues({
+            'reportHash' : null,
+            'report' : null
+          });
+          return;
+        }
+
         var filename = reportData.checkedFile.substr(
           reportData.checkedFile.lastIndexOf('/') + 1);