Move group members' table to a partial

The show file is a bit more readable now.
openSUSE · Dec 7, 2018 · cdebf34 · cdebf34
1 parent 7a4d1dc
commit cdebf34
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 23 deletions.
diff --git a/src/api/app/views/webui2/shared/_group_members_table.html.haml b/src/api/app/views/webui2/shared/_group_members_table.html.haml
@@ -0,0 +1,23 @@
+.obs-dataTable
+  %table.table.table-striped.table-bordered.w-100#group-members-table
+    %thead
+      %tr
+        %th Login Name
+        %th
+    %tbody
+      - if group.users.any?
+        - group.users.each do |user|
+          %tr
+            %td= link_to(user, user_show_path(user))
+            %td.text-nowrap
+              = link_to('#', data: { toggle: 'modal', target: "#delete-group-user-#{user.id}-modal" }) do
+                %i.fas.fa-times-circle.text-danger{ title: "Remove user '#{user}' from '#{group}' group." }
+              = render(partial: 'remove_user_modal', locals: { group: group, user: user })
+      - else
+        %tr
+          %td{ colspan: 2 } This group does not contain users.
+  - if policy(group).update?
+    = link_to('#', class: 'nav-link', data: { toggle: 'modal', target: '#add-user-modal' }, id: 'add-user') do
+      %i.fas.fa-plus-circle.text-primary
+      Add User
+    = render(partial: 'add_user_modal', locals: { group: group })
diff --git a/src/api/app/views/webui2/webui/groups/show.html.haml b/src/api/app/views/webui2/webui/groups/show.html.haml
@@ -36,29 +36,7 @@
 
       .tab-pane#group-members{ aria: { controls: 'group-members' }, role: 'tabpanel' }
         %h3 Group Members
-        .obs-dataTable
-          %table.table.table-striped.table-bordered.w-100#group-members-table
-            %thead
-              %tr
-                %th Login Name
-                %th
-            %tbody
-              - if @group.users.any?
-                - @group.users.each do |user|
-                  %tr
-                    %td= link_to(user, user_show_path(user))
-                    %td.text-nowrap
-                      = link_to('#', data: { toggle: 'modal', target: "#delete-group-user-#{user.id}-modal" }) do
-                        %i.fas.fa-times-circle.text-danger{ title: "Remove user '#{user}' from '#{@group}' group." }
-                      = render(partial: 'remove_user_modal', locals: { group: @group, user: user })
-              - else
-                %tr
-                  %td{ colspan: 2 } This group does not contain users.
-          - if policy(@group).update?
-            = link_to('#', class: 'nav-link', data: { toggle: 'modal', target: '#add-user-modal' }, id: 'add-user') do
-              %i.fas.fa-plus-circle.text-primary
-              Add User
-            = render(partial: 'add_user_modal', locals: { group: @group })
+        = render(partial: 'webui2/shared/group_members_table', locals: { group: @group })
 
 = content_for :ready_function do
   :plain