SONAR-13123 Handle WS errors when searching for issues

diff --git a/server/sonar-web/src/main/js/api/issues.ts b/server/sonar-web/src/main/js/api/issues.ts
index 357457a0ae479d427c4b40eb14dec55d5d2ec5aa..3f5f88f394808704fa00a254772239a40156de62 100644 (file)
--- a/server/sonar-web/src/main/js/api/issues.ts
+++ b/server/sonar-web/src/main/js/api/issues.ts
@@ -44,7 +44,7 @@ type FacetName =
   | 'types';
 
 export function searchIssues(query: RequestData): Promise<RawIssuesResponse> {
-  return getJSON('/api/issues/search', query);
+  return getJSON('/api/issues/search', query).catch(throwGlobalError);
 }
 
 export function getFacets(

diff --git a/server/sonar-web/src/main/js/apps/issues/components/AppContainer.tsx b/server/sonar-web/src/main/js/apps/issues/components/AppContainer.tsx
index b0c1f2b266c954e68f34853eea8ce1faf95af13e..2c22170bd485ed5f8c0881003b9dffe6bd9dcc2b 100644 (file)
--- a/server/sonar-web/src/main/js/apps/issues/components/AppContainer.tsx
+++ b/server/sonar-web/src/main/js/apps/issues/components/AppContainer.tsx
@@ -20,11 +20,11 @@
 import { connect } from 'react-redux';
 import { lazyLoadComponent } from 'sonar-ui-common/components/lazyLoadComponent';
 import { searchIssues } from '../../../api/issues';
-import throwGlobalError from '../../../app/utils/throwGlobalError';
 import { withRouter } from '../../../components/hoc/withRouter';
 import { parseIssueFromResponse } from '../../../helpers/issues';
 import { fetchBranchStatus } from '../../../store/rootActions';
 import { getCurrentUser, Store } from '../../../store/rootReducer';
+import { FetchIssuesPromise } from '../../../types/issues';
 
 const IssuesAppContainer = lazyLoadComponent(() => import('./App'), 'IssuesAppContainer');
 
@@ -38,14 +38,12 @@ const fetchIssues = (query: T.RawQuery) => {
     ...query,
     additionalFields: '_all',
     timeZone: Intl.DateTimeFormat().resolvedOptions().timeZone
-  })
-    .then(response => {
-      const parsedIssues = response.issues.map(issue =>
-        parseIssueFromResponse(issue, response.components, response.users, response.rules)
-      );
-      return { ...response, issues: parsedIssues };
-    })
-    .catch(throwGlobalError);
+  }).then(response => {
+    const parsedIssues = response.issues.map(issue =>
+      parseIssueFromResponse(issue, response.components, response.users, response.rules)
+    );
+    return { ...response, issues: parsedIssues } as FetchIssuesPromise;
+  });
 };
 
 const mapDispatchToProps = { fetchBranchStatus };