From: Jeremy Davis <jeremy.davis@sonarsource.com>
Date: Tue, 7 Jan 2020 17:23:58 +0000 (+0100)
Subject: SONAR-12726 Hide security hotspots for portfolios
X-Git-Tag: 8.2.0.32929~144
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=949f20dabb7f4799cf6dbc85a8da7eae25a93eaf;p=sonarqube.git

SONAR-12726 Hide security hotspots for portfolios
---

diff --git a/server/sonar-web/src/main/js/app/components/nav/component/ComponentNavMenu.tsx b/server/sonar-web/src/main/js/app/components/nav/component/ComponentNavMenu.tsx
index 71258e7c207..72030ec00cd 100644
--- a/server/sonar-web/src/main/js/app/components/nav/component/ComponentNavMenu.tsx
+++ b/server/sonar-web/src/main/js/app/components/nav/component/ComponentNavMenu.tsx
@@ -153,13 +153,15 @@ export class ComponentNavMenu extends React.PureComponent<Props> {
 
   renderSecurityHotspotsLink() {
     return (
-      <li>
-        <Link
-          activeClassName="active"
-          to={{ pathname: '/security_hotspots', query: this.getQuery() }}>
-          {translate('layout.security_hotspots')}
-        </Link>
-      </li>
+      !this.isPortfolio() && (
+        <li>
+          <Link
+            activeClassName="active"
+            to={{ pathname: '/security_hotspots', query: this.getQuery() }}>
+            {translate('layout.security_hotspots')}
+          </Link>
+        </li>
+      )
     );
   }
 
diff --git a/server/sonar-web/src/main/js/app/components/nav/component/__tests__/__snapshots__/ComponentNavMenu-test.tsx.snap b/server/sonar-web/src/main/js/app/components/nav/component/__tests__/__snapshots__/ComponentNavMenu-test.tsx.snap
index 1b30f2e863c..a9d856cf7c7 100644
--- a/server/sonar-web/src/main/js/app/components/nav/component/__tests__/__snapshots__/ComponentNavMenu-test.tsx.snap
+++ b/server/sonar-web/src/main/js/app/components/nav/component/__tests__/__snapshots__/ComponentNavMenu-test.tsx.snap
@@ -509,23 +509,6 @@ exports[`should work for all qualifiers 2`] = `
       issues.page
     </Link>
   </li>
-  <li>
-    <Link
-      activeClassName="active"
-      onlyActiveOnIndex={false}
-      style={Object {}}
-      to={
-        Object {
-          "pathname": "/security_hotspots",
-          "query": Object {
-            "id": "foo",
-          },
-        }
-      }
-    >
-      layout.security_hotspots
-    </Link>
-  </li>
   <li>
     <Link
       activeClassName="active"
@@ -647,23 +630,6 @@ exports[`should work for all qualifiers 3`] = `
       issues.page
     </Link>
   </li>
-  <li>
-    <Link
-      activeClassName="active"
-      onlyActiveOnIndex={false}
-      style={Object {}}
-      to={
-        Object {
-          "pathname": "/security_hotspots",
-          "query": Object {
-            "id": "foo",
-          },
-        }
-      }
-    >
-      layout.security_hotspots
-    </Link>
-  </li>
   <li>
     <Link
       activeClassName="active"