From 57835abb3734560644c9ec36f9633008e29e4616 Mon Sep 17 00:00:00 2001
From: sw <939547590@qq.com>
Date: Sat, 27 Sep 2014 16:18:29 +0800
Subject: [PATCH] =?UTF-8?q?1.=E5=A2=9E=E5=8A=A0=E7=AE=A1=E7=90=86=E5=91=98?=
 =?UTF-8?q?=E4=B8=8D=E8=83=BD=E9=80=80=E5=87=BA=E9=A1=B9=E7=9B=AE=202.?=
 =?UTF-8?q?=E5=A2=9E=E5=8A=A0=E9=80=80=E5=87=BA=E9=A1=B9=E7=9B=AE=E6=97=B6?=
 =?UTF-8?q?=E7=9A=84=E6=8F=90=E7=A4=BA?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 app/helpers/watchers_helper.rb                |  3 +-
 app/views/courses/settings/_members.html.erb  | 50 ++++++++++++-------
 app/views/layouts/_join_exit_project.html.erb |  4 +-
 config/locales/zh.yml                         |  1 +
 4 files changed, 37 insertions(+), 21 deletions(-)

diff --git a/app/helpers/watchers_helper.rb b/app/helpers/watchers_helper.rb
index b05e0b286..793581664 100644
--- a/app/helpers/watchers_helper.rb
+++ b/app/helpers/watchers_helper.rb
@@ -251,6 +251,7 @@ module WatchersHelper
   end
 
     def exit_project_link(project)
-      link_to("退出项目",exit_cur_project_path(project.id),:remote => true )
+      link_to(l(:label_exit_project),exit_cur_project_path(project.id),
+        :remote => true, :confirm => l(:lable_sure_exit_project) )
     end
   end
diff --git a/app/views/courses/settings/_members.html.erb b/app/views/courses/settings/_members.html.erb
index d0c77ae83..a62a9c613 100644
--- a/app/views/courses/settings/_members.html.erb
+++ b/app/views/courses/settings/_members.html.erb
@@ -20,26 +20,34 @@
         <% members.each do |member| %>
             <% next if member.new_record? %>
             <tr id="member-<%= member.id %>" class="<%= cycle 'odd', 'even' %> member">
-              <td class="<%= member.principal.class.name.downcase %>"><%= link_to_user member.principal %></td>
+              <td class="<%= member.principal.class.name.downcase %>">
+                <%= link_to_user member.principal %>
+              </td>
               <td class="roles">
-    <span id="member-<%= member.id %>-roles">
-      <%= h member.roles.sort.collect(&:to_s).join(', ') %>
-    </span>
-                <%= form_for(member, {:as => :membership, :remote => true, :url => membership_path(member),
-                                      :method => :put,
-                                      :html => {:id => "member-#{member.id}-roles-form", :class => 'hol'}}
+                <span id="member-<%= member.id %>-roles">
+                  <%= h member.roles.sort.collect(&:to_s).join(', ') %>
+                </span>
+                <%= form_for(member, {:as => :membership, :remote => true, 
+                                      :url => course_memberships_path(member),:method => :put,
+                                      :html => {:id => "member-#{member.id}-roles-form", 
+                                                :class => 'hol'}}
                     ) do |f| %>
-
                     <p>
                       <% roles.each do |role| %>
-                          <label><%= radio_button_tag 'membership[role_ids][]', role.id, member.roles.include?(role),
-                                                   :disabled => member.member_roles.detect { |mr| mr.role_id == role.id && !mr.inherited_from.nil? } %> <%= h role %></label><br/>
-                      <% end %></p>
+                          <label>
+                            <%= radio_button_tag 'membership[role_ids][]', role.id, member.roles.include?(role),:disabled => member.member_roles.detect { |mr| mr.role_id == role.id && !mr.inherited_from.nil? } %> 
+                            <%= h role %>
+                          </label>
+                          <br/>
+                      <% end %>
+                    </p>
                     <%= hidden_field_tag 'membership[role_ids][]', '' %>
-                    <p><%= submit_tag l(:button_change), :class => "small" %>
+                    <p>
+                      <%= submit_tag l(:button_change), :class => "small" %>
                       <%= link_to_function l(:button_cancel),
                                            "$('#member-#{member.id}-roles').show(); $('#member-#{member.id}-roles-form').hide(); return false;"
-                      %></p>
+                      %>
+                    </p>
                 <% end %>
               </td>
               <!--modified by huang for: if the user'roles is Manager that he will can't modified himself-->
@@ -71,8 +79,10 @@
       <%= form_for(@member, {:as => :membership, :url => course_memberships_path(@course), :remote => true, :method => :post}) do |f| %>
           <fieldset>
             <legend><%= l(:label_member_new) %></legend>
-
-            <p><%= label_tag "principal_search", l(:label_principal_search) %><%= text_field_tag 'principal_search', nil %></p>
+            <p>
+              <%= label_tag "principal_search", l(:label_principal_search) %>
+              <%= text_field_tag 'principal_search', nil %>
+            </p>
             <%= javascript_tag "observeSearchfield('principal_search', null, '#{ escape_javascript autocomplete_course_memberships_path(@course, :format => 'js') }')" %>
 
             <div id="principals_for_new_member">
@@ -82,10 +92,12 @@
             <!--show the roles which will select-->
             <p style="padding-top: 5px"><%= l(:label_role_plural) %>:
               <% roles.each do |role| %>
-
-                  <label><%= radio_button_tag 'membership[role_ids][]', role.id %> <%= h role %></label>
-              <% end %></p>
-
+                  <label>
+                    <%= radio_button_tag 'membership[role_ids][]', role.id %> 
+                    <%= h role %>
+                  </label>
+              <% end %>
+            </p>
             <p><%= submit_tag l(:button_add), :id => 'member-add-submit' %></p>
           </fieldset>
       <% end %>
diff --git a/app/views/layouts/_join_exit_project.html.erb b/app/views/layouts/_join_exit_project.html.erb
index c43ad55a2..310d4ecd3 100644
--- a/app/views/layouts/_join_exit_project.html.erb
+++ b/app/views/layouts/_join_exit_project.html.erb
@@ -14,7 +14,9 @@
 
 <!--添加退出项目-->
 <div style="margin-left: 20px;">
-  <% if ((User.current.member_of? @project) && User.current.login?) %>
+  <% if ((User.current.member_of? @project) && 
+    User.current.login? && 
+    @project.members.where("user_id = #{User.current.id}").first.roles.first.to_s != "Manager") %>
       <%= exit_project_link(@project) %>
   <% end %>
 </div>
\ No newline at end of file
diff --git a/config/locales/zh.yml b/config/locales/zh.yml
index fe3dd0fe1..e2ed6a045 100644
--- a/config/locales/zh.yml
+++ b/config/locales/zh.yml
@@ -2139,3 +2139,4 @@ zh:
   lable_user_active: 用户动态
   lable_bar_active: 贴吧动态
   lable_student_list_visable: 学生列表是否公开
+  lable_sure_exit_project: 是否确认退出该项目