Handle save method as it should be: return false if any errors occurs

This commit is contained in:
Jean-Baptiste Bellet
2022-07-07 09:56:14 +02:00
parent 6f228781d4
commit f71c84cea5
2 changed files with 2 additions and 1 deletions

View File

@@ -76,7 +76,7 @@ module Spree
product_set.collection.each { |p| authorize! :update, p }
if product_set.save && product_set.errors.empty?
if product_set.save
redirect_to main_app.bulk_products_api_v0_products_path(bulk_index_query)
elsif product_set.errors.present?
render json: { errors: product_set.errors }, status: :bad_request

View File

@@ -97,6 +97,7 @@ module Sets
variants_attributes.each do |attributes|
create_or_update_variant(product, attributes)
end
product.errors.empty?
end
def create_or_update_variant(product, variant_attributes)