spree_api 2.4.6 → 2.4.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: a1355738b99bc466e3299757f7f4d293a08ad45c
-  data.tar.gz: 9a81176332680c7c6802b11c395cd6c944a335c9
+  metadata.gz: 99560ebdaf455a59e76c64abf84ae1d2da1adb30
+  data.tar.gz: e9d198bf6d7a8a2131a7db49d55c61ae4d231b63
 SHA512:
-  metadata.gz: 576d4ca00833617cf383fa6c2c9ef25f5d0f82a1f3c20cbf53e2270c980dbb23aa390fa38b2945acb1aa109e9da1a1599b847fd5a39e417a746f4f8a2869276d
-  data.tar.gz: ede324c67a8f201a77f43a12b5321fae1a8a8bd72342564ef9e3cd37313c2f74bff53b6a1be2c437c50bc2c86f1d4942ed6334224bddd02054f9b1f2efccb2e9
+  metadata.gz: 94c7b6c9613e7ee865c0cc6cc0e0b761d4b077b76d7013677dacab09b032b15dccabdb3b1cb8d746557f6b66c82864bfcfedd688848c7a16da50c94a99375ba5
+  data.tar.gz: 1f4b86560857b5170b7566f641ace8b91d58f367d23dcc5bb4dfe8db02de637b19191825f3722ee378c96e02e53279e054d2f054fda207bb072f635acc070466

data/app/controllers/spree/api/checkouts_controller.rb

@@ -52,7 +52,7 @@ module Spree
         end
 
         def nested_params
-          map_nested_attributes_keys Order, params[:order] || {}
+          map_nested_attributes_keys Spree::Order, params[:order] || {}
         end
 
         # Should be overriden if you have areas of your checkout that don't match
@@ -79,7 +79,7 @@ module Spree
 
         def after_update_attributes
           if nested_params && nested_params[:coupon_code].present?
-            handler = PromotionHandler::Coupon.new(@order).apply
+            handler = Spree::PromotionHandler::Coupon.new(@order).apply
 
             if handler.error.present?
               @coupon_message = handler.error

data/app/controllers/spree/api/countries_controller.rb

@@ -5,17 +5,17 @@ module Spree
       skip_before_action :authenticate_user
 
       def index
-        @countries = Country.accessible_by(current_ability, :read).ransack(params[:q]).result.
+        @countries = Spree::Country.accessible_by(current_ability, :read).ransack(params[:q]).result.
                      includes(:states).order('name ASC').
                      page(params[:page]).per(params[:per_page])
-        country = Country.order("updated_at ASC").last
+        country = Spree::Country.order("updated_at ASC").last
         if stale?(country)
           respond_with(@countries)
         end
       end
 
       def show
-        @country = Country.accessible_by(current_ability, :read).find(params[:id])
+        @country = Spree::Country.accessible_by(current_ability, :read).find(params[:id])
         respond_with(@country)
       end
     end

data/app/controllers/spree/api/images_controller.rb

@@ -8,12 +8,12 @@ module Spree
       end
 
       def show
-        @image = Image.accessible_by(current_ability, :read).find(params[:id])
+        @image = Spree::Image.accessible_by(current_ability, :read).find(params[:id])
         respond_with(@image)
       end
 
       def create
-        authorize! :create, Image
+        authorize! :create, Spree::Image
         @image = scope.images.create(image_params)
         respond_with(@image, :status => 201, :default_template => :show)
       end

data/app/controllers/spree/api/inventory_units_controller.rb

@@ -24,7 +24,7 @@ module Spree
       private
 
       def inventory_unit
-        @inventory_unit ||= InventoryUnit.accessible_by(current_ability, :read).find(params[:id])
+        @inventory_unit ||= Spree::InventoryUnit.accessible_by(current_ability, :read).find(params[:id])
       end
 
       def prepare_event

data/app/controllers/spree/api/orders_controller.rb

@@ -7,7 +7,7 @@ module Spree
       before_action :find_order, except: [:create, :mine, :current, :index, :update]
 
       # Dynamically defines our stores checkout steps to ensure we check authorization on each step.
-      Order.checkout_steps.keys.each do |step|
+      Spree::Order.checkout_steps.keys.each do |step|
         define_method step do
           find_order
           authorize! :update, @order, params[:token]
@@ -21,7 +21,7 @@ module Spree
       end
 
       def create
-        authorize! :create, Order
+        authorize! :create, Spree::Order
         order_user = if @current_user_roles.include?('admin') && order_params[:user_id]
           Spree.user_class.find(order_params[:user_id])
         else
@@ -46,7 +46,7 @@ module Spree
 
       def index
         authorize! :index, Order
-        @orders = Order.ransack(params[:q]).result.page(params[:page]).per(params[:per_page])
+        @orders = Spree::Order.ransack(params[:q]).result.page(params[:page]).per(params[:per_page])
         respond_with(@orders)
       end
 
@@ -99,7 +99,7 @@ module Spree
         find_order
         authorize! :update, @order, order_token
         @order.coupon_code = params[:coupon_code]
-        @handler = PromotionHandler::Coupon.new(@order).apply
+        @handler = Spree::PromotionHandler::Coupon.new(@order).apply
         status = @handler.successful? ? 200 : 422
         render "spree/api/promotions/handler", :status => status
       end

data/app/controllers/spree/api/payments_controller.rb

@@ -67,7 +67,7 @@ module Spree
         end
 
         def perform_payment_action(action, *args)
-          authorize! action, Payment
+          authorize! action, Spree::Payment
           @payment.send("#{action}!", *args)
           respond_with(@payment, default_template: :show)
         end

data/app/controllers/spree/api/product_properties_controller.rb

@@ -20,7 +20,7 @@ module Spree
       end
 
       def create
-        authorize! :create, ProductProperty
+        authorize! :create, Spree::ProductProperty
         @product_property = @product.product_properties.new(product_property_params)
         if @product_property.save
           respond_with(@product_property, status: 201, default_template: :show)

data/app/controllers/spree/api/products_controller.rb

@@ -57,12 +57,12 @@ module Spree
       #   }
       #
       def create
-        authorize! :create, Product
+        authorize! :create, Spree::Product
         params[:product][:available_on] ||= Time.now
         set_up_shipping_category
 
         options = { variants_attrs: variants_params, options_attrs: option_types_params }
-        @product = Core::Importer::Product.new(nil, product_params, options).create
+        @product = Spree::Core::Importer::Product.new(nil, product_params, options).create
 
         if @product.persisted?
           respond_with(@product, :status => 201, :default_template => :show)
@@ -76,7 +76,7 @@ module Spree
         authorize! :update, @product
 
         options = { variants_attrs: variants_params, options_attrs: option_types_params }
-        @product = Core::Importer::Product.new(@product, product_params, options).update
+        @product = Spree::Core::Importer::Product.new(@product, product_params, options).update
 
         if @product.errors.empty?
           respond_with(@product.reload, :status => 200, :default_template => :show)
@@ -119,7 +119,7 @@ module Spree
 
         def set_up_shipping_category
           if shipping_category = params[:product].delete(:shipping_category)
-            id = ShippingCategory.find_or_create_by(name: shipping_category).id
+            id = Spree::ShippingCategory.find_or_create_by(name: shipping_category).id
             params[:product][:shipping_category_id] = id
           end
         end

data/app/controllers/spree/api/properties_controller.rb

@@ -25,7 +25,7 @@ module Spree
       end
 
       def create
-        authorize! :create, Property
+        authorize! :create, Spree::Property
         @property = Spree::Property.new(property_params)
         if @property.save
           respond_with(@property, status: 201, default_template: :show)

data/app/controllers/spree/api/return_authorizations_controller.rb

@@ -3,7 +3,7 @@ module Spree
     class ReturnAuthorizationsController < Spree::Api::BaseController
 
       def create
-        authorize! :create, ReturnAuthorization
+        authorize! :create, Spree::ReturnAuthorization
         @return_authorization = order.return_authorizations.build(return_authorization_params)
         if @return_authorization.save
           respond_with(@return_authorization, status: 201, default_template: :show)
@@ -19,7 +19,7 @@ module Spree
       end
 
       def index
-        authorize! :admin, ReturnAuthorization
+        authorize! :admin, Spree::ReturnAuthorization
         @return_authorizations = order.return_authorizations.accessible_by(current_ability, :read).
                                  ransack(params[:q]).result.
                                  page(params[:page]).per(params[:per_page])
@@ -27,11 +27,11 @@ module Spree
       end
 
       def new
-        authorize! :admin, ReturnAuthorization
+        authorize! :admin, Spree::ReturnAuthorization
       end
 
       def show
-        authorize! :admin, ReturnAuthorization
+        authorize! :admin, Spree::ReturnAuthorization
         @return_authorization = order.return_authorizations.accessible_by(current_ability, :read).find(params[:id])
         respond_with(@return_authorization)
       end

data/app/controllers/spree/api/states_controller.rb

@@ -27,10 +27,10 @@ module Spree
       private
         def scope
           if params[:country_id]
-            @country = Country.accessible_by(current_ability, :read).find(params[:country_id])
+            @country = Spree::Country.accessible_by(current_ability, :read).find(params[:country_id])
             return @country.states.accessible_by(current_ability, :read)
           else
-            return State.accessible_by(current_ability, :read)
+            return Spree::State.accessible_by(current_ability, :read)
           end
         end
     end

data/app/controllers/spree/api/stock_items_controller.rb

@@ -14,7 +14,7 @@ module Spree
       end
 
       def create
-        authorize! :create, StockItem
+        authorize! :create, Spree::StockItem
 
         count_on_hand = 0
         if params[:stock_item].has_key?(:count_on_hand)
@@ -31,7 +31,7 @@ module Spree
       end
 
       def update
-        @stock_item = StockItem.accessible_by(current_ability, :update).find(params[:id])
+        @stock_item = Spree::StockItem.accessible_by(current_ability, :update).find(params[:id])
 
         count_on_hand = 0
         if params[:stock_item].has_key?(:count_on_hand)
@@ -50,7 +50,7 @@ module Spree
       end
 
       def destroy
-        @stock_item = StockItem.accessible_by(current_ability, :destroy).find(params[:id])
+        @stock_item = Spree::StockItem.accessible_by(current_ability, :destroy).find(params[:id])
         @stock_item.destroy
         respond_with(@stock_item, status: 204)
       end
@@ -59,7 +59,7 @@ module Spree
 
       def stock_location
         render 'spree/api/shared/stock_location_required', status: 422 and return unless params[:stock_location_id]
-        @stock_location ||= StockLocation.accessible_by(current_ability, :read).find(params[:stock_location_id])
+        @stock_location ||= Spree::StockLocation.accessible_by(current_ability, :read).find(params[:stock_location_id])
       end
 
       def scope

data/app/controllers/spree/api/stock_locations_controller.rb

@@ -2,8 +2,8 @@ module Spree
   module Api
     class StockLocationsController < Spree::Api::BaseController
       def index
-        authorize! :read, StockLocation
-        @stock_locations = StockLocation.accessible_by(current_ability, :read).order('name ASC').ransack(params[:q]).result.page(params[:page]).per(params[:per_page])
+        authorize! :read, Spree::StockLocation
+        @stock_locations = Spree::StockLocation.accessible_by(current_ability, :read).order('name ASC').ransack(params[:q]).result.page(params[:page]).per(params[:per_page])
         respond_with(@stock_locations)
       end
 
@@ -12,8 +12,8 @@ module Spree
       end
 
       def create
-        authorize! :create, StockLocation
-        @stock_location = StockLocation.new(stock_location_params)
+        authorize! :create, Spree::StockLocation
+        @stock_location = Spree::StockLocation.new(stock_location_params)
         if @stock_location.save
           respond_with(@stock_location, status: 201, default_template: :show)
         else
@@ -39,7 +39,7 @@ module Spree
       private
 
       def stock_location
-        @stock_location ||= StockLocation.accessible_by(current_ability, :read).find(params[:id])
+        @stock_location ||= Spree::StockLocation.accessible_by(current_ability, :read).find(params[:id])
       end
 
       def stock_location_params

data/app/controllers/spree/api/stock_movements_controller.rb

@@ -4,7 +4,7 @@ module Spree
       before_action :stock_location, except: [:update, :destroy]
 
       def index
-        authorize! :read, StockMovement
+        authorize! :read, Spree::StockMovement
         @stock_movements = scope.ransack(params[:q]).result.page(params[:page]).per(params[:per_page])
         respond_with(@stock_movements)
       end
@@ -15,7 +15,7 @@ module Spree
       end
 
       def create
-        authorize! :create, StockMovement
+        authorize! :create, Spree::StockMovement
         @stock_movement = scope.new(stock_movement_params)
         if @stock_movement.save
           respond_with(@stock_movement, status: 201, default_template: :show)
@@ -28,7 +28,7 @@ module Spree
 
       def stock_location
         render 'spree/api/shared/stock_location_required', status: 422 and return unless params[:stock_location_id]
-        @stock_location ||= StockLocation.accessible_by(current_ability, :read).find(params[:stock_location_id])
+        @stock_location ||= Spree::StockLocation.accessible_by(current_ability, :read).find(params[:stock_location_id])
       end
 
       def scope

data/app/controllers/spree/api/stores_controller.rb

@@ -5,14 +5,14 @@ module Spree
       before_filter :get_store, except: [:index, :create]
 
       def index
-        authorize! :read, Store
-        @stores = Store.accessible_by(current_ability, :read).all
+        authorize! :read, Spree::Store
+        @stores = Spree::Store.accessible_by(current_ability, :read).all
         respond_with(@stores)
       end
 
       def create
-        authorize! :create, Store
-        @store = Store.new(store_params)
+        authorize! :create, Spree::Store
+        @store = Spree::Store.new(store_params)
         @store.code = params[:store][:code]
         if @store.save
           respond_with(@store, status: 201, default_template: :show)
@@ -44,7 +44,7 @@ module Spree
       private
 
       def get_store
-        @store = Store.find(params[:id])
+        @store = Spree::Store.find(params[:id])
       end
 
       def store_params

data/app/controllers/spree/api/taxonomies_controller.rb

@@ -16,8 +16,8 @@ module Spree
       end
 
       def create
-        authorize! :create, Taxonomy
-        @taxonomy = Taxonomy.new(taxonomy_params)
+        authorize! :create, Spree::Taxonomy
+        @taxonomy = Spree::Taxonomy.new(taxonomy_params)
         if @taxonomy.save
           respond_with(@taxonomy, :status => 201, :default_template => :show)
         else
@@ -43,13 +43,13 @@ module Spree
       private
 
       def taxonomies
-        @taxonomies = Taxonomy.accessible_by(current_ability, :read).order('name').includes(:root => :children).
+        @taxonomies = Spree::Taxonomy.accessible_by(current_ability, :read).order('name').includes(:root => :children).
                       ransack(params[:q]).result.
                       page(params[:page]).per(params[:per_page])
       end
 
       def taxonomy
-        @taxonomy ||= Taxonomy.accessible_by(current_ability, :read).find(params[:id])
+        @taxonomy ||= Spree::Taxonomy.accessible_by(current_ability, :read).find(params[:id])
       end
 
       def taxonomy_params

data/app/controllers/spree/api/taxons_controller.rb

@@ -26,7 +26,7 @@ module Spree
       end
 
       def create
-        authorize! :create, Taxon
+        authorize! :create, Spree::Taxon
         @taxon = Spree::Taxon.new(taxon_params)
         @taxon.taxonomy_id = params[:taxonomy_id]
         taxonomy = Spree::Taxonomy.find_by(id: params[:taxonomy_id])

data/app/controllers/spree/api/variants_controller.rb

@@ -4,7 +4,7 @@ module Spree
       before_action :product
 
       def create
-        authorize! :create, Variant
+        authorize! :create, Spree::Variant
         @variant = scope.new(variant_params)
         if @variant.save
           respond_with(@variant, status: 201, default_template: :show)
@@ -55,10 +55,10 @@ module Spree
           if @product
             variants = @product.variants_including_master
           else
-            variants = Variant
+            variants = Spree::Variant
           end
 
-          if current_ability.can?(:manage, Variant) && params[:show_deleted]
+          if current_ability.can?(:manage, Spree::Variant) && params[:show_deleted]
             variants = variants.with_deleted
           end
 

data/app/controllers/spree/api/zones_controller.rb

@@ -3,8 +3,8 @@ module Spree
     class ZonesController < Spree::Api::BaseController
 
       def create
-        authorize! :create, Zone
-        @zone = Zone.new(map_nested_attributes_keys(Spree::Zone, params[:zone]))
+        authorize! :create, Spree::Zone
+        @zone = Spree::Zone.new(map_nested_attributes_keys(Spree::Zone, params[:zone]))
         if @zone.save
           respond_with(@zone, :status => 201, :default_template => :show)
         else
@@ -19,7 +19,7 @@ module Spree
       end
 
       def index
-        @zones = Zone.accessible_by(current_ability, :read).order('name ASC').ransack(params[:q]).result.page(params[:page]).per(params[:per_page])
+        @zones = Spree::Zone.accessible_by(current_ability, :read).order('name ASC').ransack(params[:q]).result.page(params[:page]).per(params[:per_page])
         respond_with(@zones)
       end
 

data/app/views/spree/api/variants/big.v1.rabl

@@ -1,15 +1,9 @@
 object @variant
-attributes *variant_attributes
 
 cache [I18n.locale, @current_user_roles.include?('admin'), 'big_variant', root_object]
 
 extends "spree/api/variants/small"
 
-node :total_on_hand do
-  root_object.total_on_hand
-end
-
-
 child(:stock_items => :stock_items) do
   attributes :id, :count_on_hand, :stock_location_id, :backorderable
   attribute :available? => :available

data/spec/controllers/spree/api/products_controller_spec.rb

@@ -315,6 +315,13 @@ module Spree
           expect(json_response['option_types'].count).to eq(2)
         end
 
+        it "creates product with option_types ids" do
+          option_type = create(:option_type)
+          product_data.merge!(option_type_ids: [option_type.id])
+          api_post :create, product: product_data
+          expect(json_response['option_types'].first['id']).to eq option_type.id
+        end
+
         it "creates with shipping categories" do
           hash = { :name => "The Other Product",
                    :price => 19.99,

data/spec/controllers/spree/api/users_controller_spec.rb

@@ -4,7 +4,7 @@ module Spree
   describe Api::UsersController, :type => :controller do
     render_views
 
-    let(:user) { create(:user, spree_api_key: rand) }
+    let(:user) { create(:user, spree_api_key: rand.to_s) }
     let(:stranger) { create(:user, :email => 'stranger@example.com') }
     let(:attributes) { [:id, :email, :created_at, :updated_at] }
 

data/spec/controllers/spree/api/variants_controller_spec.rb

@@ -4,10 +4,11 @@ module Spree
   describe Api::VariantsController, :type => :controller do
     render_views
 
+    let(:option_value) { create(:option_value) }
     let!(:product) { create(:product) }
     let!(:variant) do
       variant = product.master
-      variant.option_values << create(:option_value)
+      variant.option_values << option_value
       variant
     end
 
@@ -166,10 +167,12 @@ module Spree
       end
 
       it "can create a new variant" do
-        api_post :create, :variant => { :sku => "12345" }
+        api_post :create, variant: { sku: "12345", option_value_ids: [option_value.id] }
+
         expect(json_response).to have_attributes(new_attributes)
         expect(response.status).to eq(201)
         expect(json_response["sku"]).to eq("12345")
+        expect(json_response["option_values"].first["id"]).to eq option_value.id
 
         expect(variant.product.variants.count).to eq(1)
       end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: spree_api
 version: !ruby/object:Gem::Version
-  version: 2.4.6
+  version: 2.4.7
 platform: ruby
 authors:
 - Ryan Bigg
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-03-10 00:00:00.000000000 Z
+date: 2015-05-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: spree_core
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 2.4.6
+        version: 2.4.7
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 2.4.6
+        version: 2.4.7
 - !ruby/object:Gem::Dependency
   name: rabl
   requirement: !ruby/object:Gem::Requirement