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 df8ecde2..1ee73892 100644
--- a/src/components/floor-plan/modal/circuitTrestle/step/type/PassivityCircuitAllocation.jsx
+++ b/src/components/floor-plan/modal/circuitTrestle/step/type/PassivityCircuitAllocation.jsx
@@ -45,6 +45,7 @@ export default function PassivityCircuitAllocation(props) {
       .getObjects()
       .filter((obj) => obj.name === POLYGON_TYPE.MODULE)
       .forEach((obj) => {
+        obj.set({ pcs: null })
         obj.on('mousedown', (e) => handleTargetModules(obj))
       })
 
@@ -134,7 +135,15 @@ export default function PassivityCircuitAllocation(props) {
         originSurfaceList.concat(originSurfaceList).forEach((surface) => {
           surfaceType[`${surface.direction}-${surface.roofMaterial.pitch}`] = surface
         })
-        
+
+        if (Object.keys(surfaceType).length > 1) {
+          swalFire({
+            text: getMessage('module.circuit.fix.not.same.roof.error'),
+            type: 'alert',
+            icon: 'warning',
+          })
+          return
+        }
 
         break
       }
@@ -341,6 +350,7 @@ export default function PassivityCircuitAllocation(props) {
           obj.circuit = null
           obj.circuitNumber = null
           obj.pcsItemId = null
+          obj.pcs = null
         })
         setCircuitNumber(minCircuitNumber)
         setTargetModules([])
@@ -367,6 +377,7 @@ export default function PassivityCircuitAllocation(props) {
           obj.circuit = null
           obj.circuitNumber = null
           obj.pcsItemId = null
+          obj.pcs = null
         })
         canvas.renderAll()
         setCircuitNumber(1)