From e5e7510a04363373fbcfcbfc90abf4aad9630ade Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jos=C3=A9=20Iv=C3=A1n=20L=C3=B3pez=20Gonz=C3=A1lez?=
 <jlopez@suse.com>
Date: Wed, 15 Nov 2023 11:23:33 +0000
Subject: [PATCH] [web] Use early return

---
 .../components/product/ProductSelector.jsx    | 37 ++++++++-----------
 1 file changed, 16 insertions(+), 21 deletions(-)

diff --git a/web/src/components/product/ProductSelector.jsx b/web/src/components/product/ProductSelector.jsx
index 05b2842bf6..311a6a7e89 100644
--- a/web/src/components/product/ProductSelector.jsx
+++ b/web/src/components/product/ProductSelector.jsx
@@ -23,32 +23,27 @@ import React from "react";
 import { Card, CardBody, Radio } from "@patternfly/react-core";
 
 import { _ } from "~/i18n";
-import { If } from "~/components/core";
 import { noop } from "~/utils";
 
 export default function ProductSelector({ value, products = [], onChange = noop }) {
+  if (products.length === 0) return <p>{_("No products available for selection")}</p>;
+
   const isSelected = (product) => product.id === value;
 
   return (
-    <If
-      condition={products.length === 0}
-      then={<p>{_("No products available for selection")}</p>}
-      else={
-        products.map((p) => (
-          <Card key={p.id} className={isSelected(p) && "selected-product"}>
-            <CardBody>
-              <Radio
-                id={p.id}
-                name="product"
-                label={p.name}
-                description={p.description}
-                isChecked={isSelected(p)}
-                onClick={() => onChange(p.id)}
-              />
-            </CardBody>
-          </Card>
-        ))
-      }
-    />
+    products.map((p) => (
+      <Card key={p.id} className={isSelected(p) && "selected-product"}>
+        <CardBody>
+          <Radio
+            id={p.id}
+            name="product"
+            label={p.name}
+            description={p.description}
+            isChecked={isSelected(p)}
+            onClick={() => onChange(p.id)}
+          />
+        </CardBody>
+      </Card>
+    ))
   );
 }