From 581151bb118d743b0af83aeb958f1df3de48f18a Mon Sep 17 00:00:00 2001
From: Andras Bacsai <5845193+andrasbacsai@users.noreply.github.com>
Date: Fri, 13 Jun 2025 16:49:27 +0200
Subject: [PATCH] refactor(resource-operations): filter out build servers from
 the server list and clean up commented-out code in the resource operations
 view

---
 .../Project/Shared/ResourceOperations.php     |  2 +-
 .../shared/resource-operations.blade.php      | 71 +++----------------
 2 files changed, 12 insertions(+), 61 deletions(-)

diff --git a/app/Livewire/Project/Shared/ResourceOperations.php b/app/Livewire/Project/Shared/ResourceOperations.php
index e19f1272d..fb19acb55 100644
--- a/app/Livewire/Project/Shared/ResourceOperations.php
+++ b/app/Livewire/Project/Shared/ResourceOperations.php
@@ -35,7 +35,7 @@ class ResourceOperations extends Component
         $this->projectUuid = data_get($parameters, 'project_uuid');
         $this->environmentUuid = data_get($parameters, 'environment_uuid');
         $this->projects = Project::ownedByCurrentTeam()->get();
-        $this->servers = currentTeam()->servers;
+        $this->servers = currentTeam()->servers->filter(fn ($server) => ! $server->isBuildServer());
     }
 
     public function toggleVolumeCloning(bool $value)
diff --git a/resources/views/livewire/project/shared/resource-operations.blade.php b/resources/views/livewire/project/shared/resource-operations.blade.php
index 4bbc28c75..f23c6eb4e 100644
--- a/resources/views/livewire/project/shared/resource-operations.blade.php
+++ b/resources/views/livewire/project/shared/resource-operations.blade.php
@@ -1,38 +1,6 @@
 <div>
     <h2>Resource Operations</h2>
     <div>You can easily make different kind of operations on this resource.</div>
-{{-- 
-    <div class="mt-8">
-        <h3 class="text-lg font-bold mb-2">Clone Volume Data</h3>
-        <div class="text-sm text-gray-600 dark:text-gray-300 mb-4">
-            Clone your volume data to the new resources volumes. This process requires a brief container downtime to ensure data consistency.
-        </div>
-        <div class="pb-4">
-            @if(!$cloneVolumeData)
-                <div wire:key="volume-disabled">
-                    <x-modal-confirmation 
-                        title="Enable Volume Data Cloning?" 
-                        buttonTitle="Enable Volume Cloning" 
-                        submitAction="toggleVolumeCloning(true)"
-                        :actions="['This will temporarily stop all the containers to copy the volume data to the new resources to ensure data consistency.', 'The process runs in the background and may take a few minutes.']" 
-                        :confirmWithPassword="false"
-                        :confirmWithText="false"
-                    />
-                </div>
-            @else
-                <div wire:key="volume-enabled" class="max-w-md">
-                    <x-forms.checkbox 
-                        label="Clone Volume Data" 
-                        id="cloneVolumeData" 
-                        wire:model="cloneVolumeData"
-                        wire:change="toggleVolumeCloning(false)"
-                        :checked="$cloneVolumeData"
-                        helper="Volume Data will be cloned to the new resources. Containers will be temporarily stopped during the cloning process." />
-                </div>
-            @endif
-        </div>
-    </div> --}}
-
     <h3>Clone</h3>
     <div class="pb-4">To another project / environment on a different / same server.</div>
     <div class="pb-4">
@@ -40,20 +8,12 @@
             @foreach ($servers->sortBy('id') as $server)
                 <h5>Server: <span class="font-bold text-dark dark:text-white">{{ $server->name }}</span></h5>
                 @foreach ($server->destinations() as $destination)
-                    <x-modal-confirmation
-                        title="Clone Resource?"
-                        buttonTitle="Clone Resource"
-                        submitAction="cloneTo({{ data_get($destination, 'id') }})"
-                        :actions="[
-                            'All containers of this resource will be duplicated and cloned to the selected destination.'
-                        ]"
-                        :confirmWithText="false"
-                        :confirmWithPassword="false"
-                        step2ButtonText="Clone Resource"
-                        dispatchEvent="true"
-                        dispatchEventType="success"
-                        dispatchEventMessage="Resource cloned to {{ $destination->name }} destination."
-                    >
+                    <x-modal-confirmation title="Clone Resource?" buttonTitle="Clone Resource"
+                        submitAction="cloneTo({{ data_get($destination, 'id') }})" :actions="[
+                            'All containers of this resource will be duplicated and cloned to the selected destination.',
+                        ]" :confirmWithText="false"
+                        :confirmWithPassword="false" step2ButtonText="Clone Resource" dispatchEvent="true" dispatchEventType="success"
+                        dispatchEventMessage="Resource cloned to {{ $destination->name }} destination.">
                         <x:slot name="content">
                             <div class="box group">
                                 <div class="flex flex-col">
@@ -80,20 +40,11 @@
                 <h5>Project: <span class="font-bold text-dark dark:text-white">{{ $project->name }}</span></h5>
 
                 @foreach ($project->environments as $environment)
-                    <x-modal-confirmation 
-                        title="Move Resource?"
-                        buttonTitle="Move Resource"
-                        submitAction="moveTo({{ data_get($environment, 'id') }})"
-                        :actions="[
-                            'All containers of this resource will be moved to the selected environment.'
-                        ]"
-                        :confirmWithText="false"
-                        :confirmWithPassword="false"
-                        step2ButtonText="Move Resource"
-                        dispatchEvent="true"
+                    <x-modal-confirmation title="Move Resource?" buttonTitle="Move Resource"
+                        submitAction="moveTo({{ data_get($environment, 'id') }})" :actions="['All containers of this resource will be moved to the selected environment.']" :confirmWithText="false"
+                        :confirmWithPassword="false" step2ButtonText="Move Resource" dispatchEvent="true"
                         dispatchEventType="success"
-                        dispatchEventMessage="Resource moved to {{ $environment->name }} environment."
-                    >
+                        dispatchEventMessage="Resource moved to {{ $environment->name }} environment.">
                         <x:slot:content>
                             <div class="box group">
                                 <div class="flex flex-col">
@@ -101,7 +52,7 @@
                                     <div class="box-description">{{ $environment->name }}</div>
                                 </div>
                             </div>
-                        </x:slot>
+                            </x:slot>
                     </x-modal-confirmation>
                 @endforeach
             @empty