From 4f1e47bb82b8f12ec68ac573568ed09841fa8c66 Mon Sep 17 00:00:00 2001
From: "Matthew K. Fu JuYuan" <matthewfu@gmail.com>
Date: Thu, 10 May 2012 14:43:05 +0800
Subject: [PATCH 1/6] Don't display admin in any premission select. Fix not
 necessary header for UT data

---
 app/controllers/admin/new_interface_module_apps_controller.rb  | 2 +-
 .../panel/announcement/back_end/approvals_controller.rb        | 1 +
 .../back_end/bulletin_category_setting_controller.rb           | 1 +
 .../announcement/back_end/approvals/_privilege_user.html.erb   | 3 +--
 .../bulletin_category_setting/_privilege_user.html.erb         | 3 +--
 .../lib/tasks/data_for_ut_list_for_posting_anc.rake            | 2 +-
 .../announcement/lib/ut_list_from_nccu_2012feb.csv             | 1 -
 .../panel/news/back_end/news_approvals_controller.rb           | 1 +
 .../news/back_end/news_bulletin_category_setting_controller.rb | 1 +
 .../news/back_end/news_approvals/_privilege_user.html.erb      | 3 +--
 .../news_bulletin_category_setting/_privilege_user.html.erb    | 3 +--
 11 files changed, 10 insertions(+), 11 deletions(-)

diff --git a/app/controllers/admin/new_interface_module_apps_controller.rb b/app/controllers/admin/new_interface_module_apps_controller.rb
index 6603334c..8cf77e5e 100644
--- a/app/controllers/admin/new_interface_module_apps_controller.rb
+++ b/app/controllers/admin/new_interface_module_apps_controller.rb
@@ -6,7 +6,7 @@ class Admin::NewInterfaceModuleAppsController < ApplicationController
   layout "new_admin"
   
   def setting
-    @sys_users = User.all
+    @sys_users = User.all(conditions: {admin: false})
     @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
diff --git a/vendor/built_in_modules/announcement/app/controllers/panel/announcement/back_end/approvals_controller.rb b/vendor/built_in_modules/announcement/app/controllers/panel/announcement/back_end/approvals_controller.rb
index 9f63a8ab..e5c92023 100644
--- a/vendor/built_in_modules/announcement/app/controllers/panel/announcement/back_end/approvals_controller.rb
+++ b/vendor/built_in_modules/announcement/app/controllers/panel/announcement/back_end/approvals_controller.rb
@@ -12,6 +12,7 @@ class Panel::Announcement::BackEnd::ApprovalsController  < OrbitBackendControlle
   # end
   
   def setting
+    @sys_users = User.all(conditions: {admin: false})
     @bulletin_categorys = BulletinCategory.all
     @options_from_collection_for_select_bulletin_categorys = @bulletin_categorys.collect{|bc| [bc.i18n_variable[I18n.locale],bc.id] }
     if params.has_key? :category
diff --git a/vendor/built_in_modules/announcement/app/controllers/panel/announcement/back_end/bulletin_category_setting_controller.rb b/vendor/built_in_modules/announcement/app/controllers/panel/announcement/back_end/bulletin_category_setting_controller.rb
index 4be5f6bc..8903adb2 100644
--- a/vendor/built_in_modules/announcement/app/controllers/panel/announcement/back_end/bulletin_category_setting_controller.rb
+++ b/vendor/built_in_modules/announcement/app/controllers/panel/announcement/back_end/bulletin_category_setting_controller.rb
@@ -12,6 +12,7 @@ class Panel::Announcement::BackEnd::BulletinCategorySettingController < OrbitBac
   # end
   
   def setting
+    @sys_users = User.all(conditions: {admin: false})
     @bulletin_categorys = []
     @bulletin_categorys << BulletinCategory.find(params[:bulletin_category_id])
     @options_from_collection_for_select_bulletin_categorys = @bulletin_categorys.collect{|bc| [bc.i18n_variable[I18n.locale],bc.id] }
diff --git a/vendor/built_in_modules/announcement/app/views/panel/announcement/back_end/approvals/_privilege_user.html.erb b/vendor/built_in_modules/announcement/app/views/panel/announcement/back_end/approvals/_privilege_user.html.erb
index c36941de..90f5cb65 100644
--- a/vendor/built_in_modules/announcement/app/views/panel/announcement/back_end/approvals/_privilege_user.html.erb
+++ b/vendor/built_in_modules/announcement/app/views/panel/announcement/back_end/approvals/_privilege_user.html.erb
@@ -1,6 +1,5 @@
 <%= content_tag :div ,:id => "users_checkbox_ary",:class => 'clear' do -%>
-<% sys_users = User.all -%>
-	<% sys_users.each do |sys_user| -%>
+	<% @sys_users.each do |sys_user| -%>
 	
 			<div class="checkblock">
 				
diff --git a/vendor/built_in_modules/announcement/app/views/panel/announcement/back_end/bulletin_category_setting/_privilege_user.html.erb b/vendor/built_in_modules/announcement/app/views/panel/announcement/back_end/bulletin_category_setting/_privilege_user.html.erb
index c36941de..90f5cb65 100644
--- a/vendor/built_in_modules/announcement/app/views/panel/announcement/back_end/bulletin_category_setting/_privilege_user.html.erb
+++ b/vendor/built_in_modules/announcement/app/views/panel/announcement/back_end/bulletin_category_setting/_privilege_user.html.erb
@@ -1,6 +1,5 @@
 <%= content_tag :div ,:id => "users_checkbox_ary",:class => 'clear' do -%>
-<% sys_users = User.all -%>
-	<% sys_users.each do |sys_user| -%>
+	<% @sys_users.each do |sys_user| -%>
 	
 			<div class="checkblock">
 				
diff --git a/vendor/built_in_modules/announcement/lib/tasks/data_for_ut_list_for_posting_anc.rake b/vendor/built_in_modules/announcement/lib/tasks/data_for_ut_list_for_posting_anc.rake
index e5bd0be2..9b8b05e5 100644
--- a/vendor/built_in_modules/announcement/lib/tasks/data_for_ut_list_for_posting_anc.rake
+++ b/vendor/built_in_modules/announcement/lib/tasks/data_for_ut_list_for_posting_anc.rake
@@ -4,9 +4,9 @@ namespace :nccu_data do
   desc "load nccu data from csv"
   task :setup_ut_list => :environment do
     require 'csv'
-
     CSV.foreach("vendor/built_in_modules/announcement/lib/ut_list_from_nccu_2012feb.csv") do |row|
       new_unit = UnitListForAnc.new(:order => row[0], :ut_code => row[1], :up_ut_code => row[2], :created_at => Time.now,:updated_at => Time.now)
+      #    order,ut_code,up_tu_code,ut_zh_tw,ut_en
       new_unit.build_title :en => row[4], :zh_tw => row[3]
       new_unit.save
     end
diff --git a/vendor/built_in_modules/announcement/lib/ut_list_from_nccu_2012feb.csv b/vendor/built_in_modules/announcement/lib/ut_list_from_nccu_2012feb.csv
index 548ad528..806d0b9a 100644
--- a/vendor/built_in_modules/announcement/lib/ut_list_from_nccu_2012feb.csv
+++ b/vendor/built_in_modules/announcement/lib/ut_list_from_nccu_2012feb.csv
@@ -1,4 +1,3 @@
-order,ut_code,up_tu_code,ut_zh_tw,ut_en
 0100,H00,H00,校長室,Office of the President
 0110,H01,H01,副校長室一,Office of the Vice President
 0110,H03,H03,副校長室二,Office of the Vice President
diff --git a/vendor/built_in_modules/news/app/controllers/panel/news/back_end/news_approvals_controller.rb b/vendor/built_in_modules/news/app/controllers/panel/news/back_end/news_approvals_controller.rb
index c6eda9d0..80838ec5 100644
--- a/vendor/built_in_modules/news/app/controllers/panel/news/back_end/news_approvals_controller.rb
+++ b/vendor/built_in_modules/news/app/controllers/panel/news/back_end/news_approvals_controller.rb
@@ -12,6 +12,7 @@ class Panel::News::BackEnd::NewsApprovalsController  < OrbitBackendController
   end
 
   def setting
+    @sys_users = User.all(conditions: {admin: false})
     @news_bulletin_categorys = NewsBulletinCategory.all
     if params.has_key?(:category_id)
       first_category = NewsBulletinCategory.find params[:category_id]
diff --git a/vendor/built_in_modules/news/app/controllers/panel/news/back_end/news_bulletin_category_setting_controller.rb b/vendor/built_in_modules/news/app/controllers/panel/news/back_end/news_bulletin_category_setting_controller.rb
index 6b92195f..6ac76973 100644
--- a/vendor/built_in_modules/news/app/controllers/panel/news/back_end/news_bulletin_category_setting_controller.rb
+++ b/vendor/built_in_modules/news/app/controllers/panel/news/back_end/news_bulletin_category_setting_controller.rb
@@ -12,6 +12,7 @@ class Panel::News::BackEnd::NewsBulletinCategorySettingController < OrbitBackend
   # end
   
   def setting
+    @sys_users = User.all(conditions: {admin: false})
     @bulletin_categorys = []
     @bulletin_categorys << NewsBulletinCategory.find(params[:news_bulletin_category_id])
     @options_from_collection_for_select_bulletin_categorys = @bulletin_categorys.collect{|bc| [bc.i18n_variable[I18n.locale],bc.id] }
diff --git a/vendor/built_in_modules/news/app/views/panel/news/back_end/news_approvals/_privilege_user.html.erb b/vendor/built_in_modules/news/app/views/panel/news/back_end/news_approvals/_privilege_user.html.erb
index 87d4d78e..d48bb850 100644
--- a/vendor/built_in_modules/news/app/views/panel/news/back_end/news_approvals/_privilege_user.html.erb
+++ b/vendor/built_in_modules/news/app/views/panel/news/back_end/news_approvals/_privilege_user.html.erb
@@ -1,6 +1,5 @@
 <%= content_tag :div ,:id => "users_checkbox_ary",:class => 'clear' do -%>
-<% sys_users = User.all -%>
-	<% sys_users.each do |user| -%>
+	<% @sys_users.each do |user| -%>
 	
 			<div class="checkblock">
 				
diff --git a/vendor/built_in_modules/news/app/views/panel/news/back_end/news_bulletin_category_setting/_privilege_user.html.erb b/vendor/built_in_modules/news/app/views/panel/news/back_end/news_bulletin_category_setting/_privilege_user.html.erb
index c36941de..90f5cb65 100644
--- a/vendor/built_in_modules/news/app/views/panel/news/back_end/news_bulletin_category_setting/_privilege_user.html.erb
+++ b/vendor/built_in_modules/news/app/views/panel/news/back_end/news_bulletin_category_setting/_privilege_user.html.erb
@@ -1,6 +1,5 @@
 <%= content_tag :div ,:id => "users_checkbox_ary",:class => 'clear' do -%>
-<% sys_users = User.all -%>
-	<% sys_users.each do |sys_user| -%>
+	<% @sys_users.each do |sys_user| -%>
 	
 			<div class="checkblock">
 				

From cb25cf60a9a4b3e5b3d600987780feb5a382f2b6 Mon Sep 17 00:00:00 2001
From: "Matthew K. Fu JuYuan" <matthewfu@gmail.com>
Date: Thu, 10 May 2012 15:37:07 +0800
Subject: [PATCH 2/6] AdBanner app_auth

---
 app/views/layouts/_side_bar.html.erb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/views/layouts/_side_bar.html.erb b/app/views/layouts/_side_bar.html.erb
index dde071c5..0a6c6912 100644
--- a/app/views/layouts/_side_bar.html.erb
+++ b/app/views/layouts/_side_bar.html.erb
@@ -54,7 +54,7 @@
 		<%#= content_tag :li, link_to(t('admin.ad.all_banners'), admin_ad_banners_path), :class => active_for_action('ad_banners', 'index') %>
 		<%#= content_tag :li, link_to(t('admin.ad.new_banner'), new_admin_ad_banner_path), :class => active_for_action('ad_banners', 'new') %>
 		<%#= content_tag :li, link_to(t('admin.ad.new_image'), new_ad_image_admin_ad_banners_path), :class => active_for_action('ad_images', 'new') %>
-				<%= content_tag :li, link_to(t('admin.module.authorization'),edit_admin_module_app_path(ModuleApp.first(conditions: {title: "ad_banners"}))), :class => active_sys_call_for_app('module_apps','edit','ad_banners')  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: "ad_banners"}))), :class => active_sys_call_for_app('new_interface_module_apps','setting','ad_banners',:module_app_id)  if (is_admin? rescue nil) %>
 	<% end -%>
 <% end %>
 

From 20a1a6e30f87f08f7453c7faddd8550efd361f18 Mon Sep 17 00:00:00 2001
From: "Matthew K. Fu JuYuan" <matthewfu@gmail.com>
Date: Fri, 11 May 2012 16:44:40 +0800
Subject: [PATCH 3/6] New Interface for App Auth and Object Auth

---
 .../module_apps_new_interface_controller.rb   | 66 +++++++++++++++++
 .../object_auths_new_interface_controller.rb  | 71 +++++++++++++++++++
 app/helpers/application_helper.rb             | 20 +++++-
 .../_modal_list.html.erb                      | 42 +++++++++++
 .../_privilege_user.html.erb                  | 25 +++++++
 .../setting.html.erb                          | 56 +++++++++++++++
 .../module_apps_new_interface/setting.js.erb  |  2 +
 .../update_setting.js.erb                     |  1 +
 .../user_list.js.erb                          |  2 +
 .../_modal_list.html.erb                      | 42 +++++++++++
 .../_privilege_user.html.erb                  | 25 +++++++
 .../setting.html.erb                          | 57 +++++++++++++++
 .../object_auths_new_interface/setting.js.erb |  2 +
 .../update_setting.js.erb                     |  1 +
 .../user_list.js.erb                          |  2 +
 app/views/layouts/_side_bar.html.erb          | 16 ++---
 config/locales/en.yml                         |  4 ++
 config/locales/zh_tw.yml                      |  6 +-
 lib/orbit_core_lib.rb                         |  4 ++
 .../back_end/bulletin_categorys_helper.rb     |  2 +-
 .../app/models/bulletin_category.rb           |  4 ++
 .../news_bulletin_categorys_helper.rb         |  2 +-
 .../news/app/models/news_bulletin_category.rb |  4 ++
 23 files changed, 442 insertions(+), 14 deletions(-)
 create mode 100644 app/controllers/admin/module_apps_new_interface_controller.rb
 create mode 100644 app/controllers/admin/object_auths_new_interface_controller.rb
 create mode 100644 app/views/admin/module_apps_new_interface/_modal_list.html.erb
 create mode 100644 app/views/admin/module_apps_new_interface/_privilege_user.html.erb
 create mode 100644 app/views/admin/module_apps_new_interface/setting.html.erb
 create mode 100644 app/views/admin/module_apps_new_interface/setting.js.erb
 create mode 100644 app/views/admin/module_apps_new_interface/update_setting.js.erb
 create mode 100644 app/views/admin/module_apps_new_interface/user_list.js.erb
 create mode 100644 app/views/admin/object_auths_new_interface/_modal_list.html.erb
 create mode 100644 app/views/admin/object_auths_new_interface/_privilege_user.html.erb
 create mode 100644 app/views/admin/object_auths_new_interface/setting.html.erb
 create mode 100644 app/views/admin/object_auths_new_interface/setting.js.erb
 create mode 100644 app/views/admin/object_auths_new_interface/update_setting.js.erb
 create mode 100644 app/views/admin/object_auths_new_interface/user_list.js.erb

diff --git a/app/controllers/admin/module_apps_new_interface_controller.rb b/app/controllers/admin/module_apps_new_interface_controller.rb
new file mode 100644
index 00000000..52292588
--- /dev/null
+++ b/app/controllers/admin/module_apps_new_interface_controller.rb
@@ -0,0 +1,66 @@
+class Admin::ModuleAppsNewInterfaceController < ApplicationController
+  before_filter :authenticate_user!
+  before_filter :is_admin?
+  include AdminHelper
+
+  layout "new_admin"
+  
+  def setting
+    @sys_users = User.all(conditions: {admin: false})
+    @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/controllers/admin/object_auths_new_interface_controller.rb b/app/controllers/admin/object_auths_new_interface_controller.rb
new file mode 100644
index 00000000..427ec722
--- /dev/null
+++ b/app/controllers/admin/object_auths_new_interface_controller.rb
@@ -0,0 +1,71 @@
+class Admin::ObjectAuthsNewInterfaceController < ApplicationController
+  include OrbitCoreLib::PermissionUnility
+  layout "new_admin"
+  before_filter :force_order
+
+  layout "new_admin"
+  
+  
+  def setting
+    @sys_users = User.all(conditions: {admin: false})
+    @ob_auth = ObjectAuth.find params[:object_auth_id]
+    @options_from_collection_for_select_ob_auth = [@ob_auth].collect{|oa| [oa.auth_obj.pp_object,oa.id] }
+    @users_array = @ob_auth.privilege_users rescue []
+
+    respond_to do |format|
+      format.html
+      format.js
+    end
+  end
+  
+  def update_setting
+    ob_auth = update_setting_by_params
+    if ob_auth.save!
+      flash[:notice] = t("admin.object_auth.update_done")
+    else
+      flash[:notice] = t("admin.object_auth.update_failed")
+    end
+  end
+  
+  def user_list
+    @ob_auth = ObjectAuth.find params[:ob_auth][:id]
+  end
+
+  protected
+  def update_setting_by_params
+    oa = ObjectAuth.find params[:ob_auth][:id]
+    user_sat = User.find params[:users].keys
+    users_to_new = user_sat - oa.auth_users
+    users_to_remove = oa.auth_users - user_sat
+
+    users_to_new.each do |new_user|
+      oa.add_user_to_privilege_list(new_user)
+    end
+
+    users_to_remove.each do |remove_user|
+      oa.remove_user_from_privilege_list(remove_user)
+    end
+    oa
+  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
+
+  def force_order
+    authenticate_user!
+    check_if_user_can_do_object_auth
+  end
+
+  def check_if_user_can_do_object_auth
+    unless  check_permission(:manager)
+      render :nothing => true, :status => 403 
+    end
+  end
+
+end
\ No newline at end of file
diff --git a/app/helpers/application_helper.rb b/app/helpers/application_helper.rb
index 844fbe14..cd7a15dd 100644
--- a/app/helpers/application_helper.rb
+++ b/app/helpers/application_helper.rb
@@ -68,9 +68,23 @@ module ApplicationHelper
     end
   end
   
-  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[field]
+  def active_for_ob_auths_object(object_class,field = :object_auth_id)
+    unless active_for_action("object_auths_new_interface","setting").nil?
+      ob_auth = ObjectAuth.find params[field]
+      ob_auth.obj_authable_type == object_class.to_s ?  'active' : nil
+    end
+  end
+  
+  def active_for_ob_auth(ob_auth_title,field = :object_auth_id)
+    unless active_for_action("module_apps_new_interface","setting").nil?
+      oa_auth = ObjectAuth.find params[field]
+      oa_auth.title == ob_auth_title ?  'active' : nil
+    end
+  end
+  
+  def active_for_app_auth(app_title ='', opt={:controller_name => 'module_apps_new_interface',:action_name=>'setting',:field => :module_app_id})
+    unless active_for_action(opt[:controller_name],opt[:action_name]).nil?
+      app = ModuleApp.find params[opt[:field]]
       app.title == app_title ? 'active' : nil
     else
       nil
diff --git a/app/views/admin/module_apps_new_interface/_modal_list.html.erb b/app/views/admin/module_apps_new_interface/_modal_list.html.erb
new file mode 100644
index 00000000..b5b0ebcd
--- /dev/null
+++ b/app/views/admin/module_apps_new_interface/_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/module_apps_new_interface/_privilege_user.html.erb b/app/views/admin/module_apps_new_interface/_privilege_user.html.erb
new file mode 100644
index 00000000..90f5cb65
--- /dev/null
+++ b/app/views/admin/module_apps_new_interface/_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/module_apps_new_interface/setting.html.erb b/app/views/admin/module_apps_new_interface/setting.html.erb
new file mode 100644
index 00000000..7d1179f0
--- /dev/null
+++ b/app/views/admin/module_apps_new_interface/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/module_apps_new_interface/setting.js.erb b/app/views/admin/module_apps_new_interface/setting.js.erb
new file mode 100644
index 00000000..0ca148da
--- /dev/null
+++ b/app/views/admin/module_apps_new_interface/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/module_apps_new_interface/update_setting.js.erb b/app/views/admin/module_apps_new_interface/update_setting.js.erb
new file mode 100644
index 00000000..7486000a
--- /dev/null
+++ b/app/views/admin/module_apps_new_interface/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/module_apps_new_interface/user_list.js.erb b/app/views/admin/module_apps_new_interface/user_list.js.erb
new file mode 100644
index 00000000..7caf1e39
--- /dev/null
+++ b/app/views/admin/module_apps_new_interface/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/admin/object_auths_new_interface/_modal_list.html.erb b/app/views/admin/object_auths_new_interface/_modal_list.html.erb
new file mode 100644
index 00000000..04f46511
--- /dev/null
+++ b/app/views/admin/object_auths_new_interface/_modal_list.html.erb
@@ -0,0 +1,42 @@
+<% if ob_auth -%>
+
+<div class="modal hide fade in" id="ob_auth-<%=ob_auth.id%>">
+  <div class="modal-header">
+    <a class="close" data-dismiss="modal">×</a>
+    <h3><%= t("admin.object_auth.list_title_of_users",:auth_title => ob_auth.title) %></h3>
+  </div>
+  <div class="modal-body">
+		<div class="clear">
+			<% ob_auth.auth_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() {
+	
+			$("#ob_auth-<%=ob_auth.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/object_auths_new_interface/_privilege_user.html.erb b/app/views/admin/object_auths_new_interface/_privilege_user.html.erb
new file mode 100644
index 00000000..90f5cb65
--- /dev/null
+++ b/app/views/admin/object_auths_new_interface/_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/object_auths_new_interface/setting.html.erb b/app/views/admin/object_auths_new_interface/setting.html.erb
new file mode 100644
index 00000000..ae5096ba
--- /dev/null
+++ b/app/views/admin/object_auths_new_interface/setting.html.erb
@@ -0,0 +1,57 @@
+<% 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 :object, @ob_auth.auth_obj.class %>
+                <%= label_tag :module,"::" %>
+                <%= select "ob_auth",'id',@options_from_collection_for_select_ob_auth %>
+								<%= search_field_tag 'user_filter' %>
+            </div>
+
+        </li>
+    </ul>
+		<%= link_to t("admin.object_auth.list_title_of_users",:auth_title => @ob_auth.title), admin_object_auth_ob_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_object_auth_ob_auth_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/object_auths_new_interface/setting.js.erb b/app/views/admin/object_auths_new_interface/setting.js.erb
new file mode 100644
index 00000000..0ca148da
--- /dev/null
+++ b/app/views/admin/object_auths_new_interface/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/object_auths_new_interface/update_setting.js.erb b/app/views/admin/object_auths_new_interface/update_setting.js.erb
new file mode 100644
index 00000000..7486000a
--- /dev/null
+++ b/app/views/admin/object_auths_new_interface/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/object_auths_new_interface/user_list.js.erb b/app/views/admin/object_auths_new_interface/user_list.js.erb
new file mode 100644
index 00000000..ebb088cc
--- /dev/null
+++ b/app/views/admin/object_auths_new_interface/user_list.js.erb
@@ -0,0 +1,2 @@
+$('#show_preview').html("<%= escape_javascript(render(:partial => 'modal_list',:locals => {:ob_auth => @ob_auth})) %>");
+var start_modal_with_id = "ob_auth-<%=@ob_auth.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 0a6c6912..7a9bcf4e 100644
--- a/app/views/layouts/_side_bar.html.erb
+++ b/app/views/layouts/_side_bar.html.erb
@@ -5,29 +5,29 @@
 	<%= 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('new_interface_module_apps','setting','Announcement',:module_app_id) ) do -%>
+<%= content_tag :li, :class => (active_for_controllers('bulletins', '/panel/announcement/back_end/tags', 'bulletin_categorys', 'approvals') ||  active_for_app_auth('Announcement') || active_for_ob_auths_object("BulletinCategory") ) 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') %>
 		<%= content_tag :li, link_to(t('announcement.add_new'), new_panel_announcement_back_end_bulletin_path), :class => active_for_action('bulletins', 'new') %>
-		<%= 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.categories'), panel_announcement_back_end_bulletin_categorys_path), :class =>( active_for_action('bulletin_categorys', 'index') || active_for_ob_auths_object("BulletinCategory") ) %>
 		<%= 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'),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) %>
+		<%= content_tag :li, link_to(t('admin.module.authorization'),admin_module_app_manager_auth_proc_path(ModuleApp.first(conditions: {title: "Announcement"}))), :class => active_for_app_auth('Announcement') 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')||  active_sys_call_for_app('new_interface_module_apps','setting','news',:module_app_id)) do -%>
+<%= content_tag :li, :class =>( active_for_controllers('news_bulletins', '/panel/news/back_end/tags', 'news_bulletin_categorys', 'news_approvals')||  active_for_app_auth('news') || active_for_ob_auths_object("NewsBulletinCategory")) 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') %>
 		<%= content_tag :li, link_to(t('announcement.add_new'), new_panel_news_back_end_news_bulletin_path), :class => active_for_action('news_bulletins', 'new') %>
-		<%= 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.categories'), panel_news_back_end_news_bulletin_categorys_path), :class => active_for_action('news_bulletin_categorys', 'index') || active_for_ob_auths_object("NewsBulletinCategory") %>
 		<%= 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'),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) %>
+		<%= content_tag :li, link_to(t('admin.module.authorization'),admin_module_app_manager_auth_proc_path(ModuleApp.first(conditions: {key: "news"}))), :class => active_for_app_auth('news') if (is_admin? rescue nil) %>
 	<% end -%>
 
 <% end -%>
@@ -47,14 +47,14 @@
 	<%= link_to content_tag(:i, nil, :class => 'icons-window-block') + t('admin.design'), admin_designs_path %>
 <% end -%>
 
-<%= content_tag :li, :class => active_for_controllers('ad_banners', 'ad_images') || active_sys_call_for_app('module_apps','edit','ad_banners') do -%>
+<%= content_tag :li, :class => active_for_controllers('ad_banners', 'ad_images') || active_for_app_auth('ad_banners') do -%>
 	<%= link_to content_tag(:i, nil, :class => 'icons-link') + t('admin.ad_banner'), admin_ad_banners_path %>
 
 	<%= content_tag :ul, :class => ("nav nav-list " + visible_for_controllers('ad_banners', 'ad_images') ) do -%>
 		<%#= content_tag :li, link_to(t('admin.ad.all_banners'), admin_ad_banners_path), :class => active_for_action('ad_banners', 'index') %>
 		<%#= content_tag :li, link_to(t('admin.ad.new_banner'), new_admin_ad_banner_path), :class => active_for_action('ad_banners', 'new') %>
 		<%#= content_tag :li, link_to(t('admin.ad.new_image'), new_ad_image_admin_ad_banners_path), :class => active_for_action('ad_images', 'new') %>
-				<%= content_tag :li, link_to(t('admin.module.authorization'),admin_module_app_manager_auth_proc_path(ModuleApp.first(conditions: {title: "ad_banners"}))), :class => active_sys_call_for_app('new_interface_module_apps','setting','ad_banners',:module_app_id)  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: "ad_banners"}))), :class => active_for_app_auth('ad_banners')  if (is_admin? rescue nil) %>
 	<% end -%>
 <% end %>
 
diff --git a/config/locales/en.yml b/config/locales/en.yml
index 07bfe189..f5407124 100644
--- a/config/locales/en.yml
+++ b/config/locales/en.yml
@@ -197,6 +197,10 @@ en:
     new_role: New user role
     news: News
     non_multilingual: Non multilingual
+    object_auth:
+      list_title_of_users: %{auth_title} Auth List
+      update_done: Update done,reulst showing in list
+      update_failed: Update failed
     options: Options
     orig_upload_file: Original filename
     page: Page
diff --git a/config/locales/zh_tw.yml b/config/locales/zh_tw.yml
index ae1b92f3..29721630 100644
--- a/config/locales/zh_tw.yml
+++ b/config/locales/zh_tw.yml
@@ -191,7 +191,11 @@ zh_tw:
     new_info: 新增用戶資料
     new_role: 新增用戶身份
     news: 新聞
-    non_multilingual: 非多種語言
+    non_multilingual: 
+    object_auth:
+      list_title_of_users: 授權清單-%{auth_title}
+      update_done: 更新完成，結果顯示於清單
+      update_failed: 更新失敗
     options: 選項
     orig_upload_file: 原上傳檔名
     page: 頁面管理
diff --git a/lib/orbit_core_lib.rb b/lib/orbit_core_lib.rb
index b92f4eca..281654a7 100644
--- a/lib/orbit_core_lib.rb
+++ b/lib/orbit_core_lib.rb
@@ -19,6 +19,10 @@ module  OrbitCoreLib
         
     end
     
+    def pp_object
+      "Object Auth method 'pp_object' need to be defined for class #{self.class}"
+    end
+    
     def get_object_auth_by_title(title)
       self.object_auths.where({title: title }).first
     end
diff --git a/vendor/built_in_modules/announcement/app/helpers/panel/announcement/back_end/bulletin_categorys_helper.rb b/vendor/built_in_modules/announcement/app/helpers/panel/announcement/back_end/bulletin_categorys_helper.rb
index 1fccbbe1..92604985 100644
--- a/vendor/built_in_modules/announcement/app/helpers/panel/announcement/back_end/bulletin_categorys_helper.rb
+++ b/vendor/built_in_modules/announcement/app/helpers/panel/announcement/back_end/bulletin_categorys_helper.rb
@@ -10,7 +10,7 @@ include ActionView::Helpers::UrlHelper
       oa = bulletin_category.get_object_auth_by_title(type)
     end
 #    link_to t('announcement.bulletin.cate_auth'), edit_admin_object_auth_path(oa)
-     link_to t('announcement.bulletin.cate_auth'),panel_announcement_back_end_bulletin_category_setting_path(bulletin_category)
+     link_to t('announcement.bulletin.cate_auth'),admin_object_auth_ob_auth_path(oa)
   end
 
 end
\ No newline at end of file
diff --git a/vendor/built_in_modules/announcement/app/models/bulletin_category.rb b/vendor/built_in_modules/announcement/app/models/bulletin_category.rb
index 7daaf6db..955d5f18 100644
--- a/vendor/built_in_modules/announcement/app/models/bulletin_category.rb
+++ b/vendor/built_in_modules/announcement/app/models/bulletin_category.rb
@@ -18,6 +18,10 @@ class BulletinCategory
   
   has_many :bulletins
   
+  def pp_object
+    i18n_variable[I18n.locale]
+  end
+  
   def self.from_id(id)
     BulletinCategory.find(id) rescue nil
   end
diff --git a/vendor/built_in_modules/news/app/helpers/panel/news/back_end/news_bulletin_categorys_helper.rb b/vendor/built_in_modules/news/app/helpers/panel/news/back_end/news_bulletin_categorys_helper.rb
index 74905640..4b896555 100644
--- a/vendor/built_in_modules/news/app/helpers/panel/news/back_end/news_bulletin_categorys_helper.rb
+++ b/vendor/built_in_modules/news/app/helpers/panel/news/back_end/news_bulletin_categorys_helper.rb
@@ -10,7 +10,7 @@ include ActionView::Helpers::UrlHelper
       oa = news_bulletin_category.get_object_auth_by_title(type)
     end
 #    link_to t('announcement.bulletin.cate_auth'), edit_admin_object_auth_path(oa)
-link_to t('announcement.bulletin.cate_auth'),panel_news_back_end_news_bulletin_category_setting_path(news_bulletin_category)
+    link_to t('announcement.bulletin.cate_auth'),admin_object_auth_ob_auth_path(oa)
 
   end
 
diff --git a/vendor/built_in_modules/news/app/models/news_bulletin_category.rb b/vendor/built_in_modules/news/app/models/news_bulletin_category.rb
index 45576589..b3dd7bc4 100644
--- a/vendor/built_in_modules/news/app/models/news_bulletin_category.rb
+++ b/vendor/built_in_modules/news/app/models/news_bulletin_category.rb
@@ -18,6 +18,10 @@ class NewsBulletinCategory
   
   has_many :news_bulletins
   
+  def pp_object
+    i18n_variable[I18n.locale]
+  end
+  
   def self.from_id(id)
     NewsBulletinCategory.find(id) rescue nil
   end

From 0a34f0277eec1b7ced7dbd7d9da3dd58b33d64e8 Mon Sep 17 00:00:00 2001
From: "Matthew K. Fu JuYuan" <matthewfu@gmail.com>
Date: Fri, 11 May 2012 16:46:36 +0800
Subject: [PATCH 4/6] Remove unless files and change routes.

---
 .../new_interface_module_apps_controller.rb   | 66 -------------------
 .../_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 -
 config/routes.rb                              | 11 +++-
 .../bulletin_category_setting_controller.rb   | 64 ------------------
 .../_modal_list.html.erb                      | 44 -------------
 .../_privilege_user.html.erb                  | 25 -------
 .../setting.html.erb                          | 56 ----------------
 .../bulletin_category_setting/setting.js.erb  |  2 -
 .../update_setting.js.erb                     |  1 -
 .../user_list.js.erb                          |  2 -
 .../announcement/config/routes.rb             |  6 +-
 ...ws_bulletin_category_setting_controller.rb | 64 ------------------
 .../_modal_list.html.erb                      | 44 -------------
 .../_privilege_user.html.erb                  | 25 -------
 .../setting.html.erb                          | 56 ----------------
 .../setting.js.erb                            |  2 -
 .../update_setting.js.erb                     |  1 -
 .../user_list.js.erb                          |  2 -
 vendor/built_in_modules/news/config/routes.rb |  6 +-
 24 files changed, 10 insertions(+), 595 deletions(-)
 delete mode 100644 app/controllers/admin/new_interface_module_apps_controller.rb
 delete mode 100644 app/views/admin/new_interface_module_apps/_modal_list.html.erb
 delete mode 100644 app/views/admin/new_interface_module_apps/_privilege_user.html.erb
 delete mode 100644 app/views/admin/new_interface_module_apps/setting.html.erb
 delete mode 100644 app/views/admin/new_interface_module_apps/setting.js.erb
 delete mode 100644 app/views/admin/new_interface_module_apps/update_setting.js.erb
 delete mode 100644 app/views/admin/new_interface_module_apps/user_list.js.erb
 delete mode 100644 vendor/built_in_modules/announcement/app/controllers/panel/announcement/back_end/bulletin_category_setting_controller.rb
 delete mode 100644 vendor/built_in_modules/announcement/app/views/panel/announcement/back_end/bulletin_category_setting/_modal_list.html.erb
 delete mode 100644 vendor/built_in_modules/announcement/app/views/panel/announcement/back_end/bulletin_category_setting/_privilege_user.html.erb
 delete mode 100644 vendor/built_in_modules/announcement/app/views/panel/announcement/back_end/bulletin_category_setting/setting.html.erb
 delete mode 100644 vendor/built_in_modules/announcement/app/views/panel/announcement/back_end/bulletin_category_setting/setting.js.erb
 delete mode 100644 vendor/built_in_modules/announcement/app/views/panel/announcement/back_end/bulletin_category_setting/update_setting.js.erb
 delete mode 100644 vendor/built_in_modules/announcement/app/views/panel/announcement/back_end/bulletin_category_setting/user_list.js.erb
 delete mode 100644 vendor/built_in_modules/news/app/controllers/panel/news/back_end/news_bulletin_category_setting_controller.rb
 delete mode 100644 vendor/built_in_modules/news/app/views/panel/news/back_end/news_bulletin_category_setting/_modal_list.html.erb
 delete mode 100644 vendor/built_in_modules/news/app/views/panel/news/back_end/news_bulletin_category_setting/_privilege_user.html.erb
 delete mode 100644 vendor/built_in_modules/news/app/views/panel/news/back_end/news_bulletin_category_setting/setting.html.erb
 delete mode 100644 vendor/built_in_modules/news/app/views/panel/news/back_end/news_bulletin_category_setting/setting.js.erb
 delete mode 100644 vendor/built_in_modules/news/app/views/panel/news/back_end/news_bulletin_category_setting/update_setting.js.erb
 delete mode 100644 vendor/built_in_modules/news/app/views/panel/news/back_end/news_bulletin_category_setting/user_list.js.erb

diff --git a/app/controllers/admin/new_interface_module_apps_controller.rb b/app/controllers/admin/new_interface_module_apps_controller.rb
deleted file mode 100644
index 8cf77e5e..00000000
--- a/app/controllers/admin/new_interface_module_apps_controller.rb
+++ /dev/null
@@ -1,66 +0,0 @@
-class Admin::NewInterfaceModuleAppsController < ApplicationController
-  before_filter :authenticate_user!
-  before_filter :is_admin?
-  include AdminHelper
-
-  layout "new_admin"
-  
-  def setting
-    @sys_users = User.all(conditions: {admin: false})
-    @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/views/admin/new_interface_module_apps/_modal_list.html.erb b/app/views/admin/new_interface_module_apps/_modal_list.html.erb
deleted file mode 100644
index b5b0ebcd..00000000
--- a/app/views/admin/new_interface_module_apps/_modal_list.html.erb
+++ /dev/null
@@ -1,42 +0,0 @@
-<% 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
deleted file mode 100644
index 90f5cb65..00000000
--- a/app/views/admin/new_interface_module_apps/_privilege_user.html.erb
+++ /dev/null
@@ -1,25 +0,0 @@
-<%= 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
deleted file mode 100644
index 7d1179f0..00000000
--- a/app/views/admin/new_interface_module_apps/setting.html.erb
+++ /dev/null
@@ -1,56 +0,0 @@
-<% 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
deleted file mode 100644
index 0ca148da..00000000
--- a/app/views/admin/new_interface_module_apps/setting.js.erb
+++ /dev/null
@@ -1,2 +0,0 @@
-$("#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
deleted file mode 100644
index 7486000a..00000000
--- a/app/views/admin/new_interface_module_apps/update_setting.js.erb
+++ /dev/null
@@ -1 +0,0 @@
-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
deleted file mode 100644
index 7caf1e39..00000000
--- a/app/views/admin/new_interface_module_apps/user_list.js.erb
+++ /dev/null
@@ -1,2 +0,0 @@
-$('#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/config/routes.rb b/config/routes.rb
index 863d17fd..065bba1d 100644
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -16,6 +16,11 @@ Orbit::Application.routes.draw do
     resources :assets
     resources :app_auths 
     resources :object_auths  do
+      match 'new_interface/:ob_type/:title/new' => "object_auths_new_interface#new" ,:as => :init_ob_auth,:via => :get
+      match 'new_interface' => "object_auths_new_interface#setting" ,:as => :ob_auth,:via => :get
+      match 'new_interface' => "object_auths_new_interface#update_setting" ,:as => :ob_auth,:via => :post
+      match 'new_interface' => "object_auths_new_interface#user_list" ,:as => :ob_auth_show,:via => :put
+      
       collection do
         match 'new/:type/:obj_id',:action => 'new',:via => "get",:as => :init
       end
@@ -61,9 +66,9 @@ Orbit::Application.routes.draw do
       end
     end
     resources :module_apps do
-      match 'manager_auth_proc' => "new_interface_module_apps#setting" ,:as => :manager_auth_proc,:via => :get
-      match 'manager_auth_proc' => "new_interface_module_apps#update_setting" ,:as => :manager_auth_proc,:via => :post
-      match 'manager_auth_proc' => "new_interface_module_apps#user_list" ,:as => :manager_auth_show,:via => :put
+      match 'manager_auth_proc' => "module_apps_new_interface#setting" ,:as => :manager_auth_proc,:via => :get
+      match 'manager_auth_proc' => "module_apps_new_interface#update_setting" ,:as => :manager_auth_proc,:via => :post
+      match 'manager_auth_proc' => "module_apps_new_interface#user_list" ,:as => :manager_auth_show,:via => :put
       resources :app_auths do
         member do
           match 'remove/:type/:target_id' ,:action=> 'remove',:via => "delete",:as =>:remove
diff --git a/vendor/built_in_modules/announcement/app/controllers/panel/announcement/back_end/bulletin_category_setting_controller.rb b/vendor/built_in_modules/announcement/app/controllers/panel/announcement/back_end/bulletin_category_setting_controller.rb
deleted file mode 100644
index 8903adb2..00000000
--- a/vendor/built_in_modules/announcement/app/controllers/panel/announcement/back_end/bulletin_category_setting_controller.rb
+++ /dev/null
@@ -1,64 +0,0 @@
-class Panel::Announcement::BackEnd::BulletinCategorySettingController < OrbitBackendController
-  before_filter :authenticate_user!
-  before_filter :for_app_manager
-  include AdminHelper
-  # layout 'admin'
-  def preview_and_approve
-    @bulletin = Bulletin.find params[:bulletin_id]
-  end
-  
-  # def approve
-  #   
-  # end
-  
-  def setting
-    @sys_users = User.all(conditions: {admin: false})
-    @bulletin_categorys = []
-    @bulletin_categorys << BulletinCategory.find(params[:bulletin_category_id])
-    @options_from_collection_for_select_bulletin_categorys = @bulletin_categorys.collect{|bc| [bc.i18n_variable[I18n.locale],bc.id] }
-    if params.has_key? :category
-      @bulletin_category = BulletinCategory.find params[:category][:id]
-    else
-      @bulletin_category =  @bulletin_categorys.first
-    end
-    preload_object_auth = @bulletin_category.object_auths.where(title: 'submit').empty?? (@bulletin_category.object_auths.create! :title=> 'submit') : @bulletin_category.object_auths.where(title: 'submit')
-    @users_array =  preload_object_auth.first.privilege_users rescue []
-    respond_to do |format|
-      format.html
-      format.js
-    end
-  end
-  
-  def update_setting
-    object_auth = update_setting_by_params
-    if object_auth.save!
-      flash[:notice] = "Update Done"
-    else
-      flash[:notice] = "Update Failed"
-    end
-  end
-  
-  def user_list
-    @bulletin_category = BulletinCategory.find params[:category][:id]
-  end
-
-  protected
-  def update_setting_by_params
-    category = BulletinCategory.find params[:category][:id]
-    privilege_users = params[:users].collect{|key,value| User.find key }  rescue []
-    object_auth_ary = category.object_auths.where(title: 'submit') || (category.object_auths.create :title=> 'submit')
-    object_auth = object_auth_ary.first
-    object_auth.privilege_users = privilege_users
-    object_auth
-  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
diff --git a/vendor/built_in_modules/announcement/app/views/panel/announcement/back_end/bulletin_category_setting/_modal_list.html.erb b/vendor/built_in_modules/announcement/app/views/panel/announcement/back_end/bulletin_category_setting/_modal_list.html.erb
deleted file mode 100644
index a73fdab2..00000000
--- a/vendor/built_in_modules/announcement/app/views/panel/announcement/back_end/bulletin_category_setting/_modal_list.html.erb
+++ /dev/null
@@ -1,44 +0,0 @@
-<% if bulletin_category -%>
-
-<div class="modal hide fade in" id="bulletin_category-<%=bulletin_category.id%>">
-  <div class="modal-header">
-    <a class="close" data-dismiss="modal">×</a>
-    <h3><%= t("announcement.bulletin.approval_user_list") %></h3>
-  </div>
-  <div class="modal-body">
-		<div class="clear">
-			<% bulletin_category.get_object_auth_by_title('submit').auth_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() {
-	
-			$("#bulletin_category-<%=bulletin_category.id%>").on("show", function () {
-			});
-		$(".modal").on("hidden", function () {
-	  $("#show_preview").remove();
-			});
-		});
-		</script>
-	</div>
-</div>
-
-
-
-<% end -%>
\ No newline at end of file
diff --git a/vendor/built_in_modules/announcement/app/views/panel/announcement/back_end/bulletin_category_setting/_privilege_user.html.erb b/vendor/built_in_modules/announcement/app/views/panel/announcement/back_end/bulletin_category_setting/_privilege_user.html.erb
deleted file mode 100644
index 90f5cb65..00000000
--- a/vendor/built_in_modules/announcement/app/views/panel/announcement/back_end/bulletin_category_setting/_privilege_user.html.erb
+++ /dev/null
@@ -1,25 +0,0 @@
-<%= 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/vendor/built_in_modules/announcement/app/views/panel/announcement/back_end/bulletin_category_setting/setting.html.erb b/vendor/built_in_modules/announcement/app/views/panel/announcement/back_end/bulletin_category_setting/setting.html.erb
deleted file mode 100644
index d20bfc27..00000000
--- a/vendor/built_in_modules/announcement/app/views/panel/announcement/back_end/bulletin_category_setting/setting.html.erb
+++ /dev/null
@@ -1,56 +0,0 @@
-<% 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 :category, t("announcement.bulletin.category") %>
-                <%= select "category",'id',@options_from_collection_for_select_bulletin_categorys %>
-								<%= search_field_tag 'user_filter' %>
-            </div>
-
-        </li>
-    </ul>
-		<%= link_to t("announcement.bulletin.approval_user_list"), panel_announcement_back_end_bulletin_category_user_list_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 panel_announcement_back_end_approval_setting_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/vendor/built_in_modules/announcement/app/views/panel/announcement/back_end/bulletin_category_setting/setting.js.erb b/vendor/built_in_modules/announcement/app/views/panel/announcement/back_end/bulletin_category_setting/setting.js.erb
deleted file mode 100644
index 0ca148da..00000000
--- a/vendor/built_in_modules/announcement/app/views/panel/announcement/back_end/bulletin_category_setting/setting.js.erb
+++ /dev/null
@@ -1,2 +0,0 @@
-$("#users_checkbox_ary").replaceWith('<%= escape_javascript(render :partial => "privilege_user", :locals => {:users => @users_array})%>');
-permissionCheckbox();
\ No newline at end of file
diff --git a/vendor/built_in_modules/announcement/app/views/panel/announcement/back_end/bulletin_category_setting/update_setting.js.erb b/vendor/built_in_modules/announcement/app/views/panel/announcement/back_end/bulletin_category_setting/update_setting.js.erb
deleted file mode 100644
index 7486000a..00000000
--- a/vendor/built_in_modules/announcement/app/views/panel/announcement/back_end/bulletin_category_setting/update_setting.js.erb
+++ /dev/null
@@ -1 +0,0 @@
-alert( "<% flash.each do |key, msg| %><%= msg %><% end%>");
\ No newline at end of file
diff --git a/vendor/built_in_modules/announcement/app/views/panel/announcement/back_end/bulletin_category_setting/user_list.js.erb b/vendor/built_in_modules/announcement/app/views/panel/announcement/back_end/bulletin_category_setting/user_list.js.erb
deleted file mode 100644
index 4c8bf25d..00000000
--- a/vendor/built_in_modules/announcement/app/views/panel/announcement/back_end/bulletin_category_setting/user_list.js.erb
+++ /dev/null
@@ -1,2 +0,0 @@
-$('#show_preview').html("<%= escape_javascript(render(:partial => 'modal_list',:locals => {:bulletin_category => @bulletin_category})) %>");
-var start_modal_with_id = "bulletin_category-<%=@bulletin_category.id%>"
\ No newline at end of file
diff --git a/vendor/built_in_modules/announcement/config/routes.rb b/vendor/built_in_modules/announcement/config/routes.rb
index c6a406b0..a9f1c496 100644
--- a/vendor/built_in_modules/announcement/config/routes.rb
+++ b/vendor/built_in_modules/announcement/config/routes.rb
@@ -21,11 +21,7 @@ Rails.application.routes.draw do
           match "file_quick_edit/:bulletin_id" => "bulletins#file_quick_edit" ,:as => :file_quick_edit
         end
 		
-        resources :bulletin_categorys do
-          match 'submit_setting' => "bulletin_category_setting#setting" ,:as => :setting,:via => :get
-          match 'submit_setting' => "bulletin_category_setting#update_setting" ,:as => :setting,:via => :post
-          match 'submit_setting' => "bulletin_category_setting#user_list" ,:as => :user_list,:via => :put
-        end
+        resources :bulletin_categorys 
 		
 		resources :bulletin_links, :controller => 'bulletin_links' do
           match "link_quick_edit/:bulletin_link_id" => "bulletin_links#link_quick_edit" ,:as => :link_quick_edit
diff --git a/vendor/built_in_modules/news/app/controllers/panel/news/back_end/news_bulletin_category_setting_controller.rb b/vendor/built_in_modules/news/app/controllers/panel/news/back_end/news_bulletin_category_setting_controller.rb
deleted file mode 100644
index 6ac76973..00000000
--- a/vendor/built_in_modules/news/app/controllers/panel/news/back_end/news_bulletin_category_setting_controller.rb
+++ /dev/null
@@ -1,64 +0,0 @@
-class Panel::News::BackEnd::NewsBulletinCategorySettingController < OrbitBackendController
-  before_filter :authenticate_user!
-  before_filter :for_app_manager
-  include AdminHelper
-  # layout 'admin'
-  # def preview_and_approve
-  #   @bulletin = Bulletin.find params[:bulletin_id]
-  # end
-  
-  # def approve
-  #   
-  # end
-  
-  def setting
-    @sys_users = User.all(conditions: {admin: false})
-    @bulletin_categorys = []
-    @bulletin_categorys << NewsBulletinCategory.find(params[:news_bulletin_category_id])
-    @options_from_collection_for_select_bulletin_categorys = @bulletin_categorys.collect{|bc| [bc.i18n_variable[I18n.locale],bc.id] }
-    if params.has_key? :category
-      @bulletin_category = NewsBulletinCategory.find params[:category][:id]
-    else
-      @bulletin_category =  @bulletin_categorys.first
-    end
-    preload_object_auth = @bulletin_category.object_auths.where(title: 'submit').empty?? (@bulletin_category.object_auths.create! :title=> 'submit') : @bulletin_category.object_auths.where(title: 'submit')
-    @users_array =  preload_object_auth.first.privilege_users rescue []
-    respond_to do |format|
-      format.html
-      format.js
-    end
-  end
-  
-  def update_setting
-    object_auth = update_setting_by_params
-    if object_auth.save!
-      flash[:notice] = "Update Done"
-    else
-      flash[:notice] = "Update Failed"
-    end
-  end
-  
-  def user_list
-    @bulletin_category = NewsBulletinCategory.find params[:category][:id]
-  end
-
-  protected
-  def update_setting_by_params
-    category = NewsBulletinCategory.find params[:category][:id]
-    privilege_users = params[:users].collect{|key,value| User.find key }  rescue []
-    object_auth_ary = category.object_auths.where(title: 'submit') || (category.object_auths.create :title=> 'submit')
-    object_auth = object_auth_ary.first
-    object_auth.privilege_users = privilege_users
-    object_auth
-  end
-  
-  def get_categorys(id = nil)
-    @bulletin_categorys = []
-    if(is_manager? || is_admin?)
-      @bulletin_categorys = (id ? NewsBulletinCategory.find(id).to_a : NewsBulletinCategory.excludes('disabled' => true))
-    elsif is_sub_manager?
-      @bulletin_categorys = NewsBulletinCategory.authed_for_user(current_user,'submit_new')
-    end
-  end
-
-end
diff --git a/vendor/built_in_modules/news/app/views/panel/news/back_end/news_bulletin_category_setting/_modal_list.html.erb b/vendor/built_in_modules/news/app/views/panel/news/back_end/news_bulletin_category_setting/_modal_list.html.erb
deleted file mode 100644
index a73fdab2..00000000
--- a/vendor/built_in_modules/news/app/views/panel/news/back_end/news_bulletin_category_setting/_modal_list.html.erb
+++ /dev/null
@@ -1,44 +0,0 @@
-<% if bulletin_category -%>
-
-<div class="modal hide fade in" id="bulletin_category-<%=bulletin_category.id%>">
-  <div class="modal-header">
-    <a class="close" data-dismiss="modal">×</a>
-    <h3><%= t("announcement.bulletin.approval_user_list") %></h3>
-  </div>
-  <div class="modal-body">
-		<div class="clear">
-			<% bulletin_category.get_object_auth_by_title('submit').auth_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() {
-	
-			$("#bulletin_category-<%=bulletin_category.id%>").on("show", function () {
-			});
-		$(".modal").on("hidden", function () {
-	  $("#show_preview").remove();
-			});
-		});
-		</script>
-	</div>
-</div>
-
-
-
-<% end -%>
\ No newline at end of file
diff --git a/vendor/built_in_modules/news/app/views/panel/news/back_end/news_bulletin_category_setting/_privilege_user.html.erb b/vendor/built_in_modules/news/app/views/panel/news/back_end/news_bulletin_category_setting/_privilege_user.html.erb
deleted file mode 100644
index 90f5cb65..00000000
--- a/vendor/built_in_modules/news/app/views/panel/news/back_end/news_bulletin_category_setting/_privilege_user.html.erb
+++ /dev/null
@@ -1,25 +0,0 @@
-<%= 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/vendor/built_in_modules/news/app/views/panel/news/back_end/news_bulletin_category_setting/setting.html.erb b/vendor/built_in_modules/news/app/views/panel/news/back_end/news_bulletin_category_setting/setting.html.erb
deleted file mode 100644
index 618f7825..00000000
--- a/vendor/built_in_modules/news/app/views/panel/news/back_end/news_bulletin_category_setting/setting.html.erb
+++ /dev/null
@@ -1,56 +0,0 @@
-<% 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 :category, t("announcement.bulletin.category") %>
-                <%= select "category",'id',@options_from_collection_for_select_bulletin_categorys %>
-								<%= search_field_tag 'user_filter' %>
-            </div>
-
-        </li>
-    </ul>
-		<%= link_to t("announcement.bulletin.approval_user_list"), panel_news_back_end_news_bulletin_category_user_list_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 panel_news_back_end_news_bulletin_category_setting_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/vendor/built_in_modules/news/app/views/panel/news/back_end/news_bulletin_category_setting/setting.js.erb b/vendor/built_in_modules/news/app/views/panel/news/back_end/news_bulletin_category_setting/setting.js.erb
deleted file mode 100644
index 0ca148da..00000000
--- a/vendor/built_in_modules/news/app/views/panel/news/back_end/news_bulletin_category_setting/setting.js.erb
+++ /dev/null
@@ -1,2 +0,0 @@
-$("#users_checkbox_ary").replaceWith('<%= escape_javascript(render :partial => "privilege_user", :locals => {:users => @users_array})%>');
-permissionCheckbox();
\ No newline at end of file
diff --git a/vendor/built_in_modules/news/app/views/panel/news/back_end/news_bulletin_category_setting/update_setting.js.erb b/vendor/built_in_modules/news/app/views/panel/news/back_end/news_bulletin_category_setting/update_setting.js.erb
deleted file mode 100644
index 7486000a..00000000
--- a/vendor/built_in_modules/news/app/views/panel/news/back_end/news_bulletin_category_setting/update_setting.js.erb
+++ /dev/null
@@ -1 +0,0 @@
-alert( "<% flash.each do |key, msg| %><%= msg %><% end%>");
\ No newline at end of file
diff --git a/vendor/built_in_modules/news/app/views/panel/news/back_end/news_bulletin_category_setting/user_list.js.erb b/vendor/built_in_modules/news/app/views/panel/news/back_end/news_bulletin_category_setting/user_list.js.erb
deleted file mode 100644
index 4c8bf25d..00000000
--- a/vendor/built_in_modules/news/app/views/panel/news/back_end/news_bulletin_category_setting/user_list.js.erb
+++ /dev/null
@@ -1,2 +0,0 @@
-$('#show_preview').html("<%= escape_javascript(render(:partial => 'modal_list',:locals => {:bulletin_category => @bulletin_category})) %>");
-var start_modal_with_id = "bulletin_category-<%=@bulletin_category.id%>"
\ No newline at end of file
diff --git a/vendor/built_in_modules/news/config/routes.rb b/vendor/built_in_modules/news/config/routes.rb
index ccefcf38..b7882d2d 100644
--- a/vendor/built_in_modules/news/config/routes.rb
+++ b/vendor/built_in_modules/news/config/routes.rb
@@ -20,11 +20,7 @@ Rails.application.routes.draw do
           match "file_quick_edit/:news_bulletin_id" => "news_bulletins#file_quick_edit" ,:as => :file_quick_edit
         end
 		
-        resources :news_bulletin_categorys do
-          match 'submit_setting' => "news_bulletin_category_setting#setting" ,:as => :setting,:via => :get
-          match 'submit_setting' => "news_bulletin_category_setting#update_setting" ,:as => :setting,:via => :post
-          match 'submit_setting' => "news_bulletin_category_setting#user_list" ,:as => :user_list,:via => :put
-        end
+        resources :news_bulletin_categorys
 		
 		resources :news_bulletin_links, :controller => 'news_bulletin_links' do
           match "link_quick_edit/:news_bulletin_link_id" => "news_bulletin_links#link_quick_edit" ,:as => :link_quick_edit

From 82d1d3fed8e5a38c34f1dbfa5e39e455b19a3a7e Mon Sep 17 00:00:00 2001
From: "Matthew K. Fu JuYuan" <matthewfu@gmail.com>
Date: Fri, 11 May 2012 16:56:32 +0800
Subject: [PATCH 5/6] Restart job

---
 app/jobs/restart_server.rb | 8 ++++++++
 config/resque_schedule.yml | 6 ++++++
 2 files changed, 14 insertions(+)
 create mode 100644 app/jobs/restart_server.rb

diff --git a/app/jobs/restart_server.rb b/app/jobs/restart_server.rb
new file mode 100644
index 00000000..53167fea
--- /dev/null
+++ b/app/jobs/restart_server.rb
@@ -0,0 +1,8 @@
+class RestartServer 
+  @queue = :high
+
+  def self.perform()
+    CronMail.time_check("Going to restarting Orbit").deliver
+    %x[touch  #{Rails.root}/tmp/restart]
+  end
+end
diff --git a/config/resque_schedule.yml b/config/resque_schedule.yml
index 6929152e..60194101 100644
--- a/config/resque_schedule.yml
+++ b/config/resque_schedule.yml
@@ -10,6 +10,12 @@ nccu_daily_ldap_sync:
   args:
   description: Sync NCCU middle site with Orbit for LDAP data
 
+nccu_daily_restart:
+  cron: * * */3 * * *
+  class: RestartServer
+  args:
+  description: Restart NCCU  site
+
 nccu_claender_sync:
   cron: * * [0,12] * * *
   class: NccuCalendar

From e696516aae2264bf16134b1d89a9a97ebde57f08 Mon Sep 17 00:00:00 2001
From: "Matthew K. Fu JuYuan" <matthewfu@gmail.com>
Date: Fri, 11 May 2012 17:07:21 +0800
Subject: [PATCH 6/6] Fix for manager and admin can see all post in
 announcement. Haven't applied to news

---
 .../announcement/back_end/bulletins_controller.rb      | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/vendor/built_in_modules/announcement/app/controllers/panel/announcement/back_end/bulletins_controller.rb b/vendor/built_in_modules/announcement/app/controllers/panel/announcement/back_end/bulletins_controller.rb
index 7d89183e..325f60f7 100644
--- a/vendor/built_in_modules/announcement/app/controllers/panel/announcement/back_end/bulletins_controller.rb
+++ b/vendor/built_in_modules/announcement/app/controllers/panel/announcement/back_end/bulletins_controller.rb
@@ -43,10 +43,12 @@ class Panel::Announcement::BackEnd::BulletinsController < OrbitBackendController
     @bulletin_file = BulletinFile.new
     @file_url = panel_announcement_back_end_bulletins_path
 	
-	  @bulletins.delete_if{ |bulletin|
-	    bulletin.is_pending == true && (!bulletin.bulletin_category.authed_users('fact_check').include?(current_user) || bulletin.create_user_id!=current_user.id)
-	    }
-
+    if(!is_admin? || !is_manager?)
+      @bulletins.delete_if{ |bulletin|
+              bulletin.is_pending == true && (!bulletin.bulletin_category.authed_users('fact_check').include?(current_user) || bulletin.create_user_id!=current_user.id)
+            }
+        end
+    
     respond_to do |format|
       format.html # index.html.erb
       format.js { }