Issuable can be assigned to author

Closes #9014

The only difference with #9014 is that I thoughed the author should also be
able to assign the issue. If this is unwanted behavior Ill revert it.

CHANGELOG

@@ -78,6 +78,7 @@ v 8.5.0 (unreleased)
   - Add label description (Nuttanart Pornprasitsakul)
   - Show label row when filtering issues or merge requests by label (Nuttanart Pornprasitsakul)
   - Add Todos
+  - Allow issues and merge requests to be assigned to the author(Zeger-Jan van de Weg)
 
 v 8.4.4
   - Update omniauth-saml gem to 1.4.2

app/assets/javascripts/users_select.js.coffee

@@ -7,6 +7,7 @@ class @UsersSelect
       @projectId = $(select).data('project-id')
       @groupId = $(select).data('group-id')
       @showCurrentUser = $(select).data('current-user')
+      @authorId = $(select).data('author-id')
       showNullUser = $(select).data('null-user')
       showAnyUser = $(select).data('any-user')
       showEmailUser = $(select).data('email-user')
@@ -112,6 +113,7 @@ class @UsersSelect
         project_id: @projectId
         group_id: @groupId
         current_user: @showCurrentUser
+        author_id: @authorId
       dataType: "json"
     ).done (users) ->
       callback(users)

app/controllers/autocomplete_controller.rb

@@ -12,8 +12,14 @@ class AutocompleteController < ApplicationController
     if params[:search].blank?
       # Include current user if available to filter by "Me"
       if params[:current_user] && current_user
-        @users = [*@users, current_user].uniq
+        @users = [*@users, current_user]
       end
+
+      if params[:author_id]
+        @users = [User.find(params[:author_id]), *@users]
+      end
+
+      @users.uniq!
     end
 
     render json: @users, only: [:name, :username, :id], methods: [:avatar_url]

app/helpers/selects_helper.rb

@@ -11,6 +11,7 @@ module SelectsHelper
     email_user    = opts[:email_user]   || false
     first_user    = opts[:first_user] && current_user ? current_user.username : false
     current_user  = opts[:current_user] || false
+    author_id     = opts[:author_id]    || false
     project       = opts[:project]      || @project
 
     html = {
@@ -21,7 +22,8 @@ module SelectsHelper
         any_user: any_user,
         email_user: email_user,
         first_user: first_user,
-        current_user: current_user
+        current_user: current_user,
+        author_id: author_id
       }
     }
 

app/models/concerns/issuable.rb

@@ -116,6 +116,11 @@ module Issuable
     assignee_id_changed?
   end
 
+  def can_assign_user?(current_user)
+    author == current_user ||
+    Ability.abilities.allowed?(current_user, :"admin_#{to_ability_name}", project)
+  end
+
   def open?
     opened? || reopened?
   end

app/services/issuable_base_service.rb

@@ -33,7 +33,7 @@ class IssuableBaseService < BaseService
     end
   end
 
-  def filter_params(issuable_ability_name = :issue)
+  def filter_params(issuable_ability_name, issuable)
     params[:assignee_id]  = "" if params[:assignee_id] == IssuableFinder::NONE
     params[:milestone_id] = "" if params[:milestone_id] == IssuableFinder::NONE
 
@@ -42,13 +42,18 @@ class IssuableBaseService < BaseService
     unless can?(current_user, ability, project)
       params.delete(:milestone_id)
       params.delete(:label_ids)
+
+      # The author of an issue can be assigned, to signal the ball being in his/her
+      # court. This allow him/her to reassign the issue back to the reviewer.
+      if issuable && !(issuable.author == current_user)
         params.delete(:assignee_id)
       end
     end
+  end
 
   def update(issuable)
     change_state(issuable)
-    filter_params
+    filter_params(issuable)
     old_labels = issuable.labels.to_a
 
     if params.present? && issuable.update_attributes(params.merge(updated_by: current_user))

app/services/issues/base_service.rb

@@ -10,8 +10,8 @@ module Issues
 
     private
 
-    def filter_params
-      super(:issue)
+    def filter_params(issuable = nil)
+      super(:issue, issuable)
     end
 
     def execute_hooks(issue, action = 'open')

app/services/merge_requests/base_service.rb

@@ -23,8 +23,8 @@ module MergeRequests
 
     private
 
-    def filter_params
-      super(:merge_request)
+    def filter_params(issueable = nil)
+      super(:merge_request, issueable)
     end
   end
 end

app/views/shared/issuable/_sidebar.html.haml

@@ -30,7 +30,7 @@
         .title
           %label
             Assignee
-          - if can?(current_user, :"admin_#{issuable.to_ability_name}", @project)
+          - if issuable.can_assign_user?(current_user)
             .pull-right
               = link_to 'Edit', '#', class: 'edit-link'
         .value
@@ -43,7 +43,10 @@
             .light None
 
         .selectbox
-          = users_select_tag("#{issuable.class.table_name.singularize}[assignee_id]", placeholder: 'Select assignee', class: 'custom-form-control js-select2 js-assignee', selected: issuable.assignee_id, project: @target_project, null_user: true, current_user: true, first_user: true)
+          = users_select_tag("#{issuable.class.table_name.singularize}[assignee_id]",
+                            placeholder: 'Select assignee', class: 'custom-form-control js-select2 js-assignee',
+                            selected: issuable.assignee_id, project: @target_project, null_user: true, current_user: true,
+                            first_user: true, author_id: issuable.author_id)
 
       .block.milestone
         .sidebar-collapsed-icon

spec/controllers/autocomplete_controller_spec.rb

@@ -143,4 +143,17 @@ describe AutocompleteController do
       it { expect(body.size).to eq 0 }
     end
   end
+
+  context 'author of issuable included' do
+    before do
+      sign_in(user)
+      get(:users, author_id: non_member.id)
+    end
+
+    let(:body) { JSON.parse(response.body) }
+
+    it 'should also return the author' do
+      expect(body.first["username"]).to eq non_member.username
+    end
+  end
 end

spec/models/concerns/issuable_spec.rb

@@ -111,6 +111,20 @@ describe Issue, "Issuable" do
     end
   end
 
+  describe "#can_assign_user?" do
+    let(:author)  { build(:user) }
+    let(:issue)   { build(:issue, author: author)}
+
+    it "Allows the author to change the assignee" do
+      expect(issue.can_assign_user?(author)).to be_truthy
+    end
+
+    it "Doesn't allow others, non-team members" do
+      other_user = build(:user)
+      expect(issue.can_assign_user?(other_user)).to be_falsey
+    end
+  end
+
   describe "votes" do
     before do
       author = create :user