shopify_api 9.0.3 → 9.0.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a9f98c9f7ec2a7834c9b4f69c4dfa346e23b220173a03690cabc753e9d0b4318
-  data.tar.gz: 5245b531946faf49fc3213e7faadc5b8372ba71ccfa8e26f14d8b2630bff0d36
+  metadata.gz: 723b8cdb25dbd053853fe820e298d98d535ee211258c04d511c43712c0e51370
+  data.tar.gz: 3306effb209ba7893725cc2b8d56556318db3bea0c7c47916acb58e2df9799e3
 SHA512:
-  metadata.gz: a8b09cb70ebd5ff98dfe08bb4adfb6f25cfa2a2f5023b7e9d703a6e97d33dd6beaac6dfbae29780cf6cab8f9de87718c1e3c5e6cdc8fe2b85b1258a9315c1fa9
-  data.tar.gz: f17ae5b2022d64fe91391dda645e9684b57449dd274f3de0ebbf8c8117072c67d78849646ba914c1432f60b4cfce46ed4b62432e7ecdb00651e1f59210d26c2a
+  metadata.gz: f42c91932eac35091ffa52485df0c82718f79a8945668bd72fdb33cbd0c0c3ca8c71daa2296b0166fbf305892a4789cdabef4356ae3b0b582bf85db68def7695
+  data.tar.gz: f4f530f8e6af923ccfc199136dc29a6ee0c82539aad6c1eb182f6bcceb1d230aa1690e6ba2043e19f1cb5d7cbd11a62a10b792f38643a9872b50504331bc2a0b

data/CHANGELOG.md

@@ -1,3 +1,7 @@
+## Version 9.0.4
+
+* Contains [#708](https://github.com/Shopify/shopify_api/pull/708) which is a revert for [#655](https://github.com/shopify/shopify_api/pull/655) due to the deprecated inventory parameters not being removed correctly in some cases
+
 ## Version 9.0.3
 
 * We now raise a `ShopifyAPI::ValidationException` exception when clients try to use `Product` and `Variant` with deprecated inventory-related fields in API version `2019-10` or later. [#655](https://github.com/shopify/shopify_api/pull/655) Deprecation and migration information can be found in the following documents:

data/lib/shopify_api/resources/product.rb

@@ -16,15 +16,6 @@ module ShopifyAPI
       end
     end
 
-    def total_inventory=(new_value)
-      raise_deprecated_inventory_call('total_inventory') unless allow_inventory_params?
-      super
-    end
-
-    def serializable_hash(options = {})
-      allow_inventory_params? ? super(options) : super(options).except('total_inventory')
-    end
-
     def collections
       CustomCollection.find(:all, :params => {:product_id => self.id})
     end
@@ -40,17 +31,5 @@ module ShopifyAPI
     def remove_from_collection(collection)
       collection.remove_product(self)
     end
-
-    private
-
-    def raise_deprecated_inventory_call(parameter)
-      raise(ShopifyAPI::ValidationException,
-        "'#{parameter}' is deprecated - see https://help.shopify.com/en/api/guides/inventory-migration-guide",
-        )
-    end
-
-    def allow_inventory_params?
-      Base.api_version < ApiVersion.find_version('2019-10')
-    end
   end
 end

data/lib/shopify_api/resources/variant.rb

@@ -4,36 +4,5 @@ module ShopifyAPI
     include DisablePrefixCheck
 
     conditional_prefix :product
-
-    def inventory_quantity_adjustment=(new_value)
-      raise_deprecated_inventory_call('inventory_quantity_adjustment') unless allow_inventory_params?
-      super
-    end
-
-    def inventory_quantity=(new_value)
-      raise_deprecated_inventory_call('inventory_quantity') unless allow_inventory_params?
-      super
-    end
-
-    def old_inventory_quantity=(new_value)
-      raise_deprecated_inventory_call('old_inventory_quantity') unless allow_inventory_params?
-      super
-    end
-
-    def serializable_hash(options = {})
-      allow_inventory_params? ? super(options) : super(options).except('inventory_quantity', 'old_inventory_quantity')
-    end
-
-    private
-
-    def raise_deprecated_inventory_call(parameter)
-      raise(ShopifyAPI::ValidationException,
-        "'#{parameter}' is deprecated - see https://help.shopify.com/en/api/guides/inventory-migration-guide",
-        )
-    end
-
-    def allow_inventory_params?
-      Base.api_version < ApiVersion.find_version('2019-10')
-    end
   end
 end

data/lib/shopify_api/version.rb

@@ -1,3 +1,3 @@
 module ShopifyAPI
-  VERSION = "9.0.3"
+  VERSION = "9.0.4"
 end

data/test/product_test.rb

@@ -57,43 +57,4 @@ class ProductTest < Test::Unit::TestCase
 
     assert_equal('100.00 - 199.00', @product.price_range)
   end
-
-  def test_deprecated_variant_inventory_fields_are_included_in_2019_07
-    ShopifyAPI::Base.api_version = '2019-07'
-    variant = @product.variants.first
-    assert variant.as_json.include?('inventory_quantity')
-  end
-
-  def test_deprecated_variant_inventory_fields_are_removed_in_2020_01
-    ShopifyAPI::Base.api_version = '2020-01'
-    variant = @product.variants.first
-    refute variant.as_json.include?('inventory_quantity')
-  end
-
-  def test_deprecated_inventory_fields_are_removed_in_2020_01
-    ShopifyAPI::Base.api_version = '2020-01'
-    refute @product.as_json.include?('total_inventory')
-  end
-
-  def test_setting_product_total_inventory_passes_in_api_before_2019_10
-    ShopifyAPI::Base.api_version = '2019-07'
-    fake("products/632910392", {:body => load_fixture('product')})
-    @product.total_inventory = 8
-  end
-
-  def test_setting_product_total_inventory_fails_in_2019_10_api
-    ShopifyAPI::Base.api_version = '2019-10'
-    fake("products/632910392", {:body => load_fixture('product')})
-    assert_raises(ShopifyAPI::ValidationException) do
-      @product.total_inventory = 8
-    end
-  end
-
-  def test_setting_product_total_inventory_fails_in_the_unstable_api
-    ShopifyAPI::Base.api_version = :unstable
-    fake("products/632910392", {:body => load_fixture('product')})
-    assert_raises(ShopifyAPI::ValidationException) do
-      @product.total_inventory = 8
-    end
-  end
 end

data/test/test_helper.rb

@@ -96,7 +96,7 @@ module Test
         body   = options.has_key?(:body) ? options.delete(:body) : load_fixture(endpoint)
         format = options.delete(:format) || :json
         method = options.delete(:method) || :get
-        api_version = options.delete(:api_version) || ShopifyAPI::Base.api_version
+        api_version = options.delete(:api_version) || ShopifyAPI::ApiVersion.find_version('2019-01')
         extension = ".#{options.delete(:extension)||'json'}" unless options[:extension]==false
         status = options.delete(:status) || 200
         url = if options.has_key?(:url)

data/test/variant_test.rb

@@ -43,93 +43,4 @@ class VariantTest < Test::Unit::TestCase
     v = ShopifyAPI::Variant.find(808950810, :params => {:product_id => 632910392})
     assert v.destroy
   end
-
-  def test_deprecated_inventory_fields_are_included_in_2019_07
-    ShopifyAPI::Base.api_version = '2019-07'
-    fake "products/632910392/variants/808950810", :method => :get, :body => load_fixture('variant')
-    variant = ShopifyAPI::Variant.find(808950810, :params => {:product_id => 632910392})
-    assert variant.as_json.include?('inventory_quantity')
-  end
-
-  def test_deprecated_inventory_fields_are_removed_in_2020_01
-    ShopifyAPI::Base.api_version = '2020-01'
-    fake "products/632910392/variants/808950810", :method => :get, :body => load_fixture('variant')
-    variant = ShopifyAPI::Variant.find(808950810, :params => {:product_id => 632910392})
-    refute variant.as_json.include?('inventory_quantity')
-  end
-
-  def test_setting_variant_inventory_quantity_adjustment_passes_in_api_before_2019_10
-    ShopifyAPI::Base.api_version = '2019-07'
-    fake "products/632910392/variants/808950810", :method => :get, :body => load_fixture('variant')
-    variant = ShopifyAPI::Variant.find(808950810, :params => {:product_id => 632910392})
-    variant.inventory_quantity_adjustment = 8
-  end
-
-  def test_setting_variant_inventory_quantity_adjustment_fails_in_2019_10_api
-    ShopifyAPI::Base.api_version = '2019-10'
-    fake "products/632910392/variants/808950810", :method => :get, :body => load_fixture('variant')
-    variant = ShopifyAPI::Variant.find(808950810, :params => {:product_id => 632910392})
-    assert_raises(ShopifyAPI::ValidationException) do
-      variant.inventory_quantity_adjustment = 8
-    end
-  end
-
-  def test_setting_variant_inventory_quantity_adjustment_fails_in_the_unstable_api
-    ShopifyAPI::Base.api_version = :unstable
-    fake "products/632910392/variants/808950810", :method => :get, :body => load_fixture('variant')
-    variant = ShopifyAPI::Variant.find(808950810, :params => {:product_id => 632910392})
-    assert_raises(ShopifyAPI::ValidationException) do
-      variant.inventory_quantity_adjustment = 8
-    end
-  end
-
-  def test_setting_variant_inventory_quantity_passes_in_api_before_2019_10
-    ShopifyAPI::Base.api_version = '2019-07'
-    fake "products/632910392/variants/808950810", :method => :get, :body => load_fixture('variant')
-    variant = ShopifyAPI::Variant.find(808950810, :params => {:product_id => 632910392})
-    variant.inventory_quantity = 8
-  end
-
-  def test_setting_variant_inventory_quantity_fails_in_2019_10_api
-    ShopifyAPI::Base.api_version = '2019-10'
-    fake "products/632910392/variants/808950810", :method => :get, :body => load_fixture('variant')
-    variant = ShopifyAPI::Variant.find(808950810, :params => {:product_id => 632910392})
-    assert_raises(ShopifyAPI::ValidationException) do
-      variant.inventory_quantity = 8
-    end
-  end
-
-  def test_setting_variant_inventory_quantity_fails_in_the_unstable_api
-    ShopifyAPI::Base.api_version = :unstable
-    fake "products/632910392/variants/808950810", :method => :get, :body => load_fixture('variant')
-    variant = ShopifyAPI::Variant.find(808950810, :params => {:product_id => 632910392})
-    assert_raises(ShopifyAPI::ValidationException) do
-      variant.inventory_quantity = 8
-    end
-  end
-
-  def test_setting_variant_old_inventory_quantity_passes_in_api_before_2019_10
-    ShopifyAPI::Base.api_version = '2019-07'
-    fake "products/632910392/variants/808950810", :method => :get, :body => load_fixture('variant')
-    variant = ShopifyAPI::Variant.find(808950810, :params => {:product_id => 632910392})
-    variant.old_inventory_quantity = 8
-  end
-
-  def test_setting_variant_old_inventory_quantity_fails_in_2019_10_api
-    ShopifyAPI::Base.api_version = '2019-10'
-    fake "products/632910392/variants/808950810", :method => :get, :body => load_fixture('variant')
-    variant = ShopifyAPI::Variant.find(808950810, :params => {:product_id => 632910392})
-    assert_raises(ShopifyAPI::ValidationException) do
-      variant.old_inventory_quantity = 8
-    end
-  end
-
-  def test_setting_variant_old_inventory_quantity_fails_in_the_unstable_api
-    ShopifyAPI::Base.api_version = :unstable
-    fake "products/632910392/variants/808950810", :method => :get, :body => load_fixture('variant')
-    variant = ShopifyAPI::Variant.find(808950810, :params => {:product_id => 632910392})
-    assert_raises(ShopifyAPI::ValidationException) do
-      variant.old_inventory_quantity = 8
-    end
-  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: shopify_api
 version: !ruby/object:Gem::Version
-  version: 9.0.3
+  version: 9.0.4
 platform: ruby
 authors:
 - Shopify