Refactoring and adding user_groups to tenant#show #105

1 files changed, 25 insertions, 21 deletions

diff --git a/app/views/user_groups/_index_core.html.haml b/app/views/user_groups/_index_core.html.haml
index d2b6e88..481a507 100644
--- a/app/views/user_groups/_index_core.html.haml
+++ b/app/views/user_groups/_index_core.html.haml
@@ -1,24 +1,28 @@
-%table
-  %tr
-    %th= t('user_groups.index.name')
-    %th= t('user_groups.index.description')
-    - if @user
-      %th= t('user_groups.index.tenant_id')
-    - else
-      %th= t('user_groups.index.members')
-
-  - reset_cycle
-  - for user_group in user_groups
-    %tr{:class => cycle('odd', 'even')}
-      %td= user_group.name
-      %td= user_group.description
+- cache(['user_groups_table', I18n.locale, current_user, @user, user_groups, User.order(:updated_at).last, User.count, UserGroupMembership.count, GsParameter.get('NUMBER_OF_SHOWN_ITEMS')]) do
+  %table
+    %tr
+      %th= t('user_groups.index.name')
+      - if user_groups.pluck(:description).uniq != [nil]
+        %th= t('user_groups.index.description')
       - if @user
-        %td= user_group.tenant
+        %th= t('user_groups.index.tenant_id')
       - else
-        %td
-          =render 'users/listing', :users => user_group.users
-          - if user_group.users.count > 1
-            %br
-          = render :partial => 'shared/create_link', :locals => {:parent => user_group, :child_class => UserGroupMembership}
+        %th= t('user_groups.index.members')
+
+    - reset_cycle
+    - for user_group in user_groups
+      %tr{:class => cycle('odd', 'even')}
+        %td= user_group.name
+        - if user_groups.pluck(:description).uniq != [nil]
+          %td= user_group.description
+        - if @user
+          %td= user_group.tenant
+        - else
+          %td
+            =render 'users/listing', :users => user_group.users
+            - if user_group.users.any?
+              %br
+            - if (user_group.tenant.user_ids - user_group.user_ids).any?
+              = render :partial => 'shared/create_link', :locals => {:parent => user_group, :child_class => UserGroupMembership}
 
-      =render :partial => 'shared/index_view_edit_destroy_part', :locals => {:parent => user_group.tenant, :child => user_group}
+        =render :partial => 'shared/index_view_edit_destroy_part', :locals => {:parent => user_group.tenant, :child => user_group}