mercadolibre_rails 0.5 → 0.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d4534f513781489dd5f68379a0e10e95370918f8376b2820de504329d16655f5
-  data.tar.gz: 5f0209ae2ed18ccd17b24c74e9681cef0e3d7705301d2bc5d7fa8c4b64008f8f
+  metadata.gz: df8b324f21f677cc5b8f45c6a37d4511523f076e99f816ac2dc5f2b98e3f6a0c
+  data.tar.gz: e717205170575157ecaee610d9f85cd6a82bab9003ddeb7b19cce045f3415c17
 SHA512:
-  metadata.gz: 8ff935a2e2005a6f7b5d885fe79a60398b98a450eda2c30c5121d2883b62c87d19bd62b09b0fe3627aa088df82e3a164ab37470cca51a6489e3ec2e1fc643832
-  data.tar.gz: c312ccc37ca12ad892dbd1ee9bdb70a40debbab1910f36a916e039981377e175248abc23ca175b77fc0e0de29cceae6584e62f75413648e4e40af2d299a8904b
+  metadata.gz: c8e100fa61d958425a3caf1e9e327469a412f10020df038f4a413ebc118118e25dd04773236e8310f39b8b8be8f4bd1dec750d6c1c2ca4e5662078af6b17f792
+  data.tar.gz: a8ece76dfaa42f9c6eca0db7ab4991dc77804416e568c01d9ef2e0009a4603345a12edf513dceabe552e3d51ee05c8c1b593fe2acbf501b57217b86c148923e7

data/app/models/mercadolibre_rails/product.rb

@@ -22,14 +22,14 @@ module MercadolibreRails
 
       return unless ml_id && site_code
 
-      mercadolibre_site = MercadolibreRails::Site.find_or_create_by!(code: site_code)
-      mercadolibre_site.products.find_or_create_by!(mercadolibre_id: ml_id)
+      mercadolibre_site = MercadolibreRails::Site.where(code: site_code).first_or_create
+      mercadolibre_site.products.where(mercadolibre_id: ml_id).first_or_create
     end
 
     # rubocop:disable Metrics/AbcSize
     def update_metadata
       ml_product = MercadolibreApi::Products::Queries::Find.run!(product_id: mercadolibre_id)
-      ml_seller = MercadolibreRails::Seller.find_or_create_by!(mercadolibre_id: ml_product[:seller_id])
+      ml_seller = MercadolibreRails::Seller.where(mercadolibre_id: ml_product[:seller_id], site: site).first_or_create
 
       update!(seller: ml_seller, title: ml_product[:title], price: ml_product[:price],
               currency_code: ml_product[:currency_id], sold_quantity: ml_product[:sold_quantity],
@@ -37,7 +37,7 @@ module MercadolibreRails
               latitude: ml_product.dig(:geolocation, :latitude), longitude: ml_product.dig(:geolocation, :longitude))
 
       ml_product[:pictures].each do |picture|
-        MercadolibreRails::Picture.find_or_create_by!(url: picture[:url], product_id: id)
+        MercadolibreRails::Picture.where(url: picture[:url], product_id: id).first_or_create
       end
     end
     # rubocop:enable Metrics/AbcSize

data/app/models/mercadolibre_rails/seller.rb

@@ -3,12 +3,28 @@ require 'mercadolibre_api'
 module MercadolibreRails
   class Seller < ApplicationRecord
     has_many :products, dependent: :destroy
+    belongs_to :site
+
+    delegate :code, to: :site, prefix: true
+    after_create_commit :create_products
 
     def self.create_from(product_url:)
       ml_product_id = MercadolibreApi::Products::Queries::GetID.run!(product_url: product_url)
       ml_seller_id = MercadolibreApi::Sellers::Queries::GetID.run!(product_id: ml_product_id)
+      ml_site_code = MercadolibreApi::Sites::Queries::GetCode.run!(product_url: product_url)
+
+      ml_site = MercadolibreRails::Site.where(code: ml_site_code).first_or_create
+      where(mercadolibre_id: ml_seller_id, site: ml_site).first_or_create
+    end
+
+    private
+
+    def create_products
+      ordered_products = MercadolibreApi::Sellers::Products::Queries::MostSold.run!(
+        seller_id: mercadolibre_id, site_code: site_code
+      )
 
-      find_or_create_by!(mercadolibre_id: ml_seller_id)
+      ordered_products.each { |product| products.where(mercadolibre_id: product[:id]).first_or_create }
     end
   end
 end

data/app/models/mercadolibre_rails/site.rb

@@ -1,6 +1,7 @@
 module MercadolibreRails
   class Site < ApplicationRecord
     has_many :products, dependent: :destroy
+    has_many :sellers, dependent: :destroy
 
     validates_presence_of :code
     validates_uniqueness_of :code

data/db/migrate/20181004143133_create_mercadolibre_rails_sellers.rb

@@ -1,7 +1,7 @@
 class CreateMercadolibreRailsSellers < ActiveRecord::Migration[5.2]
   def change
     create_table :mercadolibre_rails_sellers do |t|
-      t.integer :mercadolibre_id, null: false, index: { unique: true }
+      t.integer :mercadolibre_id, null: false
 
       t.timestamps
     end

data/db/migrate/20181022135655_add_site_id_to_mercadolibre_rails_sellers.rb

@@ -0,0 +1,6 @@
+class AddSiteIdToMercadolibreRailsSellers < ActiveRecord::Migration[5.2]
+  def change
+    add_reference :mercadolibre_rails_sellers, :site, foreign_key: true
+    add_index :mercadolibre_rails_sellers, %i[mercadolibre_id site_id], name: 'ml_id_site_id_on_sellers'
+  end
+end

data/lib/mercadolibre_rails/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module MercadolibreRails
-  VERSION = '0.5'
+  VERSION = '0.6'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: mercadolibre_rails
 version: !ruby/object:Gem::Version
-  version: '0.5'
+  version: '0.6'
 platform: ruby
 authors:
 - Pablo Gonzaga
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-10-22 00:00:00.000000000 Z
+date: 2018-10-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: mercadolibre_api
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.9'
+        version: '0.10'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.9'
+        version: '0.10'
 - !ruby/object:Gem::Dependency
   name: paper_trail
   requirement: !ruby/object:Gem::Requirement
@@ -184,6 +184,7 @@ files:
 - db/migrate/20181010142220_create_mercadolibre_rails_versions.rb
 - db/migrate/20181010142221_add_object_changes_to_mercadolibre_rails_versions.rb
 - db/migrate/20181012133312_create_mercadolibre_rails_pictures.rb
+- db/migrate/20181022135655_add_site_id_to_mercadolibre_rails_sellers.rb
 - lib/mercadolibre_rails.rb
 - lib/mercadolibre_rails/engine.rb
 - lib/mercadolibre_rails/version.rb