workarea-admin 3.4.28 → 3.4.29

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d5e5e70ec8db64536691e2c00009ec4cc3fbeafa558d063613272f4e68d9d3a8
-  data.tar.gz: fb41f84d15df3ec6e66e5005186916fec71e2e6d59b715a353254fb067edad67
+  metadata.gz: 9da8bc75f20143ab5701f535e4cce9e2b4a84f9330faab4cbd5fed620cdf6093
+  data.tar.gz: 6e06dfa75cc2b6292032eba0cd2ca07b14b2948b582e33eeecac0a1320a8646d
 SHA512:
-  metadata.gz: a6dac33cba255d0f435cdfe82510a1bb048bab67b01a39c426441eb85edf362dd7f68be808c9eaa84594d1bb1c8eb79efff56bfce50b1ecd19d40194f66f5e22
-  data.tar.gz: 9155090353b2d4b80e7860c4d14892dbcfd3a60ea83b1af751fd8baacb9f96085f234e1c541a301cabded3afea9d03720248e640fec99074496a10e956c06546
+  metadata.gz: 0b679c8124fd1b8d5ebf95d68665229a27ba2c16ca6299bac8cc578181b3e14fd2bbca99fc9bc1f0c744699121aa793f39ef2c6ea934ab6d73f995423de4ba65
+  data.tar.gz: 935c49a925308f4158eb9800bab1775ccd2e3383306126da65974436e4c59bd951f272c1730811c12cbd78bdc67f8bf79c33655fe6da07ebc5e285d416535162

data/app/controllers/workarea/admin/application_controller.rb

@@ -102,7 +102,7 @@ module Workarea
       end
 
       def track_index_filters
-        session[:last_index_path] = request.fullpath
+        session[:last_index_path] = request.fullpath unless request.xhr? || request.format.json?
       end
     end
   end

data/app/view_models/workarea/admin/pricing_sku_view_model.rb

@@ -30,14 +30,14 @@ module Workarea
       #
       # @return [Money]
       def min_price
-        sell_prices.first.sell
+        sell_prices.first&.sell
       end
 
       # The highest regular price set on this SKU.
       #
       # @return [Money]
       def max_price
-        sell_prices.last.sell
+        sell_prices.last&.sell
       end
 
       # Show a price range if the `min_price` and `max_price` are not
@@ -45,7 +45,7 @@ module Workarea
       #
       # @return [Boolean]
       def show_range?
-        min_price != max_price
+        min_price.present? && max_price.present? && min_price != max_price
       end
 
       # This SKU is considered "on sale" if it is marked as such, or if

data/app/views/workarea/admin/pricing_skus/_cards.html.haml

@@ -14,25 +14,25 @@
             %li
               %strong #{t('workarea.admin.fields.tax_code')}:
               = model.tax_code.presence || '(none)'
-            %li
-              %strong #{t('workarea.admin.fields.msrp')}:
-              - if model.msrp.blank?
-                = t('workarea.admin.cards.attributes.no_value')
-              - else
-                = number_to_currency model.msrp
             %li
               %strong #{t('workarea.admin.fields.on_sale')}:
               = model.on_sale?.to_s.titleize
             %li
               %strong #{t('workarea.admin.fields.discountable')}:
               = model.discountable?.to_s.titleize
+            %li
+              %strong #{t('workarea.admin.fields.msrp')}:
+              - if model.msrp.blank?
+                = t('workarea.admin.cards.attributes.no_value')
+              - else
+                = number_to_currency model.msrp
+            = append_partials('admin.pricing_sku_attributes_card', pricing_sku: model)
             %li
               %strong= t('workarea.admin.fields.updated_at')
               #{local_time_ago(model.updated_at)}
             %li
               %strong= t('workarea.admin.fields.created_at')
               #{local_time_ago(model.created_at)}
-            = append_partials('admin.pricing_sku_attributes_card', pricing_sku: model)
 
           = link_to edit_pricing_sku_path(model), class: 'card__button' do
             %span.button.button--small= t('workarea.admin.cards.attributes.button')

data/app/views/workarea/admin/pricing_skus/edit.html.haml

@@ -31,11 +31,6 @@
           = label_tag 'sku[tax_code]', t('workarea.admin.fields.tax_code'), class: 'property__name'
           = text_field_tag 'sku[tax_code]', @sku.tax_code, class: 'text-box'
 
-        .property
-          = label_tag 'sku_msrp', t('workarea.admin.fields.msrp'), class: 'property__name'
-          = currency_symbol
-          = text_field_tag 'sku[msrp]', @sku.msrp, class: 'text-box text-box--small'
-
         .grid
           .grid__cell.grid__cell--50.grid__cell--25-at-medium
             .property
@@ -47,6 +42,11 @@
               %span.property__name= t('workarea.admin.fields.discountable')
               = toggle_button_for 'sku[discountable]', @sku.discountable?, title_true: t('workarea.admin.pricing_skus.allow_discounting'), title_false: t('workarea.admin.pricing_skus.disallow_discounting')
 
+        .property
+          = label_tag 'sku_msrp', t('workarea.admin.fields.msrp'), class: 'property__name'
+          = currency_symbol
+          = text_field_tag 'sku[msrp]', @sku.msrp, class: 'text-box text-box--small'
+
         = append_partials('admin.pricing_sku_fields', sku: @sku)
 
         .grid.grid--huge

data/app/views/workarea/admin/pricing_skus/index.html.haml

@@ -46,6 +46,7 @@
                 = check_box_tag 'select_all', nil, false, id: 'select_all', class: 'checkbox__input', data: { bulk_action_select_all: '' }
                 = label_tag 'select_all', t('workarea.admin.bulk_actions.select_all'), class: 'checkbox__label'
             %th= t('workarea.admin.fields.sku')
+            = append_partials('admin.pricing_skus_additional_field_labels')
             %th.align-right= t('workarea.admin.fields.msrp')
             %th.align-right= t('workarea.admin.fields.sell_price')
             %th.align-center= t('workarea.admin.fields.on_sale')
@@ -61,6 +62,7 @@
               %td
                 = link_to result.id, pricing_sku_path(result)
                 = upcoming_changesets_icon_for(result)
+              = append_partials('admin.pricing_skus_additional_fields', result: result)
               %td.align-right= number_to_currency result.msrp
               %td.align-right= result.sell_price
               %td.align-center= t("workarea.admin.#{result.on_sale?}")

data/app/views/workarea/admin/pricing_skus/new.html.haml

@@ -26,13 +26,16 @@
           = label_tag 'sku[tax_code]', t('workarea.admin.fields.tax_code'), class: 'property__name'
           = text_field_tag 'sku[tax_code]', @sku.tax_code, class: 'text-box', placeholder: t('workarea.admin.pricing_skus.tax_code_placeholder')
 
-        .property
-          %span.property__name= t('workarea.admin.fields.on_sale')
-          = toggle_button_for 'sku[on_sale]', @sku.on_sale?, title_true: t('workarea.admin.pricing_skus.on_sale'), title_false: t('workarea.admin.pricing_skus.not_on_sale')
+        .grid
+          .grid__cell.grid__cell--50.grid__cell--25-at-medium
+            .property
+              %span.property__name= t('workarea.admin.fields.on_sale')
+              = toggle_button_for 'sku[on_sale]', @sku.on_sale?, title_true: t('workarea.admin.pricing_skus.on_sale'), title_false: t('workarea.admin.pricing_skus.not_on_sale')
 
-        .property
-          %span.property__name= t('workarea.admin.fields.discountable')
-          = toggle_button_for 'sku[discountable]', @sku.discountable?, title_true: t('workarea.admin.pricing_skus.allow_discounting'), title_false: t('workarea.admin.pricing_skus.disallow_discounting')
+          .grid__cell.grid__cell--50.grid__cell--25-at-medium
+            .property
+              %span.property__name= t('workarea.admin.fields.discountable')
+              = toggle_button_for 'sku[discountable]', @sku.discountable?, title_true: t('workarea.admin.pricing_skus.allow_discounting'), title_false: t('workarea.admin.pricing_skus.disallow_discounting')
 
         .property
           = label_tag 'sku_msrp', t('workarea.admin.fields.msrp'), class: 'property__name'

data/app/views/workarea/admin/users/index.html.haml

@@ -2,11 +2,13 @@
 
 .view
   .view__header
-    .grid.grid--middle.grid--center
+    .grid.grid--middle.grid--right
       .grid__cell.grid__cell--50
         .view__heading
           = link_to "↑ #{t('workarea.admin.users.index.dashboard_link')}", people_dashboards_path, class: 'view__dashboard-button'
           %h1= t('workarea.admin.users.index.title')
+      .grid__cell.grid__cell--25
+        = append_partials('admin.user_index_aux_navigation')
 
   .view__container
     .browsing-controls.browsing-controls--with-divider.browsing-controls--center{ class: ('browsing-controls--filters-displayed' unless @search.toggle_facets?) }

data/test/integration/workarea/admin/index_tracking_integration_test.rb

@@ -0,0 +1,21 @@
+require 'test_helper'
+
+module Workarea
+  module Admin
+    class IndexTrackingIntegrationTest < Workarea::IntegrationTest
+      include Admin::IntegrationTest
+
+      def test_track_index_filters
+        get admin.pricing_skus_path
+
+        assert_equal(admin.pricing_skus_path, session[:last_index_path])
+        assert_no_changes -> { session[:last_index_path] } do
+          get admin.catalog_products_path(format: :json)
+        end
+        assert_no_changes -> { session[:last_index_path] } do
+          get admin.catalog_products_path, xhr: true
+        end
+      end
+    end
+  end
+end

data/test/integration/workarea/admin/jump_to_integration_test.rb

@@ -234,6 +234,24 @@ module Workarea
         results = JSON.parse(response.body)['results']
         assert_equal(0, results.length)
       end
+
+      def test_limiting_results_by_type
+        Workarea.with_config do |config|
+          config.jump_to_type_limit = 2
+          config.jump_to_results_per_type = 2
+
+          5.times { |i| create_product(name: "Test Product #{i}") }
+          5.times { |i| create_category(name: "Test Category #{i}") }
+
+          get admin.jump_to_path(q: 'test')
+          results = JSON.parse(response.body)['results']
+
+          assert_equal(
+            %w(Products Products Categories Categories),
+            results.map { |r| r['type'] }
+          )
+        end
+      end
     end
   end
 end

data/test/view_models/workarea/admin/pricing_sku_view_model_test.rb

@@ -30,6 +30,10 @@ module Workarea
         @sku.update!(on_sale: false)
 
         assert_equal(3.to_m, @sku.min_price)
+
+        @sku.prices.destroy_all
+
+        assert_nil(@sku.min_price)
       end
 
       def test_max_price
@@ -38,6 +42,10 @@ module Workarea
         @sku.update!(on_sale: false)
 
         assert_equal(4.to_m, @sku.max_price)
+
+        @sku.prices.destroy_all
+
+        assert_nil(@sku.min_price)
       end
 
       def test_show_range?
@@ -46,6 +54,10 @@ module Workarea
         @sku.prices.last.destroy!
 
         refute(@sku.show_range?)
+
+        @sku.prices.destroy_all
+
+        refute(@sku.show_range?)
       end
 
       def test_on_sale?

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: workarea-admin
 version: !ruby/object:Gem::Version
-  version: 3.4.28
+  version: 3.4.29
 platform: ruby
 authors:
 - Ben Crouse
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-03-03 00:00:00.000000000 Z
+date: 2020-03-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: workarea-core
@@ -16,28 +16,28 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 3.4.28
+        version: 3.4.29
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 3.4.28
+        version: 3.4.29
 - !ruby/object:Gem::Dependency
   name: workarea-storefront
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 3.4.28
+        version: 3.4.29
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 3.4.28
+        version: 3.4.29
 description: Provides site administration functionality for the Workarea Commerce
   Platform.
 email:
@@ -1220,6 +1220,7 @@ files:
 - test/integration/workarea/admin/help_articles_integration_test.rb
 - test/integration/workarea/admin/impersonations_integration_test.rb
 - test/integration/workarea/admin/import_taxes_integration_test.rb
+- test/integration/workarea/admin/index_tracking_integration_test.rb
 - test/integration/workarea/admin/inventory_skus_integration_test.rb
 - test/integration/workarea/admin/jump_to_integration_test.rb
 - test/integration/workarea/admin/menus_integration_test.rb