Add UsersGroup relation to be respected by abilities and Project#team

4 files changed, 27 insertions, 15 deletions

diff --git a/app/views/groups/_new_group_member.html.haml b/app/views/groups/_new_group_member.html.haml
index 77dfda7e1d4..bc1b12f257d 100644
--- a/app/views/groups/_new_group_member.html.haml
+++ b/app/views/groups/_new_group_member.html.haml
@@ -1,8 +1,8 @@
 = form_for @team_member, as: :team_member, url: team_members_group_path(@group) do |f|
   %fieldset
-    %legend= "New Team member(s) for projects in #{@group.name}"
+    %legend= "New Group member(s) for #{@group.name}"
 
-    %h6 1. Choose people you want in the team
+    %h6 1. Choose people you want in the group
     .clearfix
       = f.label :user_ids, "People"
       .input= users_select_tag(:user_ids, multiple: true)
diff --git a/app/views/groups/_new_member.html.haml b/app/views/groups/_new_member.html.haml
index 4a762ed39ed..70c3e8cd248 100644
--- a/app/views/groups/_new_member.html.haml
+++ b/app/views/groups/_new_member.html.haml
@@ -1,8 +1,8 @@
 = form_for @team_member, as: :team_member, url: project_team_members_path(@project, @team_member) do |f|
   %fieldset
-    %legend= "New Team member(s) for #{@project.name}"
+    %legend= "New Project member(s) for #{@project.name}"
 
-    %h6 1. Choose people you want in the team
+    %h6 1. Choose people you want in the project
     .clearfix
       = f.label :user_ids, "People"
       .input= users_select_tag(:user_ids, multiple: true)
@@ -14,5 +14,5 @@
 
     .form-actions
       = hidden_field_tag :redirect_to, people_group_path(@group, project_id: @project.id)
-      = f.submit 'Add', class: "btn btn-save"
+      = f.submit 'Add users into project', class: "btn btn-create"
 
diff --git a/app/views/groups/_people_filter.html.haml b/app/views/groups/_people_filter.html.haml
index ee63743eb4f..598b1e7b979 100644
--- a/app/views/groups/_people_filter.html.haml
+++ b/app/views/groups/_people_filter.html.haml
@@ -1,6 +1,15 @@
+%fieldset
+  %legend Group members:
+  %ul.nav.nav-pills.nav-stacked
+    %li{class: ("active" if params[:project_id].blank?)}
+      = link_to people_group_path(@group) do
+        #{@group.name} Group
+        %small.pull-right= @group.users.count
+
+
 = form_tag people_group_path(@group), method: 'get' do
   %fieldset
-    %legend Projects:
+    %legend Project members:
     %ul.nav.nav-pills.nav-stacked
       - @projects.each do |project|
         %li{class: ("active" if params[:project_id] == project.id.to_s)}
@@ -10,7 +19,4 @@
       - if @projects.blank?
         %p.nothing_here_message This group has no projects yet
 
-  %fieldset
-    %hr
-    = link_to "Reset", people_group_path(@group), class: 'btn pull-right'
 
diff --git a/app/views/groups/people.html.haml b/app/views/groups/people.html.haml
index 965a761db1e..b55e1b1379e 100644
--- a/app/views/groups/people.html.haml
+++ b/app/views/groups/people.html.haml
@@ -6,12 +6,18 @@
       = render (@project ? "new_member" : "new_group_member")
     .ui-box
       %h5.title
-        #{@group.name} Team
-        %small
-          (#{@users_groups.count})
+        - if @project
+          #{@project.name} Project Members
+          %small
+            (#{@members.count})
+        - else
+          #{@group.name} Group Members
+          %small
+            (#{@members.count})
       %ul.well-list
-        - @users_groups.each do |users_group|
-          - user = users_group.user
+        - @members.each do |member|
+          - user = member.user
+          - next unless user
           %li
             = image_tag gravatar_icon(user.email, 16), class: "avatar s16"
             %strong= user.name
@@ -21,5 +27,5 @@
               - if @group.owners.include?(user)
                 %span.label.label-info Group Owner
               - else
-                = users_group.human_group_access
+                = member.human_access