From 69f2176640e4f2f75ae6e6fffe1044adb3c0aa7b Mon Sep 17 00:00:00 2001
From: "Luiz R. Rodrigues" <contato@luizrrodrigues.com.br>
Date: Thu, 12 Nov 2020 01:35:30 -0300
Subject: [PATCH] Challenge Forums - Fix logical if user registered.

---
 .../Header/TabSelector/index.jsx              | 53 ++++++++++---------
 1 file changed, 29 insertions(+), 24 deletions(-)

diff --git a/src/shared/components/challenge-detail/Header/TabSelector/index.jsx b/src/shared/components/challenge-detail/Header/TabSelector/index.jsx
index 4a3c73e159..eb95429a9d 100644
--- a/src/shared/components/challenge-detail/Header/TabSelector/index.jsx
+++ b/src/shared/components/challenge-detail/Header/TabSelector/index.jsx
@@ -176,30 +176,35 @@ export default function ChallengeViewSelector(props) {
             </a>
           ) : null
         }
-        { (hasRegistered || Boolean(roles.length))
-          && _.isEmpty(discuss)
-          ? (
-            <a
-              href={`${config.URL.FORUMS}${forumEndpoint}`}
-              styleName={getSelectorStyle(selectedView, DETAIL_TABS.CHALLENGE_FORUM)}
-              target="_blank"
-              rel="oopener noreferrer"
-            >
-              CHALLENGE FORUM
-            </a>
-          ) : (
-            discuss.map(d => (
-              <a
-                href={d.url}
-                styleName={getSelectorStyle(selectedView, DETAIL_TABS.CHALLENGE_FORUM)}
-                target="_blank"
-                rel="oopener noreferrer"
-              >
-                CHALLENGE DISCUSSION
-              </a>
-            ))
-          )
-        }
+        { (() => {
+          if (hasRegistered || Boolean(roles.length)) {
+            if (_.isEmpty(discuss)) {
+              return (
+                <a
+                  href={`${config.URL.FORUMS}${forumEndpoint}`}
+                  styleName={getSelectorStyle(selectedView, DETAIL_TABS.CHALLENGE_FORUM)}
+                  target="_blank"
+                  rel="oopener noreferrer"
+                >
+                  CHALLENGE FORUM
+                </a>
+              );
+            }
+            return (
+              discuss.map(d => (
+                <a
+                  href={d.url}
+                  styleName={getSelectorStyle(selectedView, DETAIL_TABS.CHALLENGE_FORUM)}
+                  target="_blank"
+                  rel="oopener noreferrer"
+                >
+                  CHALLENGE DISCUSSION
+                </a>
+              ))
+            );
+          }
+          return '';
+        })()}
       </div>
     </div>
   );