user friendly for editing members

This commit is contained in:
Harry Bomrah 2015-04-22 18:28:14 +08:00
parent db88b1e553
commit cf7e15d4bb
8 changed files with 26 additions and 20 deletions

View File

@ -10,6 +10,8 @@ class Admin::MembersController < OrbitMemberController
def index def index
@roles = Role.excludes(disabled: true) @roles = Role.excludes(disabled: true)
page_num = params[:page] || 1 page_num = params[:page] || 1
param_temp = request.fullpath.split("?")
@edit_params = (param_temp.count > 1 ? "?" + param_temp.last : "") rescue ""
@filter = params[:filter] @filter = params[:filter]
@mq = params[:mq] @mq = params[:mq]
@ -449,6 +451,8 @@ class Admin::MembersController < OrbitMemberController
end end
def edit def edit
param_temp = request.fullpath.split("?")
@edit_params = (param_temp.count > 1 ? "?" + param_temp.last : "") rescue ""
if has_access? if has_access?
@form_index = 0 @form_index = 0
get_info_and_roles get_info_and_roles
@ -491,6 +495,8 @@ class Admin::MembersController < OrbitMemberController
end end
def update def update
param_temp = request.fullpath.split("?")
@edit_params = (param_temp.count > 1 ? "?" + param_temp.last : "") rescue ""
respond_to do |format| respond_to do |format|
if @member.user.present? if @member.user.present?
@user = @member.user @user = @member.user
@ -529,7 +535,7 @@ class Admin::MembersController < OrbitMemberController
end end
format.html { redirect_to admin_members_path, notice: 'Successfully Updated the User' } format.html { redirect_to admin_members_path() + @edit_params, notice: 'Successfully Updated the User' }
format.json { head :no_content } format.json { head :no_content }
else else
format.html { render action: 'edit' } format.html { render action: 'edit' }
@ -585,6 +591,7 @@ class Admin::MembersController < OrbitMemberController
end end
def edit_privilege def edit_privilege
@edit_params = "?" + request.fullpath.split("?").last rescue ""
if current_user.is_admin? if current_user.is_admin?
@user = @member.user @user = @member.user
@workgroup = Workgroup.find_by(key: 'admin') @workgroup = Workgroup.find_by(key: 'admin')

View File

@ -28,9 +28,8 @@ class MembersController < ApplicationController
page_role_status.each do |status_id| page_role_status.each do |status_id|
page_roles << RoleStatus.find(status_id).role.id.to_s page_roles << RoleStatus.find(status_id).role.id.to_s
end end
roles = Role.find(page_roles.uniq).collect do |role| roles = Role.find(page_roles.uniq).collect do |role|
{ "title" => role.title, "id" => role.id, "status" => RoleStatus.where(:role_id=>role, :_id.in=>page_role_status).to_a } { "title" => role.title, "id" => role.id, "status" => RoleStatus.where(:role_id=>role, :_id.in=>page_role_status).asc(:key).to_a }
end end
end end

View File

@ -351,12 +351,12 @@ module ApplicationHelper
((controller.controller_name.eql?(controller_name) || request.fullpath.eql?(controller_name)) && controller.action_name.eql?(action_name)) ? 'active' : nil ((controller.controller_name.eql?(controller_name) || request.fullpath.eql?(controller_name)) && controller.action_name.eql?(action_name)) ? 'active' : nil
end end
def link_back(custom_class=nil) def link_back(custom_class=nil, params="")
case custom_class case custom_class
when nil when nil
link_to t('back'), get_go_back, :class => 'nav' link_to t('back'), get_go_back + params, :class => 'nav'
else else
link_to t('back'), get_go_back, :class => custom_class link_to t('back'), get_go_back + params, :class => custom_class
end end
end end

View File

@ -21,11 +21,11 @@
<%= link_to (member_for_listing.name != (member_for_listing.email) ? member_for_listing.name : member_for_listing.id),admin_member_path(member_for_listing) %> <%= link_to (member_for_listing.name != (member_for_listing.email) ? member_for_listing.name : member_for_listing.id),admin_member_path(member_for_listing) %>
<div class="quick-edit"> <div class="quick-edit">
<ul class="nav nav-pills"> <ul class="nav nav-pills">
<%= content_tag(:li, link_to(t(:edit),edit_admin_member_path(member_for_listing))) if current_user.is_admin? %> <%= content_tag(:li, link_to(t(:edit),edit_admin_member_path(member_for_listing) + @edit_params)) if current_user.is_admin? %>
<% if member_for_listing.user.present? %> <% if member_for_listing.user.present? %>
<%= content_tag(:li, link_to(t("users.setting_privilege"),admin_member_edit_privilege_path(member_for_listing))) if current_user.is_admin? and current_user.id != (member_for_listing.user.id if member_for_listing.user.present? ) %> <%= content_tag(:li, link_to(t("users.setting_privilege"),admin_member_edit_privilege_path(member_for_listing) + @edit_params)) if current_user.is_admin? and current_user.id != (member_for_listing.user.id if member_for_listing.user.present? ) %>
<% end %> <% end %>
<%= content_tag(:li, link_to(t(:delete_),admin_member_path(member_for_listing.id, at: params[:at]), data: { confirm: t('sure?') }, method: :delete, class: "text-error", remote: true)) if current_user.is_admin? %> <%= content_tag(:li, link_to(t(:delete_),admin_member_path(member_for_listing.id) + @edit_params, data: { confirm: t('sure?') }, method: :delete, class: "text-error", remote: true)) if current_user.is_admin? %>
</ul> </ul>
</div> </div>
</td> </td>

View File

@ -13,9 +13,9 @@
%> %>
<p class="<%= @user_sex%>"></p> <p class="<%= @user_sex%>"></p>
<div class="action"> <div class="action">
<%= link_to(content_tag(:i, nil, :class => 'icon-edit'),edit_admin_member_path(member_for_summary),:class=>"edit" ) if current_user.is_admin?%> <%= link_to(content_tag(:i, nil, :class => 'icon-edit'),edit_admin_member_path(member_for_summary) + @edit_params,:class=>"edit" ) if current_user.is_admin?%>
<%= link_to(content_tag(:i, nil, :class => 'icon-key'),admin_member_edit_privilege_path(member_for_summary),:class=>"key" ) if current_user.is_admin? and (current_user.id != member_for_summary.user.id if member_for_summary.user.present?) %> <%= link_to(content_tag(:i, nil, :class => 'icon-key'),admin_member_edit_privilege_path(member_for_summary) + @edit_params,:class=>"key" ) if current_user.is_admin? and (current_user.id != member_for_summary.user.id if member_for_summary.user.present?) %>
<%= link_to(content_tag(:i, nil, :class => 'icon-trash'),admin_member_path(member_for_summary.id, at: params[:at]), data: { confirm: t('sure?') }, method: :delete, class: "trash", remote: true) if current_user.is_admin? %> <%= link_to(content_tag(:i, nil, :class => 'icon-trash'),admin_member_path(member_for_summary.id) + @edit_params, data: { confirm: t('sure?') }, method: :delete, class: "trash", remote: true) if current_user.is_admin? %>
</div> </div>
<%= image_tag(member_for_summary.avatar) %> <%= image_tag(member_for_summary.avatar) %>
</div> </div>

View File

@ -13,9 +13,9 @@
<div class="member-avatar <%= @user_sex %>"> <div class="member-avatar <%= @user_sex %>">
<div class="action"> <div class="action">
<%= link_to(content_tag(:i, nil, :class => 'icon-edit'),edit_admin_member_path(member_for_thumbnail),:class=>"edit" ) if current_user.is_admin?%> <%= link_to(content_tag(:i, nil, :class => 'icon-edit'),edit_admin_member_path(member_for_thumbnail) + @edit_params,:class=>"edit" ) if current_user.is_admin?%>
<%= link_to(content_tag(:i, nil, :class => 'icon-key'),admin_member_edit_privilege_path(member_for_thumbnail),:class=>"key" ) if current_user.is_admin? and (current_user.id != member_for_thumbnail.user.id if member_for_thumbnail.user.present?) %> <%= link_to(content_tag(:i, nil, :class => 'icon-key'),admin_member_edit_privilege_path(member_for_thumbnail) + @edit_params,:class=>"key" ) if current_user.is_admin? and (current_user.id != member_for_thumbnail.user.id if member_for_thumbnail.user.present?) %>
<%= link_to(content_tag(:i, nil, :class => 'icon-trash'),admin_member_path(member_for_thumbnail.id, at: params[:at]), data: { confirm: t('sure?') }, method: :delete, class: "trash", remote: true) if current_user.is_admin? %> <%= link_to(content_tag(:i, nil, :class => 'icon-trash'),admin_member_path(member_for_thumbnail.id) + @edit_params, data: { confirm: t('sure?') }, method: :delete, class: "trash", remote: true) if current_user.is_admin? %>
</div> </div>
<%= image_tag(member_for_thumbnail.avatar) %> <%= image_tag(member_for_thumbnail.avatar) %>
</div> </div>

View File

@ -17,7 +17,7 @@
<%= javascript_include_tag "lib/member/role-forms.js" %> <%= javascript_include_tag "lib/member/role-forms.js" %>
<% end -%> <% end -%>
<%= form_for @member, :url => admin_member_path(@member), :html => { :multipart => true , :class=>"form-horizontal main-forms", :id=>"user-forms"} do |f| %> <%= form_for @member, :url => admin_member_path(@member) + @edit_params , :html => { :multipart => true , :class=>"form-horizontal main-forms", :id=>"user-forms"} do |f| %>
<%= f.error_messages %> <%= f.error_messages %>
<fieldset> <fieldset>
@ -32,7 +32,7 @@
<div class="form-actions"> <div class="form-actions">
<%= link_to t(:update_), "#", :class=>"btn btn-primary returnDecide", :onclick=>"$('#user-forms').submit()" %> <%= link_to t(:update_), "#", :class=>"btn btn-primary returnDecide", :onclick=>"$('#user-forms').submit()" %>
<%= link_back('btn') %> <%= link_back('btn',@edit_params) %>
</div> </div>
</fieldset> </fieldset>

View File

@ -12,7 +12,7 @@
<% end -%> <% end -%>
<%= form_for @member, url: admin_member_path(@member), html: { multipart: true , class: "form-horizontal main-forms", id: "user-forms"} do |f| %> <%= form_for @member, url: admin_member_path(@member) + @edit_params, html: { multipart: true , class: "form-horizontal main-forms", id: "user-forms"} do |f| %>
<fieldset> <fieldset>
<div id="basic-area" class="input-area"> <div id="basic-area" class="input-area">
@ -20,7 +20,7 @@
</div> </div>
<div class="form-actions"> <div class="form-actions">
<%= link_back('btn') %> <%= link_back('btn',@edit_params) %>
<%= hidden_field_tag 'edit_type', 'edit_privilege' %> <%= hidden_field_tag 'edit_type', 'edit_privilege' %>
<%= f.submit t(:update_), class: 'btn btn-primary' %> <%= f.submit t(:update_), class: 'btn btn-primary' %>
</div> </div>