rails 5 カート機能で追加した時にすでにカート内に同じ商品があった場合、数だけ変更する方法

Question

cart_items_controller


def create
		@cart_item = CartItem.new(cart_item_params)
		@cart_item.end_user_id = current_end_user.id
		if current_end_user.cart_items.find_by(item_id: params[:cart_item][:item_id]).present?
			current_end_user.cart_items.find_by(item_id: params[:cart_item][:item_id]).number += params[:cart_item][:number].to_i
			current_end_user.cart_items.find_by(item_id: params[:cart_item][:item_id]).save
			flash[:notice] = "Item was successfully added to cart."
			redirect_to cart_items_path


		elsif @cart_item.save
			flash[:notice] = "New Item was successfully added to cart."
			redirect_to cart_items_path
		else

			render 'end_users/items/show'
		end

現在ECサイトを作成しており、カート機能を実装しているのですが、追加した商品がすでにカート内に存在している時、同じレコードに数量だけ変化させたいのですが、

current_end_user.cart_items.find_by(item_id: params[:cart_item][:item_id]).number += params[:cart_item][:number].to_i
			current_end_user.cart_items.find_by(item_id: params[:cart_item][:item_id]).save

の箇所がうまく行きません。saveされずに元の個数で表示されます。
しかしflashメッセージはうまく表示されていてよくわからないです。
ご教授いただけたら幸いです。

Answer 1

ひとつ目のfind_byとふたつ目のfind_byが　DB上の同じレコードから作った違う2つのインスタンス　になっているのだとおもいます。
cart_item = current_end_user.cart_items.find_by(item_id: params[:cart_item][:item_id]) cart_item.number += params[:cart_item][:number].to_i cart_item.save
ではどうでしょう。

Comments

[takatin]

ありがとうございます！指摘の通り修正すれば、正常に更新されました！