From abe9f28e4e547d632acfa617a654d0689dca6308 Mon Sep 17 00:00:00 2001
From: Mirek Simek <miroslav.simek@gmail.com>
Date: Wed, 11 Dec 2024 21:44:14 +0100
Subject: [PATCH] Rendering fields only if they are filled in

---
 .../common/components/SideRequestInfo.jsx     | 84 ++++++++++---------
 1 file changed, 44 insertions(+), 40 deletions(-)

diff --git a/oarepo_requests/ui/theme/assets/semantic-ui/js/oarepo_requests_ui/common/components/SideRequestInfo.jsx b/oarepo_requests/ui/theme/assets/semantic-ui/js/oarepo_requests_ui/common/components/SideRequestInfo.jsx
index 2b3350c..08db19e 100644
--- a/oarepo_requests/ui/theme/assets/semantic-ui/js/oarepo_requests_ui/common/components/SideRequestInfo.jsx
+++ b/oarepo_requests/ui/theme/assets/semantic-ui/js/oarepo_requests_ui/common/components/SideRequestInfo.jsx
@@ -10,44 +10,48 @@ export const SideRequestInfo = ({ request }) => {
   const statusIcon = getRequestStatusIcon(request?.status_code);
   return (
     <List horizontal relaxed divided size="small">
-      <List.Item>
-        <List.Header as="h3">{i18next.t("Creator")}</List.Header>
-        <List.Content>
-          <Icon name="user circle outline" />
-          <span>
-            {_has(request, "links.created_by_html") ? (
-              <a
-                href={request.links.created_by_html}
-                target="_blank"
-                rel="noreferrer"
-              >
-                {request.created_by.label}
-              </a>
-            ) : (
-              request.created_by?.label
-            )}
-          </span>
-        </List.Content>
-      </List.Item>
-      <List.Item>
-        <List.Header as="h3">{i18next.t("Receiver")}</List.Header>
-        <List.Content>
-          <Icon name="mail outline" />
-          <span>
-            {_has(request, "links.receiver_html") ? (
-              <a
-                href={request.links.receiver_html}
-                target="_blank"
-                rel="noreferrer"
-              >
-                {request?.receiver?.label}
-              </a>
-            ) : (
-              request?.receiver?.label
-            )}
-          </span>
-        </List.Content>
-      </List.Item>
+      {request?.created_by &&
+        <List.Item>
+          <List.Header as="h3">{i18next.t("Creator")}</List.Header>
+          <List.Content>
+            <Icon name="user circle outline" />
+            <span>
+              {_has(request, "links.created_by_html") ? (
+                <a
+                  href={request.links.created_by_html}
+                  target="_blank"
+                  rel="noreferrer"
+                >
+                  {request.created_by.label}
+                </a>
+              ) : (
+                request.created_by?.label
+              )}
+            </span>
+          </List.Content>
+        </List.Item>
+      }
+      {request?.receiver &&
+        <List.Item>
+          <List.Header as="h3">{i18next.t("Receiver")}</List.Header>
+          <List.Content>
+            <Icon name="mail outline" />
+            <span>
+              {_has(request, "links.receiver_html") ? (
+                <a
+                  href={request.links.receiver_html}
+                  target="_blank"
+                  rel="noreferrer"
+                >
+                  {request?.receiver?.label}
+                </a>
+              ) : (
+                request?.receiver?.label
+              )}
+            </span>
+          </List.Content>
+        </List.Item>
+      }
       <List.Item>
         <List.Header as="h3">{i18next.t("Status")}</List.Header>
         <List.Content>
@@ -68,8 +72,8 @@ export const SideRequestInfo = ({ request }) => {
             <a href={request.links.topic_html} target="_blank" rel="noreferrer">
               {request?.topic?.label
                 ? _truncate(request?.topic?.label, {
-                    length: 350,
-                  })
+                  length: 350,
+                })
                 : i18next.t("Request topic")}
             </a>
           </List.Content>