From d46ca0e46f9964d471be8d20d907077a2cb8f6be Mon Sep 17 00:00:00 2001
From: Go MAEDA <maeda@farend.jp>
Date: Fri, 6 Sep 2024 02:57:40 +0000
Subject: [PATCH] Fix broken project list layout for projects scheduled for
 deletion (#41217).

Patch by Liane Hampe (user:liane_hampe).


git-svn-id: https://svn.redmine.org/redmine/trunk@23026 e93f8b46-1217-0410-a6f0-8f06a7374b81
---
 app/views/projects/_list.html.erb | 16 ++++++++++++----
 test/integration/projects_test.rb | 18 ++++++++++++++++++
 2 files changed, 30 insertions(+), 4 deletions(-)

diff --git a/app/views/projects/_list.html.erb b/app/views/projects/_list.html.erb
index 03c5ce059..f5eae37a6 100644
--- a/app/views/projects/_list.html.erb
+++ b/app/views/projects/_list.html.erb
@@ -41,14 +41,22 @@
     </tr>
   <% end %>
   <tr id="project-<%= entry.id %>" class="<%= cycle('odd', 'even') %> hascontextmenu <%= entry.css_classes %> <%= level > 0 ? "idnt idnt-#{level}" : nil %>">
-    <% if @admin_list && !entry.scheduled_for_deletion? %>
-      <td class="checkbox hide-when-print"><%= check_box_tag("ids[]", entry.id, false, :id => nil) %></td>
+    <% if @admin_list %>
+      <% if !entry.scheduled_for_deletion? %>
+        <td class="checkbox hide-when-print"><%= check_box_tag("ids[]", entry.id, false, :id => nil) %></td>
+      <% else %>
+        <td class="checkbox hide-when-print"></td>
+      <% end %>
     <% end %>
     <% @query.inline_columns.each do |column| %>
     <%= content_tag('td', column_content(column, entry), :class => column.css_classes) %>
     <% end %>
-    <% if @admin_list && !entry.scheduled_for_deletion? %>
-      <td class="buttons"><%= link_to_context_menu %></td>
+    <% if @admin_list %>
+      <% if !entry.scheduled_for_deletion? %>
+        <td class="buttons"><%= link_to_context_menu %></td>
+      <% else %>
+        <td class="buttons"></td>
+      <% end %>
     <% end %>
   </tr>
 <% end -%>
diff --git a/test/integration/projects_test.rb b/test/integration/projects_test.rb
index 041253230..3da9da707 100644
--- a/test/integration/projects_test.rb
+++ b/test/integration/projects_test.rb
@@ -52,4 +52,22 @@ class ProjectsTest < Redmine::IntegrationTest
       assert_response :not_found
     end
   end
+
+  def test_list_layout_when_show_projects_scheduled_for_deletion
+    project = Project.find(1)
+    project.update_attribute :status, Project::STATUS_SCHEDULED_FOR_DELETION
+
+    log_user('admin', 'admin')
+
+    get '/admin/projects', :params => { :f => ['status'], :v => { 'status' => ['10'] } }
+    assert_response :success
+
+    assert_select '#project-1' do
+      assert_select 'td.checkbox.hide-when-print'
+      assert_select 'td.name'
+      assert_select 'td.identifier'
+      assert_select 'td.short_description'
+      assert_select 'td.buttons', text: ''
+    end
+  end
 end