fix new ui member account privilege

app/controllers/admin/users_new_interface_controller.rb

@@ -181,6 +181,16 @@ class Admin::UsersNewInterfaceController <  OrbitMemberController
   def edit_passwd
     # @disable_all_field = false
     @user = User.find(params[:users_new_interface_id])
+    if current_user.id == @user.id
+      redirect_to :action => :index
+    end
+  end
+
+  def edit_privilege
+    @user = User.find(params[:users_new_interface_id])
+    if current_user.id == @user.id
+      redirect_to :action => :index
+    end
   end
   
   def update
@@ -203,6 +213,20 @@ class Admin::UsersNewInterfaceController <  OrbitMemberController
         render :action => :edit_passwd, :notice => flash.now[:error]
       end
 
+    elsif params[:edit_type] == 'edit_privilege'
+
+      @checkuser = User.where(:user_id => params[:user][:user_id]).first
+
+      if @user.admin && current_user.id == @user.id
+        flash.now[:error] = t("users.admin_change_password")
+      end
+
+      if !flash.now[:error] && @user.update_attributes(params[:user])
+        redirect_to :action => :show
+      else
+        render :action => :edit_privilege, :notice => flash.now[:error]
+      end
+
     else
 
       @user.update_attributes(params[:user])

app/controllers/application_controller.rb

@@ -140,21 +140,23 @@ class ApplicationController < ActionController::Base
     # update session if passed
     session[:locale] = params[:locale] if params[:locale]
     browser_locale = condition = nil
-    default_locale = @site.default_locale if @site.default_locale.present?
-    # check if locale is valid for non site pages
+    default_locale = condition = nil
 
+    # check if locale is valid for non site pages
     if LIST[:forbidden_item_names].include?(env['PATH_INFO'].split('/')[1].to_s)
       condition = VALID_LOCALES.include?(session[:locale])
-    elsif session[:locale]
-      condition = @site.in_use_locales.include?(session[:locale])
+    #check if the default locale is present or not
     elsif @site.default_locale.present?
       default_locale = @site.default_locale
-      condition = @site.valid_locales.include?(default_locale)
-    else
+      condition = @site.in_use_locales.include?(default_locale)
+    elsif !@site.enable_language_detection
+      #check enable langauge detection
       browser_locale = request.env['HTTP_ACCEPT_LANGUAGE'].scan(/^[a-z]{2}/).first rescue nil
       condition = @site.in_use_locales.include?(browser_locale)
+    elsif session[:locale]
+      condition = @site.in_use_locales.include?(session[:locale])
     end
-    session[:locale] = condition ? ( browser_locale || session[:locale] || default_locale) : I18n.default_locale.to_s
+    session[:locale] = condition ? (default_locale || browser_locale || session[:locale]) : I18n.default_locale.to_s
     I18n.locale = session[:locale].to_sym
     @site_in_use_locales = site_locales_default_head(@site.in_use_locales)
     @site_valid_locales = site_locales_default_head(@site.valid_locales)

app/models/site.rb

@@ -7,6 +7,7 @@ class Site
   
   field :in_use_locales, :type => Array
   field :valid_locales, :type => Array
+  field :enable_locales, :type => Array
   
   field :roaming_id
   field :private_key, :type => Binary
@@ -17,6 +18,7 @@ class Site
   field :desktop_closed, :type => Boolean, :default => false
   field :sitemap_menu_in_header, :type => Boolean, :default => false
   field :enable_terms_of_use, :type => Boolean, :default => false
+  field :enable_language_detection, :type => Boolean, :default => false
 
   field :title_always_on, :type => Boolean, :default => false
   field :dashbroad_allow_visitor, :type => Boolean, :default => false
@@ -44,6 +46,21 @@ class Site
 
   belongs_to :design
   has_many :site_metas, :autosave => true, :dependent => :destroy
+  validate :in_use_locales, :minimum_enabled_locales
+
+  def minimum_enabled_locales
+    size = self.in_use_locales.length
+    if size < 1 
+      errors.add(:size, "Needs at least one language in use")
+    end
+  end
+
+  def maximum_enabled_locales
+    size = self.in_use_locales.length
+    if size >= 2 
+      errors.add(:size, "Max 2 ")
+    end
+  end
 
   def generate_keys
     private_key = OpenSSL::PKey::RSA.generate(2048)

app/views/admin/designs/_design.html.erb

@@ -1,4 +1,3 @@
-<%= Rails.logger.info"@@@@@@@@@@@@@@@@"+design.inspect%>
 <tr class="with_action">
 	<td class="span1"><%= check_box_tag 'to_delete[]', design.id, false, :class => "checkbox_in_list" %></td>
 	<td class="span2">

app/views/admin/sites/preference.html.erb

@@ -17,7 +17,6 @@
 
           <fieldset>
 
-            
             <!-- Input Area -->
             <div class="input-area">
               <div class="nav-name"><strong>Classification</strong></div>    
@@ -127,7 +126,7 @@
                     </div>
                   </div>
                   <div class="control-group">
-                    <label class="control-label muted">Disable personal desktop</label>
+                    <label class="control-label muted"><%= I18n.t('site.enable_personal_desktop')%></label>
                     <div class="controls">
                       <%= f.check_box :desktop_closed , :class=>"toggle-check" %>
                     </div>
@@ -143,27 +142,33 @@
                       <input type="checkbox" class="toggle-check" data-disabled="true">
                     </div>
                   </div> -->
+                  <!-- Terms of Use -->
                   <div class="control-group">
-                    <label class="control-label muted">Default Language</label>
+                    <label class="control-label muted">Enable User Language Detection
+                    </label>
                     <div class="controls">
-                    <%= @site_in_use_locales %>
-
-                    <% @site_in_use_locales.each do |locale| %>
-                      <%= check_box_tag :default_locale, locale, (@site.enabled_for.nil?), :class=>"toggle-check"  %>
-                      <%= I18nVariable.from_locale(locale) %>
-                     <% end %>
+                      <%= f.check_box :enable_language_detection, :class => "toggle-check" %> 
                     </div>
                   </div>
+
                   <div class="control-group">
                     <label class="control-label muted">Enable Language</label>
                     <div class="controls">
                      <% @site_valid_locales.each do |valid_locale| %>
-                      <%= check_box_tag "site[in_use_locales][]", locale, @site.valid_locales.include?(locale), :class=>"toggle-check" %>
-						            <%= I18nVariable.from_locale(valid_locale) %>
+                      <%= check_box_tag "site[enable_locales][]", locale, @site.valid_locales.include?(locale), :class=>"toggle-check", :data => { :title => "#{I18nVariable.from_locale(locale)}" } %>
 					            <% end %>
                     </div>
                   </div>
 
+                  <div class="control-group">
+                    <label class="control-label muted">Default Language</label>
+                    <div class="controls">
+                    <% @site_in_use_locales.each do |locale| %>
+                      <%= check_box_tag :default_locale, locale, (@site.default_locale.nil?), :class=>"toggle-check", :data => { :title => "#{I18nVariable.from_locale(locale)}" }  %>
+                     <% end %>
+                    </div>
+                  </div>
+
                 </div>
 
                 <!-- OrbitBar Them -->

app/views/admin/users_new_interface/_user_basic_passwd.html.erb

@@ -39,16 +39,5 @@
         </div>
       </div>
 
-      <!-- Chec Admin -->
-      <div class="control-group">
-        <label class="control-label muted" for="">Administrator</label>
-        <div class="controls">
-          <label class="checkbox inline">
-            <%= f.check_box :admin %>
-            <%= t("admin") %>
-          </label>
-        </div>
-      </div>
-
     </div>
   </div>

app/views/admin/users_new_interface/_user_basic_privilege.html.erb

@@ -0,0 +1,32 @@
+  <%#= devise_error_messages! %>
+  <%= flash_messages %>
+
+  <div class="attributes">
+    <div class="attributes-header clearfix">
+      <h4><%=t("users.setting_privilege")%></h4>
+    </div>
+    <div class="attributes-body">
+
+      <!-- Account -->
+      <div class="control-group">
+        <label class="control-label muted" for="account">
+          <%= f.label t("users.user_id"),:class=>"control-label" %>
+        </label>
+        <div class="controls add-input">
+          <%= @user.user_id %>
+        </div>
+      </div>
+
+      <!-- Chec Admin -->
+      <div class="control-group">
+        <label class="control-label muted" for="">Administrator</label>
+        <div class="controls">
+          <label class="checkbox inline">
+            <%= f.check_box :admin %>
+            <%= t("admin") %>
+          </label>
+        </div>
+      </div>
+
+    </div>
+  </div>

app/views/admin/users_new_interface/_user_for_listing.html.erb

@@ -20,7 +20,7 @@
         <div class="quick-edit">
           <ul class="nav nav-pills">
             <%= content_tag(:li, link_to(t(:edit),edit_admin_users_new_interface_path(user_for_listing))) if is_admin? %>
-            <%= content_tag(:li, link_to(t(:Permissions),'#')) if is_admin? %>
+            <%= content_tag(:li, link_to(t("users.setting_privilege"),admin_users_new_interface_edit_privilege_path(user_for_listing))) if is_admin? and current_user.id != user_for_listing.id %>
             <%= content_tag(:li, link_to(t(:delete),admin_users_new_interface_path(user_for_listing, :at=>params[:at]), :confirm => t(:sure?), :method => :delete, :class=>"text-error", :remote => true)) if is_admin? %>
           </ul>
         </div>

app/views/admin/users_new_interface/_user_for_summary.html.erb

@@ -12,7 +12,7 @@
       <p class="<%= @user_sex%>"></p>
       <div class="action">
         <%= link_to(content_tag(:i, nil, :class => 'icon-edit'),edit_admin_users_new_interface_path(user_for_summary),:class=>"edit" ) if is_admin?%>
-        <%= link_to(content_tag(:i, nil, :class => 'icon-key'),'#',:class=>"key" ) if is_admin?%>
+        <%= link_to(content_tag(:i, nil, :class => 'icon-key'),admin_users_new_interface_edit_privilege_path(user_for_summary),:class=>"key" ) if is_admin? and current_user.id != user_for_summary.id %>
         <%= link_to(content_tag(:i, nil, :class => 'icon-trash'), admin_users_new_interface_path(user_for_summary, :at=>params[:at]), :confirm => t(:sure?), :method => :delete, :class=>"trash", :remote => true) if is_admin? %>
       </div>
       <%= image_tag(user_for_summary.avatar) %>

app/views/admin/users_new_interface/_user_for_thumbnail.html.erb

@@ -12,7 +12,7 @@
       <div class="action">
 
         <%= link_to(content_tag(:i, nil, :class => 'icon-edit'),edit_admin_users_new_interface_path(user_for_thumbnail),:class=>"edit" ) if is_admin?%>
-        <%= link_to(content_tag(:i, nil, :class => 'icon-key'),'#',:class=>"key" ) if is_admin?%>
+        <%= link_to(content_tag(:i, nil, :class => 'icon-key'),admin_users_new_interface_edit_privilege_path(user_for_thumbnail),:class=>"key" ) if is_admin? and current_user.id != user_for_thumbnail.id %>
         <%= link_to(content_tag(:i, nil, :class => 'icon-trash'), admin_users_new_interface_path(user_for_thumbnail, :at=>params[:at]), :confirm => t(:sure?), :method => :delete, :class=>"trash", :remote => true) if is_admin? %>
         
       </div>

app/views/admin/users_new_interface/edit_privilege.html.erb

@@ -0,0 +1,39 @@
+<% content_for :side_bar do %>
+  <%= render :partial => 'admin/users_new_interface/side_bar' %>
+<% end %>
+
+<% content_for :page_specific_css do -%>
+    <%= stylesheet_link_tag "lib/wrap-nav.css" %>
+    <%= stylesheet_link_tag "lib/pageslide.css" %>
+    <%= stylesheet_link_tag "lib/main-forms.css" %>
+    <%= stylesheet_link_tag "lib/fileupload.css" %>
+    <%= stylesheet_link_tag "lib/togglebox.css" %>
+<% end -%>
+<% content_for :page_specific_javascript do -%>
+    <%= javascript_include_tag "lib/jquery.tmpl.min.js" %>
+    <%= javascript_include_tag "lib/bootstrap-fileupload.js" %>
+    <%= javascript_include_tag "lib/member/role-forms.js" %>
+    <%= javascript_include_tag "lib/member/textarea-lang-btn.js" %>
+<% end -%>
+
+
+<%= form_for @user, :url => admin_users_new_interface_path(@user), :html => { :multipart => true , :class=>"form-horizontal main-forms"} do |f| %>
+
+  <fieldset>
+    <div id="basic-area" class="input-area">
+
+    <%= f.error_messages %>
+    <%= render :partial => 'user_basic_privilege', :locals => {:f => f}%>
+
+    </div>
+    
+
+    <div class="form-actions">
+      <%= link_back('btn') %>
+      <%= hidden_field_tag 'edit_type', 'edit_privilege' %>
+      <%= link_to t(:update_), "#", :class=>"btn btn-primary returnDecide", :onclick=>"$('#edit_user_#{@user.id}').submit()" %>
+    </div>
+
+  </fieldset>  
+
+<% end -%>

app/views/admin/users_new_interface/show.html.erb

@@ -21,6 +21,7 @@
               <div class="btn-group">
                   <%= link_to("<i class='icon-edit'></i> #{t(:edit)}".html_safe,edit_admin_users_new_interface_path(@user),:class=>"btn btn-mini" ) if is_admin?%>
                   <%= link_to("<i class='icons-cycle'></i> #{t("users.change_passwd")}".html_safe,admin_users_new_interface_edit_passwd_path(@user),:class=>"btn btn-mini" ) if is_admin?%>
+                  <%= link_to("<i class='icons-lock-open'></i> #{t("users.setting_privilege")}".html_safe,admin_users_new_interface_edit_privilege_path(@user),:class=>"btn btn-mini" ) if is_admin? and current_user.id != @user.id %>
               </div>
             </div>
 

app/views/layouts/_left_menu.html.erb

@@ -21,11 +21,13 @@
   </li>
 
   <!-- Desktop -->
+  <% if !@site.desktop_closed%>
   <li id="orbit-desktop">
     <a href="<%= desktop_path %>" role="button">
       <i class="icons-screen"></i> <span class="hide"><%= t(:desktop) %></span>
     </a>
   </li>
+  <% end %>
   
 </ul>
 <% if @site.default_image.present?%>

config/locales/en.yml

@@ -391,6 +391,7 @@ en:
     footer_help: Footer Guide
     frontend_closed: Frontend Closed
     frontend_open: Frontend Open
+    enable_personal_desktop: Enable Personal Desktop
     header: Site header
     info: Site information
     keywords: Site keywords
@@ -486,7 +487,8 @@ en:
   users:
     admin_change_password: You cannot change your own password here!
     avatar: Profile Picture
-    change_passwd: Change password
+    change_passwd: Password
+    setting_privilege: Privilege
     email: Email
     first_name: First Name
     last_name: Last Name

config/locales/zh_tw.yml

@@ -385,8 +385,9 @@ zh_tw:
     thumbnail: 縮圖
   site:
     frontend_closed: 前台關閉？
-    frontend_open:  前台關閉 ?
+    frontend_open: 開啟前台
     backend_openness_on: 開啟開放式後台（訪客可遊覽）
+    enable_personal_desktop: 開啟個人桌面
     default_image: 預設圖像
     description: 網站描述
     description_help: 網站描述說明
@@ -488,6 +489,7 @@ zh_tw:
     admin_change_password: 您不能在此處修改自己的密碼！
     avatar: 大頭貼照
     change_passwd: 變更密碼
+    setting_privilege: 權限設定
     email: 電子郵件
     first_name: 名字
     male: 男性

config/routes.rb

@@ -221,6 +221,7 @@ Orbit::Application.routes.draw do
         post 'update_order_list'
       end
       match "edit_passwd" => "users_new_interface#edit_passwd" ,:as => :edit_passwd
+      match "edit_privilege" => "users_new_interface#edit_privilege" ,:as => :edit_privilege
     end
 
     resources :member_selects do