diff --git a/app/assets/javascripts/admin/bulk_product_update.js.coffee b/app/assets/javascripts/admin/bulk_product_update.js.coffee index 5ec6eb52f8..bc7d01066b 100644 --- a/app/assets/javascripts/admin/bulk_product_update.js.coffee +++ b/app/assets/javascripts/admin/bulk_product_update.js.coffee @@ -4,13 +4,15 @@ angular.module("ofn.admin").controller "AdminProductEditCtrl", ($scope, $timeout $scope.StatusMessage = StatusMessage $scope.columns = - producer: {name: "Producer", visible: true} - name: {name: "Name", visible: true} - unit: {name: "Unit", visible: true} - price: {name: "Price", visible: true} - on_hand: {name: "On Hand", visible: true} - category: {name: "Category", visible: false} - available_on: {name: "Available On", visible: false} + producer: {name: "Producer", visible: true} + sku: {name: "SKU", visible: false} + name: {name: "Name", visible: true} + unit: {name: "Unit", visible: true} + price: {name: "Price", visible: true} + on_hand: {name: "On Hand", visible: true} + category: {name: "Category", visible: false} + inherits_properties: {name: "Inherits Properties?", visible: false} + available_on: {name: "Available On", visible: false} $scope.variant_unit_options = VariantUnitManager.variantUnitOptions() @@ -288,6 +290,9 @@ filterSubmitProducts = (productsToFilter) -> filteredMaster ?= { id: product.master.id } filteredMaster.display_as = product.master.display_as + if product.hasOwnProperty("sku") + filteredProduct.sku = product.sku + hasUpdatableProperty = true if product.hasOwnProperty("name") filteredProduct.name = product.name hasUpdatableProperty = true @@ -310,6 +315,9 @@ filterSubmitProducts = (productsToFilter) -> if product.hasOwnProperty("category_id") filteredProduct.primary_taxon_id = product.category_id hasUpdatableProperty = true + if product.hasOwnProperty("inherits_properties") + filteredProduct.inherits_properties = product.inherits_properties + hasUpdatableProperty = true if product.hasOwnProperty("available_on") filteredProduct.available_on = product.available_on hasUpdatableProperty = true diff --git a/app/assets/javascripts/darkswarm/directives/active_selector.js.coffee b/app/assets/javascripts/darkswarm/directives/active_selector.js.coffee index 563b5a1716..46be338610 100644 --- a/app/assets/javascripts/darkswarm/directives/active_selector.js.coffee +++ b/app/assets/javascripts/darkswarm/directives/active_selector.js.coffee @@ -6,10 +6,10 @@ Darkswarm.directive "activeSelector", -> replace: true templateUrl: 'active_selector.html' link: (scope, elem, attr)-> - scope.selector.emit = scope.emit - elem.bind "click", -> - scope.$apply -> - scope.selector.active = !scope.selector.active - # This function is a convention, e.g. a callback on the scope applied when active changes - scope.emit() if scope.emit - + unless scope.readOnly && scope.readOnly() + scope.selector.emit = scope.emit + elem.bind "click", -> + scope.$apply -> + scope.selector.active = !scope.selector.active + # This function is a convention, e.g. a callback on the scope applied when active changes + scope.emit() if scope.emit diff --git a/app/assets/javascripts/darkswarm/directives/filter_selector.js.coffee b/app/assets/javascripts/darkswarm/directives/filter_selector.js.coffee index b964219925..95aee1b413 100644 --- a/app/assets/javascripts/darkswarm/directives/filter_selector.js.coffee +++ b/app/assets/javascripts/darkswarm/directives/filter_selector.js.coffee @@ -5,7 +5,7 @@ Darkswarm.directive "filterSelector", (FilterSelectorsService)-> replace: true scope: objects: "&" - activeSelectors: "=" + activeSelectors: "=?" allSelectors: "=?" # Optional templateUrl: "filter_selector.html" @@ -13,6 +13,9 @@ Darkswarm.directive "filterSelector", (FilterSelectorsService)-> selectors_by_id = {} selectors = null # To get scoping/closure right + scope.readOnly = -> + !attr.activeSelectors? + scope.emit = -> scope.activeSelectors = selectors.filter (selector)-> selector.active @@ -23,10 +26,10 @@ Darkswarm.directive "filterSelector", (FilterSelectorsService)-> # when data has been loaded, in order to pass # selectors up scope.$on 'loadFilterSelectors', -> - scope.allSelectors = scope.selectors() + scope.allSelectors = scope.selectors() if attr.allSelectors? scope.$watchCollection "selectors()", (newValue, oldValue) -> - scope.allSelectors = scope.selectors() + scope.allSelectors = scope.selectors() if attr.allSelectors? # Build a list of selectors scope.selectors = -> diff --git a/app/assets/javascripts/darkswarm/directives/single_line_selectors.coffee b/app/assets/javascripts/darkswarm/directives/single_line_selectors.coffee index 97e03cb689..93896a41f9 100644 --- a/app/assets/javascripts/darkswarm/directives/single_line_selectors.coffee +++ b/app/assets/javascripts/darkswarm/directives/single_line_selectors.coffee @@ -4,6 +4,7 @@ Darkswarm.directive 'singleLineSelectors', ($timeout, $filter) -> scope: objects: "&" activeSelectors: "=" + selectorName: "@activeSelectors" link: (scope,element,attrs) -> scope.fitting = false diff --git a/app/assets/javascripts/darkswarm/filters/properties_with_values_of.js.coffee b/app/assets/javascripts/darkswarm/filters/properties_with_values_of.js.coffee new file mode 100644 index 0000000000..7cdf962160 --- /dev/null +++ b/app/assets/javascripts/darkswarm/filters/properties_with_values_of.js.coffee @@ -0,0 +1,7 @@ +Darkswarm.filter 'propertiesWithValuesOf', -> + (objects)-> + propertiesWithValues = {} + for object in objects + for property in object.properties_with_values + propertiesWithValues[property.id] = property + propertiesWithValues diff --git a/app/assets/javascripts/darkswarm/services/products.js.coffee b/app/assets/javascripts/darkswarm/services/products.js.coffee index db8bbbb7ee..a07ae1f466 100644 --- a/app/assets/javascripts/darkswarm/services/products.js.coffee +++ b/app/assets/javascripts/darkswarm/services/products.js.coffee @@ -21,6 +21,8 @@ Darkswarm.factory 'Products', ($resource, Enterprises, Dereferencer, Taxons, Pro for product in @products product.supplier = Enterprises.enterprises_by_id[product.supplier.id] Dereferencer.dereference product.taxons, Taxons.taxons_by_id + + product.properties = angular.copy(product.properties_with_values) Dereferencer.dereference product.properties, Properties.properties_by_id # May return different objects! If the variant has already been registered diff --git a/app/assets/javascripts/templates/active_selector.html.haml b/app/assets/javascripts/templates/active_selector.html.haml index 7c9412760e..87fb066636 100644 --- a/app/assets/javascripts/templates/active_selector.html.haml +++ b/app/assets/javascripts/templates/active_selector.html.haml @@ -1,2 +1,3 @@ %li{ ng: { class: "{active: selector.active}" } } - %a{ ng: { transclude: true, class: "{active: selector.active}" } } + %a{ "tooltip" => "{{selector.object.value}}", "tooltip-placement" => "bottom", + ng: { transclude: true, class: "{active: selector.active, 'has-tip': selector.object.value}" } } diff --git a/app/assets/javascripts/templates/filter_selector.html.haml b/app/assets/javascripts/templates/filter_selector.html.haml index cddcd2f523..ecd4fb7d6d 100644 --- a/app/assets/javascripts/templates/filter_selector.html.haml +++ b/app/assets/javascripts/templates/filter_selector.html.haml @@ -1,3 +1,4 @@ -%active-selector{ ng: { repeat: "selector in selectors()", show: "ifDefined(selector.fits, true)" } } - %render-svg{path: "{{selector.object.icon}}", ng: { if: "selector.object.icon"} } - %span {{ selector.object.name }} +%div{ style: "display: inline-block" } + %active-selector{ ng: { repeat: "selector in selectors()", show: "ifDefined(selector.fits, true)" } } + %render-svg{path: "{{selector.object.icon}}", ng: { if: "selector.object.icon"} } + %span {{ selector.object.name }} diff --git a/app/assets/javascripts/templates/product_modal.html.haml b/app/assets/javascripts/templates/product_modal.html.haml index 1b59838e53..bf66bd4aea 100644 --- a/app/assets/javascripts/templates/product_modal.html.haml +++ b/app/assets/javascripts/templates/product_modal.html.haml @@ -6,24 +6,15 @@ %em from %span.avenir {{ enterprise.name }} + %br - -# TODO: Add product taxons and properties here - -# / TODO: Rob - add in taxons and properties and property pop-overs - -# / %render-svg{path: "{{product.primary_taxon.icon}}"} - -# .pad-top - -# %span.filter-shopfront.taxon-selectors - -# %ul.inline-block - -# %li - -# %a.button.tiny.disabled Grains - -# %li - -# %a.button.tiny.disabled Dairy - -# - -# %span.filter-shopfront.property-selectors.pad-top - -# %ul.inline-block - -# %li - -# %a.button.tiny Organic certified - -# / TODO: Rob - need popover, use will's directive or this? http://pineconellc.github.io/angular-foundation/ + .filter-shopfront.taxon-selectors.inline-block + %ul + %filter-selector{ objects: "[product] | taxonsOf" } + .filter-shopfront.property-selectors.inline-block + %ul + %filter-selector{ objects: "[product] | propertiesWithValuesOf" } %div{"ng-if" => "product.description"} %hr diff --git a/app/assets/javascripts/templates/single_line_selectors.html.haml b/app/assets/javascripts/templates/single_line_selectors.html.haml index e1e96e904a..d54ce57bbb 100644 --- a/app/assets/javascripts/templates/single_line_selectors.html.haml +++ b/app/assets/javascripts/templates/single_line_selectors.html.haml @@ -3,12 +3,12 @@ %filter-selector{objects: "objects()", "active-selectors" => "activeSelectors", "all-selectors" => "allSelectors" } %li.more{ ng: { show: "overFlowSelectors().length > 0 || fitting" } } - %a.dropdown{ data: { dropdown: "show-more" }, ng: { class: "{active: selectedOverFlowSelectors().length > 0}" } } + %a.dropdown{ data: { dropdown: "{{ 'show-more-' + selectorName }}" }, ng: { class: "{active: selectedOverFlowSelectors().length > 0}" } } %span + {{ overFlowSelectors().length }} more %i.ofn-i_052-point-down - .f-dropdown.text-right.content#show-more + .f-dropdown.text-right.content{ ng: { attr: { id: "{{ 'show-more-' + selectorName }}" } } } %ul %active-selector{ ng: { repeat: "selector in overFlowSelectors()", hide: "selector.fits" } } - %render-svg{path: "{{selector.object.icon}}"} + %render-svg{path: "{{selector.object.icon}}", ng: { if: "selector.object.icon"}} %span {{ selector.object.name }} diff --git a/app/assets/stylesheets/darkswarm/_shop-filters.css.sass b/app/assets/stylesheets/darkswarm/_shop-filters.css.sass index 0d59f260f4..3b31481ee1 100644 --- a/app/assets/stylesheets/darkswarm/_shop-filters.css.sass +++ b/app/assets/stylesheets/darkswarm/_shop-filters.css.sass @@ -4,7 +4,7 @@ @import animations @mixin filter-selector($base-clr, $border-clr, $hover-clr) - ul.inline-block + &.inline-block, ul.inline-block display: inline-block li @@ -14,7 +14,7 @@ margin: 0 0 0.25rem 0.25rem &:hover, &:focus background: transparent - &.active + &.active box-shadow: none a, a.button @@ -48,10 +48,10 @@ svg path fill: $hover-clr - + &.disabled opacity: 0.6 - + &:hover, &:focus border-color: $border-clr color: $base-clr @@ -118,5 +118,3 @@ // Shopfront properties &.property-selectors @include filter-selector(#666, #ccc, #777) - - diff --git a/app/controllers/spree/api/products_controller_decorator.rb b/app/controllers/spree/api/products_controller_decorator.rb index 0186c6e8df..6e9799c86d 100644 --- a/app/controllers/spree/api/products_controller_decorator.rb +++ b/app/controllers/spree/api/products_controller_decorator.rb @@ -65,7 +65,7 @@ Spree::Api::ProductsController.class_eval do end def render_paged_products(products) - render text: { products: ActiveModel::ArraySerializer.new(products, each_serializer: Spree::Api::ProductSerializer), pages: products.num_pages }.to_json + render text: { products: ActiveModel::ArraySerializer.new(products, each_serializer: Api::Admin::ProductSerializer), pages: products.num_pages }.to_json end end diff --git a/app/helpers/admin/injection_helper.rb b/app/helpers/admin/injection_helper.rb index 14c4f9c20d..36ddccdb9a 100644 --- a/app/helpers/admin/injection_helper.rb +++ b/app/helpers/admin/injection_helper.rb @@ -47,7 +47,7 @@ module Admin end def admin_inject_products - admin_inject_json_ams_array "ofn.admin", "products", @products, Spree::Api::ProductSerializer + admin_inject_json_ams_array "ofn.admin", "products", @products, Api::Admin::ProductSerializer end def admin_inject_taxons diff --git a/app/models/spree/product_decorator.rb b/app/models/spree/product_decorator.rb index f7834dc3ee..e9b99c9b7a 100644 --- a/app/models/spree/product_decorator.rb +++ b/app/models/spree/product_decorator.rb @@ -18,8 +18,10 @@ Spree::Product.class_eval do delegate_belongs_to :master, :unit_value, :unit_description delegate :images_attributes=, :display_as=, to: :master - attr_accessible :supplier_id, :primary_taxon_id, :distributor_ids, :product_distributions_attributes, :group_buy, :group_buy_unit_size - attr_accessible :variant_unit, :variant_unit_scale, :variant_unit_name, :unit_value, :unit_description, :notes, :images_attributes, :display_as + attr_accessible :supplier_id, :primary_taxon_id, :distributor_ids, :product_distributions_attributes + attr_accessible :group_buy, :group_buy_unit_size, :unit_description, :notes, :images_attributes, :display_as + attr_accessible :variant_unit, :variant_unit_scale, :variant_unit_name, :unit_value + attr_accessible :inherits_properties, :sku # validates_presence_of :variants, unless: :new_record?, message: "Product must have at least one variant" validates_presence_of :supplier @@ -107,19 +109,21 @@ Spree::Product.class_eval do # -- Methods - def properties_h + def properties_including_inherited # Product properties override producer properties - ps = supplier.producer_properties.inject(product_properties) do |properties, property| - if properties.find { |p| p.property.presentation == property.property.presentation } - properties - else - properties + [property] + ps = product_properties.all + + if inherits_properties + supplier.producer_properties.each do |producer_property| + unless ps.find { |product_property| product_property.property.presentation == producer_property.property.presentation } + ps << producer_property + end end end ps. sort_by { |pp| pp.position }. - map { |pp| {presentation: pp.property.presentation, value: pp.value} } + map { |pp| {id: pp.property.id, name: pp.property.presentation, value: pp.value} } end def in_distributor?(distributor) diff --git a/app/overrides/spree/admin/product_properties/index/add_producer_properties_warning_and_table.html.haml.deface b/app/overrides/spree/admin/product_properties/index/add_producer_properties_warning_and_table.html.haml.deface new file mode 100644 index 0000000000..8d2e598c43 --- /dev/null +++ b/app/overrides/spree/admin/product_properties/index/add_producer_properties_warning_and_table.html.haml.deface @@ -0,0 +1,26 @@ +/ insert_after 'table.index.sortable' + +=f.check_box :inherits_properties +=f.label :inherits_properties, "Inherit properties from #{@product.supplier.name}? (unless overridden above)" +%br +%br + +#inherited_properties + %table.index + %thead + %tr{"data-hook" => "producer_properties_header"} + %th= t(:inherited_property) + %th= t(:value) + %th.actions + %tbody#producer_properties{"data-hook" => ""} + - @product.supplier.producer_properties.each do |producer_property| + %tr + %td= producer_property.property.presentation + %td= producer_property.value + %td.actions + +:coffee + $(document).ready -> + $("#inherited_properties").toggle $("input#product_inherits_properties").is(':checked') + $("input#product_inherits_properties").change -> + $("#inherited_properties").toggle $(this).is(':checked') diff --git a/app/serializers/spree/api/product_serializer.rb b/app/serializers/api/admin/product_serializer.rb similarity index 63% rename from app/serializers/spree/api/product_serializer.rb rename to app/serializers/api/admin/product_serializer.rb index 8c1a331936..b1f233c044 100644 --- a/app/serializers/spree/api/product_serializer.rb +++ b/app/serializers/api/admin/product_serializer.rb @@ -1,12 +1,12 @@ -class Spree::Api::ProductSerializer < ActiveModel::Serializer - attributes :id, :name, :variant_unit, :variant_unit_scale, :variant_unit_name, :on_demand +class Api::Admin::ProductSerializer < ActiveModel::Serializer + attributes :id, :name, :sku, :variant_unit, :variant_unit_scale, :variant_unit_name, :on_demand, :inherits_properties attributes :on_hand, :price, :available_on, :permalink_live has_one :supplier, key: :producer_id, embed: :id has_one :primary_taxon, key: :category_id, embed: :id - has_many :variants, key: :variants, serializer: Spree::Api::VariantSerializer # embed: ids - has_one :master, serializer: Spree::Api::VariantSerializer + has_many :variants, key: :variants, serializer: Api::Admin::VariantSerializer # embed: ids + has_one :master, serializer: Api::Admin::VariantSerializer def on_hand object.on_hand.nil? ? 0 : object.on_hand.to_f.finite? ? object.on_hand : "On demand" diff --git a/app/serializers/spree/api/variant_serializer.rb b/app/serializers/api/admin/variant_serializer.rb similarity index 88% rename from app/serializers/spree/api/variant_serializer.rb rename to app/serializers/api/admin/variant_serializer.rb index a2342de6e4..26ad2d7f37 100644 --- a/app/serializers/spree/api/variant_serializer.rb +++ b/app/serializers/api/admin/variant_serializer.rb @@ -1,4 +1,4 @@ -class Spree::Api::VariantSerializer < ActiveModel::Serializer +class Api::Admin::VariantSerializer < ActiveModel::Serializer attributes :id, :options_text, :unit_value, :unit_description, :unit_to_display, :on_demand, :display_as, :display_name, :name_to_display attributes :on_hand, :price diff --git a/app/serializers/api/product_serializer.rb b/app/serializers/api/product_serializer.rb index 78f7593e4f..0de794796b 100644 --- a/app/serializers/api/product_serializer.rb +++ b/app/serializers/api/product_serializer.rb @@ -31,17 +31,20 @@ class Api::CachedProductSerializer < ActiveModel::Serializer #delegate :cache_key, to: :object attributes :id, :name, :permalink, :count_on_hand, :on_demand, :group_buy, - :notes, :description + :notes, :description, :properties_with_values has_many :variants, serializer: Api::VariantSerializer has_many :taxons, serializer: Api::IdSerializer - has_many :properties, serializer: Api::IdSerializer has_many :images, serializer: Api::ImageSerializer has_one :supplier, serializer: Api::IdSerializer has_one :primary_taxon, serializer: Api::TaxonSerializer has_one :master, serializer: Api::VariantSerializer + def properties_with_values + object.properties_including_inherited + end + def variants # We use the in_stock? method here instead of the in_stock scope because we need to # look up the stock as overridden by VariantOverrides, and the scope method is not affected diff --git a/app/views/home/_filters.html.haml b/app/views/home/_filters.html.haml index 7fcda4bbf4..6b339c13c9 100644 --- a/app/views/home/_filters.html.haml +++ b/app/views/home/_filters.html.haml @@ -1,21 +1,22 @@ .row - = render partial: 'shared/components/filter_controls' + -# = render partial: 'shared/components/filter_controls' + .small-12.medium-6.columns   = render partial: 'shared/components/show_profiles' -.row.animate-show{"ng-show" => "filtersActive"} - .small-12.columns - .row.filter-box - .small-12.large-9.columns - %h5.tdhead - .light Filter by - Type - %ul.small-block-grid-2.medium-block-grid-4.large-block-grid-5 - %filter-selector{objects: "Enterprises.hubs | searchEnterprises:query | taxonsOf", "active-selectors" => "activeTaxons"} - .small-12.large-3.columns - %h5.tdhead - .light Filter by - Delivery - %ul.small-block-grid-2.medium-block-grid-4.large-block-grid-2 - %shipping-type-selector{results: "shippingTypes"} - -= render partial: 'shared/components/filter_box' +-# .row.animate-show{"ng-show" => "filtersActive"} +-# .small-12.columns +-# .row.filter-box +-# .small-12.large-9.columns +-# %h5.tdhead +-# .light Filter by +-# Type +-# %ul.small-block-grid-2.medium-block-grid-4.large-block-grid-5 +-# %filter-selector{objects: "Enterprises.hubs | searchEnterprises:query | taxonsOf", "active-selectors" => "activeTaxons"} +-# .small-12.large-3.columns +-# %h5.tdhead +-# .light Filter by +-# Delivery +-# %ul.small-block-grid-2.medium-block-grid-4.large-block-grid-2 +-# %shipping-type-selector{results: "shippingTypes"} +-# +-# = render partial: 'shared/components/filter_box' diff --git a/app/views/producers/_filters.html.haml b/app/views/producers/_filters.html.haml index b548bd90c4..ec23c05b92 100644 --- a/app/views/producers/_filters.html.haml +++ b/app/views/producers/_filters.html.haml @@ -1,15 +1,15 @@ -.row - = render partial: 'shared/components/filter_controls' - .small-12.medium-6.columns.text-right -   - -.row.animate-show{"ng-show" => "filtersActive"} - .small-12.columns - .row.filter-box - .small-12.columns - %h5.tdhead - .light Filter by - Type - %ul.small-block-grid-2.medium-block-grid-4.large-block-grid-6 - %filter-selector{objects: "Enterprises.producers | searchEnterprises:query | taxonsOf", "active-selectors" => "activeTaxons"} - = render partial: 'shared/components/filter_box' +-# .row +-# = render partial: 'shared/components/filter_controls' +-# .small-12.medium-6.columns.text-right +-#   +-# +-# .row.animate-show{"ng-show" => "filtersActive"} +-# .small-12.columns +-# .row.filter-box +-# .small-12.columns +-# %h5.tdhead +-# .light Filter by +-# Type +-# %ul.small-block-grid-2.medium-block-grid-4.large-block-grid-6 +-# %filter-selector{objects: "Enterprises.producers | searchEnterprises:query | taxonsOf", "active-selectors" => "activeTaxons"} +-# = render partial: 'shared/components/filter_box' diff --git a/app/views/spree/admin/products/bulk_edit/_products_head.html.haml b/app/views/spree/admin/products/bulk_edit/_products_head.html.haml index d6be2a88e0..2e37da2bc8 100644 --- a/app/views/spree/admin/products/bulk_edit/_products_head.html.haml +++ b/app/views/spree/admin/products/bulk_edit/_products_head.html.haml @@ -1,12 +1,14 @@ %colgroup %col.actions %col.producer{ ng: { show: 'columns.producer.visible' } } + %col.sku{ ng: { show: 'columns.sku.visible' } } %col.name{ ng: { show: 'columns.name.visible' } } %col.unit{ ng: { show: 'columns.unit.visible' } } %col.display_as{ ng: { show: 'columns.unit.visible' } } %col.price{ ng: { show: 'columns.price.visible'} } %col.on_hand{ ng: { show: 'columns.on_hand.visible' } } %col.category{ ng: { show: 'columns.category.visible' } } + %col.inherits_properties{ ng: { show: 'columns.inherits_properties.visible' } } %col.available_on{ ng: { show: 'columns.available_on.visible' } } %col.actions %col.actions @@ -16,12 +18,14 @@ %tr %th.left-actions %th.producer{ 'ng-show' => 'columns.producer.visible' } Producer + %th.sku{ 'ng-show' => 'columns.sku.visible' } SKU %th.name{ 'ng-show' => 'columns.name.visible' } Name %th.unit{ 'ng-show' => 'columns.unit.visible' } Unit / Value %th.display_as{ 'ng-show' => 'columns.unit.visible' } Display As %th.price{ 'ng-show' => 'columns.price.visible' } Price %th.on_hand{ 'ng-show' => 'columns.on_hand.visible' } On Hand %th.category{ 'ng-show' => 'columns.category.visible' } Category + %th.inherits_properties{ 'ng-show' => 'columns.inherits_properties.visible' } Inherits Properties? %th.available_on{ 'ng-show' => 'columns.available_on.visible' } Av. On %th.actions %th.actions diff --git a/app/views/spree/admin/products/bulk_edit/_products_product.html.haml b/app/views/spree/admin/products/bulk_edit/_products_product.html.haml index bf527ba507..0eb9ff5f58 100644 --- a/app/views/spree/admin/products/bulk_edit/_products_product.html.haml +++ b/app/views/spree/admin/products/bulk_edit/_products_product.html.haml @@ -4,6 +4,8 @@ %a{ :class => "add-variant icon-plus-sign", 'ng-click' => "addVariant(product)", 'ng-show' => "!hasVariants(product) && hasUnit(product)" } %td.producer{ 'ng-show' => 'columns.producer.visible' } %select.select2.fullwidth{ 'ng-model' => 'product.producer_id', :name => 'producer_id', 'ofn-track-product' => 'producer_id', 'ng-options' => 'producer.id as producer.name for producer in producers' } + %td.sku{ 'ng-show' => 'columns.sku.visible' } + %input{ 'ng-model' => "product.sku", :name => 'product_sku', 'ofn-track-product' => 'sku', :type => 'text' } %td.name{ 'ng-show' => 'columns.name.visible' } %input{ 'ng-model' => "product.name", :name => 'product_name', 'ofn-track-product' => 'name', :type => 'text' } %td.unit{ 'ng-show' => 'columns.unit.visible' } @@ -20,6 +22,8 @@ %input.field{ 'ng-model' => 'product.on_hand', :name => 'on_hand', 'ofn-track-product' => 'on_hand', 'ng-hide' => 'hasVariants(product) || product.on_demand', :type => 'number' } %td.category{ 'ng-if' => 'columns.category.visible' } %input.fullwidth{ :type => 'text', id: "p{{product.id}}_category_id", 'ng-model' => 'product.category_id', 'ofn-taxon-autocomplete' => '', 'ofn-track-product' => 'category_id', 'multiple-selection' => 'false', placeholder: 'Category' } + %td.inherits_properties{ 'ng-show' => 'columns.inherits_properties.visible' } + %input{ 'ng-model' => 'product.inherits_properties', :name => 'inherits_properties', 'ofn-track-product' => 'inherits_properties', type: "checkbox" } %td.available_on{ 'ng-show' => 'columns.available_on.visible' } %input{ 'ng-model' => 'product.available_on', :name => 'available_on', 'ofn-track-product' => 'available_on', 'datetimepicker' => 'product.available_on', type: "text" } %td.actions diff --git a/app/views/spree/admin/products/bulk_edit/_products_variant.html.haml b/app/views/spree/admin/products/bulk_edit/_products_variant.html.haml index 8825cec46c..3a783329c0 100644 --- a/app/views/spree/admin/products/bulk_edit/_products_variant.html.haml +++ b/app/views/spree/admin/products/bulk_edit/_products_variant.html.haml @@ -3,6 +3,7 @@ %a{ :class => "variant-item icon-caret-right", 'ng-hide' => "$last" } %a{ :class => "add-variant icon-plus-sign", 'ng-click' => "addVariant(product)", 'ng-show' => "$last" } %td{ 'ng-show' => 'columns.producer.visible' } + %td{ 'ng-show' => 'columns.sku.visible' } %td{ 'ng-show' => 'columns.name.visible' } %input{ 'ng-model' => 'variant.display_name', :name => 'variant_display_name', 'ofn-track-variant' => 'display_name', :type => 'text', placeholder: "{{ product.name }}" } %td.unit_value{ 'ng-show' => 'columns.unit.visible' } @@ -15,6 +16,7 @@ %input.field{ 'ng-model' => 'variant.on_hand', 'ng-change' => 'updateOnHand(product)', :name => 'variant_on_hand', 'ng-hide' => 'variant.on_demand', 'ofn-track-variant' => 'on_hand', :type => 'number' } %span{ 'ng-bind' => 'variant.on_hand', :name => 'variant_on_hand', 'ng-show' => 'variant.on_demand' } %td{ 'ng-show' => 'columns.category.visible' } + %td{ 'ng-show' => 'columns.inherits_properties.visible' } %td{ 'ng-show' => 'columns.available_on.visible' } %td.actions %a{ 'ng-click' => 'editWarn(product,variant)', :class => "edit-variant icon-edit no-text", 'ng-show' => "variantSaved(variant)" } diff --git a/db/migrate/20150422014819_add_inherits_properties_to_product.rb b/db/migrate/20150422014819_add_inherits_properties_to_product.rb new file mode 100644 index 0000000000..aba8bbbe45 --- /dev/null +++ b/db/migrate/20150422014819_add_inherits_properties_to_product.rb @@ -0,0 +1,5 @@ +class AddInheritsPropertiesToProduct < ActiveRecord::Migration + def change + add_column :spree_products, :inherits_properties, :boolean, null: false, default: true + end +end diff --git a/db/migrate/20150424025907_add_default_and_not_null_to_producer_properties_position.rb b/db/migrate/20150424025907_add_default_and_not_null_to_producer_properties_position.rb new file mode 100644 index 0000000000..8c5a2d1f2c --- /dev/null +++ b/db/migrate/20150424025907_add_default_and_not_null_to_producer_properties_position.rb @@ -0,0 +1,9 @@ +class AddDefaultAndNotNullToProducerPropertiesPosition < ActiveRecord::Migration + def change + ProducerProperty.where(position: nil).each do |producer_property| + producer_property.update_attribute(:position, 0) + end + + change_column :producer_properties, :position, :integer, null: false, default: 0 + end +end diff --git a/db/schema.rb b/db/schema.rb index f30408c2ac..cb8b503583 100644 --- a/db/schema.rb +++ b/db/schema.rb @@ -11,7 +11,7 @@ # # It's strongly recommended to check this file into your version control system. -ActiveRecord::Schema.define(:version => 20150410043302) do +ActiveRecord::Schema.define(:version => 20150424025907) do create_table "adjustment_metadata", :force => true do |t| t.integer "adjustment_id" @@ -373,9 +373,9 @@ ActiveRecord::Schema.define(:version => 20150410043302) do t.string "value" t.integer "producer_id" t.integer "property_id" - t.integer "position" - t.datetime "created_at", :null => false - t.datetime "updated_at", :null => false + t.integer "position", :default => 0, :null => false + t.datetime "created_at", :null => false + t.datetime "updated_at", :null => false end add_index "producer_properties", ["position"], :name => "index_producer_properties_on_position" @@ -766,6 +766,7 @@ ActiveRecord::Schema.define(:version => 20150410043302) do t.string "variant_unit_name" t.text "notes" t.integer "primary_taxon_id", :null => false + t.boolean "inherits_properties", :default => true, :null => false end add_index "spree_products", ["available_on"], :name => "index_products_on_available_on" diff --git a/script/ci/includes.sh b/script/ci/includes.sh index 4b07dacfdc..ee59a732d7 100644 --- a/script/ci/includes.sh +++ b/script/ci/includes.sh @@ -11,3 +11,17 @@ function exit_unless_master_merged { exit 1 fi } + +function drop_and_recreate_database { + # Adapted from: http://stackoverflow.com/questions/12924466/capistrano-with-postgresql-error-database-is-being-accessed-by-other-users + psql -U openfoodweb postgres < pg_backend_pid() +AND datname='$1'; +DROP DATABASE $1; +CREATE DATABASE $1; +EOF +} diff --git a/script/ci/load_staging_baseline.sh b/script/ci/load_staging_baseline.sh index 0a59b66d3b..ac7d88569f 100755 --- a/script/ci/load_staging_baseline.sh +++ b/script/ci/load_staging_baseline.sh @@ -5,22 +5,23 @@ # current database. set -e - cd /home/openfoodweb/apps/openfoodweb/current +source ./script/ci/includes.sh -echo "Stopping unicorn..." +echo "Stopping unicorn and delayed job..." service unicorn_openfoodweb stop +RAILS_ENV=staging script/delayed_job -i 0 stop echo "Backing up current data..." mkdir -p db/backup pg_dump -h localhost -U openfoodweb openfoodweb_production |gzip > db/backup/staging-`date +%Y%m%d%H%M%S`.sql.gz echo "Loading baseline data..." -dropdb -U openfoodweb -w openfoodweb_production -createdb -U openfoodweb -w openfoodweb_production +drop_and_recreate_database "openfoodweb_production" gunzip -c db/backup/staging-baseline.sql.gz |psql -h localhost -U openfoodweb openfoodweb_production echo "Restarting unicorn..." service unicorn_openfoodweb start +# Delayed job is restarted by monit echo "Done!" diff --git a/spec/features/admin/bulk_product_update_spec.rb b/spec/features/admin/bulk_product_update_spec.rb index f5cc856da2..079279e5cf 100644 --- a/spec/features/admin/bulk_product_update_spec.rb +++ b/spec/features/admin/bulk_product_update_spec.rb @@ -231,6 +231,55 @@ feature %q{ expect(page).to have_selector "a.edit-variant", visible: true end + scenario "updating product attributes" do + s1 = FactoryGirl.create(:supplier_enterprise) + s2 = FactoryGirl.create(:supplier_enterprise) + t1 = FactoryGirl.create(:taxon) + t2 = FactoryGirl.create(:taxon) + p = FactoryGirl.create(:product, supplier: s1, available_on: Date.today, variant_unit: 'volume', variant_unit_scale: 1, primary_taxon: t2, sku: "OLD SKU") + + login_to_admin_section + + visit '/admin/products/bulk_edit' + + first("div#columns_dropdown", :text => "COLUMNS").click + first("div#columns_dropdown div.menu div.menu_item", text: "Available On").click + first("div#columns_dropdown div.menu div.menu_item", text: "Category").click + first("div#columns_dropdown div.menu div.menu_item", text: "Inherits Properties?").click + first("div#columns_dropdown div.menu div.menu_item", text: "SKU").click + + within "tr#p_#{p.id}" do + expect(page).to have_field "product_name", with: p.name + expect(page).to have_select "producer_id", selected: s1.name + expect(page).to have_field "available_on", with: p.available_on.strftime("%F %T") + expect(page).to have_select2 "p#{p.id}_category_id", selected: t2.name + expect(page).to have_select "variant_unit_with_scale", selected: "Volume (L)" + expect(page).to have_checked_field "inherits_properties" + expect(page).to have_field "product_sku", with: p.sku + + fill_in "product_name", with: "Big Bag Of Potatoes" + select s2.name, :from => 'producer_id' + fill_in "available_on", with: (3.days.ago.beginning_of_day).strftime("%F %T") + select "Weight (kg)", from: "variant_unit_with_scale" + select2_select t1.name, from: "p#{p.id}_category_id" + uncheck "inherits_properties" + fill_in "product_sku", with: "NEW SKU" + end + + click_button 'Save Changes' + expect(page.find("#status-message")).to have_content "Changes saved." + + p.reload + expect(p.name).to eq "Big Bag Of Potatoes" + expect(p.supplier).to eq s2 + expect(p.variant_unit).to eq "weight" + expect(p.variant_unit_scale).to eq 1000 # Kg + expect(p.available_on).to eq 3.days.ago.beginning_of_day + expect(p.primary_taxon).to eq t1 + expect(p.inherits_properties).to be false + expect(p.sku).to eq "NEW SKU" + end + scenario "updating a product with a variant unit of 'items'" do p = FactoryGirl.create(:product, variant_unit: 'weight', variant_unit_scale: 1000) diff --git a/spec/javascripts/unit/darkswarm/services/products_spec.js.coffee b/spec/javascripts/unit/darkswarm/services/products_spec.js.coffee index 2cb23522e2..590156260d 100644 --- a/spec/javascripts/unit/darkswarm/services/products_spec.js.coffee +++ b/spec/javascripts/unit/darkswarm/services/products_spec.js.coffee @@ -69,7 +69,7 @@ describe 'Products service', -> expect(Products.products[0].taxons[1]).toBe taxons[0] it "dereferences properties", -> - product.properties = [1] + product.properties_with_values = [1] $httpBackend.expectGET("/shop/products").respond([product]) $httpBackend.flush() expect(Products.products[0].properties[1]).toBe properties[0] diff --git a/spec/models/spree/product_spec.rb b/spec/models/spree/product_spec.rb index d6e814fed2..c30a5920ca 100644 --- a/spec/models/spree/product_spec.rb +++ b/spec/models/spree/product_spec.rb @@ -353,8 +353,9 @@ module Spree it "returns product properties as a hash" do product = create(:simple_product) product.set_property 'Organic Certified', 'NASAA 12345' + property = product.properties.last - product.properties_h.should == [{presentation: 'Organic Certified', value: 'NASAA 12345'}] + product.properties_including_inherited.should == [{id: property.id, name: "Organic Certified", value: 'NASAA 12345'}] end it "returns producer properties as a hash" do @@ -362,8 +363,9 @@ module Spree product = create(:simple_product, supplier: supplier) supplier.set_producer_property 'Organic Certified', 'NASAA 54321' + property = supplier.properties.last - product.properties_h.should == [{presentation: 'Organic Certified', value: 'NASAA 54321'}] + product.properties_including_inherited.should == [{id: property.id, name: "Organic Certified", value: 'NASAA 54321'}] end it "overrides producer properties with product properties" do @@ -372,8 +374,32 @@ module Spree product.set_property 'Organic Certified', 'NASAA 12345' supplier.set_producer_property 'Organic Certified', 'NASAA 54321' + property = product.properties.last - product.properties_h.should == [{presentation: 'Organic Certified', value: 'NASAA 12345'}] + product.properties_including_inherited.should == [{id: property.id, name: "Organic Certified", value: 'NASAA 12345'}] + end + + context "when product has an inherit_properties value set to true" do + let(:supplier) { create(:supplier_enterprise) } + let(:product) { create(:simple_product, supplier: supplier, inherits_properties: true) } + + it "inherits producer properties" do + supplier.set_producer_property 'Organic Certified', 'NASAA 54321' + property = supplier.properties.last + + product.properties_including_inherited.should == [{id: property.id, name: "Organic Certified", value: 'NASAA 54321'}] + end + end + + context "when product has an inherit_properties value set to true" do + let(:supplier) { create(:supplier_enterprise) } + let(:product) { create(:simple_product, supplier: supplier, inherits_properties: false) } + + it "does not inherit producer properties" do + supplier.set_producer_property 'Organic Certified', 'NASAA 54321' + + product.properties_including_inherited.should == [] + end end it "sorts by position" do @@ -388,10 +414,10 @@ module Spree product.product_properties.create!({property_id: pc.id, value: '3', position: 3}, {without_protection: true}) supplier.producer_properties.create!({property_id: pb.id, value: '2', position: 2}, {without_protection: true}) - product.properties_h.should == - [{presentation: 'A', value: '1'}, - {presentation: 'B', value: '2'}, - {presentation: 'C', value: '3'}] + product.properties_including_inherited.should == + [{id: pa.id, name: "A", value: '1'}, + {id: pb.id, name: "B", value: '2'}, + {id: pc.id, name: "C", value: '3'}] end end diff --git a/spec/serializers/spree/product_serializer_spec.rb b/spec/serializers/spree/product_serializer_spec.rb index bbb65c1c7e..ab6883f9c5 100644 --- a/spec/serializers/spree/product_serializer_spec.rb +++ b/spec/serializers/spree/product_serializer_spec.rb @@ -1,7 +1,7 @@ -describe Spree::Api::ProductSerializer do +describe Api::Admin::ProductSerializer do let(:product) { create(:simple_product) } it "serializes a product" do - serializer = Spree::Api::ProductSerializer.new product + serializer = Api::Admin::ProductSerializer.new product serializer.to_json.should match product.name end -end \ No newline at end of file +end diff --git a/spec/serializers/spree/variant_serializer_spec.rb b/spec/serializers/spree/variant_serializer_spec.rb index e0f26d3423..8f6b1e0bf5 100644 --- a/spec/serializers/spree/variant_serializer_spec.rb +++ b/spec/serializers/spree/variant_serializer_spec.rb @@ -1,7 +1,7 @@ -describe Spree::Api::VariantSerializer do +describe Api::Admin::VariantSerializer do let(:variant) { create(:variant) } it "serializes a variant" do - serializer = Spree::Api::VariantSerializer.new variant + serializer = Api::Admin::VariantSerializer.new variant serializer.to_json.should match variant.options_text end -end \ No newline at end of file +end