From 87ebcc48e7a0b9f32deed5f04ad2c7455b821604 Mon Sep 17 00:00:00 2001
From: "Matthew K. Fu JuYuan" <matthewfu@gmail.com>
Date: Thu, 10 May 2012 11:51:44 +0800
Subject: [PATCH] Appauth new interface for setting managers. Fix showing
 sidebar for News and Announcement when setting managers at module app page.

---
 app/assets/javascripts/.DS_Store              | Bin 6148 -> 6148 bytes
 app/assets/javascripts/inc/search.js          |  34 ---------
 .../admin/module_apps_controller.rb           |   4 ++
 .../new_interface_module_apps_controller.rb   |  66 ++++++++++++++++++
 app/helpers/application_helper.rb             |   4 +-
 .../_modal_list.html.erb                      |  42 +++++++++++
 .../_privilege_user.html.erb                  |  25 +++++++
 .../setting.html.erb                          |  56 +++++++++++++++
 .../new_interface_module_apps/setting.js.erb  |   2 +
 .../update_setting.js.erb                     |   1 +
 .../user_list.js.erb                          |   2 +
 app/views/layouts/_side_bar.html.erb          |   8 +--
 dump.rdb                                      | Bin 1530 -> 716 bytes
 13 files changed, 204 insertions(+), 40 deletions(-)
 delete mode 100644 app/assets/javascripts/inc/search.js
 create mode 100644 app/controllers/admin/new_interface_module_apps_controller.rb
 create mode 100644 app/views/admin/new_interface_module_apps/_modal_list.html.erb
 create mode 100644 app/views/admin/new_interface_module_apps/_privilege_user.html.erb
 create mode 100644 app/views/admin/new_interface_module_apps/setting.html.erb
 create mode 100644 app/views/admin/new_interface_module_apps/setting.js.erb
 create mode 100644 app/views/admin/new_interface_module_apps/update_setting.js.erb
 create mode 100644 app/views/admin/new_interface_module_apps/user_list.js.erb

diff --git a/app/assets/javascripts/.DS_Store b/app/assets/javascripts/.DS_Store
index 51582ef8aa6a5ea474683aee9d3de008e14c257f..9134f2a58171c1a69637b1eef035b67fcfa72004 100644
GIT binary patch
delta 139
zcmZoMXfc=|#>B)qu~2NHo+2a5#DLw41(+BaStj!^$_QpM<S`^Cl@}Kz<>V(ZFfgp2
zY{{6cBT-##WM-(NU}9oWtD{hDZf*c%o0`<ta&mBT3cThJRnoKZ2uZA|t*dXCe2h_^
ov1zk1qde<ob`E|HpjDe0nZ7em<`=Q#U}RtbA_j)d0U}$N0kr%fy8r+H

delta 67
zcmZoMXfc=|#>B`mu~2NHo+2aj#DLw5%#(STWH!HLl49L_f%z!oW_AvK4xp0F2bsS!
VPv#e~<X`{-Mg|6^%>g1?m;q!45S;)3

diff --git a/app/assets/javascripts/inc/search.js b/app/assets/javascripts/inc/search.js
deleted file mode 100644
index 70507fc6..00000000
--- a/app/assets/javascripts/inc/search.js
+++ /dev/null
@@ -1,34 +0,0 @@
-// JavaScript Document
-
-// can copy code to any of ur desired javascsript 
-
-//extended jquery to search fast.
-$.extend($.expr[':'], {
-	'containsi': function (elem, i, match, array) {
-		return (elem.textContent || elem.innerText || '').toLowerCase().indexOf((match[3] || "").toLowerCase()) >= 0;
-	}
-});
-var interval,sval;
-$(document).ready(function(){
-	$("#user_filter").keyup(function(){
-		sval = $(this).val();
-		$(".checkbox").popover("hide");
-		$("div.checkblock").hide();
-		clearInterval(interval);
-		interval = setInterval(waitForSearch,1000);
-	})
-})
-var waitForSearch = function(){
-	if(sval){
-		var totalfoundbyname = $("div#users_checkbox_ary label.member-name:containsi("+sval+")").length
-		if(totalfoundbyname!=0){
-			$("div#users_checkbox_ary label.member-name:containsi("+sval+")").parent().parent().show();				
-		}else if(totalfoundbyname==0){
-			$("div#users_checkbox_ary div.for_unit:containsi("+sval+")").parent().show();
-		}	
-	}else{
-		$(".checkbox").popover('hide');	
-		$("div.checkblock").show();
-	}
-	clearInterval(interval);
-}
\ No newline at end of file
diff --git a/app/controllers/admin/module_apps_controller.rb b/app/controllers/admin/module_apps_controller.rb
index e44be291..c6a8d2b2 100644
--- a/app/controllers/admin/module_apps_controller.rb
+++ b/app/controllers/admin/module_apps_controller.rb
@@ -17,6 +17,10 @@ class Admin::ModuleAppsController < ApplicationController
     end
   end
   
+  def module_auth_proc
+    @module_app = ModuleApp.find(params[:id])
+  end
+  
   def edit
     @module_app = ModuleApp.find(params[:id])
   end
diff --git a/app/controllers/admin/new_interface_module_apps_controller.rb b/app/controllers/admin/new_interface_module_apps_controller.rb
new file mode 100644
index 00000000..6603334c
--- /dev/null
+++ b/app/controllers/admin/new_interface_module_apps_controller.rb
@@ -0,0 +1,66 @@
+class Admin::NewInterfaceModuleAppsController < ApplicationController
+  before_filter :authenticate_user!
+  before_filter :is_admin?
+  include AdminHelper
+
+  layout "new_admin"
+  
+  def setting
+    @sys_users = User.all
+    @module_app = ModuleApp.find(params[:module_app_id])
+    @options_from_collection_for_select_bulletin_categorys = [@module_app].collect{|ma| [ma.title,ma.id] }
+    # if params.has_key? :category
+    #   @bulletin_category = BulletinCategory.find params[:category][:id]
+    # else
+    #   @bulletin_category =  @bulletin_categorys.first
+    # end
+    @users_array = @module_app.managing_users rescue []
+
+    respond_to do |format|
+      format.html
+      format.js
+    end
+  end
+  
+  def update_setting
+    module_app = update_setting_by_params
+    if module_app.save!
+      flash[:notice] = "Update Done"
+    else
+      flash[:notice] = "Update Failed"
+    end
+  end
+  
+  def user_list
+    @module_app = ModuleApp.find params[:module_app][:id]
+  end
+
+  protected
+  def update_setting_by_params
+    ma = ModuleApp.find params[:module_app][:id]
+    user_sat = User.find params[:users].keys
+    users_to_new = user_sat - ma.managing_users
+    users_to_remove = ma.managing_users - user_sat
+
+    users_to_new.each do |new_user|
+      ma.assign_manager(new_user,current_user)
+    end
+
+    users_to_remove.each do |remove_user|
+      ma.remove_manager(remove_user)
+    end
+    ma
+  end
+  
+  def get_categorys(id = nil)
+    @bulletin_categorys = []
+    if(is_manager? || is_admin?)
+      @bulletin_categorys = (id ? BulletinCategory.find(id).to_a : BulletinCategory.excludes('disabled' => true))
+    elsif is_sub_manager?
+      @bulletin_categorys = BulletinCategory.authed_for_user(current_user,'submit_new')
+    end
+  end
+
+
+
+end
\ No newline at end of file
diff --git a/app/helpers/application_helper.rb b/app/helpers/application_helper.rb
index 74d3e662..844fbe14 100644
--- a/app/helpers/application_helper.rb
+++ b/app/helpers/application_helper.rb
@@ -68,9 +68,9 @@ module ApplicationHelper
     end
   end
   
-  def active_sys_call_for_app(controller_name,action_name,app_title)
+  def active_sys_call_for_app(controller_name,action_name,app_title,field = :id)
     unless active_for_action(controller_name,action_name).nil?
-      app = ModuleApp.find params[:id]
+      app = ModuleApp.find params[field]
       app.title == app_title ? 'active' : nil
     else
       nil
diff --git a/app/views/admin/new_interface_module_apps/_modal_list.html.erb b/app/views/admin/new_interface_module_apps/_modal_list.html.erb
new file mode 100644
index 00000000..b5b0ebcd
--- /dev/null
+++ b/app/views/admin/new_interface_module_apps/_modal_list.html.erb
@@ -0,0 +1,42 @@
+<% if module_app -%>
+
+<div class="modal hide fade in" id="module_app-<%=module_app.id%>">
+  <div class="modal-header">
+    <a class="close" data-dismiss="modal">×</a>
+    <h3><%= t("admin.user_role.auth.manager") %></h3>
+  </div>
+  <div class="modal-body">
+		<div class="clear">
+			<% module_app.managing_users.each do |user| %>
+			<div class="checkbox clear checked">
+					<div class='member-avatar'>
+						<% if user.avatar? %>
+							<%= image_tag(user.avatar.thumb.url,:class => "member-img") %>
+						<% else %>
+							<%= image_tag "person.png",:class => "member-img" %>
+						<% end %>	
+					</div>
+					<%= label_tag "lab-user-#{user.id}", (user.name rescue ''),:class=>"member-name",:id=>nil -%>
+			</div>
+			<% end -%>
+		<divl>
+  </div>
+  <div class="modal-footer">
+    <a href="#" class="btn" data-dismiss="modal"><%= t("modal.close") %></a>
+  </div>
+  <div>
+  	<script type="text/javascript" src="/static/kernel.js"></script>
+		<script type="text/javascript">
+			$(document).ready(function() {
+	
+			$("#module_app-<%=module_app.id%>").on("show", function () {
+			});
+		$(".modal").on("hidden", function () {
+	  $("#show_preview").remove();
+			});
+		});
+		</script>
+	</div>
+</div>
+
+<% end -%>
\ No newline at end of file
diff --git a/app/views/admin/new_interface_module_apps/_privilege_user.html.erb b/app/views/admin/new_interface_module_apps/_privilege_user.html.erb
new file mode 100644
index 00000000..90f5cb65
--- /dev/null
+++ b/app/views/admin/new_interface_module_apps/_privilege_user.html.erb
@@ -0,0 +1,25 @@
+<%= content_tag :div ,:id => "users_checkbox_ary",:class => 'clear' do -%>
+	<% @sys_users.each do |sys_user| -%>
+	
+			<div class="checkblock">
+				
+				<% sys_user.sub_roles.each do |sr| %>
+					<div class="for_unit" style="display:none;"> <%= sr.key %></div>
+				<% end %>
+				<%= content_tag :div,:data=>{'original-title'=>t('announcement.bulletin.approval_setting_window_title'),:content => "#{sys_user.sub_roles.collect{|sr| sr.i18n_variable[I18n.locale]}.join(',')}"},:class=>"checkbox clear" do %>
+					<div class="check-icon">
+					</div>
+					<div class='member-avatar'>
+						<% if sys_user.avatar? %>
+							<%= image_tag(sys_user.avatar.thumb.url,:class => "member-img") %>
+						<% else %>
+							<%= image_tag "person.png",:class => "member-img" %>
+						<% end %>	
+					</div>
+					<%= label_tag "lab-user-#{sys_user.id}", (sys_user.name rescue ''),:class=>"member-name",:id=>nil -%>
+					<%= check_box_tag "[users][#{sys_user.id}]", 'true',users.include?(sys_user),:class => "check"   -%>
+				<%end -%>
+			</div>
+
+	<% end -%>
+<% end -%>
\ No newline at end of file
diff --git a/app/views/admin/new_interface_module_apps/setting.html.erb b/app/views/admin/new_interface_module_apps/setting.html.erb
new file mode 100644
index 00000000..7d1179f0
--- /dev/null
+++ b/app/views/admin/new_interface_module_apps/setting.html.erb
@@ -0,0 +1,56 @@
+<% content_for :page_specific_css do %>
+	<%= stylesheet_link_tag "inc/permission-checkbox"  %>
+<% end %>
+<% content_for :page_specific_javascript do %>
+	<%= javascript_include_tag "inc/permission-checkbox"  %>
+	<%= javascript_include_tag "inc/search"  %>
+	<%= javascript_include_tag "inc/modal-preview"  %>
+<% end %>
+<%#= label_tag :fact_check_setting, t("announcement.bulletin.fact_check_setting") %>
+<%= form_tag('', :remote => true,:class => "prevent_enter_submit_form")  %>
+<div class="subnav clear">
+    <ul class="nav nav-pills filter pull-left">
+        <li class="accordion-group">
+            <div class="form-search" style="margin: 5px 10px;">
+                <%= label_tag :module, t("module") %>
+                <%= select "module_app",'id',@options_from_collection_for_select_bulletin_categorys %>
+								<%= search_field_tag 'user_filter' %>
+            </div>
+
+        </li>
+    </ul>
+		<%= link_to t("admin.user_role.auth.manager"), admin_module_app_manager_auth_show_path , :class=>'preview_trigger btn btn-success pull-right'%>	
+
+</div>
+<%#= label_tag :role, t("admin.roles") %>
+<div class="clear">
+<%= content_tag :div do -%>
+	<% form_tag admin_module_app_manager_auth_proc_path  do %>
+		<%= render :partial => "privilege_user", :locals => {:users => @users_array} %>
+	<div class="form-actions form-fixed pagination-right">
+	<%= submit_tag "Update", :class => 'btn btn-primary' %>
+	</div>
+	<% end -%>
+<% end -%>
+</div>
+
+<script type="text/javascript" charset="utf-8">
+var availableTags = [];
+$(document).ready(function() {
+	
+	$(".prevent_enter_submit_form").bind("keypress", function(e) {
+		if (e.keyCode == 13) {
+			return false;
+		}
+	});
+   
+	$('#category_id').change(function() {
+		$.ajax({
+			type: 'GET',
+			dataType: "script",
+			url:$(this).parents("from").attr("href"),
+			data:$(this).parents("form").serialize()
+		});
+	});
+});
+</script>
\ No newline at end of file
diff --git a/app/views/admin/new_interface_module_apps/setting.js.erb b/app/views/admin/new_interface_module_apps/setting.js.erb
new file mode 100644
index 00000000..0ca148da
--- /dev/null
+++ b/app/views/admin/new_interface_module_apps/setting.js.erb
@@ -0,0 +1,2 @@
+$("#users_checkbox_ary").replaceWith('<%= escape_javascript(render :partial => "privilege_user", :locals => {:users => @users_array})%>');
+permissionCheckbox();
\ No newline at end of file
diff --git a/app/views/admin/new_interface_module_apps/update_setting.js.erb b/app/views/admin/new_interface_module_apps/update_setting.js.erb
new file mode 100644
index 00000000..7486000a
--- /dev/null
+++ b/app/views/admin/new_interface_module_apps/update_setting.js.erb
@@ -0,0 +1 @@
+alert( "<% flash.each do |key, msg| %><%= msg %><% end%>");
\ No newline at end of file
diff --git a/app/views/admin/new_interface_module_apps/user_list.js.erb b/app/views/admin/new_interface_module_apps/user_list.js.erb
new file mode 100644
index 00000000..7caf1e39
--- /dev/null
+++ b/app/views/admin/new_interface_module_apps/user_list.js.erb
@@ -0,0 +1,2 @@
+$('#show_preview').html("<%= escape_javascript(render(:partial => 'modal_list',:locals => {:module_app => @module_app})) %>");
+var start_modal_with_id = "module_app-<%=@module_app.id%>"
\ No newline at end of file
diff --git a/app/views/layouts/_side_bar.html.erb b/app/views/layouts/_side_bar.html.erb
index 691b330a..dde071c5 100644
--- a/app/views/layouts/_side_bar.html.erb
+++ b/app/views/layouts/_side_bar.html.erb
@@ -5,7 +5,7 @@
 	<%= javascript_include_tag "/static/kernel.js"  %>
 <% end %>
 
-<%= content_tag :li, :class => active_for_controllers('bulletins', '/panel/announcement/back_end/tags', 'bulletin_categorys', 'approvals') ||  active_sys_call_for_app('module_apps','edit','Announcement')  do -%>
+<%= content_tag :li, :class => (active_for_controllers('bulletins', '/panel/announcement/back_end/tags', 'bulletin_categorys', 'approvals') ||  active_sys_call_for_app('new_interface_module_apps','setting','Announcement',:module_app_id) ) do -%>
 	<%= link_to content_tag(:i, nil, :class => 'icons-announcement') + t('admin.announcement'), panel_announcement_back_end_bulletins_path %>
 	<%= content_tag :ul, :class => ("nav nav-list " + visible_for_controllers('bulletins', '/panel/announcement/back_end/tags', 'bulletin_categorys', 'approvals')) do -%>
 		<%= content_tag :li, link_to(t('announcement.all_articles'), panel_announcement_back_end_bulletins_path), :class => active_for_action('bulletins', 'index') %>
@@ -13,13 +13,13 @@
 		<%= content_tag :li, link_to(t('announcement.categories'), panel_announcement_back_end_bulletin_categorys_path), :class => active_for_action('bulletin_categorys', 'index') %>
 		<%= content_tag :li, link_to(t('announcement.tags'), panel_announcement_back_end_tags_path), :class => active_for_action('/panel/announcement/back_end/tags', 'index') %>
 		<%= content_tag :li, link_to(t('announcement.bulletin.approval_setting'), panel_announcement_back_end_approval_setting_path), :class => active_for_action('approvals', 'setting')  if (is_manager? rescue nil) %>
-		<%= content_tag :li, link_to(t('admin.module.authorization'),edit_admin_module_app_path(ModuleApp.first(conditions: {title: "Announcement"}))), :class => active_sys_call_for_app('module_apps','edit','Announcement') if (is_admin? rescue nil) %>
+		<%= content_tag :li, link_to(t('admin.module.authorization'),admin_module_app_manager_auth_proc_path(ModuleApp.first(conditions: {title: "Announcement"}))), :class => active_sys_call_for_app('new_interface_module_apps','setting','Announcement',:module_app_id) if (is_admin? rescue nil) %>
 
 	<% end -%>
 
 <% end -%>
 
-<%= content_tag :li, :class => active_for_controllers('news_bulletins', '/panel/news/back_end/tags', 'news_bulletin_categorys', 'news_approvals') do -%>
+<%= content_tag :li, :class =>( active_for_controllers('news_bulletins', '/panel/news/back_end/tags', 'news_bulletin_categorys', 'news_approvals')||  active_sys_call_for_app('new_interface_module_apps','setting','news',:module_app_id)) do -%>
 	<%= link_to content_tag(:i, nil, :class => 'icons-announcement') + t('admin.news'), panel_news_back_end_news_bulletins_path %>
 	<%= content_tag :ul, :class => ("nav nav-list " + visible_for_controllers('news_bulletins', '/panel/news/back_end/tags', 'news_bulletin_categorys', 'news_approvals')) do -%>
 		<%= content_tag :li, link_to(t('announcement.all_articles'), panel_news_back_end_news_bulletins_path), :class => active_for_action('news_bulletins', 'index') %>
@@ -27,7 +27,7 @@
 		<%= content_tag :li, link_to(t('announcement.categories'), panel_news_back_end_news_bulletin_categorys_path), :class => active_for_action('news_bulletin_categorys', 'index') %>
 		<%= content_tag :li, link_to(t('announcement.tags'), panel_news_back_end_tags_path), :class => active_for_action('/panel/news/back_end/tags', 'index') %>
 		<%= content_tag :li, link_to(t('announcement.bulletin.approval_setting'), panel_news_back_end_news_approval_setting_path), :class => active_for_action('news_approvals', 'setting')  if (is_manager? rescue nil) %>
-		<%= content_tag :li, link_to(t('admin.module.authorization'),edit_admin_module_app_path(ModuleApp.first(conditions: {key: "news"}))), :class => active_sys_call_for_app('module_apps','edit','news') if (is_admin? rescue nil) %>
+		<%= content_tag :li, link_to(t('admin.module.authorization'),admin_module_app_manager_auth_proc_path(ModuleApp.first(conditions: {key: "news"}))), :class => active_sys_call_for_app('new_interface_module_apps','setting','news',:module_app_id) if (is_admin? rescue nil) %>
 	<% end -%>
 
 <% end -%>
diff --git a/dump.rdb b/dump.rdb
index a3c1b42c46ad075453c0ee3cb39529b5802dd944..748293b8fe7d0506c035ed140046de46e5d7b109 100644
GIT binary patch
delta 27
jcmeyxeTH>HEu-?p2NDyf)iJ5COne}@S)XwZBjbMnnG*_X

literal 1530
zcmbW1PjA~s5XC9_PmRQZn*xE0wunb-z)k{LT+$LP5iMdvc7q-o6mf#0DI94@jwRYl
zF0)+9c8U~6i=6T$dhQ|T+^QhoB8NagZn^Z(dt1tqBclLy=#ev=d4u<U*n2zM-|SLK
z&A+umQDX8qMQ7LRpr3M-JI5@HIXYP}D|2yFSo(>0oFxfy$k(X6&ix=nQX)*Yv(k@y
zPxULK9cLqnKHUl0ovCb6`()He6jRPV3po0{qW0;i`eB(<sZ=YrMSEIak6JxVzdrHQ
zi@G@qbUjnJcJZcI3J;S6{a4qgbG0k^cbS8YIcnylxoYlQjpq04^+`MJ2|M$9wz)B3
zF284+wcI(|toO76nV6R|phPEQw(fdf+F%?bpUI1OeKG!J_aySR-HXk_IeSC8^w(Q&
z8-HcM4nLbrWzbsGv<zA`XbsGoQ>D)OVg(wz_p?1QJK=))#2INoX=bPapX(Xluy7^g
zyF~EV!&n6(f~EN_*-k@@)Q*%w!it8ikCq&wWPm;>0iHyr5WdU60^uH@r{E}<R~?Zm
zVBm7$9YS@^W!|BZ%tK&H#CVXPOpHSoC}EUkH#FK9CWcJilQpwkzgD)^VyjL!Wy*u7
z?TN72LFS(>PbERb;0~<Ju9q!?2X61`XNJoXEKAoh&H9EDM9}haKZvjyB0MkntMF@o
zuRGnw2xal(e(PAbsNMRrcteZHlveCNe6Q6`A8WGsPQ(7gsIXCK_4PPa{U5k)Q>(?@
zmzUv&1suD`lbb;9P5a2PU3}A@d!}y-&s)Bf7~h_JFXUkoBY8h#2<Zmux0zWM$Hswh
z$*X0dH+}b)-&+UsG0J@*!&S0ZB1iC*hK)xHAyaCN1=6Y6R<-WjB7aN`-j(Isvt&RF
zdHQ<G-nu<G;)+8T>;F6Axw|aHVZ3A%ZyF74^dSq7ap*D`Fqb1_2!uF@*N0a2%1PBs
zU{7C#eONqg>rR7jb}yC4IBxl$xHHSnXd0WQA*_6GZEoaRVr)KI$XpvFYoEoMSqFUf
c+`2C{+<Y;%u1>8D>fC~#&#Z$d=hpr6FTfG)_y7O^