diff --git a/app/controllers/members_controller.rb b/app/controllers/members_controller.rb index c88ea52df..5545f4186 100644 --- a/app/controllers/members_controller.rb +++ b/app/controllers/members_controller.rb @@ -5,7 +5,7 @@ def destroy find_member toggle_active_posts @member.destroy - redirect_to users_path + redirect_to manage_users_path end def toggle_manager diff --git a/app/controllers/users_controller.rb b/app/controllers/users_controller.rb index 42aa7abaf..ee0af5f60 100644 --- a/app/controllers/users_controller.rb +++ b/app/controllers/users_controller.rb @@ -61,7 +61,7 @@ def search_and_load_members(members_scope) @search = members_scope.ransack(search_params) @members = - @search.result.eager_load(:account, :user).page(params[:page]).per(25) + @search.result.eager_load(:account, :user).page(params[:page]).per(20) @member_view_models = @members.map { |m| MemberDecorator.new(m, self.class.helpers) } diff --git a/app/views/users/_member_card.html.erb b/app/views/users/_member_card.html.erb index e750c9dfd..c6ed0c62c 100644 --- a/app/views/users/_member_card.html.erb +++ b/app/views/users/_member_card.html.erb @@ -1,34 +1,36 @@ -
-
-
<%= member.avatar_img(48) %>
-
-

<%= member.link_to_self %>

-
- <%= "##{member.member_uid} — " %> - <% if member.last_sign_in_at %> - <%= t('.active_ago', time: distance_of_time_in_words(Time.now, member.last_sign_in_at)) %> - <% else %> - <%= t('.no_activity') %> - <% end %> +
+
+
+
<%= member.avatar_img(48) %>
+
+

<%= member.link_to_self %>

+
+ <%= "##{member.member_uid} — " %> + <% if member.last_sign_in_at %> + <%= t('.active_ago', time: distance_of_time_in_words(Time.now, member.last_sign_in_at)) %> + <% else %> + <%= t('.no_activity') %> + <% end %> +
-
-
-
- <%= member.description %> -
-
-
- - <%= phone_to member.phone %> -
-
- - <%= member.mail_to %> +
+
+ <%= member.description %>
-
- <%= "Balance: " %> - <%= member.account_balance %> +
+
+ + <%= phone_to member.phone %> +
+
+ + <%= member.mail_to %> +
+
+ <%= "Balance: " %> + <%= member.account_balance %> +
diff --git a/app/views/users/_user_row.html.erb b/app/views/users/_user_row.html.erb new file mode 100644 index 000000000..29af9c8d2 --- /dev/null +++ b/app/views/users/_user_row.html.erb @@ -0,0 +1,22 @@ +<%= content_tag(:tr, class: member.row_css_class) do %> + <%= member.avatar_img %> + <%= member.member_uid %> + + <%= member.inactive_icon %> + <%= member.link_to_self %> + + <%= member.mail_to %> + <%= phone_to member.phone %> + <%= member.account_balance %> + <% if current_user.manages?(current_organization) %> + + <% if member.active? %> + <%= render 'toggle_manager_link', member: member if can_toggle_manager?(member) %> + <% else %> + <%= render 'cancel_membership_link', member: member if can_cancel_member?(member) %> + <% end %> + + <%= render 'toggle_active_link', member: member if can_toggle_active?(member) %> + + <% end %> +<% end %> \ No newline at end of file diff --git a/app/views/users/_user_rows.html.erb b/app/views/users/_user_rows.html.erb deleted file mode 100644 index e34632d1d..000000000 --- a/app/views/users/_user_rows.html.erb +++ /dev/null @@ -1,24 +0,0 @@ -<% members.each do |member| %> - <%= content_tag(:tr, class: member.row_css_class) do %> - <%= member.avatar_img %> - <%= member.member_uid %> - - <%= member.inactive_icon %> - <%= member.link_to_self %> - - <%= member.mail_to %> - <%= phone_to member.phone %> - <%= member.account_balance %> - <% if current_user.manages?(current_organization) %> - - <% if member.active? %> - <%= render 'toggle_manager_link', member: member if can_toggle_manager?(member) %> - <% else %> - <%= render 'cancel_membership_link', member: member if can_cancel_member?(member) %> - <% end %> - - <%= render 'toggle_active_link', member: member if can_toggle_active?(member) %> - - <% end %> - <% end %> -<% end %> diff --git a/app/views/users/index.html.erb b/app/views/users/index.html.erb index 8cde20cd7..3d901a9dd 100644 --- a/app/views/users/index.html.erb +++ b/app/views/users/index.html.erb @@ -32,11 +32,7 @@
- <% @member_view_models.each do |member| %> -
- <%= render 'member_card', member: member %> -
- <% end %> + <%= render partial: 'member_card', collection: @member_view_models, as: :member %>
diff --git a/app/views/users/manage.html.erb b/app/views/users/manage.html.erb index 9c0cee757..a02f58d4f 100644 --- a/app/views/users/manage.html.erb +++ b/app/views/users/manage.html.erb @@ -68,7 +68,7 @@ - <%= render "user_rows", members: @member_view_models %> + <%= render partial: "user_row", collection: @member_view_models, as: :member %>