From f62ba0f73cd465d36d74e1a2fd8ff76027b7d685 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jonne=20Ha=C3=9F?= Date: Sun, 28 Dec 2014 21:40:07 +0100 Subject: [PATCH] Don't split contact_ids when they already are an array --- app/controllers/conversations_controller.rb | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/app/controllers/conversations_controller.rb b/app/controllers/conversations_controller.rb index 56b263133..201b816d1 100644 --- a/app/controllers/conversations_controller.rb +++ b/app/controllers/conversations_controller.rb @@ -40,9 +40,12 @@ class ConversationsController < ApplicationController end def create + contact_ids = params[:contact_ids] + # Can't split nil - if params[:contact_ids] - person_ids = current_user.contacts.where(id: params[:contact_ids].split(',')).pluck(:person_id) + if contact_ids + contact_ids = contact_ids.split(',') if contact_ids.is_a? String + person_ids = current_user.contacts.where(id: contact_ids).pluck(:person_id) end opts = params.require(:conversation).permit(:subject)