diff options
| author | Valery Sizov <valery@gitlab.com> | 2017-05-05 20:06:22 +0300 |
|---|---|---|
| committer | Valery Sizov <valery@gitlab.com> | 2017-05-05 20:07:35 +0300 |
| commit | 0b7aabe302195348e06cea68937457c0c905de6c (patch) | |
| tree | b4834044534ae3a62a9adbf75147dc4cba81230c | |
| parent | 7389bb98167551a850be52e6683ecc93c4734a05 (diff) | |
Multiple issue assignee: fix for CE restrictions
| -rw-r--r-- | app/services/issues/base_service.rb | 2 | ||||
| -rw-r--r-- | spec/services/issues/update_service_spec.rb | 4 |
2 files changed, 3 insertions, 3 deletions
diff --git a/app/services/issues/base_service.rb b/app/services/issues/base_service.rb index bcd196bfa21..34199eb5d13 100644 --- a/app/services/issues/base_service.rb +++ b/app/services/issues/base_service.rb @@ -25,7 +25,7 @@ module Issues return if params[:assignee_ids].blank? # The number of assignees is limited by one for GitLab CE - params[:assignee_ids].slice!(0, 1) + params[:assignee_ids] = params[:assignee_ids][0, 1] assignee_ids = params[:assignee_ids].select { |assignee_id| assignee_can_read?(issuable, assignee_id) } diff --git a/spec/services/issues/update_service_spec.rb b/spec/services/issues/update_service_spec.rb index 1797a23ee8a..6633ac10236 100644 --- a/spec/services/issues/update_service_spec.rb +++ b/spec/services/issues/update_service_spec.rb @@ -40,7 +40,7 @@ describe Issues::UpdateService, services: true do { title: 'New title', description: 'Also please fix', - assignee_ids: [user2.id, user3.id], + assignee_ids: [user2.id], state_event: 'close', label_ids: [label.id], due_date: Date.tomorrow @@ -53,7 +53,7 @@ describe Issues::UpdateService, services: true do expect(issue).to be_valid expect(issue.title).to eq 'New title' expect(issue.description).to eq 'Also please fix' - expect(issue.assignees).to match_array([user2, user3]) + expect(issue.assignees).to match_array([user2]) expect(issue).to be_closed expect(issue.labels).to match_array [label] expect(issue.due_date).to eq Date.tomorrow |