From cd7d4e4f27190d9925e662478bab32da953b7e08 Mon Sep 17 00:00:00 2001
From: Matt Ray <github@mattray.dev>
Date: Tue, 20 Jun 2023 15:24:03 +1000
Subject: [PATCH] Fix for "86:5  error  Arrow function expected no return value
  consistent-return"

Signed-off-by: Matt Ray <github@mattray.dev>
---
 .../AllocationChart/SummaryChart.js           | 92 ++++++++++---------
 1 file changed, 51 insertions(+), 41 deletions(-)

diff --git a/plugins/opencost/src/components/AllocationChart/SummaryChart.js b/plugins/opencost/src/components/AllocationChart/SummaryChart.js
index 977ee395a3..9ac59cdbca 100644
--- a/plugins/opencost/src/components/AllocationChart/SummaryChart.js
+++ b/plugins/opencost/src/components/AllocationChart/SummaryChart.js
@@ -14,81 +14,89 @@
  * limitations under the License.
  */
 
-import React from 'react'
-import { ResponsiveContainer, PieChart, Pie, Cell } from 'recharts'
+import React from 'react';
+import { ResponsiveContainer, PieChart, Pie, Cell } from 'recharts';
 import { primary, greyscale, browns } from '../../constants/colors';
 import { toCurrency } from '../../util';
 
 function toPieData(top, other, idle) {
-  const slices = []
+  const slices = [];
 
   for (const i in top) {
     if (Object.hasOwn(top, i)) {
-      const allocation = top[i]
-      const fill = allocation.name === "__unallocated__"
-            ? "#212121"
-            : primary[i % primary.length]
+      const allocation = top[i];
+      const fill =
+        allocation.name === '__unallocated__'
+          ? '#212121'
+          : primary[i % primary.length];
 
       slices.push({
         name: allocation.name,
         value: allocation.totalCost,
         fill: fill,
-      })
+      });
     }
   }
 
   for (const i in other) {
     if (Object.hasOwn(other, i)) {
-      const allocation = other[i]
-      const fill = browns[i % browns.length]
+      const allocation = other[i];
+      const fill = browns[i % browns.length];
       slices.push({
         name: allocation.name,
         value: allocation.totalCost,
         fill: fill,
-      })
+      });
     }
   }
 
   for (const i in idle) {
     if (Object.hasOwn(idle, i)) {
-      const allocation = idle[i]
-      const fill = greyscale[i % greyscale.length]
+      const allocation = idle[i];
+      const fill = greyscale[i % greyscale.length];
       slices.push({
         name: allocation.name,
         value: allocation.totalCost,
         fill: fill,
-      })
+      });
     }
   }
 
-  return slices
+  return slices;
 }
 
 const SummaryChart = ({ top, other, idle, currency, height }) => {
-  const pieData = toPieData(top, other, idle)
+  const pieData = toPieData(top, other, idle);
 
-  const renderLabel = (params) => {
-    const {
-      cx, cy, midAngle, outerRadius, percent, name, fill, value
-    } = params
+  const renderLabel = params => {
+    const { cx, cy, midAngle, outerRadius, percent, name, fill, value } =
+      params;
 
-    const RADIAN = Math.PI / 180
-    const radius = outerRadius * 1.1
-    let x = cx + radius * Math.cos(-midAngle * RADIAN)
-    x += x > cx ? 2 : -2
-    const y = cy + radius * Math.sin(-midAngle * RADIAN)
+    const RADIAN = Math.PI / 180;
+    const radius = outerRadius * 1.1;
+    let x = cx + radius * Math.cos(-midAngle * RADIAN);
+    x += x > cx ? 2 : -2;
+    const y = cy + radius * Math.sin(-midAngle * RADIAN);
     // y -= Math.min(Math.abs(2 / Math.cos(-midAngle * RADIAN)), 8)
 
     if (percent < 0.02) {
-      return
+      return undefined;
     }
 
     return (
-      <text x={x} y={y} fill={fill} textAnchor={x > cx ? 'start' : 'end'} dominantBaseline="central">
-        {`${name}: ${toCurrency(value, currency)} (${(percent * 100).toFixed(1)}%)`}
+      <text
+        x={x}
+        y={y}
+        fill={fill}
+        textAnchor={x > cx ? 'start' : 'end'}
+        dominantBaseline="central"
+      >
+        {`${name}: ${toCurrency(value, currency)} (${(percent * 100).toFixed(
+          1,
+        )}%)`}
       </text>
-    )
-  }
+    );
+  };
 
   return (
     <ResponsiveContainer width="100%" height={height}>
@@ -100,19 +108,21 @@ const SummaryChart = ({ top, other, idle, currency, height }) => {
           label={renderLabel}
           labelLine
           // niko: if tooltips error, try disabling animation
-        // isAnimationActive={false}
-        animationDuration={400}
-        cy="90%"
-        outerRadius="140%"
-        innerRadius="60%"
-        startAngle={180}
-        endAngle={0}
+          // isAnimationActive={false}
+          animationDuration={400}
+          cy="90%"
+          outerRadius="140%"
+          innerRadius="60%"
+          startAngle={180}
+          endAngle={0}
         >
-        {pieData.map((datum, i) => <Cell key={i} fill={datum.fill} />)}
+          {pieData.map((datum, i) => (
+            <Cell key={i} fill={datum.fill} />
+          ))}
         </Pie>
       </PieChart>
     </ResponsiveContainer>
-        )
-  }
+  );
+};
 
-  export default SummaryChart
+export default SummaryChart;