diff --git a/app/controllers/spree/api/products_controller_decorator.rb b/app/controllers/spree/api/products_controller_decorator.rb index ee3cc9e70b..2101ca6f61 100644 --- a/app/controllers/spree/api/products_controller_decorator.rb +++ b/app/controllers/spree/api/products_controller_decorator.rb @@ -76,7 +76,11 @@ Spree::Api::ProductsController.class_eval do end def render_paged_products(products) - render text: { products: ActiveModel::ArraySerializer.new(products, each_serializer: Api::Admin::ProductSerializer), pages: products.num_pages }.to_json - end + serializer = ActiveModel::ArraySerializer.new( + products, + each_serializer: Api::Admin::ProductSerializer + ) + render text: { products: serializer, pages: products.num_pages }.to_json + end end diff --git a/app/serializers/api/admin/product_serializer.rb b/app/serializers/api/admin/product_serializer.rb index 8af5701b94..27b914de29 100644 --- a/app/serializers/api/admin/product_serializer.rb +++ b/app/serializers/api/admin/product_serializer.rb @@ -9,11 +9,19 @@ class Api::Admin::ProductSerializer < ActiveModel::Serializer has_one :master, serializer: Api::Admin::VariantSerializer def image_url - object.images.present? ? object.images.first.attachment.url(:product) : "/assets/noimage/product.png" + if object.images.present? + object.images.first.attachment.url(:product) + else + "/assets/noimage/product.png" + end end def thumb_url - object.images.present? ? object.images.first.attachment.url(:mini) : "/assets/noimage/mini.png" + if object.images.present? + object.images.first.attachment.url(:mini) + else + "/assets/noimage/mini.png" + end end def on_hand