From a9453d1b65926ac7642e7104cd068d524098cc19 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=EA=B9=80=EB=AF=BC=EC=8B=9D?=
 <43837214+Minsiki@users.noreply.github.com>
Date: Tue, 4 Feb 2025 09:50:22 +0900
Subject: [PATCH] =?UTF-8?q?-=20=EC=88=98=EB=8F=99=ED=95=A0=EB=8B=B9?=
 =?UTF-8?q?=EC=8B=9C=20=EC=A4=91=EB=B3=B5=20=EB=B2=88=ED=98=B8=20=EC=82=AD?=
 =?UTF-8?q?=EC=A0=9C=20-=20=EC=A7=80=EB=B6=95=EB=A9=B4=20=EC=A0=95?=
 =?UTF-8?q?=EB=A0=AC=20=EC=B6=94=EA=B0=80?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../circuitTrestle/CircuitTrestleSetting.jsx  | 41 +++++++++----------
 .../step/type/PassivityCircuitAllocation.jsx  |  3 ++
 2 files changed, 23 insertions(+), 21 deletions(-)

diff --git a/src/components/floor-plan/modal/circuitTrestle/CircuitTrestleSetting.jsx b/src/components/floor-plan/modal/circuitTrestle/CircuitTrestleSetting.jsx
index 3783e033..e648511c 100644
--- a/src/components/floor-plan/modal/circuitTrestle/CircuitTrestleSetting.jsx
+++ b/src/components/floor-plan/modal/circuitTrestle/CircuitTrestleSetting.jsx
@@ -159,27 +159,26 @@ export default function CircuitTrestleSetting({ id }) {
   }
 
   const getRoofSurfaceList = () => {
-    return canvas
-      .getObjects()
-      .filter((obj) => obj.name === POLYGON_TYPE.MODULE_SETUP_SURFACE)
-      .map((obj) => {
-        return {
-          roofSurfaceId: obj.id,
-          roofSurface: canvas
-            .getObjects()
-            .filter((o) => o.id === obj.parentId)[0]
-            .directionText.replace(/[0-9]/g, ''),
-          roofSurfaceIncl: canvas.getObjects().filter((o) => o.id === obj.parentId)[0].roofMaterial.pitch,
-          moduleList: getModuleList(obj).map((module) => {
-            return {
-              itemId: module.moduleInfo.itemId,
-              circuit: module.circuitNumber ? module.circuitNumber : null,
-              pcsItemId: module.circuit ? module.circuit?.pcsItemId : null,
-              uniqueId: module.id ? module.id : null,
-            }
-          }),
-        }
-      })
+    const roofSurfaceList = canvas.getObjects().filter((obj) => obj.name === POLYGON_TYPE.MODULE_SETUP_SURFACE)
+    roofSurfaceList.sort((a, b) => a.left - b.left || b.top - a.top)
+    return roofSurfaceList.map((obj) => {
+      return {
+        roofSurfaceId: obj.id,
+        roofSurface: canvas
+          .getObjects()
+          .filter((o) => o.id === obj.parentId)[0]
+          .directionText.replace(/[0-9]/g, ''),
+        roofSurfaceIncl: canvas.getObjects().filter((o) => o.id === obj.parentId)[0].roofMaterial.pitch,
+        moduleList: getModuleList(obj).map((module) => {
+          return {
+            itemId: module.moduleInfo.itemId,
+            circuit: module.circuitNumber ? module.circuitNumber : null,
+            pcsItemId: module.circuit ? module.circuit?.pcsItemId : null,
+            uniqueId: module.id ? module.id : null,
+          }
+        }),
+      }
+    })
   }
 
   const getModuleList = (surface) => {
diff --git a/src/components/floor-plan/modal/circuitTrestle/step/type/PassivityCircuitAllocation.jsx b/src/components/floor-plan/modal/circuitTrestle/step/type/PassivityCircuitAllocation.jsx
index 32b15236..77404733 100644
--- a/src/components/floor-plan/modal/circuitTrestle/step/type/PassivityCircuitAllocation.jsx
+++ b/src/components/floor-plan/modal/circuitTrestle/step/type/PassivityCircuitAllocation.jsx
@@ -206,6 +206,9 @@ export default function PassivityCircuitAllocation(props) {
       .getObjects()
       .filter((obj) => targetModules.includes(obj.id))
       .forEach((obj) => {
+        if (obj.circuit) {
+          canvas.remove(obj.circuit)
+        }
         const moduleCircuitText = new fabric.Text(getCircuitNumber(), {
           left: obj.left + obj.width / 2,
           top: obj.top + obj.height / 2,