spree_api 3.7.14.1 → 4.0.0.beta

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (139) hide show
  1. checksums.yaml +4 -4
  2. data/Gemfile +1 -1
  3. data/app/controllers/spree/api/base_controller.rb +3 -3
  4. data/app/controllers/spree/api/v1/addresses_controller.rb +2 -2
  5. data/app/controllers/spree/api/v1/countries_controller.rb +2 -2
  6. data/app/controllers/spree/api/v1/credit_cards_controller.rb +2 -2
  7. data/app/controllers/spree/api/v1/images_controller.rb +3 -3
  8. data/app/controllers/spree/api/v1/inventory_units_controller.rb +2 -2
  9. data/app/controllers/spree/api/v1/option_types_controller.rb +4 -4
  10. data/app/controllers/spree/api/v1/option_values_controller.rb +3 -3
  11. data/app/controllers/spree/api/v1/payments_controller.rb +2 -2
  12. data/app/controllers/spree/api/v1/product_properties_controller.rb +4 -4
  13. data/app/controllers/spree/api/v1/properties_controller.rb +4 -4
  14. data/app/controllers/spree/api/v1/return_authorizations_controller.rb +4 -4
  15. data/app/controllers/spree/api/v1/shipments_controller.rb +3 -3
  16. data/app/controllers/spree/api/v1/states_controller.rb +3 -3
  17. data/app/controllers/spree/api/v1/stock_items_controller.rb +2 -2
  18. data/app/controllers/spree/api/v1/stock_locations_controller.rb +5 -4
  19. data/app/controllers/spree/api/v1/stock_movements_controller.rb +3 -3
  20. data/app/controllers/spree/api/v1/stores_controller.rb +4 -4
  21. data/app/controllers/spree/api/v1/taxonomies_controller.rb +3 -3
  22. data/app/controllers/spree/api/v1/taxons_controller.rb +5 -5
  23. data/app/controllers/spree/api/v1/users_controller.rb +3 -3
  24. data/app/controllers/spree/api/v1/variants_controller.rb +3 -3
  25. data/app/controllers/spree/api/v1/zones_controller.rb +3 -3
  26. data/app/controllers/spree/api/v2/base_controller.rb +1 -6
  27. data/app/controllers/spree/api/v2/storefront/account/credit_cards_controller.rb +1 -1
  28. data/app/controllers/spree/api/v2/storefront/countries_controller.rb +2 -1
  29. data/app/controllers/spree/api/v2/storefront/order_status_controller.rb +0 -6
  30. data/app/controllers/spree/api/v2/storefront/products_controller.rb +1 -1
  31. data/app/controllers/spree/api/v2/storefront/taxons_controller.rb +1 -1
  32. data/app/views/spree/api/errors/{gateway_error.v1.rabl → gateway_error.rabl} +0 -0
  33. data/app/views/spree/api/errors/{invalid_api_key.v1.rabl → invalid_api_key.rabl} +0 -0
  34. data/app/views/spree/api/errors/{invalid_resource.v1.rabl → invalid_resource.rabl} +0 -0
  35. data/app/views/spree/api/errors/{must_specify_api_key.v1.rabl → must_specify_api_key.rabl} +0 -0
  36. data/app/views/spree/api/errors/{not_found.v1.rabl → not_found.rabl} +0 -0
  37. data/app/views/spree/api/errors/{unauthorized.v1.rabl → unauthorized.rabl} +0 -0
  38. data/app/views/spree/api/v1/addresses/{show.v1.rabl → show.rabl} +0 -0
  39. data/app/views/spree/api/v1/adjustments/{show.v1.rabl → show.rabl} +0 -0
  40. data/app/views/spree/api/v1/countries/{index.v1.rabl → index.rabl} +0 -0
  41. data/app/views/spree/api/v1/countries/{show.v1.rabl → show.rabl} +0 -0
  42. data/app/views/spree/api/v1/credit_cards/{index.v1.rabl → index.rabl} +0 -0
  43. data/app/views/spree/api/v1/credit_cards/{show.v1.rabl → show.rabl} +0 -0
  44. data/app/views/spree/api/v1/customer_returns/{index.v1.rabl → index.rabl} +0 -0
  45. data/app/views/spree/api/v1/images/{index.v1.rabl → index.rabl} +0 -0
  46. data/app/views/spree/api/v1/images/{new.v1.rabl → new.rabl} +0 -0
  47. data/app/views/spree/api/v1/images/{show.v1.rabl → show.rabl} +0 -0
  48. data/app/views/spree/api/v1/line_items/{new.v1.rabl → new.rabl} +0 -0
  49. data/app/views/spree/api/v1/line_items/{show.v1.rabl → show.rabl} +0 -0
  50. data/app/views/spree/api/v1/option_types/{index.v1.rabl → index.rabl} +0 -0
  51. data/app/views/spree/api/v1/option_types/{new.v1.rabl → new.rabl} +0 -0
  52. data/app/views/spree/api/v1/option_types/{show.v1.rabl → show.rabl} +0 -0
  53. data/app/views/spree/api/v1/option_values/{index.v1.rabl → index.rabl} +0 -0
  54. data/app/views/spree/api/v1/option_values/{new.v1.rabl → new.rabl} +0 -0
  55. data/app/views/spree/api/v1/option_values/{show.v1.rabl → show.rabl} +0 -0
  56. data/app/views/spree/api/v1/orders/{address.v1.rabl → address.rabl} +0 -0
  57. data/app/views/spree/api/v1/orders/{canceled.v1.rabl → canceled.rabl} +0 -0
  58. data/app/views/spree/api/v1/orders/{cart.v1.rabl → cart.rabl} +0 -0
  59. data/app/views/spree/api/v1/orders/{complete.v1.rabl → complete.rabl} +0 -0
  60. data/app/views/spree/api/v1/orders/{could_not_apply_coupon.v1.rabl → could_not_apply_coupon.rabl} +0 -0
  61. data/app/views/spree/api/v1/orders/{could_not_transition.v1.rabl → could_not_transition.rabl} +0 -0
  62. data/app/views/spree/api/v1/orders/{index.v1.rabl → index.rabl} +0 -0
  63. data/app/views/spree/api/v1/orders/{insufficient_quantity.v1.rabl → insufficient_quantity.rabl} +0 -0
  64. data/app/views/spree/api/v1/orders/{invalid_shipping_method.v1.rabl → invalid_shipping_method.rabl} +0 -0
  65. data/app/views/spree/api/v1/orders/{mine.v1.rabl → mine.rabl} +0 -0
  66. data/app/views/spree/api/v1/orders/{order.v1.rabl → order.rabl} +0 -0
  67. data/app/views/spree/api/v1/orders/{payment.v1.rabl → payment.rabl} +0 -0
  68. data/app/views/spree/api/v1/orders/{show.v1.rabl → show.rabl} +0 -0
  69. data/app/views/spree/api/v1/payments/{credit_over_limit.v1.rabl → credit_over_limit.rabl} +0 -0
  70. data/app/views/spree/api/v1/payments/{index.v1.rabl → index.rabl} +0 -0
  71. data/app/views/spree/api/v1/payments/{new.v1.rabl → new.rabl} +0 -0
  72. data/app/views/spree/api/v1/payments/{show.v1.rabl → show.rabl} +0 -0
  73. data/app/views/spree/api/v1/payments/{update_forbidden.v1.rabl → update_forbidden.rabl} +0 -0
  74. data/app/views/spree/api/v1/product_properties/{index.v1.rabl → index.rabl} +0 -0
  75. data/app/views/spree/api/v1/product_properties/{new.v1.rabl → new.rabl} +0 -0
  76. data/app/views/spree/api/v1/product_properties/{show.v1.rabl → show.rabl} +0 -0
  77. data/app/views/spree/api/v1/products/{index.v1.rabl → index.rabl} +0 -0
  78. data/app/views/spree/api/v1/products/{new.v1.rabl → new.rabl} +0 -0
  79. data/app/views/spree/api/v1/products/{product.v1.rabl → product.rabl} +0 -0
  80. data/app/views/spree/api/v1/products/{show.v1.rabl → show.rabl} +0 -0
  81. data/app/views/spree/api/v1/promotions/{handler.v1.rabl → handler.rabl} +0 -0
  82. data/app/views/spree/api/v1/promotions/{show.v1.rabl → show.rabl} +0 -0
  83. data/app/views/spree/api/v1/properties/{index.v1.rabl → index.rabl} +0 -0
  84. data/app/views/spree/api/v1/properties/{new.v1.rabl → new.rabl} +0 -0
  85. data/app/views/spree/api/v1/properties/{show.v1.rabl → show.rabl} +0 -0
  86. data/app/views/spree/api/v1/reimbursements/{index.v1.rabl → index.rabl} +0 -0
  87. data/app/views/spree/api/v1/return_authorizations/{index.v1.rabl → index.rabl} +0 -0
  88. data/app/views/spree/api/v1/return_authorizations/{new.v1.rabl → new.rabl} +0 -0
  89. data/app/views/spree/api/v1/return_authorizations/{show.v1.rabl → show.rabl} +0 -0
  90. data/app/views/spree/api/v1/shared/{stock_location_required.v1.rabl → stock_location_required.rabl} +0 -0
  91. data/app/views/spree/api/v1/shipments/{big.v1.rabl → big.rabl} +0 -0
  92. data/app/views/spree/api/v1/shipments/{cannot_ready_shipment.v1.rabl → cannot_ready_shipment.rabl} +0 -0
  93. data/app/views/spree/api/v1/shipments/{mine.v1.rabl → mine.rabl} +0 -0
  94. data/app/views/spree/api/v1/shipments/{show.v1.rabl → show.rabl} +0 -0
  95. data/app/views/spree/api/v1/shipments/{small.v1.rabl → small.rabl} +0 -0
  96. data/app/views/spree/api/v1/shipping_rates/{show.v1.rabl → show.rabl} +0 -0
  97. data/app/views/spree/api/v1/states/{index.v1.rabl → index.rabl} +0 -0
  98. data/app/views/spree/api/v1/states/{show.v1.rabl → show.rabl} +0 -0
  99. data/app/views/spree/api/v1/stock_items/{index.v1.rabl → index.rabl} +0 -0
  100. data/app/views/spree/api/v1/stock_items/{show.v1.rabl → show.rabl} +0 -0
  101. data/app/views/spree/api/v1/stock_locations/{index.v1.rabl → index.rabl} +0 -0
  102. data/app/views/spree/api/v1/stock_locations/{show.v1.rabl → show.rabl} +0 -0
  103. data/app/views/spree/api/v1/stock_movements/{index.v1.rabl → index.rabl} +0 -0
  104. data/app/views/spree/api/v1/stock_movements/{show.v1.rabl → show.rabl} +0 -0
  105. data/app/views/spree/api/v1/stores/{index.v1.rabl → index.rabl} +0 -0
  106. data/app/views/spree/api/v1/stores/{show.v1.rabl → show.rabl} +0 -0
  107. data/app/views/spree/api/v1/tags/{index.v1.rabl → index.rabl} +0 -0
  108. data/app/views/spree/api/v1/taxonomies/{index.v1.rabl → index.rabl} +0 -0
  109. data/app/views/spree/api/v1/taxonomies/{nested.v1.rabl → nested.rabl} +0 -0
  110. data/app/views/spree/api/v1/taxonomies/{new.v1.rabl → new.rabl} +0 -0
  111. data/app/views/spree/api/v1/taxonomies/{show.v1.rabl → show.rabl} +0 -0
  112. data/app/views/spree/api/v1/taxons/{index.v1.rabl → index.rabl} +0 -0
  113. data/app/views/spree/api/v1/taxons/{new.v1.rabl → new.rabl} +0 -0
  114. data/app/views/spree/api/v1/taxons/{show.v1.rabl → show.rabl} +0 -0
  115. data/app/views/spree/api/v1/taxons/{taxons.v1.rabl → taxons.rabl} +0 -0
  116. data/app/views/spree/api/v1/users/{index.v1.rabl → index.rabl} +0 -0
  117. data/app/views/spree/api/v1/users/{new.v1.rabl → new.rabl} +0 -0
  118. data/app/views/spree/api/v1/users/{show.v1.rabl → show.rabl} +0 -0
  119. data/app/views/spree/api/v1/variants/{big.v1.rabl → big.rabl} +0 -0
  120. data/app/views/spree/api/v1/variants/{index.v1.rabl → index.rabl} +0 -0
  121. data/app/views/spree/api/v1/variants/{new.v1.rabl → new.rabl} +0 -0
  122. data/app/views/spree/api/v1/variants/{show.v1.rabl → show.rabl} +0 -0
  123. data/app/views/spree/api/v1/variants/{small.v1.rabl → small.rabl} +0 -0
  124. data/app/views/spree/api/v1/zones/{index.v1.rabl → index.rabl} +0 -0
  125. data/app/views/spree/api/v1/zones/{show.v1.rabl → show.rabl} +0 -0
  126. data/config/initializers/doorkeeper.rb +12 -0
  127. data/config/initializers/rabl_rails6_fix.rb +16 -0
  128. data/config/routes.rb +0 -2
  129. data/docs/v2/storefront/index.yaml +36 -31
  130. data/lib/spree/api/engine.rb +0 -19
  131. data/lib/spree/api/testing_support/v2/base.rb +1 -1
  132. data/lib/spree_api.rb +0 -1
  133. data/spree_api.gemspec +2 -3
  134. metadata +108 -126
  135. data/app/controllers/spree/api/v1/tags_controller.rb +0 -28
  136. data/app/models/doorkeeper/access_grant_decorator.rb +0 -3
  137. data/app/models/doorkeeper/access_token_decorator.rb +0 -3
  138. data/app/models/doorkeeper/application_decorator.rb +0 -3
  139. data/config/initializers/metal_load_paths.rb +0 -1
checksums.yaml CHANGED
@@ -1,7 +1,7 @@
1
1
  ---
2
2
  SHA256:
3
- metadata.gz: e95b4dd375ca3352b36e5cf43149af4c4bd1a2888b129dc0062e22a4cc851fa4
4
- data.tar.gz: 216504eec54335aa7127682fc5f2c40b51f2b2c748e86473a96321e966db0db8
3
+ metadata.gz: 72f06e43e51c1833d120d602a9558aad7f5ccf2e21bfc2d98bb72a34f1dfd86d
4
+ data.tar.gz: a81013706d6a7e712cc183bffb7d2222bf94e07da568c943daad27e5d3fd7a59
5
5
  SHA512:
6
- metadata.gz: 2f9ed96506e51d38bcd329cc2f9d472da590f48e2b35cdda9aa4c5ed072e32d01084dd98f843b3fbc68f7afa8e7e5f764e866d539c2816577721bbf885d1148c
7
- data.tar.gz: d24344cf64fb701559e67e4b768c1c23149c702c74c8c029ab5994428dd71d667f9fdd855eae98a6cce1f5fe26546ef948a9042e5e15c3d0079032a898376673
6
+ metadata.gz: 93248446a291211f90371a8f6f1355193497e0bae48fa2e9c91e45273d2385d38dbbf108b0b554ac805d993e207d14056a6d0b4962fef56242b9c793755d3e05
7
+ data.tar.gz: 69a6cbd99a6faf0a3e4f7ae450027d99c02ddb7454400f29587d7f866540fff57e3ddb257fe7479c85d67299b06743f67af5b40f4b5ebce583e1ae2fe0476364
data/Gemfile CHANGED
@@ -1,4 +1,4 @@
1
- eval(File.read(File.dirname(__FILE__) + '/../common_spree_dependencies.rb'))
1
+ eval_gemfile('../common_spree_dependencies.rb')
2
2
 
3
3
  gem 'spree_core', path: '../core'
4
4
 
@@ -133,12 +133,12 @@ module Spree
133
133
 
134
134
  def product_scope
135
135
  if @current_user_roles.include?('admin')
136
- scope = Product.with_deleted.accessible_by(current_ability, :read).includes(*product_includes)
136
+ scope = Product.with_deleted.accessible_by(current_ability, :show).includes(*product_includes)
137
137
 
138
138
  scope = scope.not_deleted unless params[:show_deleted]
139
139
  scope = scope.not_discontinued unless params[:show_discontinued]
140
140
  else
141
- scope = Product.accessible_by(current_ability, :read).active.includes(*product_includes)
141
+ scope = Product.accessible_by(current_ability, :show).active.includes(*product_includes)
142
142
  end
143
143
 
144
144
  scope
@@ -158,7 +158,7 @@ module Spree
158
158
 
159
159
  def authorize_for_order
160
160
  @order = Spree::Order.find_by(number: order_id)
161
- authorize! :read, @order, order_token
161
+ authorize! :show, @order, order_token
162
162
  end
163
163
  end
164
164
  end
@@ -5,7 +5,7 @@ module Spree
5
5
  before_action :find_order
6
6
 
7
7
  def show
8
- authorize! :read, @order, order_token
8
+ authorize! :show, @order, order_token
9
9
  @address = find_address
10
10
  respond_with(@address)
11
11
  end
@@ -14,7 +14,7 @@ module Spree
14
14
  authorize! :update, @order, order_token
15
15
  @address = find_address
16
16
 
17
- if @address.update_attributes(address_params)
17
+ if @address.update(address_params)
18
18
  respond_with(@address, default_template: :show)
19
19
  else
20
20
  invalid_resource!(@address)
@@ -5,7 +5,7 @@ module Spree
5
5
  skip_before_action :authenticate_user
6
6
 
7
7
  def index
8
- @countries = Country.accessible_by(current_ability, :read).ransack(params[:q]).result.
8
+ @countries = Country.accessible_by(current_ability).ransack(params[:q]).result.
9
9
  order('name ASC').
10
10
  page(params[:page]).per(params[:per_page])
11
11
  country = Country.order('updated_at ASC').last
@@ -13,7 +13,7 @@ module Spree
13
13
  end
14
14
 
15
15
  def show
16
- @country = Country.accessible_by(current_ability, :read).find(params[:id])
16
+ @country = Country.accessible_by(current_ability, :show).find(params[:id])
17
17
  respond_with(@country)
18
18
  end
19
19
  end
@@ -7,7 +7,7 @@ module Spree
7
7
  def index
8
8
  @credit_cards = user.
9
9
  credit_cards.
10
- accessible_by(current_ability, :read).
10
+ accessible_by(current_ability).
11
11
  with_payment_profile.
12
12
  ransack(params[:q]).result.page(params[:page]).per(params[:per_page])
13
13
  respond_with(@credit_cards)
@@ -17,7 +17,7 @@ module Spree
17
17
 
18
18
  def user
19
19
  if params[:user_id].present?
20
- @user ||= Spree.user_class.accessible_by(current_ability, :read).find(params[:user_id])
20
+ @user ||= Spree.user_class.accessible_by(current_ability, :show).find(params[:user_id])
21
21
  end
22
22
  end
23
23
  end
@@ -3,12 +3,12 @@ module Spree
3
3
  module V1
4
4
  class ImagesController < Spree::Api::BaseController
5
5
  def index
6
- @images = scope.images.accessible_by(current_ability, :read)
6
+ @images = scope.images.accessible_by(current_ability)
7
7
  respond_with(@images)
8
8
  end
9
9
 
10
10
  def show
11
- @image = Image.accessible_by(current_ability, :read).find(params[:id])
11
+ @image = Image.accessible_by(current_ability, :show).find(params[:id])
12
12
  respond_with(@image)
13
13
  end
14
14
 
@@ -26,7 +26,7 @@ module Spree
26
26
 
27
27
  def update
28
28
  @image = scope.images.accessible_by(current_ability, :update).find(params[:id])
29
- if @image.update_attributes(image_params)
29
+ if @image.update(image_params)
30
30
  respond_with(@image, default_template: :show)
31
31
  else
32
32
  invalid_resource!(@image)
@@ -13,7 +13,7 @@ module Spree
13
13
  authorize! :update, inventory_unit.order
14
14
 
15
15
  inventory_unit.transaction do
16
- if inventory_unit.update_attributes(inventory_unit_params)
16
+ if inventory_unit.update(inventory_unit_params)
17
17
  fire
18
18
  render :show, status: 200
19
19
  else
@@ -25,7 +25,7 @@ module Spree
25
25
  private
26
26
 
27
27
  def inventory_unit
28
- @inventory_unit ||= InventoryUnit.accessible_by(current_ability, :read).find(params[:id])
28
+ @inventory_unit ||= InventoryUnit.accessible_by(current_ability, :show).find(params[:id])
29
29
  end
30
30
 
31
31
  def prepare_event
@@ -6,19 +6,19 @@ module Spree
6
6
  @option_types = if params[:ids]
7
7
  Spree::OptionType.
8
8
  includes(:option_values).
9
- accessible_by(current_ability, :read).
9
+ accessible_by(current_ability).
10
10
  where(id: params[:ids].split(','))
11
11
  else
12
12
  Spree::OptionType.
13
13
  includes(:option_values).
14
- accessible_by(current_ability, :read).
14
+ accessible_by(current_ability).
15
15
  load.ransack(params[:q]).result
16
16
  end
17
17
  respond_with(@option_types)
18
18
  end
19
19
 
20
20
  def show
21
- @option_type = Spree::OptionType.accessible_by(current_ability, :read).find(params[:id])
21
+ @option_type = Spree::OptionType.accessible_by(current_ability, :show).find(params[:id])
22
22
  respond_with(@option_type)
23
23
  end
24
24
 
@@ -36,7 +36,7 @@ module Spree
36
36
 
37
37
  def update
38
38
  @option_type = Spree::OptionType.accessible_by(current_ability, :update).find(params[:id])
39
- if @option_type.update_attributes(option_type_params)
39
+ if @option_type.update(option_type_params)
40
40
  render :show
41
41
  else
42
42
  invalid_resource!(@option_type)
@@ -30,7 +30,7 @@ module Spree
30
30
 
31
31
  def update
32
32
  @option_value = scope.accessible_by(current_ability, :update).find(params[:id])
33
- if @option_value.update_attributes(option_value_params)
33
+ if @option_value.update(option_value_params)
34
34
  render :show
35
35
  else
36
36
  invalid_resource!(@option_value)
@@ -47,9 +47,9 @@ module Spree
47
47
 
48
48
  def scope
49
49
  @scope ||= if params[:option_type_id]
50
- Spree::OptionType.find(params[:option_type_id]).option_values.accessible_by(current_ability, :read)
50
+ Spree::OptionType.find(params[:option_type_id]).option_values.accessible_by(current_ability, :show)
51
51
  else
52
- Spree::OptionValue.accessible_by(current_ability, :read).load
52
+ Spree::OptionValue.accessible_by(current_ability, :show).load
53
53
  end
54
54
  end
55
55
 
@@ -29,7 +29,7 @@ module Spree
29
29
  authorize! params[:action], @payment
30
30
  if !@payment.editable?
31
31
  render 'update_forbidden', status: 403
32
- elsif @payment.update_attributes(payment_params)
32
+ elsif @payment.update(payment_params)
33
33
  respond_with(@payment, default_template: :show)
34
34
  else
35
35
  invalid_resource!(@payment)
@@ -60,7 +60,7 @@ module Spree
60
60
 
61
61
  def find_order
62
62
  @order = Spree::Order.find_by!(number: order_id)
63
- authorize! :read, @order, order_token
63
+ authorize! :show, @order, order_token
64
64
  end
65
65
 
66
66
  def find_payment
@@ -6,7 +6,7 @@ module Spree
6
6
  before_action :product_property, only: [:show, :update, :destroy]
7
7
 
8
8
  def index
9
- @product_properties = @product.product_properties.accessible_by(current_ability, :read).
9
+ @product_properties = @product.product_properties.accessible_by(current_ability).
10
10
  ransack(params[:q]).result.
11
11
  page(params[:page]).per(params[:per_page])
12
12
  respond_with(@product_properties)
@@ -31,7 +31,7 @@ module Spree
31
31
  def update
32
32
  authorize! :update, @product_property
33
33
 
34
- if @product_property.update_attributes(product_property_params)
34
+ if @product_property.update(product_property_params)
35
35
  respond_with(@product_property, status: 200, default_template: :show)
36
36
  else
37
37
  invalid_resource!(@product_property)
@@ -51,7 +51,7 @@ module Spree
51
51
  end
52
52
 
53
53
  def authorize_product!
54
- authorize! :read, @product
54
+ authorize! :show, @product
55
55
  end
56
56
 
57
57
  def product_property
@@ -60,7 +60,7 @@ module Spree
60
60
  @product_property ||= @product.product_properties.includes(:property).where(spree_properties: { name: params[:id] }).first
61
61
  raise ActiveRecord::RecordNotFound unless @product_property
62
62
 
63
- authorize! :read, @product_property
63
+ authorize! :show, @product_property
64
64
  end
65
65
  end
66
66
 
@@ -5,7 +5,7 @@ module Spree
5
5
  before_action :find_property, only: [:show, :update, :destroy]
6
6
 
7
7
  def index
8
- @properties = Spree::Property.accessible_by(current_ability, :read)
8
+ @properties = Spree::Property.accessible_by(current_ability)
9
9
 
10
10
  @properties = if params[:ids]
11
11
  @properties.where(id: params[:ids].split(',').flatten)
@@ -36,7 +36,7 @@ module Spree
36
36
  def update
37
37
  if @property
38
38
  authorize! :update, @property
39
- @property.update_attributes(property_params)
39
+ @property.update(property_params)
40
40
  respond_with(@property, status: 200, default_template: :show)
41
41
  else
42
42
  invalid_resource!(@property)
@@ -56,9 +56,9 @@ module Spree
56
56
  private
57
57
 
58
58
  def find_property
59
- @property = Spree::Property.accessible_by(current_ability, :read).find(params[:id])
59
+ @property = Spree::Property.accessible_by(current_ability, :show).find(params[:id])
60
60
  rescue ActiveRecord::RecordNotFound
61
- @property = Spree::Property.accessible_by(current_ability, :read).find_by!(name: params[:id])
61
+ @property = Spree::Property.accessible_by(current_ability, :show).find_by!(name: params[:id])
62
62
  end
63
63
 
64
64
  def property_params
@@ -20,7 +20,7 @@ module Spree
20
20
 
21
21
  def index
22
22
  authorize! :admin, ReturnAuthorization
23
- @return_authorizations = order.return_authorizations.accessible_by(current_ability, :read).
23
+ @return_authorizations = order.return_authorizations.accessible_by(current_ability).
24
24
  ransack(params[:q]).result.
25
25
  page(params[:page]).per(params[:per_page])
26
26
  respond_with(@return_authorizations)
@@ -32,13 +32,13 @@ module Spree
32
32
 
33
33
  def show
34
34
  authorize! :admin, ReturnAuthorization
35
- @return_authorization = order.return_authorizations.accessible_by(current_ability, :read).find(params[:id])
35
+ @return_authorization = order.return_authorizations.accessible_by(current_ability, :show).find(params[:id])
36
36
  respond_with(@return_authorization)
37
37
  end
38
38
 
39
39
  def update
40
40
  @return_authorization = order.return_authorizations.accessible_by(current_ability, :update).find(params[:id])
41
- if @return_authorization.update_attributes(return_authorization_params)
41
+ if @return_authorization.update(return_authorization_params)
42
42
  respond_with(@return_authorization, default_template: :show)
43
43
  else
44
44
  invalid_resource!(@return_authorization)
@@ -58,7 +58,7 @@ module Spree
58
58
 
59
59
  def order
60
60
  @order ||= Spree::Order.find_by!(number: order_id)
61
- authorize! :read, @order
61
+ authorize! :show, @order
62
62
  end
63
63
 
64
64
  def return_authorization_params
@@ -20,7 +20,7 @@ module Spree
20
20
 
21
21
  def create
22
22
  @order = Spree::Order.find_by!(number: params.fetch(:shipment).fetch(:order_id))
23
- authorize! :read, @order
23
+ authorize! :show, @order
24
24
  authorize! :create, Shipment
25
25
  quantity = params[:quantity].to_i
26
26
  @shipment = @order.shipments.create(stock_location_id: params.fetch(:stock_location_id))
@@ -126,13 +126,13 @@ module Spree
126
126
  @original_shipment = Spree::Shipment.find_by!(number: params[:original_shipment_number])
127
127
  @variant = Spree::Variant.find(params[:variant_id])
128
128
  @quantity = params[:quantity].to_i
129
- authorize! :read, @original_shipment
129
+ authorize! :show, @original_shipment
130
130
  authorize! :create, Shipment
131
131
  end
132
132
 
133
133
  def find_and_update_shipment
134
134
  @shipment = Spree::Shipment.accessible_by(current_ability, :update).readonly(false).find_by!(number: params[:id])
135
- @shipment.update_attributes(shipment_params)
135
+ @shipment.update(shipment_params)
136
136
  @shipment.reload
137
137
  end
138
138
 
@@ -24,10 +24,10 @@ module Spree
24
24
 
25
25
  def scope
26
26
  if params[:country_id]
27
- @country = Country.accessible_by(current_ability, :read).find(params[:country_id])
28
- @country.states.accessible_by(current_ability, :read).order('name ASC')
27
+ @country = Country.accessible_by(current_ability, :show).find(params[:country_id])
28
+ @country.states.accessible_by(current_ability).order('name ASC')
29
29
  else
30
- State.accessible_by(current_ability, :read).order('name ASC')
30
+ State.accessible_by(current_ability).order('name ASC')
31
31
  end
32
32
  end
33
33
  end
@@ -65,12 +65,12 @@ module Spree
65
65
 
66
66
  def stock_location
67
67
  render 'spree/api/v1/shared/stock_location_required', status: 422 and return unless params[:stock_location_id]
68
- @stock_location ||= StockLocation.accessible_by(current_ability, :read).find(params[:stock_location_id])
68
+ @stock_location ||= StockLocation.accessible_by(current_ability, :show).find(params[:stock_location_id])
69
69
  end
70
70
 
71
71
  def scope
72
72
  includes = { variant: [{ option_values: :option_type }, :product] }
73
- @stock_location.stock_items.accessible_by(current_ability, :read).includes(includes)
73
+ @stock_location.stock_items.accessible_by(current_ability, :show).includes(includes)
74
74
  end
75
75
 
76
76
  def stock_item_params
@@ -3,8 +3,9 @@ module Spree
3
3
  module V1
4
4
  class StockLocationsController < Spree::Api::BaseController
5
5
  def index
6
- authorize! :read, StockLocation
7
- @stock_locations = StockLocation.accessible_by(current_ability, :read).order('name ASC').ransack(params[:q]).result.page(params[:page]).per(params[:per_page])
6
+ authorize! :index, StockLocation
7
+ @stock_locations = StockLocation.accessible_by(current_ability).order('name ASC').
8
+ ransack(params[:q]).result.page(params[:page]).per(params[:per_page])
8
9
  respond_with(@stock_locations)
9
10
  end
10
11
 
@@ -24,7 +25,7 @@ module Spree
24
25
 
25
26
  def update
26
27
  authorize! :update, stock_location
27
- if stock_location.update_attributes(stock_location_params)
28
+ if stock_location.update(stock_location_params)
28
29
  respond_with(stock_location, status: 200, default_template: :show)
29
30
  else
30
31
  invalid_resource!(stock_location)
@@ -40,7 +41,7 @@ module Spree
40
41
  private
41
42
 
42
43
  def stock_location
43
- @stock_location ||= StockLocation.accessible_by(current_ability, :read).find(params[:id])
44
+ @stock_location ||= StockLocation.accessible_by(current_ability, :show).find(params[:id])
44
45
  end
45
46
 
46
47
  def stock_location_params
@@ -5,7 +5,7 @@ module Spree
5
5
  before_action :stock_location, except: [:update, :destroy]
6
6
 
7
7
  def index
8
- authorize! :read, StockMovement
8
+ authorize! :index, StockMovement
9
9
  @stock_movements = scope.ransack(params[:q]).result.page(params[:page]).per(params[:per_page])
10
10
  respond_with(@stock_movements)
11
11
  end
@@ -29,11 +29,11 @@ module Spree
29
29
 
30
30
  def stock_location
31
31
  render 'spree/api/v1/shared/stock_location_required', status: 422 and return unless params[:stock_location_id]
32
- @stock_location ||= StockLocation.accessible_by(current_ability, :read).find(params[:stock_location_id])
32
+ @stock_location ||= StockLocation.accessible_by(current_ability, :show).find(params[:stock_location_id])
33
33
  end
34
34
 
35
35
  def scope
36
- @stock_location.stock_movements.accessible_by(current_ability, :read)
36
+ @stock_location.stock_movements.accessible_by(current_ability, :show)
37
37
  end
38
38
 
39
39
  def stock_movement_params
@@ -5,8 +5,8 @@ module Spree
5
5
  before_action :get_store, except: [:index, :create]
6
6
 
7
7
  def index
8
- authorize! :read, Store
9
- @stores = Store.accessible_by(current_ability, :read).all
8
+ authorize! :index, Store
9
+ @stores = Store.accessible_by(current_ability).all
10
10
  respond_with(@stores)
11
11
  end
12
12
 
@@ -23,7 +23,7 @@ module Spree
23
23
 
24
24
  def update
25
25
  authorize! :update, @store
26
- if @store.update_attributes(store_params)
26
+ if @store.update(store_params)
27
27
  respond_with(@store, status: 200, default_template: :show)
28
28
  else
29
29
  invalid_resource!(@store)
@@ -31,7 +31,7 @@ module Spree
31
31
  end
32
32
 
33
33
  def show
34
- authorize! :read, @store
34
+ authorize! :show, @store
35
35
  respond_with(@store)
36
36
  end
37
37
 
@@ -29,7 +29,7 @@ module Spree
29
29
 
30
30
  def update
31
31
  authorize! :update, taxonomy
32
- if taxonomy.update_attributes(taxonomy_params)
32
+ if taxonomy.update(taxonomy_params)
33
33
  respond_with(taxonomy, status: 200, default_template: :show)
34
34
  else
35
35
  invalid_resource!(taxonomy)
@@ -45,13 +45,13 @@ module Spree
45
45
  private
46
46
 
47
47
  def taxonomies
48
- @taxonomies = Taxonomy.accessible_by(current_ability, :read).order('name').includes(root: :children).
48
+ @taxonomies = Taxonomy.accessible_by(current_ability).order('name').includes(root: :children).
49
49
  ransack(params[:q]).result.
50
50
  page(params[:page]).per(params[:per_page])
51
51
  end
52
52
 
53
53
  def taxonomy
54
- @taxonomy ||= Taxonomy.accessible_by(current_ability, :read).find(params[:id])
54
+ @taxonomy ||= Taxonomy.accessible_by(current_ability, :show).find(params[:id])
55
55
  end
56
56
 
57
57
  def taxonomy_params
@@ -6,9 +6,9 @@ module Spree
6
6
  @taxons = if taxonomy
7
7
  taxonomy.root.children
8
8
  elsif params[:ids]
9
- Spree::Taxon.includes(:children).accessible_by(current_ability, :read).where(id: params[:ids].split(','))
9
+ Spree::Taxon.includes(:children).accessible_by(current_ability).where(id: params[:ids].split(','))
10
10
  else
11
- Spree::Taxon.includes(:children).accessible_by(current_ability, :read).order(:taxonomy_id, :lft)
11
+ Spree::Taxon.includes(:children).accessible_by(current_ability).order(:taxonomy_id, :lft)
12
12
  end
13
13
  @taxons = @taxons.ransack(params[:q]).result
14
14
  @taxons = @taxons.page(params[:page]).per(params[:per_page])
@@ -48,7 +48,7 @@ module Spree
48
48
 
49
49
  def update
50
50
  authorize! :update, taxon
51
- if taxon.update_attributes(taxon_params)
51
+ if taxon.update(taxon_params)
52
52
  respond_with(taxon, status: 200, default_template: :show)
53
53
  else
54
54
  invalid_resource!(taxon)
@@ -74,12 +74,12 @@ module Spree
74
74
 
75
75
  def taxonomy
76
76
  if params[:taxonomy_id].present?
77
- @taxonomy ||= Spree::Taxonomy.accessible_by(current_ability, :read).find(params[:taxonomy_id])
77
+ @taxonomy ||= Spree::Taxonomy.accessible_by(current_ability, :show).find(params[:taxonomy_id])
78
78
  end
79
79
  end
80
80
 
81
81
  def taxon
82
- @taxon ||= taxonomy.taxons.accessible_by(current_ability, :read).find(params[:id])
82
+ @taxon ||= taxonomy.taxons.accessible_by(current_ability, :show).find(params[:id])
83
83
  end
84
84
 
85
85
  def taxon_params
@@ -5,7 +5,7 @@ module Spree
5
5
  rescue_from Spree::Core::DestroyWithOrdersError, with: :error_during_processing
6
6
 
7
7
  def index
8
- @users = Spree.user_class.accessible_by(current_ability, :read)
8
+ @users = Spree.user_class.accessible_by(current_ability, :show)
9
9
 
10
10
  @users = if params[:ids]
11
11
  @users.ransack(id_in: params[:ids].split(','))
@@ -37,7 +37,7 @@ module Spree
37
37
 
38
38
  def update
39
39
  authorize! :update, user
40
- if user.update_attributes(user_params)
40
+ if user.update(user_params)
41
41
  respond_with(user, status: 200, default_template: :show)
42
42
  else
43
43
  invalid_resource!(user)
@@ -53,7 +53,7 @@ module Spree
53
53
  private
54
54
 
55
55
  def user
56
- @user ||= Spree.user_class.accessible_by(current_ability, :read).find(params[:id])
56
+ @user ||= Spree.user_class.accessible_by(current_ability, :show).find(params[:id])
57
57
  end
58
58
 
59
59
  def user_params
@@ -38,7 +38,7 @@ module Spree
38
38
 
39
39
  def update
40
40
  @variant = scope.accessible_by(current_ability, :update).find(params[:id])
41
- if @variant.update_attributes(variant_params)
41
+ if @variant.update(variant_params)
42
42
  respond_with(@variant, status: 200, default_template: :show)
43
43
  else
44
44
  invalid_resource!(@product)
@@ -49,7 +49,7 @@ module Spree
49
49
 
50
50
  def product
51
51
  if params[:product_id]
52
- @product ||= Spree::Product.accessible_by(current_ability, :read).
52
+ @product ||= Spree::Product.accessible_by(current_ability, :show).
53
53
  friendly.find(params[:product_id])
54
54
  end
55
55
  end
@@ -65,7 +65,7 @@ module Spree
65
65
  variants = variants.with_deleted
66
66
  end
67
67
 
68
- variants.eligible.accessible_by(current_ability, :read)
68
+ variants.eligible.accessible_by(current_ability)
69
69
  end
70
70
 
71
71
  def variant_params
@@ -19,7 +19,7 @@ module Spree
19
19
  end
20
20
 
21
21
  def index
22
- @zones = Zone.accessible_by(current_ability, :read).order('name ASC').ransack(params[:q]).result.page(params[:page]).per(params[:per_page])
22
+ @zones = Zone.accessible_by(current_ability).order('name ASC').ransack(params[:q]).result.page(params[:page]).per(params[:per_page])
23
23
  respond_with(@zones)
24
24
  end
25
25
 
@@ -29,7 +29,7 @@ module Spree
29
29
 
30
30
  def update
31
31
  authorize! :update, zone
32
- if zone.update_attributes(zone_params)
32
+ if zone.update(zone_params)
33
33
  respond_with(zone, status: 200, default_template: :show)
34
34
  else
35
35
  invalid_resource!(zone)
@@ -47,7 +47,7 @@ module Spree
47
47
  end
48
48
 
49
49
  def zone
50
- @zone ||= Spree::Zone.accessible_by(current_ability, :read).find(params[:id])
50
+ @zone ||= Spree::Zone.accessible_by(current_ability, :show).find(params[:id])
51
51
  end
52
52
  end
53
53
  end
@@ -55,12 +55,7 @@ module Spree
55
55
  end
56
56
 
57
57
  def spree_current_user
58
- return nil unless doorkeeper_token
59
- return @spree_current_user if @spree_current_user
60
-
61
- doorkeeper_authorize!
62
-
63
- @spree_current_user ||= Spree.user_class.find_by(id: doorkeeper_token.resource_owner_id)
58
+ @spree_current_user ||= Spree.user_class.find_by(id: doorkeeper_token.resource_owner_id) if doorkeeper_token
64
59
  end
65
60
 
66
61
  def spree_authorize!(action, subject, *args)
@@ -41,7 +41,7 @@ module Spree
41
41
  end
42
42
 
43
43
  def scope
44
- spree_current_user.credit_cards.accessible_by(current_ability, :read)
44
+ spree_current_user.credit_cards.accessible_by(current_ability, :show)
45
45
  end
46
46
  end
47
47
  end
@@ -36,6 +36,7 @@ module Spree
36
36
  return scope.default if params[:iso] == 'default'
37
37
 
38
38
  scope.find_by(iso: params[:iso]&.upcase) ||
39
+ scope.find_by(id: params[:iso]&.upcase) ||
39
40
  scope.find_by(iso3: params[:iso]&.upcase)
40
41
  end
41
42
 
@@ -52,7 +53,7 @@ module Spree
52
53
  end
53
54
 
54
55
  def scope
55
- Spree::Country.accessible_by(current_ability, :read)
56
+ Spree::Country.accessible_by(current_ability, :show)
56
57
  end
57
58
  end
58
59
  end