products: fix dependent plugins

Braulio Bhavamitra
1 parent 74e81649
Showing 19 changed files with 292 additions and 276 deletions Show diff stats
plugins/metadata/install.rb
plugins/metadata/lib/ext/product.rb
plugins/metadata/lib/ext/products_plugin/product.rb
plugins/metadata/test/functional/manage_products_controller_test.rb
plugins/metadata/test/functional/products_plugin/page_controller_test.rb
plugins/orders/install.rb
plugins/orders/lib/ext/product.rb
plugins/orders/lib/ext/products_plugin/product.rb
plugins/orders/models/orders_plugin/item.rb
plugins/orders_cycle/lib/ext/product.rb
plugins/orders_cycle/lib/ext/products_plugin/product.rb
plugins/pg_search/db/migrate/20130320010063_create_indexes_for_search.rb
plugins/shopping_cart/views/shopping_cart_plugin/send_request.js.erb
plugins/sniffer/install.rb
plugins/statistics/install.rb
plugins/suppliers/install.rb
plugins/suppliers/lib/ext/price_detail.rb
plugins/suppliers/lib/ext/product.rb
plugins/suppliers/lib/ext/products_plugin/product.rb
@@ -0,0 +1,5 @@
+if ENV['CI']
+  system 'script/noosfero-plugins -q enable products'
+  exit $?.exitstatus
+end
+
@@ -1,34 +0,0 @@
-require_dependency 'product'
-
-class Product
-
-  metadata_spec namespace: :og, tags: {
-    type: proc{ |p, plugin| plugin.context.params[:og_type] || MetadataPlugin.og_types[:product] || :product },
-    url: proc do |p, plugin|
-      url = p.url.merge! profile: p.profile.identifier, og_type: plugin.context.params[:og_type]
-      plugin.og_url_for url
-    end,
-    gr_hascurrencyvalue: proc{ |p, plugin| p.price.to_f },
-    gr_hascurrency: proc{ |p, plugin| p.environment.currency_unit },
-    title: proc{ |p, plugin| "#{p.name} - #{p.profile.name}" if p },
-    description: proc{ |p, plugin| ActionView::Base.full_sanitizer.sanitize p.description },
-
-    image: proc do |p, plugin|
-      img = "#{p.environment.top_url}#{p.image.public_filename}".html_safe if p.image
-      img = "#{p.environment.top_url}#{p.profile.image.public_filename}".html_safe if img.blank? and p.profile.image
-      img ||= MetadataPlugin.config[:open_graph][:environment_logo] rescue nil if img.blank?
-      img
-    end,
-    'image:type' => proc{ |p, plugin| p.image.content_type if p.image },
-    'image:height' => proc{ |p, plugin| p.image.height if p.image },
-    'image:width' => proc{ |p, plugin| p.image.width if p.image },
-
-    see_also: [],
-    site_name: proc{ |p, plugin| plugin.og_url_for p.profile.url },
-    updated_time: proc{ |p, plugin| p.updated_at.iso8601 if p.updated_at },
-
-    'locale:locale' => proc{ |p, plugin| p.environment.default_language },
-    'locale:alternate' => proc{ |p, plugin| p.environment.languages - [p.environment.default_language] if p.environment.languages },
-  }
-
-end
@@ -0,0 +1,38 @@
+if defined? ProductsPlugin
+  require_dependency 'products_plugin/product'
+
+  module ProductsPlugin
+    class Product
+
+      metadata_spec namespace: :og, tags: {
+        type: proc{ |p, plugin| plugin.context.params[:og_type] || MetadataPlugin.og_types[:product] || :product },
+        url: proc do |p, plugin|
+          url = p.url.merge! profile: p.profile.identifier, og_type: plugin.context.params[:og_type]
+          plugin.og_url_for url
+        end,
+        gr_hascurrencyvalue: proc{ |p, plugin| p.price.to_f },
+        gr_hascurrency: proc{ |p, plugin| p.environment.currency_unit },
+        title: proc{ |p, plugin| "#{p.name} - #{p.profile.name}" if p },
+        description: proc{ |p, plugin| ActionView::Base.full_sanitizer.sanitize p.description },
+
+        image: proc do |p, plugin|
+          img = "#{p.environment.top_url}#{p.image.public_filename}".html_safe if p.image
+          img = "#{p.environment.top_url}#{p.profile.image.public_filename}".html_safe if img.blank? and p.profile.image
+          img ||= MetadataPlugin.config[:open_graph][:environment_logo] rescue nil if img.blank?
+          img
+        end,
+        'image:type' => proc{ |p, plugin| p.image.content_type if p.image },
+        'image:height' => proc{ |p, plugin| p.image.height if p.image },
+        'image:width' => proc{ |p, plugin| p.image.width if p.image },
+
+        see_also: [],
+        site_name: proc{ |p, plugin| plugin.og_url_for p.profile.url },
+        updated_time: proc{ |p, plugin| p.updated_at.iso8601 if p.updated_at },
+
+        'locale:locale' => proc{ |p, plugin| p.environment.default_language },
+        'locale:alternate' => proc{ |p, plugin| p.environment.languages - [p.environment.default_language] if p.environment.languages },
+      }
+
+    end
+  end
+end
@@ -1,24 +0,0 @@
-require 'test_helper'
-require 'home_controller'
-
-class ManageProductsControllerTest < ActionController::TestCase
-
-  def setup
-    @controller = ManageProductsController.new
-    @request    = ActionController::TestRequest.new
-    @response   = ActionController::TestResponse.new
-    @enterprise = fast_create(Enterprise, name: 'test', identifier: 'test_ent')
-    @user = create_user_with_permission('test_user', 'manage_products', @enterprise)
-    @environment = @enterprise.environment
-    @environment.enable('products_for_enterprises')
-    login_as :test_user
-
-    @environment.enabled_plugins += ['MetadataPlugin']
-    @environment.save!
-  end
-
-  should "not crash on new products" do
-    get :new, profile: @enterprise.identifier
-  end
-
-end
@@ -0,0 +1,24 @@
+require 'test_helper'
+
+if defined? ProductsPlugin
+  module ProductsPlugin
+    class ManageProductsControllerTest < ActionController::TestCase
+
+      def setup
+        @controller = PageController.new
+        @request    = ActionController::TestRequest.new
+        @response   = ActionController::TestResponse.new
+        @enterprise = fast_create(Enterprise, name: 'test', identifier: 'test_ent')
+        @user = create_user_with_permission('test_user', 'manage_products', @enterprise)
+        login_as :test_user
+
+        @enterprise.environment.enable_plugin 'MetadataPlugin'
+      end
+
+      should "not crash on new products" do
+        get :new, profile: @enterprise.identifier
+      end
+
+    end
+  end
+end
-system "script/noosfero-plugins -q enable delivery"
+system 'script/noosfero-plugins -q enable products delivery'
 exit $?.exitstatus
@@ -1,16 +0,0 @@
-require_dependency 'product'
-
-class Product
-
-  has_many :items, class_name: 'OrdersPlugin::Item', foreign_key: :product_id, dependent: :destroy
-
-  has_many :orders, through: :items
-  has_many :sales, through: :items
-  has_many :purchases, through: :items
-
-  attr_accessor :quantity_ordered
-
-  extend CurrencyHelper::ClassMethods
-  instance_exec &OrdersPlugin::Item::DefineTotals
-
-end
@@ -0,0 +1,18 @@
+require_dependency 'products_plugin/product'
+
+module ProductsPlugin
+  class Product
+
+    has_many :items, class_name: 'OrdersPlugin::Item', foreign_key: :product_id, dependent: :destroy
+
+    has_many :orders, through: :items
+    has_many :sales, through: :items
+    has_many :purchases, through: :items
+
+    attr_accessor :quantity_ordered
+
+    extend CurrencyHelper::ClassMethods
+    instance_exec &OrdersPlugin::Item::DefineTotals
+
+  end
+end
@@ -39,7 +39,7 @@ class OrdersPlugin::Item &lt; ApplicationRecord
   belongs_to :sale, class_name: '::OrdersPlugin::Sale', foreign_key: :order_id, touch: true
   belongs_to :purchase, class_name: '::OrdersPlugin::Purchase', foreign_key: :order_id, touch: true
-  belongs_to :product
+  belongs_to :product, class_name: '::ProductsPlugin::Product'
   has_one :supplier, through: :product
   has_one :profile, through: :order
@@ -1,12 +0,0 @@
-require_dependency 'product'
-
-# based on orders/lib/ext/product.rb
-class Product
-
-  has_many :orders_cycles_items, class_name: 'OrdersCyclePlugin::Item', foreign_key: :product_id
-
-  has_many :orders_cycles_orders, through: :orders_cycles_items, source: :order
-  has_many :orders_cycles_sales, through: :orders_cycles_items, source: :sale
-  has_many :orders_cycles_purchases, through: :orders_cycles_items, source: :purchase
-
-end
@@ -0,0 +1,14 @@
+require_dependency 'products_plugin/product'
+
+# based on orders/lib/ext/product.rb
+module ProductsPlugin
+  class Product
+
+    has_many :orders_cycles_items, class_name: 'OrdersCyclePlugin::Item', foreign_key: :product_id
+
+    has_many :orders_cycles_orders, through: :orders_cycles_items, source: :order
+    has_many :orders_cycles_sales, through: :orders_cycles_items, source: :sale
+    has_many :orders_cycles_purchases, through: :orders_cycles_items, source: :purchase
+
+  end
+end
 class CreateIndexesForSearch < ActiveRecord::Migration
-  SEARCHABLES = %w[ article comment qualifier national_region certifier profile license scrap category ]
+  SEARCHABLES = %w[ article comment national_region profile license scrap category ]
   KLASSES = SEARCHABLES.map {|searchable| searchable.camelize.constantize }
   def self.up
     KLASSES.each do |klass|
 <% if @success %>
-  window.location.href = <%= url_for(controller: :catalog, profile: @profile.identifier).to_json %>
+  window.location.href = <%= url_for(controller: 'products_plugin/catalog', profile: @profile.identifier).to_json %>
 <% else %>
   display_notice(<%= "#{_ 'Error'}: #{@error}".to_json %>)
 <% end %>
@@ -0,0 +1,3 @@
+system 'script/noosfero-plugins -q enable products'
+exit $?.exitstatus
+
@@ -0,0 +1,5 @@
+if ENV['CI']
+  system 'script/noosfero-plugins -q enable products'
+  exit $?.exitstatus
+end
+
@@ -0,0 +1,3 @@
+system 'script/noosfero-plugins -q enable products'
+exit $?.exitstatus
+
@@ -1,8 +0,0 @@
-require_dependency 'price_detail'
-
-class PriceDetail
-
-  # should be on core, used by SuppliersPlugin::Import
-  attr_accessible :production_cost
-
-end
@@ -1,178 +0,0 @@
-require_dependency 'product'
-
-# FIXME: The lines bellow should be on the core
-class Product
-
-  extend CurrencyHelper::ClassMethods
-  has_currency :price
-  has_currency :discount
-
-  scope :alphabetically, -> { order 'products.name ASC' }
-
-  scope :available, -> { where available: true }
-  scope :unavailable, -> { where 'products.available <> true' }
-  scope :archived, -> { where archived: true }
-  scope :unarchived, -> { where 'products.archived <> true' }
-
-  scope :with_available, -> (available) { where available: available }
-  scope :with_price, -> { where 'products.price > 0' }
-
-  # FIXME: transliterate input and name column
-  scope :name_like, -> (name) { where "name ILIKE ?", "%#{name}%" }
-  scope :with_product_category_id, -> (id) { where product_category_id: id }
-
-  scope :by_profile, -> (profile) { where profile_id: profile.id }
-  scope :by_profile_id, -> (profile_id) { where profile_id: profile_id }
-
-  def self.product_categories_of products
-    ProductCategory.find products.collect(&:product_category_id).compact.select{ |id| not id.zero? }
-  end
-
-  attr_accessible :external_id
-  settings_items :external_id, type: String, default: nil
-
-  # should be on core, used by SuppliersPlugin::Import
-  attr_accessible :price_details
-
-end
-
-class Product
-
-  attr_accessible :from_products, :from_product, :supplier_id, :supplier
-
-  has_many :sources_from_products, foreign_key: :to_product_id, class_name: 'SuppliersPlugin::SourceProduct', dependent: :destroy
-  has_one  :sources_from_product,  foreign_key: :to_product_id, class_name: 'SuppliersPlugin::SourceProduct'
-  has_many :sources_to_products, foreign_key: :from_product_id, class_name: 'SuppliersPlugin::SourceProduct', dependent: :destroy
-  has_one  :sources_to_product,  foreign_key: :from_product_id, class_name: 'SuppliersPlugin::SourceProduct'
-  has_many :to_products, -> { order 'id ASC' }, through: :sources_to_products
-  has_one  :to_product, -> { order 'id ASC' },  through: :sources_to_product,  autosave: true
-  has_many :from_products, -> { order 'id ASC' }, through: :sources_from_products
-  has_one  :from_product, -> { order 'id ASC' },  through: :sources_from_product, autosave: true
-
-  has_many :sources_from_2x_products, through: :from_products, source: :sources_from_products
-  has_one  :sources_from_2x_product,  through: :from_product,  source: :sources_from_product
-  has_many :sources_to_2x_products,   through: :to_products,   source: :sources_to_products
-  has_one  :sources_to_2x_product,    through: :to_product,    source: :sources_to_product
-  has_many :from_2x_products, through: :sources_from_2x_products, source: :from_product
-  has_one  :from_2x_product,  through: :sources_from_2x_product,  source: :from_product
-  has_many :to_2x_products,   through: :sources_to_2x_products,   source: :to_product
-  has_one  :to_2x_product,    through: :sources_to_2x_product,    source: :to_product
-
-  # semantic alias for supplier_from_product(s)
-  has_many :sources_supplier_products, foreign_key: :to_product_id, class_name: 'SuppliersPlugin::SourceProduct'
-  has_one  :sources_supplier_product,  foreign_key: :to_product_id, class_name: 'SuppliersPlugin::SourceProduct'
-  has_many :supplier_products, -> { order 'id ASC' }, through: :sources_supplier_products, source: :from_product
-  has_one  :supplier_product, -> { order 'id ASC' },  through: :sources_supplier_product,  source: :from_product, autosave: true
-  has_many :suppliers, -> { distinct.order 'id ASC' }, through: :sources_supplier_products
-  has_one  :supplier, -> { order 'id ASC' },  through: :sources_supplier_product
-
-  has_many :consumers, -> { distinct.order 'id ASC' }, through: :to_products, source: :profile
-  has_one  :consumer, -> { order 'id ASC' },  through: :to_product,  source: :profile
-
-  # overhide original, FIXME: rename to available_and_supplier_active
-  scope :available, -> {
-    joins(:suppliers).
-    where 'products.available = ? AND suppliers_plugin_suppliers.active = ?', true, true
-  }
-  scope :unavailable, -> {
-    where 'products.available <> ? OR suppliers_plugin_suppliers.active <> ?', true, true
-  }
-  scope :with_available, -> (available) {
-    op = if available then '=' else '<>' end
-    cond = if available then 'AND' else 'OR' end
-    where "products.available #{op} ? #{cond} suppliers_plugin_suppliers.active #{op} ?", true, true
-  }
-
-  scope :fp_name_like, -> (name) { where "from_products_products.name ILIKE ?", "%#{name}%" }
-  scope :fp_with_product_category_id, -> (id) { where 'from_products_products.product_category_id = ?', id }
-
-  # prefer distributed_products has_many to use DistributedProduct scopes and eager loading
-  scope :distributed, -> { where type: 'SuppliersPlugin::DistributedProduct'}
-  scope :own, -> { where type: nil }
-  scope :supplied, -> {
-    # this remove duplicates and allow sorting on the fields, unlike distinct
-    group('products.id').
-    where type: [nil, 'SuppliersPlugin::DistributedProduct']
-  }
-  scope :supplied_for_count, -> {
-    distinct.
-    where type: [nil, 'SuppliersPlugin::DistributedProduct']
-  }
-
-  scope :from_supplier, -> (supplier) { joins(:suppliers).where 'suppliers_plugin_suppliers.id = ?', supplier.id }
-  scope :from_supplier_id, -> (supplier_id) { joins(:suppliers).where 'suppliers_plugin_suppliers.id = ?', supplier_id }
-
-  after_create :distribute_to_consumers
-
-  def own?
-    self.class == Product
-  end
-  def distributed?
-    self.class == SuppliersPlugin::DistributedProduct
-  end
-  def supplied?
-    self.own? or self.distributed?
-  end
-
-  def supplier
-    # FIXME: use self.suppliers when rails support for nested preload comes
-    @supplier ||= self.sources_supplier_product.supplier rescue nil
-    @supplier ||= self.profile.self_supplier rescue nil
-  end
-  def supplier= value
-    @supplier = value
-  end
-  def supplier_id
-    self.supplier.id
-  end
-  def supplier_id= id
-    @supplier = profile.environment.profiles.find id
-  end
-
-  def supplier_dummy?
-    self.supplier ? self.supplier.dummy? : self.profile.dummy?
-  end
-
-  def distribute_to_consumer consumer, attrs = {}
-    distributed_product = consumer.distributed_products.where(profile_id: consumer.id, from_products_products: {id: self.id}).first
-    distributed_product ||= SuppliersPlugin::DistributedProduct.create! profile: consumer, from_product: self
-    distributed_product.update! attrs if attrs.present?
-    distributed_product
-  end
-
-  def destroy_dependent
-    self.to_products.each do |to_product|
-      to_product.destroy if to_product.respond_to? :dependent? and to_product.dependent?
-    end
-  end
-
-  # before_destroy and after_destroy don't work,
-  # see http://stackoverflow.com/questions/14175330/associations-not-loaded-in-before-destroy-callback
-  def destroy
-    self.class.transaction do
-      self.destroy_dependent
-      super
-    end
-  end
-
-  def diff from = self.from_product
-    return @changed_attrs if @changed_attrs
-    @changed_attrs = []
-    SuppliersPlugin::BaseProduct::CORE_DEFAULT_ATTRIBUTES.each do |attr|
-      @changed_attrs << attr if self[attr].present? and self[attr] != from[attr]
-    end
-    @changed_attrs
-  end
-
-  protected
-
-  def distribute_to_consumers
-    # shopping_cart creates products without a profile...
-    return unless self.profile
-
-    self.profile.consumers.except_people.except_self.each do |consumer|
-      self.distribute_to_consumer consumer.profile
-    end
-  end
-
-end
@@ -0,0 +1,178 @@
+require_dependency 'products_plugin/product'
+
+# FIXME: The lines bellow should be on the core
+class Product
+
+  extend CurrencyHelper::ClassMethods
+  has_currency :price
+  has_currency :discount
+
+  scope :alphabetically, -> { order 'products.name ASC' }
+
+  scope :available, -> { where available: true }
+  scope :unavailable, -> { where 'products.available <> true' }
+  scope :archived, -> { where archived: true }
+  scope :unarchived, -> { where 'products.archived <> true' }
+
+  scope :with_available, -> (available) { where available: available }
+  scope :with_price, -> { where 'products.price > 0' }
+
+  # FIXME: transliterate input and name column
+  scope :name_like, -> (name) { where "name ILIKE ?", "%#{name}%" }
+  scope :with_product_category_id, -> (id) { where product_category_id: id }
+
+  scope :by_profile, -> (profile) { where profile_id: profile.id }
+  scope :by_profile_id, -> (profile_id) { where profile_id: profile_id }
+
+  def self.product_categories_of products
+    ProductCategory.find products.collect(&:product_category_id).compact.select{ |id| not id.zero? }
+  end
+
+  attr_accessible :external_id
+  settings_items :external_id, type: String, default: nil
+
+  # should be on core, used by SuppliersPlugin::Import
+  attr_accessible :price_details
+
+end
+
+class Product
+
+  attr_accessible :from_products, :from_product, :supplier_id, :supplier
+
+  has_many :sources_from_products, foreign_key: :to_product_id, class_name: 'SuppliersPlugin::SourceProduct', dependent: :destroy
+  has_one  :sources_from_product,  foreign_key: :to_product_id, class_name: 'SuppliersPlugin::SourceProduct'
+  has_many :sources_to_products, foreign_key: :from_product_id, class_name: 'SuppliersPlugin::SourceProduct', dependent: :destroy
+  has_one  :sources_to_product,  foreign_key: :from_product_id, class_name: 'SuppliersPlugin::SourceProduct'
+  has_many :to_products, -> { order 'id ASC' }, through: :sources_to_products
+  has_one  :to_product, -> { order 'id ASC' },  through: :sources_to_product,  autosave: true
+  has_many :from_products, -> { order 'id ASC' }, through: :sources_from_products
+  has_one  :from_product, -> { order 'id ASC' },  through: :sources_from_product, autosave: true
+
+  has_many :sources_from_2x_products, through: :from_products, source: :sources_from_products
+  has_one  :sources_from_2x_product,  through: :from_product,  source: :sources_from_product
+  has_many :sources_to_2x_products,   through: :to_products,   source: :sources_to_products
+  has_one  :sources_to_2x_product,    through: :to_product,    source: :sources_to_product
+  has_many :from_2x_products, through: :sources_from_2x_products, source: :from_product
+  has_one  :from_2x_product,  through: :sources_from_2x_product,  source: :from_product
+  has_many :to_2x_products,   through: :sources_to_2x_products,   source: :to_product
+  has_one  :to_2x_product,    through: :sources_to_2x_product,    source: :to_product
+
+  # semantic alias for supplier_from_product(s)
+  has_many :sources_supplier_products, foreign_key: :to_product_id, class_name: 'SuppliersPlugin::SourceProduct'
+  has_one  :sources_supplier_product,  foreign_key: :to_product_id, class_name: 'SuppliersPlugin::SourceProduct'
+  has_many :supplier_products, -> { order 'id ASC' }, through: :sources_supplier_products, source: :from_product
+  has_one  :supplier_product, -> { order 'id ASC' },  through: :sources_supplier_product,  source: :from_product, autosave: true
+  has_many :suppliers, -> { distinct.order 'id ASC' }, through: :sources_supplier_products
+  has_one  :supplier, -> { order 'id ASC' },  through: :sources_supplier_product
+
+  has_many :consumers, -> { distinct.order 'id ASC' }, through: :to_products, source: :profile
+  has_one  :consumer, -> { order 'id ASC' },  through: :to_product,  source: :profile
+
+  # overhide original, FIXME: rename to available_and_supplier_active
+  scope :available, -> {
+    joins(:suppliers).
+    where 'products.available = ? AND suppliers_plugin_suppliers.active = ?', true, true
+  }
+  scope :unavailable, -> {
+    where 'products.available <> ? OR suppliers_plugin_suppliers.active <> ?', true, true
+  }
+  scope :with_available, -> (available) {
+    op = if available then '=' else '<>' end
+    cond = if available then 'AND' else 'OR' end
+    where "products.available #{op} ? #{cond} suppliers_plugin_suppliers.active #{op} ?", true, true
+  }
+
+  scope :fp_name_like, -> (name) { where "from_products_products.name ILIKE ?", "%#{name}%" }
+  scope :fp_with_product_category_id, -> (id) { where 'from_products_products.product_category_id = ?', id }
+
+  # prefer distributed_products has_many to use DistributedProduct scopes and eager loading
+  scope :distributed, -> { where type: 'SuppliersPlugin::DistributedProduct'}
+  scope :own, -> { where type: nil }
+  scope :supplied, -> {
+    # this remove duplicates and allow sorting on the fields, unlike distinct
+    group('products.id').
+    where type: [nil, 'SuppliersPlugin::DistributedProduct']
+  }
+  scope :supplied_for_count, -> {
+    distinct.
+    where type: [nil, 'SuppliersPlugin::DistributedProduct']
+  }
+
+  scope :from_supplier, -> (supplier) { joins(:suppliers).where 'suppliers_plugin_suppliers.id = ?', supplier.id }
+  scope :from_supplier_id, -> (supplier_id) { joins(:suppliers).where 'suppliers_plugin_suppliers.id = ?', supplier_id }
+
+  after_create :distribute_to_consumers
+
+  def own?
+    self.class == Product
+  end
+  def distributed?
+    self.class == SuppliersPlugin::DistributedProduct
+  end
+  def supplied?
+    self.own? or self.distributed?
+  end
+
+  def supplier
+    # FIXME: use self.suppliers when rails support for nested preload comes
+    @supplier ||= self.sources_supplier_product.supplier rescue nil
+    @supplier ||= self.profile.self_supplier rescue nil
+  end
+  def supplier= value
+    @supplier = value
+  end
+  def supplier_id
+    self.supplier.id
+  end
+  def supplier_id= id
+    @supplier = profile.environment.profiles.find id
+  end
+
+  def supplier_dummy?
+    self.supplier ? self.supplier.dummy? : self.profile.dummy?
+  end
+
+  def distribute_to_consumer consumer, attrs = {}
+    distributed_product = consumer.distributed_products.where(profile_id: consumer.id, from_products_products: {id: self.id}).first
+    distributed_product ||= SuppliersPlugin::DistributedProduct.create! profile: consumer, from_product: self
+    distributed_product.update! attrs if attrs.present?
+    distributed_product
+  end
+
+  def destroy_dependent
+    self.to_products.each do |to_product|
+      to_product.destroy if to_product.respond_to? :dependent? and to_product.dependent?
+    end
+  end
+
+  # before_destroy and after_destroy don't work,
+  # see http://stackoverflow.com/questions/14175330/associations-not-loaded-in-before-destroy-callback
+  def destroy
+    self.class.transaction do
+      self.destroy_dependent
+      super
+    end
+  end
+
+  def diff from = self.from_product
+    return @changed_attrs if @changed_attrs
+    @changed_attrs = []
+    SuppliersPlugin::BaseProduct::CORE_DEFAULT_ATTRIBUTES.each do |attr|
+      @changed_attrs << attr if self[attr].present? and self[attr] != from[attr]
+    end
+    @changed_attrs
+  end
+
+  protected
+
+  def distribute_to_consumers
+    # shopping_cart creates products without a profile...
+    return unless self.profile
+
+    self.profile.consumers.except_people.except_self.each do |consumer|
+      self.distribute_to_consumer consumer.profile
+    end
+  end
+
+end
@@ -0,0 +1,5 @@		@@ -0,0 +1,5 @@
	1	+if ENV['CI']
	2	+ system 'script/noosfero-plugins -q enable products'
	3	+ exit $?.exitstatus
	4	+end
	5	+
	@@ -1,34 +0,0 @@	@@ -1,34 +0,0 @@
1	-require_dependency 'product'
2	-
3	-class Product
4	-
5	- metadata_spec namespace: :og, tags: {
6	- type: proc{ \|p, plugin\| plugin.context.params[:og_type] \|\| MetadataPlugin.og_types[:product] \|\| :product },
7	- url: proc do \|p, plugin\|
8	- url = p.url.merge! profile: p.profile.identifier, og_type: plugin.context.params[:og_type]
9	- plugin.og_url_for url
10	- end,
11	- gr_hascurrencyvalue: proc{ \|p, plugin\| p.price.to_f },
12	- gr_hascurrency: proc{ \|p, plugin\| p.environment.currency_unit },
13	- title: proc{ \|p, plugin\| "#{p.name} - #{p.profile.name}" if p },
14	- description: proc{ \|p, plugin\| ActionView::Base.full_sanitizer.sanitize p.description },
15	-
16	- image: proc do \|p, plugin\|
17	- img = "#{p.environment.top_url}#{p.image.public_filename}".html_safe if p.image
18	- img = "#{p.environment.top_url}#{p.profile.image.public_filename}".html_safe if img.blank? and p.profile.image
19	- img \|\|= MetadataPlugin.config[:open_graph][:environment_logo] rescue nil if img.blank?
20	- img
21	- end,
22	- 'image:type' => proc{ \|p, plugin\| p.image.content_type if p.image },
23	- 'image:height' => proc{ \|p, plugin\| p.image.height if p.image },
24	- 'image:width' => proc{ \|p, plugin\| p.image.width if p.image },
25	-
26	- see_also: [],
27	- site_name: proc{ \|p, plugin\| plugin.og_url_for p.profile.url },
28	- updated_time: proc{ \|p, plugin\| p.updated_at.iso8601 if p.updated_at },
29	-
30	- 'locale:locale' => proc{ \|p, plugin\| p.environment.default_language },
31	- 'locale:alternate' => proc{ \|p, plugin\| p.environment.languages - [p.environment.default_language] if p.environment.languages },
32	- }
33	-
34	-end
@@ -0,0 +1,38 @@		@@ -0,0 +1,38 @@
	1	+if defined? ProductsPlugin
	2	+ require_dependency 'products_plugin/product'
	3	+
	4	+ module ProductsPlugin
	5	+ class Product
	6	+
	7	+ metadata_spec namespace: :og, tags: {
	8	+ type: proc{ \|p, plugin\| plugin.context.params[:og_type] \|\| MetadataPlugin.og_types[:product] \|\| :product },
	9	+ url: proc do \|p, plugin\|
	10	+ url = p.url.merge! profile: p.profile.identifier, og_type: plugin.context.params[:og_type]
	11	+ plugin.og_url_for url
	12	+ end,
	13	+ gr_hascurrencyvalue: proc{ \|p, plugin\| p.price.to_f },
	14	+ gr_hascurrency: proc{ \|p, plugin\| p.environment.currency_unit },
	15	+ title: proc{ \|p, plugin\| "#{p.name} - #{p.profile.name}" if p },
	16	+ description: proc{ \|p, plugin\| ActionView::Base.full_sanitizer.sanitize p.description },
	17	+
	18	+ image: proc do \|p, plugin\|
	19	+ img = "#{p.environment.top_url}#{p.image.public_filename}".html_safe if p.image
	20	+ img = "#{p.environment.top_url}#{p.profile.image.public_filename}".html_safe if img.blank? and p.profile.image
	21	+ img \|\|= MetadataPlugin.config[:open_graph][:environment_logo] rescue nil if img.blank?
	22	+ img
	23	+ end,
	24	+ 'image:type' => proc{ \|p, plugin\| p.image.content_type if p.image },
	25	+ 'image:height' => proc{ \|p, plugin\| p.image.height if p.image },
	26	+ 'image:width' => proc{ \|p, plugin\| p.image.width if p.image },
	27	+
	28	+ see_also: [],
	29	+ site_name: proc{ \|p, plugin\| plugin.og_url_for p.profile.url },
	30	+ updated_time: proc{ \|p, plugin\| p.updated_at.iso8601 if p.updated_at },
	31	+
	32	+ 'locale:locale' => proc{ \|p, plugin\| p.environment.default_language },
	33	+ 'locale:alternate' => proc{ \|p, plugin\| p.environment.languages - [p.environment.default_language] if p.environment.languages },
	34	+ }
	35	+
	36	+ end
	37	+ end
	38	+end
	@@ -1,24 +0,0 @@	@@ -1,24 +0,0 @@
1	-require 'test_helper'
2	-require 'home_controller'
3	-
4	-class ManageProductsControllerTest < ActionController::TestCase
5	-
6	- def setup
7	- @controller = ManageProductsController.new
8	- @request = ActionController::TestRequest.new
9	- @response = ActionController::TestResponse.new
10	- @enterprise = fast_create(Enterprise, name: 'test', identifier: 'test_ent')
11	- @user = create_user_with_permission('test_user', 'manage_products', @enterprise)
12	- @environment = @enterprise.environment
13	- @environment.enable('products_for_enterprises')
14	- login_as :test_user
15	-
16	- @environment.enabled_plugins += ['MetadataPlugin']
17	- @environment.save!
18	- end
19	-
20	- should "not crash on new products" do
21	- get :new, profile: @enterprise.identifier
22	- end
23	-
24	-end
@@ -0,0 +1,24 @@		@@ -0,0 +1,24 @@
	1	+require 'test_helper'
	2	+
	3	+if defined? ProductsPlugin
	4	+ module ProductsPlugin
	5	+ class ManageProductsControllerTest < ActionController::TestCase
	6	+
	7	+ def setup
	8	+ @controller = PageController.new
	9	+ @request = ActionController::TestRequest.new
	10	+ @response = ActionController::TestResponse.new
	11	+ @enterprise = fast_create(Enterprise, name: 'test', identifier: 'test_ent')
	12	+ @user = create_user_with_permission('test_user', 'manage_products', @enterprise)
	13	+ login_as :test_user
	14	+
	15	+ @enterprise.environment.enable_plugin 'MetadataPlugin'
	16	+ end
	17	+
	18	+ should "not crash on new products" do
	19	+ get :new, profile: @enterprise.identifier
	20	+ end
	21	+
	22	+ end
	23	+ end
	24	+end
1	-system "script/noosfero-plugins -q enable delivery"	1	+system 'script/noosfero-plugins -q enable products delivery'
2	exit $?.exitstatus	2	exit $?.exitstatus
3		3
	@@ -1,16 +0,0 @@	@@ -1,16 +0,0 @@
1	-require_dependency 'product'
2	-
3	-class Product
4	-
5	- has_many :items, class_name: 'OrdersPlugin::Item', foreign_key: :product_id, dependent: :destroy
6	-
7	- has_many :orders, through: :items
8	- has_many :sales, through: :items
9	- has_many :purchases, through: :items
10	-
11	- attr_accessor :quantity_ordered
12	-
13	- extend CurrencyHelper::ClassMethods
14	- instance_exec &OrdersPlugin::Item::DefineTotals
15	-
16	-end
@@ -0,0 +1,18 @@		@@ -0,0 +1,18 @@
	1	+require_dependency 'products_plugin/product'
	2	+
	3	+module ProductsPlugin
	4	+ class Product
	5	+
	6	+ has_many :items, class_name: 'OrdersPlugin::Item', foreign_key: :product_id, dependent: :destroy
	7	+
	8	+ has_many :orders, through: :items
	9	+ has_many :sales, through: :items
	10	+ has_many :purchases, through: :items
	11	+
	12	+ attr_accessor :quantity_ordered
	13	+
	14	+ extend CurrencyHelper::ClassMethods
	15	+ instance_exec &OrdersPlugin::Item::DefineTotals
	16	+
	17	+ end
	18	+end
	@@ -39,7 +39,7 @@ class OrdersPlugin::Item < ApplicationRecord		@@ -39,7 +39,7 @@ class OrdersPlugin::Item < ApplicationRecord
39	belongs_to :sale, class_name: '::OrdersPlugin::Sale', foreign_key: :order_id, touch: true	39	belongs_to :sale, class_name: '::OrdersPlugin::Sale', foreign_key: :order_id, touch: true
40	belongs_to :purchase, class_name: '::OrdersPlugin::Purchase', foreign_key: :order_id, touch: true	40	belongs_to :purchase, class_name: '::OrdersPlugin::Purchase', foreign_key: :order_id, touch: true
41		41
42	- belongs_to :product	42	+ belongs_to :product, class_name: '::ProductsPlugin::Product'
43	has_one :supplier, through: :product	43	has_one :supplier, through: :product
44		44
45	has_one :profile, through: :order	45	has_one :profile, through: :order
	@@ -1,12 +0,0 @@	@@ -1,12 +0,0 @@
1	-require_dependency 'product'
2	-
3	-# based on orders/lib/ext/product.rb
4	-class Product
5	-
6	- has_many :orders_cycles_items, class_name: 'OrdersCyclePlugin::Item', foreign_key: :product_id
7	-
8	- has_many :orders_cycles_orders, through: :orders_cycles_items, source: :order
9	- has_many :orders_cycles_sales, through: :orders_cycles_items, source: :sale
10	- has_many :orders_cycles_purchases, through: :orders_cycles_items, source: :purchase
11	-
12	-end