From 5d984b75119fd09c67e1886a64b0ecf1055e5283 Mon Sep 17 00:00:00 2001
From: B_Rass <b.rasschaert@pantographe.studio>
Date: Wed, 22 Jan 2025 17:44:13 +0100
Subject: [PATCH 01/13] Create PowerDistributionUnitsController and its views

---
 .../power_distribution_units_controller.rb    | 107 ++++++
 app/controllers/servers_controller.rb         |  68 ++--
 app/models/modele.rb                          |   2 +
 app/models/server.rb                          |  18 +-
 app/views/layouts/_sidebar.html.erb           |   8 +-
 .../power_distribution_units/_form.html.erb   | 167 ++++++++
 .../duplicate.html.erb                        |  26 ++
 .../power_distribution_units/edit.html.erb    |  23 ++
 .../power_distribution_units/index.html.erb   | 157 ++++++++
 .../index.json.jbuilder                       |   7 +
 .../power_distribution_units/new.html.erb     |  14 +
 .../power_distribution_units/show.html.erb    | 356 ++++++++++++++++++
 .../show.json.jbuilder                        |   8 +
 app/views/servers/index.html.erb              |   2 +-
 app/views/servers/show.html.erb               |  42 ++-
 config/locales/activerecord.fr.yml            |   1 +
 config/locales/en.yml                         |  37 ++
 config/locales/fr.yml                         |  37 +-
 config/routes.rb                              |   6 +
 .../power_distribution_units_request_spec.rb  | 229 +++++++++++
 spec/requests/servers_request_spec.rb         |   3 +
 test/fixtures/documents.yml                   |   4 +
 test/fixtures/servers.yml                     |   1 +
 23 files changed, 1258 insertions(+), 65 deletions(-)
 create mode 100644 app/controllers/power_distribution_units_controller.rb
 create mode 100644 app/views/power_distribution_units/_form.html.erb
 create mode 100644 app/views/power_distribution_units/duplicate.html.erb
 create mode 100644 app/views/power_distribution_units/edit.html.erb
 create mode 100644 app/views/power_distribution_units/index.html.erb
 create mode 100644 app/views/power_distribution_units/index.json.jbuilder
 create mode 100644 app/views/power_distribution_units/new.html.erb
 create mode 100644 app/views/power_distribution_units/show.html.erb
 create mode 100644 app/views/power_distribution_units/show.json.jbuilder
 create mode 100644 spec/requests/power_distribution_units_request_spec.rb

diff --git a/app/controllers/power_distribution_units_controller.rb b/app/controllers/power_distribution_units_controller.rb
new file mode 100644
index 000000000..cb1bba77c
--- /dev/null
+++ b/app/controllers/power_distribution_units_controller.rb
@@ -0,0 +1,107 @@
+# frozen_string_literal: true
+
+class PowerDistributionUnitsController < ServersController
+  before_action :set_pdu, only: %i[show edit update destroy destroy_connections]
+
+  def index
+    @pdus = Server.only_pdus.includes(:frame, :room, :islet, bay: :frames, modele: :category)
+      .references(:room, :islet, :bay, modele: :category)
+      .order(:name)
+    @filter = ProcessorFilter.new(@pdus, params)
+
+    @pdus = @filter.results
+    @search_params = search_params
+
+    respond_to do |format|
+      format.json
+      format.html { @pagy, @pdus = pagy(@pdus) }
+    end
+  end
+
+  def show; end
+
+  def new
+    @pdu = Server.new
+  end
+
+  def edit; end
+
+  def create
+    @pdu = Server.new(pdu_params)
+
+    respond_to do |format|
+      if @pdu.save
+        format.html { redirect_to power_distribution_unit_path(@pdu), notice: t(".flashes.created") }
+        format.json { render :show, status: :created, location: @pdu }
+      else
+        format.html { render :new }
+        format.json { render json: @pdu.errors, status: :unprocessable_entity }
+      end
+    end
+  end
+
+  def update
+    respond_to do |format|
+      if @pdu.update(pdu_params)
+        format.html { redirect_to power_distribution_unit_path(@pdu), notice: t(".flashes.updated") }
+        format.json { render :show, status: :ok, location: @pdu }
+      else
+        format.html { render :edit }
+        format.json { render json: @pdu.errors, status: :unprocessable_entity }
+      end
+    end
+  end
+
+  def destroy
+    respond_to do |format|
+      if @pdu.destroy
+        format.html { redirect_to power_distribution_units_path(search_params), notice: t(".flashes.destroyed") }
+        format.json { head :no_content }
+      else
+        format.html { redirect_to power_distribution_units_path(search_params), alert: t(".flashes.not_destroyed") }
+        format.json { head :bad_request }
+      end
+    end
+  end
+
+  def duplicate
+    @original_pdu = Server.friendly.find(params[:id].to_s.downcase)
+    @pdu = @original_pdu.deep_dup
+  end
+
+  def destroy_connections
+    if @pdu.destroy_connections!
+      flash[:notice] = t(".flashes.connections_destroyed")
+    else
+      flash[:alert] = t(".flashes.connections_not_destroyed")
+    end
+
+    redirect_to power_distribution_unit_path(@pdu)
+  end
+
+  private
+
+  def set_pdu
+    @pdu = Server.includes(cards: [ports: [:connection, :cable], card_type: [:port_type]])
+      .find_with_numero_or_friendly_id(params[:id].to_s.downcase)
+  end
+
+  def pdu_params
+    params.require(:power_distribution_unit)
+      .permit(:photo, :comment, :position, :frame_id, :gestion_id, :fc_futur, :rj45_cm, :name, :modele_id, :numero,
+              :critique, :domaine_id, :fc_total, :fc_utilise, :rj45_total, :rj45_utilise, :rj45_futur,
+              :ipmi_utilise, :ipmi_futur, :ipmi_dedie,
+              :frame, # TODO: Check if it should be removed or if it's used somewhere
+              cards_attributes: [
+                :composant_id, :card_type_id, :_destroy, :id, :twin_card_id, :orientation, :name,
+                :first_position,
+              ],
+              documents_attributes: [:document, :id, :_destroy])
+  end
+
+  def search_params
+    params.permit(:sort, :sort_by, :page, :per_page, :q,
+                  network_types: [], bay_ids: [], islet_ids: [], room_ids: [], frame_ids: [], cluster_ids: [],
+                  gestion_ids: [], domaine_ids: [], modele_ids: [], server_state_ids: [], stack_ids: [])
+  end
+end
diff --git a/app/controllers/servers_controller.rb b/app/controllers/servers_controller.rb
index 42fbfe2a3..21fd34a61 100644
--- a/app/controllers/servers_controller.rb
+++ b/app/controllers/servers_controller.rb
@@ -13,7 +13,7 @@ def index
       logger.warn("DEPRECATION WARNING: Search with 'name' is now deprecated. Use 'q' instead.")
     end
 
-    @servers = Server.includes(:frame, :room, :islet, bay: :frames, modele: :category)
+    @servers = Server.no_pdus.includes(:frame, :room, :islet, bay: :frames, modele: :category)
       .references(:room, :islet, :bay, modele: :category)
       .order(:name)
     @filter = ProcessorFilter.new(@servers, params)
@@ -27,27 +27,6 @@ def index
     end
   end
 
-  def grid
-    @servers = ServersGrid.new(params[:servers_grid])
-  end
-
-  def sort
-    room = Room.find_by_name(params[:room]) unless params[:room].include?('non ')
-    frame = room.frames.where('islets.name = ? AND frames.name = ?', params[:islet], params[:frame]).first
-    positions = params[:positions].split(',')
-
-    params[:server].each_with_index do |id, index|
-      if positions[index].present?
-        server = Server.find_by_id(id)
-        new_params = { position: positions[index] }
-        new_params[:frame_id] = frame.id if frame.present?
-
-        server.update(new_params)
-      end
-    end if params[:server].present?
-    head :ok # render empty body, status only
-  end
-
   def show; end
 
   def new
@@ -82,6 +61,39 @@ def update
     end
   end
 
+  def destroy
+    respond_to do |format|
+      if @server.destroy
+        format.html { redirect_to servers_path(search_params), notice: t(".flashes.destroyed") }
+        format.json { head :no_content }
+      else
+        format.html { redirect_to servers_path(search_params), alert: t(".flashes.not_destroyed") }
+        format.json { head :bad_request }
+      end
+    end
+  end
+
+  def grid
+    @servers = ServersGrid.new(params[:servers_grid])
+  end
+
+  def sort
+    room = Room.find_by_name(params[:room]) unless params[:room].include?('non ')
+    frame = room.frames.where('islets.name = ? AND frames.name = ?', params[:islet], params[:frame]).first
+    positions = params[:positions].split(',')
+
+    params[:server].each_with_index do |id, index|
+      if positions[index].present?
+        server = Server.find_by_id(id)
+        new_params = { position: positions[index] }
+        new_params[:frame_id] = frame.id if frame.present?
+
+        server.update(new_params)
+      end
+    end if params[:server].present?
+    head :ok # render empty body, status only
+  end
+
   def import_csv; end
 
   def import
@@ -96,18 +108,6 @@ def import
     end
   end
 
-  def destroy
-    respond_to do |format|
-      if @server.destroy
-        format.html { redirect_to servers_path(search_params), notice: t(".flashes.destroyed") }
-        format.json { head :no_content }
-      else
-        format.html { redirect_to servers_path(search_params), alert: t(".flashes.not_destroyed") }
-        format.json { head :bad_request }
-      end
-    end
-  end
-
   def duplicate
     @original_server = Server.friendly.find(params[:id].to_s.downcase)
     @server = @original_server.deep_dup
diff --git a/app/models/modele.rb b/app/models/modele.rb
index dfdf3c9c9..56b55ee3c 100644
--- a/app/models/modele.rb
+++ b/app/models/modele.rb
@@ -24,6 +24,8 @@ class Modele < ApplicationRecord
   scope :sorted, -> { order(:name) }
   scope :with_servers, -> { joins(:servers).uniq }
   scope :glpi_synchronizable, -> { where(category: Category.glpi_synchronizable) }
+  scope :no_pdus, -> { joins(:category).where("categories.name<>'Pdu'") }
+  scope :only_pdus, -> { joins(:category).where("categories.name='Pdu'").order(:name) }
 
   def self.all_sorted
     Modele.includes(:manufacturer).all.sort { |f1, f2| f1.name_with_brand.capitalize <=> f2.name_with_brand.capitalize }
diff --git a/app/models/server.rb b/app/models/server.rb
index e680d2951..8b1f559a6 100644
--- a/app/models/server.rb
+++ b/app/models/server.rb
@@ -40,23 +40,23 @@ class Server < ApplicationRecord
   validate :validate_network_types_values
 
   accepts_nested_attributes_for :cards,
-                                :allow_destroy => true,
-                                :reject_if => :all_blank
+                                allow_destroy: true,
+                                reject_if: :all_blank
   accepts_nested_attributes_for :disks,
-                                :allow_destroy => true,
-                                :reject_if => :all_blank
+                                allow_destroy: true,
+                                reject_if: :all_blank
   accepts_nested_attributes_for :memory_components,
-                                :allow_destroy => true,
-                                :reject_if => :all_blank
+                                allow_destroy: true,
+                                reject_if: :all_blank
   accepts_nested_attributes_for :documents,
-                                :allow_destroy => true,
-                                :reject_if => :all_blank
+                                allow_destroy: true,
+                                reject_if: :all_blank
 
   normalizes :network_types, with: ->(values) { values.compact_blank }
 
   before_create :set_default_network_types
 
-  scope :sorted, -> { order(:position => :desc) }
+  scope :sorted, -> { order(position: :desc) }
   scope :sorted_by_name, -> { order('LOWER(name) ASC') }
 
   scope :glpi_synchronizable, -> { joins(modele: :category).merge(Modele.glpi_synchronizable) }
diff --git a/app/views/layouts/_sidebar.html.erb b/app/views/layouts/_sidebar.html.erb
index 5c6b5c515..004c4dc33 100644
--- a/app/views/layouts/_sidebar.html.erb
+++ b/app/views/layouts/_sidebar.html.erb
@@ -45,7 +45,7 @@
           </li>
         </ul>
       </li>
-      <!-- Equipement -->
+      <!-- Equipments -->
       <li class="mydcim-sidebar-links-group py-2">
         <strong class="mydcim-sidebar-links-heading d-flex align-items-center fw-semibold">
           <i class="bi bi-hdd-rack me-2 text-primary"></i>
@@ -58,6 +58,12 @@
                         class: class_names("mydcim-sidebar-links-link d-inline-block rounded",
                                           active: controller.controller_name == "servers") %>
           </li>
+          <li>
+            <%= link_to t("power_distribution_units.index.title"),
+                        power_distribution_units_path,
+                        class: class_names("mydcim-sidebar-links-link d-inline-block rounded",
+                                          active: controller.controller_name == "power_distribution_units") %>
+          </li>
           <li>
             <%= link_to AirConditioner.model_name.human.pluralize,
                         air_conditioners_path,
diff --git a/app/views/power_distribution_units/_form.html.erb b/app/views/power_distribution_units/_form.html.erb
new file mode 100644
index 000000000..399edbfc4
--- /dev/null
+++ b/app/views/power_distribution_units/_form.html.erb
@@ -0,0 +1,167 @@
+<%
+  is_edit = controller.action_name == "edit"
+  is_new = !is_edit
+
+  dynamic_url = is_edit ? power_distribution_unit_path(@pdu) : power_distribution_units_path
+%>
+
+<%= form_for(@pdu, as: :power_distribution_unit,
+                   url: dynamic_url,
+                   html: { class: "col-12 col-md-10 col-lg-8 mx-auto", role: "form" }) do |f| %>
+  <%= render FormErrorsComponent.new(@pdu) %>
+
+  <%= render CardComponent.new(extra_classes: "bg-body-tertiary") do |card| %>
+    <div class="row g-3">
+      <fieldset class="col-md-6 col-12">
+        <%= f.label :name, class: "form-label" %>
+        <%= f.text_field :name, class: "form-control" %>
+      </fieldset>
+
+      <fieldset class="col-md-6 col-12 mt-md-3 mt-4">
+        <%= f.label :numero, class: "form-label" %>
+        <%= f.text_field :numero, class: "form-control" %>
+      </fieldset>
+
+      <fieldset class="col-md-6 col-12 mt-4">
+        <%= f.label :domaine_id, class: "form-label" %>
+        <%= f.collection_select(:domaine_id, Domaine.sorted, :id,
+                                                             :name,
+                                                             { prompt: true, include_blank: true },
+                                                             { class: "form-select",
+                                                               data: { controller: "select" } }) %>
+      </fieldset>
+
+      <fieldset class="col-md-6 col-12 mt-4">
+        <%= f.label :gestion_id, class: "form-label" %>
+        <%= f.collection_select(:gestion_id, Gestion.sorted, :id,
+                                                            :name,
+                                                            { prompt: true, include_blank: true },
+                                                            { class: "form-select",
+                                                              data: { controller: "select" } }) %>
+      </fieldset>
+
+      <fieldset class="col-md-6 col-12 mt-4">
+        <%= f.label :modele_id, class: "form-label" %>
+        <%= f.collection_select(:modele_id, Modele.only_pdus.sorted, :id,
+                                                                     :name_with_brand,
+                                                                     { prompt: true, include_blank: false },
+                                                                     { class: "form-select",
+                                                                       data: { controller: "select" } }) %>
+      </fieldset>
+
+      <fieldset class="col-md-6 col-12 mt-4">
+        <%= f.label :stack_id, class: "form-label" %>
+        <%= f.collection_select(:stack_id, Stack.order(:id), :id, :name, { prompt: true, include_blank: true },
+                                                                         { class: "form-select" }) %>
+      </fieldset>
+
+      <fieldset class="col-12 mt-4 align-self-center">
+        <div class="form-check">
+          <%= f.check_box :critique, class: "form-check-input" %>
+          <%= f.label :critique, class: "form-check-label" %>
+        </div>
+      </fieldset>
+    </div>
+  <% end %>
+
+  <%= render CardComponent.new(type: :primary, extra_classes: "mt-4 bg-body-tertiary") do |card| %>
+    <% card.with_header do %>
+      <%= t("layouts.sidebar.location.title") %>
+    <% end %>
+
+    <div class="row g-3">
+      <fieldset class="col-12">
+        <%= f.label :frame_id, class: "form-label" %>
+        <%= f.collection_select(:frame_id, Frame.all_sorted, :id,
+                                                            :name_with_room_and_islet,
+                                                            { prompt: true, include_blank: false },
+                                                            { class: "form-select", data: { controller: "select" } }) %>
+      </fieldset>
+    </div>
+  <% end %>
+
+  <!-- TODO: update -->
+  <% if @pdu.modele %>
+    <% @pdu.modele.enclosures.each do |enclosure| %>
+      <div class="col-12 mt-4" data-controller="nested-form">
+        <template data-nested-form-target="template">
+          <%= f.fields_for :cards, f.object.cards.klass.new, child_index: '__NEW_RECORD__' do |card_fields| %>
+            <%= render partial: "card_fields", locals: { f: card_fields, server: @pdu, enclosure: enclosure } %>
+          <% end %>
+        </template>
+
+        <%= render CardComponent.new(extra_classes: "bg-body") do |card| %>
+          <% card.with_header do %>
+            <%= "#{Enclosure.model_name.human} #{enclosure.position}" %>
+          <% end %>
+
+          <div class="row-gap-3 d-flex flex-column">
+            <% cards = @pdu.cards.select { |c| c.composant.enclosure == enclosure }
+                                    .sort_by! { |c| c.composant.position } %>
+            <%= f.fields_for :cards, cards do |card_fields| %>
+              <%= render partial: "card_fields", locals: { f: card_fields, server: @pdu, enclosure: enclosure } %>
+            <% end %>
+
+            <div data-nested-form-target="target" class="d-none"></div>
+          </div>
+
+          <% card.with_footer do %>
+            <button type="button" class="btn btn-outline-success btn-sm mx-auto" data-action="nested-form#add">
+              <span class="bi bi-plus-lg" aria-hidden="true"></span>
+              <span><%= t(".add_card") %></span>
+            </button>
+          <% end %>
+        <% end %>
+      </div>
+    <% end %>
+  <% end %>
+
+  <%= render CardComponent.new(extra_classes: "bg-body-tertiary mt-4") do |card| %>
+    <fieldset class="col-12">
+      <%= f.label :comment, class: "form-label" %>
+      <%= f.text_area :comment, class: "form-control" %>
+    </fieldset>
+  <% end %>
+
+  <fieldset class="col-12 mt-4" data-controller="nested-form">
+    <template data-nested-form-target="template">
+      <%= f.fields_for :documents, f.object.documents.klass.new, child_index: '__NEW_RECORD__' do |document_fields| %>
+        <%= render partial: "document_fields", locals: { f: document_fields } %>
+      <% end %>
+    </template>
+
+    <%= render CardComponent.new(extra_classes: "bg-body-tertiary") do |card| %>
+      <% card.with_header do %>
+        <%= f.label :documents %>
+      <% end %>
+
+      <%= f.fields_for :documents do |document_fields| %>
+        <%= render partial: "document_fields", locals: { f:  document_fields } %>
+      <% end %>
+
+      <div data-nested-form-target="target"></div>
+
+      <button type="button" class="btn btn-outline-success btn-sm" data-action="nested-form#add">
+        <span class="bi bi-plus-lg" aria-hidden="true"></span>
+        <span><%= t(".add_document") %></span>
+      </button>
+    <% end %>
+  </fieldset>
+
+  <%= render CardComponent.new(extra_classes: "bg-body-tertiary mt-4") do |card| %>
+    <fieldset class="col-12">
+      <%= f.label :photo, class: "form-label" %>
+      <%= f.file_field :photo, class: "form-control" %>
+
+      <% if @pdu.photo.attached? %>
+        <%= image_tag @pdu.photo.representation(resize_to_limit: [200, 200]), class: "ms-0 mt-2" %>
+      <% end %>
+    </fieldset>
+  <% end %>
+
+  <%#= render Form::ActionsComponent.new(f) %>
+  <div class="col-12 py-4 mt-4 text-end sticky-bottom bg-body-tertiary border-top">
+    <%= link_to(t("action.cancel"), dynamic_url, class: "btn btn-outline-secondary me-2") %>
+    <%= f.submit(class: class_names("btn", 'btn-info': is_edit, 'btn-success': is_new)) %>
+  </div>
+<% end %>
diff --git a/app/views/power_distribution_units/duplicate.html.erb b/app/views/power_distribution_units/duplicate.html.erb
new file mode 100644
index 000000000..7b333f880
--- /dev/null
+++ b/app/views/power_distribution_units/duplicate.html.erb
@@ -0,0 +1,26 @@
+<% provide(:title, "#{t("power_distribution_units.index.title")} | #{@original_pdu.name} | #{t("action.duplicate")}") %>
+<% breadcrumb_steps = {
+    t("power_distribution_units.index.title") => power_distribution_units_path,
+    @original_pdu.name => power_distribution_unit_path(@original_pdu),
+    t("action.duplicate") => ""
+} %>
+
+<%#= render Page::HeadingComponent.new(
+  resource: @original_pdu, title: t(".title", pdu_name: @original_pdu.name), breadcrumb_steps:
+) %>
+<%= render Page::HeadingComponent.new(
+  title: t(".title", pdu_name: @original_pdu.name), breadcrumb_steps:, back_button_url: power_distribution_units_path
+) do |heading| %>
+  <% heading.with_right_content do %>
+    <div class="align-self-center d-inline-flex">
+      <%= render ButtonComponent.new(t("action.show"),
+                                    url: power_distribution_unit_path(@original_pdu),
+                                    variant: :primary,
+                                    icon: "eye",
+                                    is_responsive: true) %>
+    </div>
+  <% end %>
+<% end %>
+<div class="col-12 p-4 border-top">
+  <%= render "form" %>
+</div>
diff --git a/app/views/power_distribution_units/edit.html.erb b/app/views/power_distribution_units/edit.html.erb
new file mode 100644
index 000000000..7c9d6cf03
--- /dev/null
+++ b/app/views/power_distribution_units/edit.html.erb
@@ -0,0 +1,23 @@
+<% provide(:title, "#{t("power_distribution_units.index.title")} | #{@pdu.name} | #{t(".title")}") %>
+<% breadcrumb_steps = {
+    t("power_distribution_units.index.title") => power_distribution_units_path, # TODO: prefer use ActiveRecord translation of Modele name
+    @pdu.name => power_distribution_unit_path(@pdu),
+    t("action.edit") => ""
+} %>
+
+<%= render Page::HeadingComponent.new(
+  title: t(".title"), breadcrumb_steps:, back_button_url: power_distribution_unit_path(@pdu)
+) do |heading| %>
+  <% heading.with_right_content do %>
+    <div class="align-self-center d-inline-flex">
+      <%= render ButtonComponent.new(t("action.show"),
+                                    url: power_distribution_unit_path(@pdu),
+                                    variant: :primary,
+                                    icon: "eye",
+                                    is_responsive: true) %>
+    </div>
+  <% end %>
+<% end %>
+<div class="col-12 p-4 border-top">
+  <%= render "form" %>
+</div>
diff --git a/app/views/power_distribution_units/index.html.erb b/app/views/power_distribution_units/index.html.erb
new file mode 100644
index 000000000..0ee0bb92b
--- /dev/null
+++ b/app/views/power_distribution_units/index.html.erb
@@ -0,0 +1,157 @@
+<% provide(:title, t(".title")) # TODO: prefer use ActiveRecord translation of Modele name %>
+
+<%= render Page::HeadingComponent.new(title: t(".title"), breadcrumb_steps: { t(".title") => "" } ) do |heading| %>
+  <% heading.with_right_content do %>
+    <%= render ButtonComponent.new(t(".new_power_distributon_unit"),
+                                    url: new_power_distribution_unit_path,
+                                    variant: :success,
+                                    icon: "plus-lg",
+                                    is_responsive: true) %>
+  <% end %>
+<% end %>
+
+<div class="p-4 border-top d-flex flex-column row-gap-4">
+  <%= render FilterComponent.new(@filter) do |c| %>
+    <% c.with_form do |f| %>
+      <div class="w-100">
+        <fieldset class="form-floating">
+          <%= f.text_field :q, class: "form-control", placeholder: t("filters.placeholder.search") %>
+          <%= f.label :q %>
+        </fieldset>
+      </div>
+
+      <div>
+        <fieldset class="form-floating">
+          <%= f.collection_select(:room_ids, Room.order(:name),
+                                  :id,
+                                  :name,
+                                  { prompt: true, multiple: true },
+                                  { class: "form-select", data: { controller: :select } }) %>
+          <%= f.label :room_ids %>
+        </fieldset>
+      </div>
+
+      <div>
+        <fieldset class="form-floating">
+          <%= f.collection_select(:islet_ids, Islet.sorted,
+                                  :id,
+                                  :name_with_room,
+                                  { prompt: true, multiple: true },
+                                  { class: "form-select", data: { controller: :select } }) %>
+          <%= f.label :islet_ids %>
+        </fieldset>
+      </div>
+
+      <div>
+        <fieldset class="form-floating">
+          <%= f.collection_select(:bay_ids, Bay.sorted,
+                                  :id,
+                                  :to_s,
+                                  { prompt: true, multiple: true },
+                                  { class: "form-select", data: { controller: :select } }) %>
+          <%= f.label :bay_ids %>
+        </fieldset>
+      </div>
+
+      <div>
+        <fieldset class="form-floating">
+          <%= f.collection_select(:frame_ids, Frame.order(:name),
+                                  :id,
+                                  :name,
+                                  { prompt: true, multiple: true },
+                                  { class: "form-select", data: { controller: :select } }) %>
+          <%= f.label :frame_ids %>
+        </fieldset>
+      </div>
+
+      <div>
+        <fieldset class="form-floating">
+          <%= f.collection_select(:modele_ids, Modele.only_pdus.sorted,
+                                  :id,
+                                  :name,
+                                  { prompt: true, multiple: true },
+                                  { class: "form-select", data: { controller: :select } }) %>
+          <%= f.label :modele_ids %>
+        </fieldset>
+      </div>
+
+      <div>
+        <fieldset class="form-floating">
+          <%= f.collection_select(:gestion_ids, Gestion.sorted,
+                                  :id,
+                                  :name,
+                                  { prompt: true, multiple: true },
+                                  { class: "form-select", data: { controller: :select } }) %>
+          <%= f.label :gestion_ids %>
+        </fieldset>
+      </div>
+
+      <div>
+        <fieldset class="form-floating">
+          <%= f.collection_select(:domaine_ids, Domaine.sorted,
+                                  :id,
+                                  :name,
+                                  { prompt: true, multiple: true },
+                                  { class: "form-select", data: { controller: :select } }) %>
+          <%= f.label :domaine_ids %>
+        </fieldset>
+      </div>
+    <% end %>
+  <% end %>
+
+  <%= turbo_frame_tag(dom_id(Server, :table), data: { turbo_action: :advance }) do %>
+    <div class="d-flex flex-column row-gap-4">
+      <%= render List::DataTableComponent.new(@pdus) do |table| %>
+        <% table.with_column(Server.human_attribute_name(:name), sort_by: :name) do |pdu| %>
+          <%= link_to pdu.name, power_distribution_unit_path(pdu), class: "fw-bold", data: { turbo_frame: :_top } %>
+        <% end %>
+
+        <% table.with_column(Server.human_attribute_name(:numero), sort_by: :numero) do |pdu| %>
+          <%= link_to pdu.numero, power_distribution_unit_path(pdu), class: "fw-bold", data: { turbo_frame: :_top } %>
+        <% end %>
+
+        <% table.with_column(Server.human_attribute_name(:room), sort_by: :"rooms.name") do |pdu| %>
+          <%= link_to pdu.room, room_path(pdu.room), data: { turbo_frame: :_top } if pdu.room %>
+        <% end %>
+
+        <% table.with_column(Islet.model_name.human, sort_by: :"islets.name") do |pdu| %>
+          <%= link_to pdu.islet, islet_path(pdu.islet), data: { turbo_frame: :_top } if pdu.islet %>
+        <% end %>
+
+        <% table.with_column(Bay.model_name.human, sort_by: :"bays.id") do |pdu| %>
+          <%= link_to pdu.bay, bay_path(pdu.bay), data: { turbo_frame: :_top } if pdu.bay %>
+        <% end %>
+
+        <% table.with_column(style: "min-width: 100px; width: 100px") do |pdu| %>
+          <div class="btn-group btn-group-sm" role="group" aria-label="...">
+            <%= link_to duplicate_power_distribution_unit_path(pdu), class: "btn btn-success", data: { turbo_frame: :_top } do %>
+              <span class="bi bi-copy" title="<%= t("action.duplicate") %>" aria-hidden="true"
+                                                                            data-controller="tooltip"
+                                                                            data-bs-placement="left"></span>
+              <span class="visually-hidden"><%= t("action.duplicate") %></span>
+            <% end %>
+            <%= link_to edit_power_distribution_unit_path(pdu), class: "btn btn-info", data: { turbo_frame: :_top } do %>
+              <span class="bi bi-pencil" title="<%= t("action.edit") %>" aria-hidden="true"
+                                                                        data-controller="tooltip"
+                                                                        data-bs-placement="left"></span>
+              <span class="visually-hidden"><%= t("action.edit") %></span>
+            <% end %>
+            <%= link_to power_distribution_unit_path(pdu, @search_params),
+                        method: :delete,
+                        data: { turbo_frame: :_top, confirm: t("action.confirm") },
+                        class: "btn btn-danger" do %>
+              <span class="bi bi-trash" title="<%= t("action.delete") %>" aria-hidden="true"
+                                                                          data-controller="tooltip"
+                                                                          data-bs-placement="left"></span>
+              <span class="visually-hidden"><%= t("action.delete") %></span>
+            <% end %>
+          </div>
+        <% end %>
+      <% end %>
+
+      <% if @pagy.pages > 1 %>
+        <%== pagy_bootstrap_nav(@pagy) %>
+      <% end %>
+    </div>
+  <% end %>
+</div>
diff --git a/app/views/power_distribution_units/index.json.jbuilder b/app/views/power_distribution_units/index.json.jbuilder
new file mode 100644
index 000000000..20474a8b7
--- /dev/null
+++ b/app/views/power_distribution_units/index.json.jbuilder
@@ -0,0 +1,7 @@
+# frozen_string_literal: true
+
+json.array!(@pdus) do |pdu|
+  json.extract! pdu, :id, :name, :modele_id,
+                :numero, :cluster, :critique
+  json.url power_distribution_unit_url(pdu, format: :json)
+end
diff --git a/app/views/power_distribution_units/new.html.erb b/app/views/power_distribution_units/new.html.erb
new file mode 100644
index 000000000..b6add08e4
--- /dev/null
+++ b/app/views/power_distribution_units/new.html.erb
@@ -0,0 +1,14 @@
+<% provide(:title, "#{t("power_distribution_units.index.title")} | #{t("action.create")}") # TODO: prefer use ActiveRecord translation of Modele name %>
+<% breadcrumb_steps = {
+  t("power_distribution_units.index.title") => power_distribution_units_path,
+  t("action.create") => ""
+} %>
+
+<%#= render Page::HeadingNewComponent.new(resource: @server, title: t(".title"), breadcrumb_steps:) %>
+<%= render Page::HeadingComponent.new(
+  title: t(".title"), breadcrumb_steps:, back_button_url: power_distribution_units_path
+) %>
+
+<div class="col-12 p-4 border-top">
+  <%= render "form" %>
+</div>
diff --git a/app/views/power_distribution_units/show.html.erb b/app/views/power_distribution_units/show.html.erb
new file mode 100644
index 000000000..fec2d8569
--- /dev/null
+++ b/app/views/power_distribution_units/show.html.erb
@@ -0,0 +1,356 @@
+<% title = @pdu.name %>
+<% provide(:title, "#{t("power_distribution_units.index.title")} | #{title}") %>
+
+<% breadcrumb_steps = {
+  t("power_distribution_units.index.title") => power_distribution_units_path, # TODO: prefer use ActiveRecord translation of Modele name
+  @pdu.name => ""
+} %>
+
+<%#<%= render Page::HeadingShowComponent.new(resource: @pdu, title: title, breadcrumb_steps:) do |heading| %>
+  <%#<% heading.with_extra_buttons do %>
+    <%#<%= render ButtonComponent.new(t("action.duplicate"),
+                                  url: duplicate_power_distribution_unit_path(@pdu.slug),
+                                  variant: :success,
+                                  icon: "copy",
+                                  is_responsive: true,
+                                  extra_classes: "me-2") %>
+  <% # end %>
+<% # end %>
+
+<%= render Page::HeadingComponent.new(
+  title: title, breadcrumb_steps:, back_button_url: power_distribution_units_path
+) do |heading| %>
+  <% heading.with_right_content do %>
+    <div class="align-self-center d-inline-flex">
+      <%= render ButtonComponent.new(t("action.duplicate"),
+                                    url: duplicate_power_distribution_unit_path(@pdu.slug),
+                                    variant: :success,
+                                    icon: "copy",
+                                    is_responsive: true,
+                                    extra_classes: "me-2") %>
+      <%= render ButtonComponent.new(t("action.edit"),
+                                    url: edit_power_distribution_unit_path(@pdu.slug),
+                                    variant: :info,
+                                    icon: "pencil",
+                                    is_responsive: true,
+                                    extra_classes: "me-2") %>
+    </div>
+  <% end %>
+<% end %>
+
+<div class="page-show__tabs" id="pduShowTabs">
+  <ul class="nav nav-tabs ps-4">
+    <li class="nav-item" role="presentation">
+      <button class="nav-link active"
+              id="details-tab"
+              data-bs-toggle="tab"
+              data-bs-target="#details-tab-pane"
+              type="button"
+              role="tab"
+              aria-controls="details-tab-pane"
+              aria-selected="true">
+        <%= t("show.tabs.details") %>
+      </button>
+    </li>
+    <li class="nav-item" role="presentation">
+      <button class="nav-link"
+              id="connections-tab"
+              data-bs-toggle="tab"
+              data-bs-target="#connections-tab-pane"
+              type="button"
+              role="tab"
+              aria-controls="connections-tab-pane"
+              aria-selected="true">
+        <%= Connection.model_name.human.pluralize %>
+      </button>
+    </li>
+  </ul>
+</div>
+<div class="col-12 p-4 tab-content" id="pduShowTabsContent">
+  <div class="tab-pane fade show active" id="details-tab-pane" role="tabpanel" aria-labelledby="details-tab" tabindex="0">
+    <div class="mb-4">
+      <% bgModeleColor = define_background_color(server: @pdu, mode: params["bg"]) %>
+      <div class="<%= class_names("rounded-top-1 border border-secondary-subtle p-2 mb-2",
+                                  colorized_background: bgModeleColor.present?) %>"
+            style="background-color:<%= bgModeleColor %> !important;">
+        <%= render partial: "servers/draw_server_compact_front", locals: { server: @pdu } %>
+      </div>
+
+      <div class="server rounded-bottom-1 bg-body border border-secondary-subtle p-2 mb-2">
+        <span id="back_server_<%= @pdu.id %>" class="no-decoration-link d-flex flex-row-reverse flex-wrap">
+            <%= render partial: "servers/draw_server_compact", locals: { server: @pdu } %>
+        </span>
+      </div>
+    </div>
+
+    <% if @pdu.comment.present? %>
+      <div class="mb-4">
+        <div class="alert alert-warning">
+          <%= @pdu.comment %>
+        </div>
+      </div>
+    <% end %>
+
+    <div class="row row-cols-1 row-cols-lg-2 row-gap-4">
+      <div class="col">
+        <%= render CardComponent.new do |card| %>
+          <dl class="show-page_dl d-grid row-gap-2 mb-0">
+            <dt class="pb-2"><%= Modele.human_attribute_name(:nb_elts) %></dt>
+            <dd class="mb-0 pb-2 ps-3"><%= @pdu.modele.try(:nb_elts) %></dd>
+
+            <dt class="pb-2"><%= Architecture.model_name.human %></dt>
+            <dd class="mb-0 pb-2 ps-3">
+              <% if @pdu.modele&.architecture.present? %>
+                <%= link_to @pdu.modele.architecture, architecture_path(@pdu.modele.architecture) %>
+              <% end %>
+            </dd>
+
+            <dt class="pb-2"><%= Manufacturer.model_name.human %></dt>
+            <dd class="mb-0 pb-2 ps-3">
+              <% if @pdu.modele&.manufacturer.present? %>
+                <%= link_to @pdu.modele.manufacturer, manufacturer_path(@pdu.modele.manufacturer) %>
+              <% end %>
+              <% if (documentation_url = @pdu.documentation_url).present? %>
+                (
+                  <%= link_to t(".documentation"), documentation_url, target: :_blank, rel: :noopener %>
+                )
+              <% end %>
+            </dd>
+
+            <dt class="pb-2"><%= Modele.model_name.human %></dt>
+            <dd class="mb-0 pb-2 ps-3">
+              <%= link_to @pdu.modele, modele_path(@pdu.modele) if @pdu.modele.try(:name) %>
+            </dd>
+
+            <dt class="pb-2"><%= Server.human_attribute_name(:numero) %></dt>
+            <dd class="mb-0 pb-2 ps-3"><%= @pdu.try(:numero) %></dd>
+
+            <dt class="pb-2"><%= Server.human_attribute_name(:critique) %></dt>
+            <dd class="mb-0 pb-2 ps-3">
+              <% if @pdu.critique.present? %>
+                <span class="<%= class_names("text-uppercase text-danger": @pdu.critique) %>">
+                  <%= t("boolean.#{@pdu.critique}") %>
+                </span>
+              <% end %>
+            </dd>
+
+            <dt class="pb-2"><%= Domaine.model_name.human %></dt>
+            <dd class="mb-0 pb-2 ps-3"><%= link_to_if @pdu.domaine, @pdu.domaine %></dd>
+
+            <dt class="pb-2"><%= Gestion.model_name.human %></dt>
+            <dd class="mb-0 pb-2 ps-3"><%= link_to_if @pdu.gestion, @pdu.gestion %></dd>
+
+            <dt class="pb-2"><%= Card.model_name.human.pluralize %></dt>
+            <dd class="mb-0 pb-2 ps-3">
+              <% @pdu.modele.composants.ordered.slots.each do |slot| %>
+                <% @pdu.cards.where(:composant_id => slot.id).each do |card| %>
+                  <div>
+                    <%= card.composant.name ? card.composant.name : card.composant.position %>
+                    :
+                    <%= "#{card.card_type.try(:name)} (#{card.card_type.try(:port_quantity)}#{card.card_type.try(:port_type).try(:name)})" %>
+                  </div>
+                <% end %>
+              <% end if @pdu.modele.present? %>
+            </dd>
+          </dl>
+        <% end %>
+
+        <% if @pdu.photo.attached? %>
+          <!-- Card shown on lg, in 2nd position (Grid isn't working as Masonry) -->
+          <%= render CardComponent.new(extra_classes: "mt-4 d-none d-lg-flex") do |card| %>
+            <% card.with_header do %>
+              <%= Server.human_attribute_name(:photo) %>
+            <% end %>
+
+            <%= link_to @pdu.photo, rel: :noopener, target: :_blank do %>
+              <%= image_tag @pdu.photo.representation(resize_to_limit: [1200, 1200]), class: "w-100" %>
+            <% end %>
+          <% end %>
+        <% end %>
+      </div>
+
+      <div class="col">
+        <%= render CardComponent.new(type: :primary) do |card| %>
+          <% card.with_header do %>
+            <%= t("layouts.sidebar.location.title") %>
+          <% end %>
+
+          <dl class="show-page_dl d-grid row-gap-2 mb-0">
+            <dt class="pb-2"><%= Site.model_name.human %></dt>
+            <dd class="mb-0 pb-2 ps-3"><%= link_to @pdu.room.site, site_path(@pdu.room.site) %></dd>
+
+            <dt class="pb-2"><%= Room.model_name.human %></dt>
+            <dd class="mb-0 pb-2 ps-3 d-flex justify-content-between">
+              <%= link_to @pdu.room.try(:name), @pdu.room %>
+              <%= link_to visualization_room_path(@pdu.room),
+                          class: "btn btn-primary btn-sm",
+                          data: { turbo_frame: :_top } do %>
+                <span class="bi bi-diagram-3-fill"
+                      aria-hidden="true"
+                      data-controller="tooltip"
+                      data-bs-placement="left"
+                      title="<%= t("visualization.title") %>"></span>
+                <span class="visually-hidden"><%= t("visualization.title") %></span>
+              <% end %>
+            </dd>
+
+            <dt class="pb-2"><%= Islet.model_name.human %></dt>
+            <dd class="mb-0 pb-2 ps-3 d-flex justify-content-between">
+              <%= link_to @pdu.islet.name_with_room, islet_path(@pdu.islet) %>
+              <%= link_to visualization_room_path(@pdu.room, islet: @pdu.islet.name),
+                          class: "btn btn-primary btn-sm",
+                          data: { turbo_frame: :_top } do %>
+                <span class="bi bi-diagram-3-fill"
+                      aria-hidden="true"
+                      data-controller="tooltip"
+                      data-bs-placement="left"
+                      title="<%= t("visualization.title") %>"></span>
+                <span class="visually-hidden"><%= t("visualization.title") %></span>
+              <% end %>
+            </dd>
+
+            <dt class="pb-2"><%= Bay.model_name.human %></dt>
+            <dd class="mb-0 pb-2 ps-3 d-flex justify-content-between">
+              <%= link_to @pdu.bay, bay_path(@pdu.bay) %>
+              <%= link_to visualization_bay_path(@pdu.bay),
+                          class: "btn btn-primary btn-sm",
+                          data: { turbo_frame: :_top } do %>
+                <span class="bi bi-diagram-3-fill"
+                      aria-hidden="true"
+                      data-controller="tooltip"
+                      data-bs-placement="left"
+                      title="<%= t("visualization.title") %>"></span>
+                <span class="visually-hidden"><%= t("visualization.title") %></span>
+              <% end %>
+            </dd>
+
+            <dt class="pb-2"><%= Frame.model_name.human %></dt>
+            <dd class="mb-0 pb-2 ps-3 d-flex justify-content-between">
+              <%= link_to @pdu.frame.try(:name), frame_path(@pdu.frame) %>
+              <%= link_to visualization_frame_path(@pdu.frame),
+                          class: "btn btn-primary btn-sm",
+                          data: { turbo_frame: :_top } do %>
+                <span class="bi bi-diagram-3-fill"
+                      aria-hidden="true"
+                      data-controller="tooltip"
+                      data-bs-placement="left"
+                      title="<%= t("visualization.title") %>"></span>
+                <span class="visually-hidden"><%= t("visualization.title") %></span>
+              <% end %>
+            </dd>
+          </dl>
+        <% end %>
+
+        <% if @pdu.photo.attached? %>
+          <!-- Card shown on mobile, in 3rd position (Grid isn't working as Masonry) -->
+          <%= render CardComponent.new(extra_classes: "d-lg-none mt-4") do |card| %>
+            <% card.with_header do %>
+              <%= Server.human_attribute_name(:photo) %>
+            <% end %>
+
+            <%= link_to @pdu.photo, rel: :noopener, target: :_blank do %>
+              <%= image_tag @pdu.photo.representation(resize_to_limit: [1200, 1200]), class: "w-100" %>
+            <% end %>
+          <% end %>
+        <% end %>
+
+        <%= render CardComponent.new(extra_classes: "mt-4") do |card| %>
+          <% card.with_header do %>
+            <%= t(".glpi_data") %>
+          <% end %>
+          <%
+            begin
+              Timeout.timeout(3) do
+                client = GlpiClient.new
+                computer = client.computer(serial: Rails.env.production? ? @pdu.numero : "AZERTY") %>
+              <% if computer.present? %>
+                <dl class="show-page_dl d-grid row-gap-2 mb-0">
+                  <dt class="pb-2"><%= t("glpi.computer.name") %></dt>
+                  <dd class="mb-0 pb-2 ps-3"><%= computer.name %></dd>
+
+                  <dt class="pb-2"><%= t("glpi.computer.id") %></dt>
+                  <dd class="mb-0 pb-2 ps-3"><%= computer.id %></dd>
+
+                  <dt class="pb-2"><%= t("glpi.computer.serial") %></dt>
+                  <dd class="mb-0 pb-2 ps-3"><%= computer.serial %></dd>
+
+                  <dt class="pb-2"><%= t("glpi.computer.contact") %></dt>
+                  <dd class="mb-0 pb-2 ps-3"><%= computer.contact %></dd>
+                </dl>
+                <dl class="show-page_dl d-grid row-gap-2 pt-3 border-top mb-0">
+                  <dt class="pb-2"><%= t("glpi.computer.disks_count") %></dt>
+                  <dd class="mb-0 pb-2 ps-3"><%= computer.hard_drives&.size %></dd>
+
+                  <dt class="pb-2"><%= t("glpi.computer.disks_capacity") %></dt>
+                  <dd class="mb-0 pb-2 ps-3"><%= number_to_human_size(computer.hard_drives_total_capacity * 1048576) %></dd>
+
+                  <dt class="pb-2"><%= t("glpi.computer.memories_count") %></dt>
+                  <dd class="mb-0 pb-2 ps-3"><%= computer.memories&.size %></dd>
+
+                  <dt class="pb-2"><%= t("glpi.computer.memories_total") %></dt>
+                  <dd class="mb-0 pb-2 ps-3"><%= number_to_human_size(computer.memories_total_size * 1048576) %></dd>
+
+                  <dt class="pb-2"><%= t("glpi.computer.cpus") %></dt>
+                  <dd class="mb-0 pb-2 ps-3">
+                    <%= computer.processors.group_by { |p| p[1]["designation"] }
+                                            .map { |designation, processors| "#{processors.size} x #{designation}" }
+                                            .join(", ") %>
+                  </dd>
+                </dl>
+                <% card.with_footer do %>
+                  <%= link_to t(".glpi_visit_page"),
+                              "#{Rails.application.credentials.glpi_url}/front/computer.form.php?id=#{computer.id}",
+                              target: :_blank %>
+                <% end %>
+              <% else %>
+                <%= t(".no_match_serial", serial: @pdu.numero) %>
+              <% end %>
+            <% end
+                rescue Timeout::Error => e
+                Rails.logger.warn "WARNING: couldn't get GLPI data because the operation timed out"
+                rescue Exception => e
+                Rails.logger.warn "WARNING: couldn't get GLPI data because of an error: #{e.message}" %>
+            <%= t(".glpi_connection_error") %> : <pre><code><%= e&.message.to_s %></code></pre>
+          <% end %>
+        <% end %>
+
+        <% if @pdu.documents.present? %>
+          <%= render CardComponent.new(extra_classes: "mt-4") do |card| %>
+            <% card.with_header do %>
+              <%= t(".attached_documents") %>
+            <% end %>
+            <ul>
+              <% @pdu.documents.each do |doc| %>
+                <%- next unless doc.document.present? %>
+                <li>
+                  <%= link_to(doc.document.metadata["filename"], doc.document_url, { target: :_blank })  %>
+                </li>
+              <% end %>
+            </ul>
+          <% end %>
+        <% end %>
+      </div>
+    </div>
+
+    <%= render ChangelogEntries::ObjectListComponent.new(@pdu) %>
+  </div>
+  <div class="tab-pane fade" id="connections-tab-pane" role="tabpanel" aria-labelledby="connections-tab" tabindex="0">
+    <%= render HasManyTurboFrameComponent.new(
+      Connection.model_name.human.pluralize,
+      url: connections_path(server_ids: @pdu.id),
+      frame_id: dom_id(Connection, :table),
+      extra_classes: "bg-body-tertiary"
+    ) do |c| %>
+      <% c.with_actions do %>
+        <%= render ButtonComponent.new(t(".destroy_connections"),
+                                          url: destroy_connections_server_path(@pdu),
+                                          icon: "trash",
+                                          variant: :danger,
+                                          size: :sm,
+                                          data: { confirm: t(".destroy_connections_confirmation") }) %>
+      <% end %>
+    <% end %>
+  </div>
+</div>
+
+<%= render "ports/modal_edit_port" %>
diff --git a/app/views/power_distribution_units/show.json.jbuilder b/app/views/power_distribution_units/show.json.jbuilder
new file mode 100644
index 000000000..bafaeef1f
--- /dev/null
+++ b/app/views/power_distribution_units/show.json.jbuilder
@@ -0,0 +1,8 @@
+# frozen_string_literal: true
+
+json.extract! @pdu, :id, :name, :modele, :numero, :critique, :domaine, :gestion, :created_at, :updated_at
+
+json.cards @pdu.cards do |card|
+  json.extract! card, :id, :name, :first_position, :orientation, :card_type_id, :composant_id, :twin_card_id
+  json.ports card.ports.order(:position), :id, :position, :vlans, :color, :cablename
+end
diff --git a/app/views/servers/index.html.erb b/app/views/servers/index.html.erb
index 20d6b2a9b..234403414 100644
--- a/app/views/servers/index.html.erb
+++ b/app/views/servers/index.html.erb
@@ -80,7 +80,7 @@
 
       <div>
         <fieldset class="form-floating">
-          <%= f.collection_select(:modele_ids, Modele.sorted,
+          <%= f.collection_select(:modele_ids, Modele.no_pdus.sorted,
                                   :id,
                                   :name,
                                   { prompt: true, multiple: true },
diff --git a/app/views/servers/show.html.erb b/app/views/servers/show.html.erb
index 179211b65..a0d8ec026 100644
--- a/app/views/servers/show.html.erb
+++ b/app/views/servers/show.html.erb
@@ -8,20 +8,18 @@
 
 <%= render Page::HeadingShowComponent.new(resource: @server, title: title, breadcrumb_steps:) do |heading| %>
   <% heading.with_extra_buttons do %>
-    <% if @server.is_not_a_pdu? %>
-      <%= render ButtonComponent.new(t("action.move"),
-                                    url: new_move_path(server_id: @server.id),
-                                    variant: :outline_primary,
-                                    icon: "box-seam",
-                                    is_responsive: true,
-                                    extra_classes: "me-2",
-                                    aria: { hidden: true },
-                                    data: {
-                                      controller: "tooltip",
-                                      bs_placement: :bottom,
-                                      tooltip_title: t(".moves_cta_tooltip")
-                                    }) %>
-    <% end %>
+    <%= render ButtonComponent.new(t("action.move"),
+                                  url: new_move_path(server_id: @server.id),
+                                  variant: :outline_primary,
+                                  icon: "box-seam",
+                                  is_responsive: true,
+                                  extra_classes: "me-2",
+                                  aria: { hidden: true },
+                                  data: {
+                                    controller: "tooltip",
+                                    bs_placement: :bottom,
+                                    tooltip_title: t(".moves_cta_tooltip")
+                                  }) %>
     <%= render ButtonComponent.new(t("action.duplicate"),
                                   url: duplicate_server_path(@server.slug),
                                   variant: :success,
@@ -105,7 +103,9 @@
           <dl class="show-page_dl d-grid row-gap-2 mb-0">
             <dt class="pb-2"><%= Modele.model_name.human %></dt>
             <dd class="mb-0 pb-2 ps-3">
-              <%= link_to @server.modele, modele_path(@server.modele) if @server.modele.try(:name) %>
+              <% if @server.modele&.architecture.present? %>
+                <%= link_to @server.modele.architecture, architecture_path(@server.modele.architecture) %>
+              <% end %>
             </dd>
 
             <dt class="pb-2"><%= Category.model_name.human %></dt>
@@ -113,7 +113,9 @@
 
             <dt class="pb-2"><%= Manufacturer.model_name.human %></dt>
             <dd class="mb-0 pb-2 ps-3">
-              <%= link_to @server.modele.try(:manufacturer), manufacturer_path(@server.modele.try(:manufacturer)) %>
+              <% if @server.modele&.manufacturer.present? %>
+                <%= link_to @server.modele.manufacturer, manufacturer_path(@server.modele.manufacturer) %>
+              <% end %>
               <% if (documentation_url = @server.documentation_url).present? %>
                 (
                   <%= link_to t(".documentation"), documentation_url, target: :_blank, rel: :noopener %>
@@ -148,9 +150,11 @@
 
             <dt class="pb-2"><%= Server.human_attribute_name(:critique) %></dt>
             <dd class="mb-0 pb-2 ps-3">
-              <span class="<%= class_names("text-uppercase text-danger": @server.critique) %>">
-                <%= t("boolean.#{@server.critique}") %>
-              </span>
+              <% if @server.critique.present? %>
+                <span class="<%= class_names("text-uppercase text-danger": @server.critique) %>">
+                  <%= t("boolean.#{@server.critique}") %>
+                </span>
+              <% end %>
             </dd>
 
             <dt class="pb-2"><%= ServerState.model_name.human %></dt>
diff --git a/config/locales/activerecord.fr.yml b/config/locales/activerecord.fr.yml
index d709db9b4..f6a87383b 100644
--- a/config/locales/activerecord.fr.yml
+++ b/config/locales/activerecord.fr.yml
@@ -110,6 +110,7 @@ fr:
         photo: Photo
         frame: Châssis
         span: Slots
+        u: Nb de U
       site:
         name: Nom
         description: Description
diff --git a/config/locales/en.yml b/config/locales/en.yml
index 8b5b761dd..4bb97f835 100644
--- a/config/locales/en.yml
+++ b/config/locales/en.yml
@@ -19,6 +19,43 @@ en:
       edit:
         title: Change your password
 
+  power_distribution_units:
+    index:
+      title: PDUs
+      new_power_distributon_unit: Add a PDU
+    show:
+      documentation: Documentation
+      attached_documents: "Attached documents:"
+      glpi_data: GLPI data
+      glpi_visit_page: See the PDU page on GLPI
+      no_match_serial: "No match for this serial number: %{serial}"
+      glpi_connection_error: "Problem connecting to GLPI :"
+      destroy_connections: Delete connections
+      destroy_connections_confirmation: Can you confirm that you want to remove all connections from this PDU?
+    new:
+      title: Add a PDU
+    create:
+      flashes:
+        created: The PDU has been successfully created.
+    form:
+      add_card: Add a card
+      add_document: Add an attached document
+    edit:
+      title: Modify PDU
+      delete_confirmation: All data concerning this PDU will be deleted, along with all associated connections. Would you like to confirm?
+    update:
+      flashes:
+        updated: The PDU has been successfully updated.
+    duplicate:
+      title: Duplicate PDU %{pdu_name}
+    destroy:
+      flashes:
+        destroyed: The PDU has been deleted.
+        not_destroyed: An error occurred while deleting the PDU.
+    flashes:
+      connections_destroyed: The connections have been removed.
+      connections_not_destroyed: An error prevented connections from being deleted.
+
   rooms:
     action_buttons:
       caption:
diff --git a/config/locales/fr.yml b/config/locales/fr.yml
index daec8f7a6..12f7042a3 100644
--- a/config/locales/fr.yml
+++ b/config/locales/fr.yml
@@ -483,7 +483,6 @@ fr:
       sync_with_external_apps: Synchroniser avec GLPI
     show:
       back_to_bay: Baie %{name}
-      nb_u: Nb de U
       documentation: Documentation
       attached_documents: "Documents attachés :"
       glpi_data: Données GLPI
@@ -530,6 +529,42 @@ fr:
       memories_count: Nombre de barettes
       memories_total: Ram totale
       cpus: Processeurs
+  power_distribution_units:
+    index:
+      title: PDUs
+      new_power_distributon_unit: Ajouter un PDU
+    show:
+      documentation: Documentation
+      attached_documents: "Documents attachés :"
+      glpi_data: Données GLPI
+      glpi_visit_page: Voir la page de la machine sur GLPI
+      no_match_serial: "Aucune correspondance pour ce numéro de série : %{serial}"
+      glpi_connection_error: "Problème lors de la connexion à GLPI :"
+      destroy_connections: Supprimer les connexions
+      destroy_connections_confirmation: Confirmez-vous vouloir supprimer toutes les connections de ce PDU ?
+    new:
+      title: Ajouter un PDU
+    create:
+      flashes:
+        created: Le PDU a été créé avec succès.
+    form:
+      add_card: Ajouter une carte
+      add_document: Ajouter une pièce jointe
+    edit:
+      title: Modifier le PDU
+      delete_confirmation: Toutes les données concernant ce PDU seront supprimées, ainsi que toutes les connexions associées. Souhaitez-vous confirmer ?
+    update:
+      flashes:
+        updated: Le PDU a été mis à jour avec succès.
+    duplicate:
+      title: Dupliquer le PDU %{pdu_name}
+    destroy:
+      flashes:
+        destroyed: Le PDU a bien été supprimé.
+        not_destroyed: Une erreur est survenue lors de la suppression du PDU.
+    flashes:
+      connections_destroyed: Les connexions ont été supprimées.
+      connections_not_destroyed: Une erreur a empêché la suppression des connexions.
   frames:
     coupled_frame_label: Châssis couplé à
     no_coupled_frame: Aucun châssis couplé.
diff --git a/config/routes.rb b/config/routes.rb
index 32a888efb..569731b1a 100644
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -61,6 +61,12 @@
       get :destroy_connections
     end
   end
+  resources :power_distribution_units do
+    member do
+      get :duplicate
+      get :destroy_connections
+    end
+  end
 
   resources :servers_grids, only: [:index]
   resources :card_types
diff --git a/spec/requests/power_distribution_units_request_spec.rb b/spec/requests/power_distribution_units_request_spec.rb
new file mode 100644
index 000000000..6b852dc3c
--- /dev/null
+++ b/spec/requests/power_distribution_units_request_spec.rb
@@ -0,0 +1,229 @@
+# frozen_string_literal: true
+
+require "rails_helper"
+
+RSpec.describe "/power_distribution_units" do
+  let(:pdu) { servers(:pdu) }
+  let(:pdu2) { Server.create(name: "PDU 2", frame_id: 2, modele_id: 3, numero: "PDU_2") }
+  let(:server) { servers(:one) }
+
+  before do
+    sign_in users(:one)
+
+    pdu.save
+    server.save
+    pdu2
+  end
+
+  describe "GET /index" do
+    before { get power_distribution_units_path }
+
+    it { expect(response).to have_http_status(:success) }
+    it { expect(response).to render_template(:index) }
+    it { expect(response.body).to include(pdu.name) }
+    it { expect(response.body).to include(pdu2.name) }
+    it { expect(response.body).not_to include(server.name) }
+
+    context "when searching on name" do
+      before { get power_distribution_units_path(q: "PDU_FRAME") }
+
+      it { expect(response).to have_http_status(:success) }
+      it { expect(response).to render_template(:index) }
+      it { expect(response.body).to include(pdu.name) }
+      it { expect(response.body).not_to include(pdu2.name) }
+      it { expect(response.body).not_to include(server.name) }
+    end
+  end
+
+  describe "GET /show" do
+    before { get power_distribution_unit_path(pdu) }
+
+    it { expect(response).to have_http_status(:success) }
+    it { expect(response).to render_template(:show) }
+    it { expect(response.body).to include(pdu.name) }
+
+    context "when using id" do
+      before { get power_distribution_unit_path(pdu.id) }
+
+      it { expect(response).to have_http_status(:success) }
+    end
+
+    context "when using name" do
+      before { get power_distribution_unit_path(pdu.name) }
+
+      it { expect(response).to render_template(:show) }
+    end
+
+    context "when using serial number" do
+      before { get power_distribution_unit_path(pdu.numero) }
+
+      it { expect(response).to render_template(:show) }
+    end
+
+    context "with an id not set" do
+      it { expect { get power_distribution_unit_path("unknown-id") }.to raise_error(ActiveRecord::RecordNotFound) }
+    end
+  end
+
+  describe "GET /new" do
+    before { get new_power_distribution_unit_path }
+
+    it { expect(response).to have_http_status(:success) }
+    it { expect(response).to render_template(:new) }
+  end
+
+  describe "GET /duplicate" do
+    before { get duplicate_power_distribution_unit_path(pdu) }
+
+    it { expect(response).to have_http_status(:success) }
+    it { expect(response).to render_template(:duplicate) }
+  end
+
+  describe "POST /create" do
+    context "with valid parameters" do
+      let(:valid_attributes) do
+        pdu.attributes.except(["id", "numero", "name"]).merge(numero: "new_numero", name: "NewServerName")
+      end
+
+      it "creates a new PDU" do
+        expect do
+          post power_distribution_units_path, params: { power_distribution_unit: valid_attributes }
+        end.to change(Server, :count).by(1)
+      end
+
+      it "redirects to the created pdu" do
+        post power_distribution_units_path, params: { power_distribution_unit: valid_attributes }
+        expect(response).to redirect_to(power_distribution_unit_path(assigns(:pdu)))
+      end
+    end
+
+    context "with invalid parameters" do
+      let(:invalid_attributes) { { name: "" } }
+
+      it "does not create a new PDU without attributes" do
+        expect { post power_distribution_units_path, params: { power_distribution_unit: {} } }
+          .to raise_error(ActionController::ParameterMissing)
+      end
+
+      it "does not create a new PDU without parameters" do
+        expect { post power_distribution_units_path, params: {} }
+          .to raise_error(ActionController::ParameterMissing)
+      end
+
+      it "does not create a new PDU with invalid parameters" do
+        post power_distribution_units_path, params: { power_distribution_unit: invalid_attributes }
+        expect(response).to render_template(:new)
+      end
+    end
+  end
+
+  describe "GET /edit" do
+    before { get edit_power_distribution_unit_path(pdu) }
+
+    it { expect(response).to have_http_status(:success) }
+    it { expect(response).to render_template(:edit) }
+  end
+
+  describe "PATCH /update" do
+    context "with valid parameters" do
+      let(:new_attributes) { pdu.attributes.except("name").merge(name: "New name") }
+
+      it :aggregate_failures do # rubocop:disable RSpec/ExampleLength
+        patch power_distribution_unit_path(pdu), params: { power_distribution_unit: new_attributes }
+        pdu.reload
+        assigns(:pdu).reload
+
+        expect(response).to be_redirection
+        expect(response).to redirect_to(power_distribution_unit_path(assigns(:pdu)))
+        expect(pdu.name).to eq(new_attributes[:name])
+
+        # old name should continue to work
+        get power_distribution_unit_path("PDU_FRAME-1_A")
+        expect(response).to have_http_status(:success)
+        expect(pdu).to eq(assigns(:pdu))
+      end
+
+      # it "does update cards in a pdu", :aggregate_failures do
+      #   patch power_distribution_unit_path(pdu), params: { power_distribution_unit: {
+      #                                                           cards_attributes: { id: 3,
+      #                                                             composant_id: 1,
+      #                                                             twin_card_id: 2,
+      #                                                             orientation: "lr-td" } } }
+
+      #   assigns(:pdu).reload
+      #   pdu.reload
+
+      #   expect(response).to be_redirection
+      #   expect(response).to redirect_to(power_distribution_unit_path(assigns(:pdu)))
+
+      #   # Test new card
+      #   get power_distribution_unit_path(pdu)
+      #   expect(response).to have_http_status(:success)
+      #   expect(pdu).to eq(assigns(:pdu))
+      #   expect(Card.find(1).twin_card_id).to eq 2
+
+      #   # Test twin card
+      #   expect(Card.find(2).twin_card_id).to eq 1
+      # end
+    end
+
+    context "with invalid parameters" do
+      it "does not update a PDU without attributes" do
+        expect { patch power_distribution_unit_path(pdu), params: { power_distribution_unit: {} } }
+          .to raise_error(ActionController::ParameterMissing)
+      end
+
+      it "does not update a PDU without parameters" do
+        expect { patch power_distribution_unit_path(pdu), params: {} }
+          .to raise_error(ActionController::ParameterMissing)
+      end
+
+      it "does not update a PDU with invalid parameters", :aggregate_failures do
+        patch power_distribution_unit_path(pdu), params: { power_distribution_unit: { name: "" } }
+
+        expect(response).to render_template(:edit)
+      end
+    end
+  end
+
+  describe "DELETE /destroy" do
+    context "with a pdu without association" do
+      it "destroys the requested pdu" do
+        expect do
+          delete power_distribution_unit_path(pdu2)
+        end.to change(Server, :count).by(-1)
+      end
+
+      it "redirects to the pdus list" do
+        delete power_distribution_unit_path(pdu2)
+        expect(response).to redirect_to(power_distribution_units_path)
+      end
+
+      it "redirects to the pdus list and keep params" do
+        delete power_distribution_unit_path(pdu2, params: { sort: "asc", sort_by: "rooms.name" })
+        expect(response).to redirect_to(power_distribution_units_path({ sort: "asc", sort_by: "rooms.name" }))
+      end
+    end
+
+    context "with a pdu with association" do
+      it "does not destroy the requested pdu" do
+        expect do
+          delete power_distribution_unit_path(pdu)
+        end.not_to change(Server, :count)
+      end
+
+      it "redirects to the pdus list" do
+        delete power_distribution_unit_path(pdu)
+        expect(response).to redirect_to(power_distribution_units_path)
+      end
+    end
+  end
+
+  describe "GET /destroy_connections" do
+    before { get destroy_connections_power_distribution_unit_path(pdu) }
+
+    it { expect(response).to have_http_status(:redirect) }
+    it { expect(pdu.connections.count).to be_zero }
+    it { expect(flash[:notice]).to be_present }
+  end
+end
diff --git a/spec/requests/servers_request_spec.rb b/spec/requests/servers_request_spec.rb
index 6a5c2f654..607d37f0c 100644
--- a/spec/requests/servers_request_spec.rb
+++ b/spec/requests/servers_request_spec.rb
@@ -5,12 +5,14 @@
 RSpec.describe "/servers" do
   let(:server) { servers(:one) }
   let(:server2) { servers(:two) }
+  let(:pdu) { servers(:pdu) }
 
   before do
     sign_in users(:one)
 
     server.save!
     server2.save!
+    pdu.save!
   end
 
   describe "GET /index" do
@@ -20,6 +22,7 @@
     it { expect(response).to render_template(:index) }
     it { expect(response.body).to include(server.name) }
     it { expect(response.body).to include(server2.name) }
+    it { expect(response.body).not_to include(pdu.name) }
 
     context "when searching on name" do
       before { get servers_path(q: "ServerName1") }
diff --git a/test/fixtures/documents.yml b/test/fixtures/documents.yml
index 66b70d7eb..a8277c87a 100644
--- a/test/fixtures/documents.yml
+++ b/test/fixtures/documents.yml
@@ -7,3 +7,7 @@ one:
 two:
   id: 2
   server_id: 1
+
+three:
+  id: 3
+  server_id: 3
diff --git a/test/fixtures/servers.yml b/test/fixtures/servers.yml
index df40b033e..4f02f1d1c 100644
--- a/test/fixtures/servers.yml
+++ b/test/fixtures/servers.yml
@@ -49,6 +49,7 @@ pdu:
   modele_id: 3
   side: right
   color: B
+  numero: PDU_1
 
 with_cluster:
   frame_id: 1

From 804ecefbef00a0d78ce1495672c907cbde242752 Mon Sep 17 00:00:00 2001
From: B_Rass <b.rasschaert@pantographe.studio>
Date: Wed, 22 Jan 2025 18:03:43 +0100
Subject: [PATCH 02/13] Fixing tests

---
 app/controllers/power_distribution_units_controller.rb | 2 +-
 app/views/power_distribution_units/_form.html.erb      | 8 ++++----
 test/fixtures/servers.yml                              | 2 +-
 test/models/server_test.rb                             | 7 +++++--
 4 files changed, 11 insertions(+), 8 deletions(-)

diff --git a/app/controllers/power_distribution_units_controller.rb b/app/controllers/power_distribution_units_controller.rb
index cb1bba77c..6dc784a24 100644
--- a/app/controllers/power_distribution_units_controller.rb
+++ b/app/controllers/power_distribution_units_controller.rb
@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-class PowerDistributionUnitsController < ServersController
+class PowerDistributionUnitsController < ApplicationController
   before_action :set_pdu, only: %i[show edit update destroy destroy_connections]
 
   def index
diff --git a/app/views/power_distribution_units/_form.html.erb b/app/views/power_distribution_units/_form.html.erb
index 399edbfc4..52e1898f5 100644
--- a/app/views/power_distribution_units/_form.html.erb
+++ b/app/views/power_distribution_units/_form.html.erb
@@ -86,7 +86,7 @@
       <div class="col-12 mt-4" data-controller="nested-form">
         <template data-nested-form-target="template">
           <%= f.fields_for :cards, f.object.cards.klass.new, child_index: '__NEW_RECORD__' do |card_fields| %>
-            <%= render partial: "card_fields", locals: { f: card_fields, server: @pdu, enclosure: enclosure } %>
+            <%= render partial: "servers/card_fields", locals: { f: card_fields, server: @pdu, enclosure: enclosure } %>
           <% end %>
         </template>
 
@@ -99,7 +99,7 @@
             <% cards = @pdu.cards.select { |c| c.composant.enclosure == enclosure }
                                     .sort_by! { |c| c.composant.position } %>
             <%= f.fields_for :cards, cards do |card_fields| %>
-              <%= render partial: "card_fields", locals: { f: card_fields, server: @pdu, enclosure: enclosure } %>
+              <%= render partial: "servers/card_fields", locals: { f: card_fields, server: @pdu, enclosure: enclosure } %>
             <% end %>
 
             <div data-nested-form-target="target" class="d-none"></div>
@@ -126,7 +126,7 @@
   <fieldset class="col-12 mt-4" data-controller="nested-form">
     <template data-nested-form-target="template">
       <%= f.fields_for :documents, f.object.documents.klass.new, child_index: '__NEW_RECORD__' do |document_fields| %>
-        <%= render partial: "document_fields", locals: { f: document_fields } %>
+        <%= render partial: "servers/document_fields", locals: { f: document_fields } %>
       <% end %>
     </template>
 
@@ -136,7 +136,7 @@
       <% end %>
 
       <%= f.fields_for :documents do |document_fields| %>
-        <%= render partial: "document_fields", locals: { f:  document_fields } %>
+        <%= render partial: "servers/document_fields", locals: { f:  document_fields } %>
       <% end %>
 
       <div data-nested-form-target="target"></div>
diff --git a/test/fixtures/servers.yml b/test/fixtures/servers.yml
index 4f02f1d1c..67e6fb96e 100644
--- a/test/fixtures/servers.yml
+++ b/test/fixtures/servers.yml
@@ -49,7 +49,7 @@ pdu:
   modele_id: 3
   side: right
   color: B
-  numero: PDU_1
+  numero: pdu_frame_1
 
 with_cluster:
   frame_id: 1
diff --git a/test/models/server_test.rb b/test/models/server_test.rb
index 1ee946ac2..75d232bf1 100644
--- a/test/models/server_test.rb
+++ b/test/models/server_test.rb
@@ -8,13 +8,16 @@ class ServerTest < ActiveSupport::TestCase
     refute_includes Server.only_pdus, servers(:one)
   end
 
-  test 'scope no_puds' do
+  test 'scope no_pdus' do
     refute_includes Server.no_pdus, servers(:pdu)
     assert_includes Server.no_pdus, servers(:one)
   end
 
   test '#documentation_url with no documentation_url value on manufacturer' do
-    assert_nil servers(:pdu).documentation_url
+    server = servers(:pdu)
+    server.numero = nil
+
+    assert_nil server.documentation_url
   end
 
   test '#documentation_url with no numero value on manufacturer' do

From 995f06a585e3411ac3f4de46bcd953384347945b Mon Sep 17 00:00:00 2001
From: B_Rass <b.rasschaert@pantographe.studio>
Date: Wed, 22 Jan 2025 18:09:32 +0100
Subject: [PATCH 03/13] Fix after rebase

---
 app/views/power_distribution_units/show.html.erb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/views/power_distribution_units/show.html.erb b/app/views/power_distribution_units/show.html.erb
index fec2d8569..ac1eaee80 100644
--- a/app/views/power_distribution_units/show.html.erb
+++ b/app/views/power_distribution_units/show.html.erb
@@ -337,7 +337,7 @@
   <div class="tab-pane fade" id="connections-tab-pane" role="tabpanel" aria-labelledby="connections-tab" tabindex="0">
     <%= render HasManyTurboFrameComponent.new(
       Connection.model_name.human.pluralize,
-      url: connections_path(server_ids: @pdu.id),
+      url: cables_path(server_ids: @pdu.id),
       frame_id: dom_id(Connection, :table),
       extra_classes: "bg-body-tertiary"
     ) do |c| %>

From bb342452c7f44bd6be1e5513a12e7b8dd42e71a5 Mon Sep 17 00:00:00 2001
From: B_Rass <b.rasschaert@pantographe.studio>
Date: Thu, 23 Jan 2025 18:59:33 +0100
Subject: [PATCH 04/13] Remove field stack on form

---
 app/views/power_distribution_units/_form.html.erb | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/app/views/power_distribution_units/_form.html.erb b/app/views/power_distribution_units/_form.html.erb
index 52e1898f5..fc701309f 100644
--- a/app/views/power_distribution_units/_form.html.erb
+++ b/app/views/power_distribution_units/_form.html.erb
@@ -49,12 +49,6 @@
                                                                        data: { controller: "select" } }) %>
       </fieldset>
 
-      <fieldset class="col-md-6 col-12 mt-4">
-        <%= f.label :stack_id, class: "form-label" %>
-        <%= f.collection_select(:stack_id, Stack.order(:id), :id, :name, { prompt: true, include_blank: true },
-                                                                         { class: "form-select" }) %>
-      </fieldset>
-
       <fieldset class="col-12 mt-4 align-self-center">
         <div class="form-check">
           <%= f.check_box :critique, class: "form-check-input" %>

From 8b2652864131bc4bb1b3b703156bb884a270fc27 Mon Sep 17 00:00:00 2001
From: B_Rass <b.rasschaert@pantographe.studio>
Date: Fri, 24 Jan 2025 11:08:52 +0100
Subject: [PATCH 05/13] Remove Critique & Domaine from PDUs

---
 .../power_distribution_units/_form.html.erb      | 16 ----------------
 .../power_distribution_units/index.html.erb      | 11 -----------
 app/views/power_distribution_units/show.html.erb | 12 ------------
 3 files changed, 39 deletions(-)

diff --git a/app/views/power_distribution_units/_form.html.erb b/app/views/power_distribution_units/_form.html.erb
index fc701309f..1f4548e08 100644
--- a/app/views/power_distribution_units/_form.html.erb
+++ b/app/views/power_distribution_units/_form.html.erb
@@ -22,15 +22,6 @@
         <%= f.text_field :numero, class: "form-control" %>
       </fieldset>
 
-      <fieldset class="col-md-6 col-12 mt-4">
-        <%= f.label :domaine_id, class: "form-label" %>
-        <%= f.collection_select(:domaine_id, Domaine.sorted, :id,
-                                                             :name,
-                                                             { prompt: true, include_blank: true },
-                                                             { class: "form-select",
-                                                               data: { controller: "select" } }) %>
-      </fieldset>
-
       <fieldset class="col-md-6 col-12 mt-4">
         <%= f.label :gestion_id, class: "form-label" %>
         <%= f.collection_select(:gestion_id, Gestion.sorted, :id,
@@ -48,13 +39,6 @@
                                                                      { class: "form-select",
                                                                        data: { controller: "select" } }) %>
       </fieldset>
-
-      <fieldset class="col-12 mt-4 align-self-center">
-        <div class="form-check">
-          <%= f.check_box :critique, class: "form-check-input" %>
-          <%= f.label :critique, class: "form-check-label" %>
-        </div>
-      </fieldset>
     </div>
   <% end %>
 
diff --git a/app/views/power_distribution_units/index.html.erb b/app/views/power_distribution_units/index.html.erb
index 0ee0bb92b..acd911416 100644
--- a/app/views/power_distribution_units/index.html.erb
+++ b/app/views/power_distribution_units/index.html.erb
@@ -85,17 +85,6 @@
           <%= f.label :gestion_ids %>
         </fieldset>
       </div>
-
-      <div>
-        <fieldset class="form-floating">
-          <%= f.collection_select(:domaine_ids, Domaine.sorted,
-                                  :id,
-                                  :name,
-                                  { prompt: true, multiple: true },
-                                  { class: "form-select", data: { controller: :select } }) %>
-          <%= f.label :domaine_ids %>
-        </fieldset>
-      </div>
     <% end %>
   <% end %>
 
diff --git a/app/views/power_distribution_units/show.html.erb b/app/views/power_distribution_units/show.html.erb
index ac1eaee80..ace0a33a5 100644
--- a/app/views/power_distribution_units/show.html.erb
+++ b/app/views/power_distribution_units/show.html.erb
@@ -125,18 +125,6 @@
             <dt class="pb-2"><%= Server.human_attribute_name(:numero) %></dt>
             <dd class="mb-0 pb-2 ps-3"><%= @pdu.try(:numero) %></dd>
 
-            <dt class="pb-2"><%= Server.human_attribute_name(:critique) %></dt>
-            <dd class="mb-0 pb-2 ps-3">
-              <% if @pdu.critique.present? %>
-                <span class="<%= class_names("text-uppercase text-danger": @pdu.critique) %>">
-                  <%= t("boolean.#{@pdu.critique}") %>
-                </span>
-              <% end %>
-            </dd>
-
-            <dt class="pb-2"><%= Domaine.model_name.human %></dt>
-            <dd class="mb-0 pb-2 ps-3"><%= link_to_if @pdu.domaine, @pdu.domaine %></dd>
-
             <dt class="pb-2"><%= Gestion.model_name.human %></dt>
             <dd class="mb-0 pb-2 ps-3"><%= link_to_if @pdu.gestion, @pdu.gestion %></dd>
 

From 8104713fd561393c8b12d5a01d087137bdc40ec7 Mon Sep 17 00:00:00 2001
From: B_Rass <b.rasschaert@pantographe.studio>
Date: Wed, 29 Jan 2025 18:19:35 +0100
Subject: [PATCH 06/13] Rebase rework

---
 app/views/servers/show.html.erb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/views/servers/show.html.erb b/app/views/servers/show.html.erb
index a0d8ec026..af56fe055 100644
--- a/app/views/servers/show.html.erb
+++ b/app/views/servers/show.html.erb
@@ -131,7 +131,7 @@
               <%= link_to @server.modele.try(:architecture), architecture_path(@server.modele.try(:architecture)) %>
             </dd>
 
-            <dt class="pb-2"><%= t(".nb_u") %></dt>
+            <dt class="pb-2"><%= Server.human_attribute_name(:u) %></dt>
             <dd class="mb-0 pb-2 ps-3"><%= @server.modele.try(:u) %></dd>
 
             <dt class="pb-2"><%= Server.human_attribute_name(:network_types) %></dt>

From a1df0c322c6ec49e80da07e265ff0ccade3e8db2 Mon Sep 17 00:00:00 2001
From: B_Rass <b.rasschaert@pantographe.studio>
Date: Thu, 30 Jan 2025 16:38:38 +0100
Subject: [PATCH 07/13] Fix regression: cable color css class on Cables#index

---
 app/views/cables/index.html.erb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/views/cables/index.html.erb b/app/views/cables/index.html.erb
index c7f53034b..f104e65ee 100644
--- a/app/views/cables/index.html.erb
+++ b/app/views/cables/index.html.erb
@@ -97,7 +97,7 @@
                 <%= cable.decorated.server_connected_with_link(from_connection, from: true) %>
                 <span class="d-inline-flex align-items-center justify-content-center col-4 col-xl-3">
                   <%= cable.decorated.draw_port(from_connection) %>
-                  <hr class="cable border #{color} border-t px-3 opacity-100">
+                  <hr class="cable border <%= cable.color %> border-t px-3 opacity-100">
                   <%= cable.decorated.draw_port(to_connection) %>
                 </span>
                 <%= cable.decorated.server_connected_with_link(to_connection) %>

From 34c9ca65eb2159d5656f6be212a7979de744981a Mon Sep 17 00:00:00 2001
From: B_Rass <b.rasschaert@pantographe.studio>
Date: Thu, 30 Jan 2025 17:25:48 +0100
Subject: [PATCH 08/13] Update PDU views with last blocks added

---
 .../power_distribution_units_controller.rb    | 22 ++---
 .../power_distribution_units/_form.html.erb   | 93 ++++++++++++++-----
 .../power_distribution_units/show.html.erb    | 79 +++++++++++-----
 app/views/servers/_form.html.erb              |  2 +-
 .../power_distribution_units_request_spec.rb  | 39 ++++----
 test/fixtures/cards.yml                       |  9 ++
 6 files changed, 166 insertions(+), 78 deletions(-)

diff --git a/app/controllers/power_distribution_units_controller.rb b/app/controllers/power_distribution_units_controller.rb
index 6dc784a24..76211e761 100644
--- a/app/controllers/power_distribution_units_controller.rb
+++ b/app/controllers/power_distribution_units_controller.rb
@@ -82,21 +82,21 @@ def destroy_connections
   private
 
   def set_pdu
-    @pdu = Server.includes(cards: [ports: [:connection, :cable], card_type: [:port_type]])
+    @pdu = Server.includes(cards: [ports: %i[connection cable], card_type: [:port_type]])
       .find_with_numero_or_friendly_id(params[:id].to_s.downcase)
   end
 
   def pdu_params
-    params.require(:power_distribution_unit)
-      .permit(:photo, :comment, :position, :frame_id, :gestion_id, :fc_futur, :rj45_cm, :name, :modele_id, :numero,
-              :critique, :domaine_id, :fc_total, :fc_utilise, :rj45_total, :rj45_utilise, :rj45_futur,
-              :ipmi_utilise, :ipmi_futur, :ipmi_dedie,
-              :frame, # TODO: Check if it should be removed or if it's used somewhere
-              cards_attributes: [
-                :composant_id, :card_type_id, :_destroy, :id, :twin_card_id, :orientation, :name,
-                :first_position,
-              ],
-              documents_attributes: [:document, :id, :_destroy])
+    params.expect(
+      power_distribution_unit: [
+        :photo, :comment, :position, :frame_id, :gestion_id, :fc_futur, :rj45_cm, :name, :modele_id, :numero, :critique,
+        :domaine_id, :fc_total, :fc_utilise, :rj45_total, :rj45_utilise, :rj45_futur, :ipmi_utilise, :ipmi_futur,
+        :ipmi_dedie,
+        :frame, # TODO: Check if it should be removed or if it's used somewhere
+        { cards_attributes: [%i[composant_id card_type_id twin_card_id orientation name first_position _destroy id]] },
+        { documents_attributes: [%i[document id _destroy]] },
+      ]
+    )
   end
 
   def search_params
diff --git a/app/views/power_distribution_units/_form.html.erb b/app/views/power_distribution_units/_form.html.erb
index 1f4548e08..8b5cf414e 100644
--- a/app/views/power_distribution_units/_form.html.erb
+++ b/app/views/power_distribution_units/_form.html.erb
@@ -11,33 +11,50 @@
   <%= render FormErrorsComponent.new(@pdu) %>
 
   <%= render CardComponent.new(extra_classes: "bg-body-tertiary") do |card| %>
+    <% card.with_header do %>
+      <%= t("show.cards.identification") %>
+    <% end %>
+
     <div class="row g-3">
-      <fieldset class="col-md-6 col-12">
+      <fieldset class="col-12">
         <%= f.label :name, class: "form-label" %>
         <%= f.text_field :name, class: "form-control" %>
       </fieldset>
+    </div>
+  <% end %>
 
-      <fieldset class="col-md-6 col-12 mt-md-3 mt-4">
-        <%= f.label :numero, class: "form-label" %>
-        <%= f.text_field :numero, class: "form-control" %>
-      </fieldset>
+  <%= render CardComponent.new(extra_classes: "mt-4 bg-body-tertiary") do |card| %>
+    <% card.with_header do %>
+      <%= Modele.model_name.human %>
+    <% end %>
 
-      <fieldset class="col-md-6 col-12 mt-4">
-        <%= f.label :gestion_id, class: "form-label" %>
-        <%= f.collection_select(:gestion_id, Gestion.sorted, :id,
-                                                            :name,
-                                                            { prompt: true, include_blank: true },
-                                                            { class: "form-select",
-                                                              data: { controller: "select" } }) %>
+    <div class="row g-3">
+      <fieldset class="col-12">
+        <%= f.label :modele_id, class: "form-label" %>
+        <%= f.collection_select(:modele_id,
+                                Modele.only_pdus.sorted,
+                                :id,
+                                :name_with_brand,
+                                { prompt: true, include_blank: false },
+                                {
+                                  class: "form-select",
+                                  data: { controller: "select" },
+                                  placeholder: t("select.placeholder")
+                                })
+        %>
       </fieldset>
+    </div>
+  <% end %>
 
-      <fieldset class="col-md-6 col-12 mt-4">
-        <%= f.label :modele_id, class: "form-label" %>
-        <%= f.collection_select(:modele_id, Modele.only_pdus.sorted, :id,
-                                                                     :name_with_brand,
-                                                                     { prompt: true, include_blank: false },
-                                                                     { class: "form-select",
-                                                                       data: { controller: "select" } }) %>
+  <%= render CardComponent.new(extra_classes: "mt-4 bg-body-tertiary") do |card| %>
+    <% card.with_header do %>
+      <%= t("show.cards.features") %>
+    <% end %>
+
+    <div class="row g-3">
+      <fieldset class="col-12">
+        <%= f.label :numero, class: "form-label" %>
+        <%= f.text_field :numero, class: "form-control" %>
       </fieldset>
     </div>
   <% end %>
@@ -50,10 +67,40 @@
     <div class="row g-3">
       <fieldset class="col-12">
         <%= f.label :frame_id, class: "form-label" %>
-        <%= f.collection_select(:frame_id, Frame.all_sorted, :id,
-                                                            :name_with_room_and_islet,
-                                                            { prompt: true, include_blank: false },
-                                                            { class: "form-select", data: { controller: "select" } }) %>
+        <%= f.collection_select(:frame_id,
+                                Frame.all_sorted,
+                                :id,
+                                :name_with_room_and_islet,
+                                { prompt: true, include_blank: false },
+                                {
+                                  class: "form-select",
+                                  data: { controller: "select" },
+                                  placeholder: t("select.placeholder")
+                                })
+        %>
+      </fieldset>
+    </div>
+  <% end %>
+
+  <%= render CardComponent.new(type: :success, extra_classes: "mt-4 bg-body-tertiary") do |card| %>
+    <% card.with_header do %>
+      <%= t("layouts.sidebar.exploitation.title") %>
+    <% end %>
+
+    <div class="row g-3">
+      <fieldset class="col-12">
+        <%= f.label :gestion_id, class: "form-label" %>
+        <%= f.collection_select(:gestion_id,
+                                Gestion.sorted,
+                                :id,
+                                :name,
+                                { prompt: true, include_blank: true },
+                                {
+                                  class: "form-select",
+                                  data: { controller: "select" },
+                                  placeholder: t("select.placeholder")
+                                })
+        %>
       </fieldset>
     </div>
   <% end %>
diff --git a/app/views/power_distribution_units/show.html.erb b/app/views/power_distribution_units/show.html.erb
index ace0a33a5..4a6cb5724 100644
--- a/app/views/power_distribution_units/show.html.erb
+++ b/app/views/power_distribution_units/show.html.erb
@@ -94,15 +94,25 @@
     <div class="row row-cols-1 row-cols-lg-2 row-gap-4">
       <div class="col">
         <%= render CardComponent.new do |card| %>
+          <% card.with_header do %>
+            <%= t("show.cards.identification") %>
+          <% end %>
+
           <dl class="show-page_dl d-grid row-gap-2 mb-0">
-            <dt class="pb-2"><%= Modele.human_attribute_name(:nb_elts) %></dt>
-            <dd class="mb-0 pb-2 ps-3"><%= @pdu.modele.try(:nb_elts) %></dd>
+            <dt class="pb-2"><%= Server.human_attribute_name(:name) %></dt>
+            <dd class="mb-0 pb-2 ps-3"><%= @pdu.name %></dd>
+          </dl>
+        <% end %>
 
-            <dt class="pb-2"><%= Architecture.model_name.human %></dt>
+        <%= render CardComponent.new(extra_classes: "mt-4") do |card| %>
+          <% card.with_header do %>
+            <%= Modele.model_name.human %>
+          <% end %>
+
+          <dl class="show-page_dl d-grid row-gap-2 mb-0">
+            <dt class="pb-2"><%= Modele.model_name.human %></dt>
             <dd class="mb-0 pb-2 ps-3">
-              <% if @pdu.modele&.architecture.present? %>
-                <%= link_to @pdu.modele.architecture, architecture_path(@pdu.modele.architecture) %>
-              <% end %>
+              <%= link_to @pdu.modele, modele_path(@pdu.modele) if @pdu.modele.try(:name) %>
             </dd>
 
             <dt class="pb-2"><%= Manufacturer.model_name.human %></dt>
@@ -117,17 +127,27 @@
               <% end %>
             </dd>
 
-            <dt class="pb-2"><%= Modele.model_name.human %></dt>
+            <dt class="pb-2"><%= Modele.human_attribute_name(:nb_elts) %></dt>
+            <dd class="mb-0 pb-2 ps-3"><%= @pdu.modele.try(:nb_elts) %></dd>
+
+            <dt class="pb-2"><%= Architecture.model_name.human %></dt>
             <dd class="mb-0 pb-2 ps-3">
-              <%= link_to @pdu.modele, modele_path(@pdu.modele) if @pdu.modele.try(:name) %>
+              <% if @pdu.modele&.architecture.present? %>
+                <%= link_to @pdu.modele.architecture, architecture_path(@pdu.modele.architecture) %>
+              <% end %>
             </dd>
+          </dl>
+        <% end %>
 
+        <%= render CardComponent.new(extra_classes: "mt-4") do |card| %>
+          <% card.with_header do %>
+            <%= t("show.cards.features") %>
+          <% end %>
+
+          <dl class="show-page_dl d-grid row-gap-2 mb-0">
             <dt class="pb-2"><%= Server.human_attribute_name(:numero) %></dt>
             <dd class="mb-0 pb-2 ps-3"><%= @pdu.try(:numero) %></dd>
 
-            <dt class="pb-2"><%= Gestion.model_name.human %></dt>
-            <dd class="mb-0 pb-2 ps-3"><%= link_to_if @pdu.gestion, @pdu.gestion %></dd>
-
             <dt class="pb-2"><%= Card.model_name.human.pluralize %></dt>
             <dd class="mb-0 pb-2 ps-3">
               <% @pdu.modele.composants.ordered.slots.each do |slot| %>
@@ -143,6 +163,17 @@
           </dl>
         <% end %>
 
+        <%= render CardComponent.new(type: :success, extra_classes: "mt-4") do |card| %>
+          <% card.with_header do %>
+            <%= t("layouts.sidebar.exploitation.title") %>
+          <% end %>
+
+          <dl class="show-page_dl d-grid row-gap-2 mb-0">
+            <dt class="pb-2"><%= Gestion.model_name.human %></dt>
+            <dd class="mb-0 pb-2 ps-3"><%= link_to_if @pdu.gestion, @pdu.gestion %></dd>
+          </dl>
+        <% end %>
+
         <% if @pdu.photo.attached? %>
           <!-- Card shown on lg, in 2nd position (Grid isn't working as Masonry) -->
           <%= render CardComponent.new(extra_classes: "mt-4 d-none d-lg-flex") do |card| %>
@@ -229,19 +260,6 @@
           </dl>
         <% end %>
 
-        <% if @pdu.photo.attached? %>
-          <!-- Card shown on mobile, in 3rd position (Grid isn't working as Masonry) -->
-          <%= render CardComponent.new(extra_classes: "d-lg-none mt-4") do |card| %>
-            <% card.with_header do %>
-              <%= Server.human_attribute_name(:photo) %>
-            <% end %>
-
-            <%= link_to @pdu.photo, rel: :noopener, target: :_blank do %>
-              <%= image_tag @pdu.photo.representation(resize_to_limit: [1200, 1200]), class: "w-100" %>
-            <% end %>
-          <% end %>
-        <% end %>
-
         <%= render CardComponent.new(extra_classes: "mt-4") do |card| %>
           <% card.with_header do %>
             <%= t(".glpi_data") %>
@@ -302,6 +320,19 @@
           <% end %>
         <% end %>
 
+        <% if @pdu.photo.attached? %>
+          <!-- Card shown on mobile, in 3rd position (Grid isn't working as Masonry) -->
+          <%= render CardComponent.new(extra_classes: "d-lg-none mt-4") do |card| %>
+            <% card.with_header do %>
+              <%= Server.human_attribute_name(:photo) %>
+            <% end %>
+
+            <%= link_to @pdu.photo, rel: :noopener, target: :_blank do %>
+              <%= image_tag @pdu.photo.representation(resize_to_limit: [1200, 1200]), class: "w-100" %>
+            <% end %>
+          <% end %>
+        <% end %>
+
         <% if @pdu.documents.present? %>
           <%= render CardComponent.new(extra_classes: "mt-4") do |card| %>
             <% card.with_header do %>
diff --git a/app/views/servers/_form.html.erb b/app/views/servers/_form.html.erb
index 03d31bfbe..e87a28e0e 100644
--- a/app/views/servers/_form.html.erb
+++ b/app/views/servers/_form.html.erb
@@ -23,7 +23,7 @@
       <fieldset class="col-md-6 col-12">
         <%= f.label :modele_id, class: "form-label" %>
         <%= f.collection_select(:modele_id,
-                                Modele.all_sorted,
+                                Modele.no_pdus.sorted,
                                 :id,
                                 :name_with_brand,
                                 { prompt: true, include_blank: false },
diff --git a/spec/requests/power_distribution_units_request_spec.rb b/spec/requests/power_distribution_units_request_spec.rb
index 6b852dc3c..a71f1a68a 100644
--- a/spec/requests/power_distribution_units_request_spec.rb
+++ b/spec/requests/power_distribution_units_request_spec.rb
@@ -82,7 +82,7 @@
   describe "POST /create" do
     context "with valid parameters" do
       let(:valid_attributes) do
-        pdu.attributes.except(["id", "numero", "name"]).merge(numero: "new_numero", name: "NewServerName")
+        pdu.attributes.except(%w[id numero name]).merge(numero: "new_numero", name: "NewServerName")
       end
 
       it "creates a new PDU" do
@@ -143,28 +143,29 @@
         expect(pdu).to eq(assigns(:pdu))
       end
 
-      # it "does update cards in a pdu", :aggregate_failures do
-      #   patch power_distribution_unit_path(pdu), params: { power_distribution_unit: {
-      #                                                           cards_attributes: { id: 3,
-      #                                                             composant_id: 1,
-      #                                                             twin_card_id: 2,
-      #                                                             orientation: "lr-td" } } }
+      it "does update cards in a server", :aggregate_failures do # rubocop:disable RSpec/ExampleLength
+        patch power_distribution_unit_path(pdu), params: {
+          power_distribution_unit: {
+            name: pdu.name,
+            cards_attributes: [{ id: 7, composant_id: 1, twin_card_id: 2, orientation: "lr-td" }]
+          }
+        }
 
-      #   assigns(:pdu).reload
-      #   pdu.reload
+        assigns(:pdu).reload
+        pdu.reload
 
-      #   expect(response).to be_redirection
-      #   expect(response).to redirect_to(power_distribution_unit_path(assigns(:pdu)))
+        expect(response).to be_redirection
+        expect(response).to redirect_to(power_distribution_unit_path(assigns(:pdu)))
 
-      #   # Test new card
-      #   get power_distribution_unit_path(pdu)
-      #   expect(response).to have_http_status(:success)
-      #   expect(pdu).to eq(assigns(:pdu))
-      #   expect(Card.find(1).twin_card_id).to eq 2
+        # Test new card
+        get power_distribution_unit_path(pdu)
+        expect(response).to have_http_status(:success)
+        expect(pdu).to eq(assigns(:pdu))
+        expect(Card.find(7).twin_card_id).to eq 2
 
-      #   # Test twin card
-      #   expect(Card.find(2).twin_card_id).to eq 1
-      # end
+        # Test twin card
+        expect(Card.find(2).twin_card_id).to eq 7
+      end
     end
 
     context "with invalid parameters" do
diff --git a/test/fixtures/cards.yml b/test/fixtures/cards.yml
index 566512b0b..532629a4c 100644
--- a/test/fixtures/cards.yml
+++ b/test/fixtures/cards.yml
@@ -51,3 +51,12 @@ six:
   name: CardIPMI
   twin_card_id:
   orientation: "lr-td"
+
+seven:
+  id: 7
+  card_type_id: 4
+  server_id: 3
+  composant_id: 4
+  name: CardAlim
+  twin_card_id:
+  orientation: "lr-td"

From c5076ef829286f3f16262c5e4087d5ad2da87d23 Mon Sep 17 00:00:00 2001
From: B_Rass <b.rasschaert@pantographe.studio>
Date: Wed, 5 Feb 2025 15:55:20 +0100
Subject: [PATCH 09/13] Review rework

---
 app/controllers/power_distribution_units_controller.rb | 3 +--
 app/views/servers/show.html.erb                        | 4 ++--
 spec/requests/servers_request_spec.rb                  | 2 --
 test/fixtures/servers.yml                              | 1 +
 4 files changed, 4 insertions(+), 6 deletions(-)

diff --git a/app/controllers/power_distribution_units_controller.rb b/app/controllers/power_distribution_units_controller.rb
index 76211e761..688652504 100644
--- a/app/controllers/power_distribution_units_controller.rb
+++ b/app/controllers/power_distribution_units_controller.rb
@@ -82,8 +82,7 @@ def destroy_connections
   private
 
   def set_pdu
-    @pdu = Server.includes(cards: [ports: %i[connection cable], card_type: [:port_type]])
-      .find_with_numero_or_friendly_id(params[:id].to_s.downcase)
+    @pdu = Server.find_with_numero_or_friendly_id(params[:id].to_s.downcase)
   end
 
   def pdu_params
diff --git a/app/views/servers/show.html.erb b/app/views/servers/show.html.erb
index af56fe055..78572a6cc 100644
--- a/app/views/servers/show.html.erb
+++ b/app/views/servers/show.html.erb
@@ -103,8 +103,8 @@
           <dl class="show-page_dl d-grid row-gap-2 mb-0">
             <dt class="pb-2"><%= Modele.model_name.human %></dt>
             <dd class="mb-0 pb-2 ps-3">
-              <% if @server.modele&.architecture.present? %>
-                <%= link_to @server.modele.architecture, architecture_path(@server.modele.architecture) %>
+              <% if @server.modele.present? %>
+                <%= link_to @server.modele, modele_path(@server.modele) %>
               <% end %>
             </dd>
 
diff --git a/spec/requests/servers_request_spec.rb b/spec/requests/servers_request_spec.rb
index 607d37f0c..da93cb79a 100644
--- a/spec/requests/servers_request_spec.rb
+++ b/spec/requests/servers_request_spec.rb
@@ -11,8 +11,6 @@
     sign_in users(:one)
 
     server.save!
-    server2.save!
-    pdu.save!
   end
 
   describe "GET /index" do
diff --git a/test/fixtures/servers.yml b/test/fixtures/servers.yml
index 67e6fb96e..5be5ec9ec 100644
--- a/test/fixtures/servers.yml
+++ b/test/fixtures/servers.yml
@@ -4,6 +4,7 @@ one:
   id: 1
   frame_id: 1
   name: ServerName1
+  slug: servername1
   modele_id: 1
   numero: CZ31535FEY
   critique: false

From 5a6000f0d79a07f0293102b4e55b23f42ee863c2 Mon Sep 17 00:00:00 2001
From: B_Rass <b.rasschaert@pantographe.studio>
Date: Wed, 5 Feb 2025 16:02:01 +0100
Subject: [PATCH 10/13] Fixing tests

---
 test/controllers/disks_controller_test.rb             | 4 ++--
 test/controllers/memory_components_controller_test.rb | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/test/controllers/disks_controller_test.rb b/test/controllers/disks_controller_test.rb
index 6da815ee7..f080c2735 100644
--- a/test/controllers/disks_controller_test.rb
+++ b/test/controllers/disks_controller_test.rb
@@ -24,7 +24,7 @@ class DisksControllerTest < ActionController::TestCase
       post :create, params: { disk: { disk_type_id: @disk.disk_type_id, quantity: @disk.quantity, server_id: @disk.server_id } }
     end
 
-    assert_redirected_to server_path(@disk.server_id)
+    assert_redirected_to server_path(@disk.server)
   end
 
   test "should show disk" do
@@ -39,7 +39,7 @@ class DisksControllerTest < ActionController::TestCase
 
   test "should update disk" do
     patch :update, params: { id: @disk, disk: { disk_type_id: @disk.disk_type_id, quantity: @disk.quantity, server_id: @disk.server_id } }
-    assert_redirected_to server_path(@disk.server_id)
+    assert_redirected_to server_path(@disk.server)
   end
 
   test "should destroy disk" do
diff --git a/test/controllers/memory_components_controller_test.rb b/test/controllers/memory_components_controller_test.rb
index c7428948a..b3cac1f67 100644
--- a/test/controllers/memory_components_controller_test.rb
+++ b/test/controllers/memory_components_controller_test.rb
@@ -24,7 +24,7 @@ class MemoryComponentsControllerTest < ActionController::TestCase
       post :create, params: { memory_component: { memory_type_id: @memory_component.memory_type_id, quantity: @memory_component.quantity, server_id: @memory_component.server_id } }
     end
 
-    assert_redirected_to server_path(@memory_component.server_id)
+    assert_redirected_to server_path(@memory_component.server)
   end
 
   test "should show memory_component" do
@@ -39,7 +39,7 @@ class MemoryComponentsControllerTest < ActionController::TestCase
 
   test "should update memory_component" do
     patch :update, params: { id: @memory_component, memory_component: { memory_type_id: @memory_component.memory_type_id, quantity: @memory_component.quantity, server_id: @memory_component.server_id } }
-    assert_redirected_to server_path(@memory_component.server_id)
+    assert_redirected_to server_path(@memory_component.server)
   end
 
   test "should destroy memory_component" do

From 6277b1f5a38ac190bbc2e95f5b51a7203bb5b8b9 Mon Sep 17 00:00:00 2001
From: B_Rass <b.rasschaert@pantographe.studio>
Date: Wed, 5 Feb 2025 17:37:14 +0100
Subject: [PATCH 11/13] Rebase changes

---
 app/controllers/power_distribution_units_controller.rb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/controllers/power_distribution_units_controller.rb b/app/controllers/power_distribution_units_controller.rb
index 688652504..48571091e 100644
--- a/app/controllers/power_distribution_units_controller.rb
+++ b/app/controllers/power_distribution_units_controller.rb
@@ -82,7 +82,7 @@ def destroy_connections
   private
 
   def set_pdu
-    @pdu = Server.find_with_numero_or_friendly_id(params[:id].to_s.downcase)
+    @pdu = Server.friendly_find_by_numero_or_name(params[:id].to_s.downcase)
   end
 
   def pdu_params

From fd07ad1f148539220feae4de5e7377bd760cc43f Mon Sep 17 00:00:00 2001
From: B_Rass <b.rasschaert@pantographe.studio>
Date: Wed, 5 Feb 2025 17:42:47 +0100
Subject: [PATCH 12/13] Review rework

---
 app/views/power_distribution_units/_form.html.erb | 2 +-
 app/views/servers/_form.html.erb                  | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/app/views/power_distribution_units/_form.html.erb b/app/views/power_distribution_units/_form.html.erb
index 8b5cf414e..e4680c28e 100644
--- a/app/views/power_distribution_units/_form.html.erb
+++ b/app/views/power_distribution_units/_form.html.erb
@@ -32,7 +32,7 @@
       <fieldset class="col-12">
         <%= f.label :modele_id, class: "form-label" %>
         <%= f.collection_select(:modele_id,
-                                Modele.only_pdus.sorted,
+                                Modele.only_pdus.all_sorted,
                                 :id,
                                 :name_with_brand,
                                 { prompt: true, include_blank: false },
diff --git a/app/views/servers/_form.html.erb b/app/views/servers/_form.html.erb
index e87a28e0e..c78362cd7 100644
--- a/app/views/servers/_form.html.erb
+++ b/app/views/servers/_form.html.erb
@@ -23,7 +23,7 @@
       <fieldset class="col-md-6 col-12">
         <%= f.label :modele_id, class: "form-label" %>
         <%= f.collection_select(:modele_id,
-                                Modele.no_pdus.sorted,
+                                Modele.no_pdus.all_sorted,
                                 :id,
                                 :name_with_brand,
                                 { prompt: true, include_blank: false },

From 572c7965d48a56653b3e81e9e4b3da780ddc0bc1 Mon Sep 17 00:00:00 2001
From: B-Rass <B-Rass@users.noreply.github.com>
Date: Wed, 5 Feb 2025 17:59:03 +0100
Subject: [PATCH 13/13] Update
 app/controllers/power_distribution_units_controller.rb

Co-authored-by: Nicolas Brousse <nicolas-brousse@users.noreply.github.com>
---
 app/controllers/power_distribution_units_controller.rb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/controllers/power_distribution_units_controller.rb b/app/controllers/power_distribution_units_controller.rb
index 48571091e..bd7b1666b 100644
--- a/app/controllers/power_distribution_units_controller.rb
+++ b/app/controllers/power_distribution_units_controller.rb
@@ -82,7 +82,7 @@ def destroy_connections
   private
 
   def set_pdu
-    @pdu = Server.friendly_find_by_numero_or_name(params[:id].to_s.downcase)
+    @pdu = Server.friendly_find_by_numero_or_name(params[:id])
   end
 
   def pdu_params