From de2c6a8717677bbeb7f0600c7022b5d26fa6bd28 Mon Sep 17 00:00:00 2001
From: Luis Ramos <luisramos0@gmail.com>
Date: Tue, 10 Mar 2020 11:21:36 +0000
Subject: [PATCH] Revert previous commits and use collection_singular_ids=(ids)
 setter instead

---
 app/controllers/api/enterprises_controller.rb | 12 +-----------
 1 file changed, 1 insertion(+), 11 deletions(-)

diff --git a/app/controllers/api/enterprises_controller.rb b/app/controllers/api/enterprises_controller.rb
index 603724c780..7c545e5181 100644
--- a/app/controllers/api/enterprises_controller.rb
+++ b/app/controllers/api/enterprises_controller.rb
@@ -15,7 +15,7 @@ module Api
       user_ids = params[:enterprise].delete(:user_ids)
       @enterprise = Enterprise.new(params[:enterprise])
       if @enterprise.save
-        save_enterprise_users(user_ids)
+        @enterprise.user_ids = user_ids
         render text: @enterprise.id, status: :created
       else
         invalid_resource!(@enterprise)
@@ -76,15 +76,5 @@ module Api
     def override_visible
       params[:enterprise][:visible] = false
     end
-
-    def save_enterprise_users(user_ids)
-      return unless user_ids
-
-      user_ids.each do |user_id|
-        next if @enterprise.user_ids.include? user_id.to_i
-
-        @enterprise.users << Spree::User.find(user_id)
-      end
-    end
   end
 end