Spree v4.0

app/controllers/spree/admin/base_controller_decorator.rb

@@ -1,4 +1,4 @@
-Spree::Admin::BaseController.class_eval do
+module Spree::Admin::BaseControllerDecorator
   def authorize_admin
     begin
       if params[:id]
@@ -25,3 +25,5 @@ def new_action?
       NEW_ACTIONS.include?(params[:action].to_sym)
     end
 end
+
+::Spree::BaseController.prepend(Spree::BaseControllerDecorator)

app/models/spree/ability_decorator.rb

@@ -1,6 +1,5 @@
 module Spree
-  Ability.class_eval do
-
+  module AbilityDecorator
     def initialize(user)
       self.clear_aliased_actions
 
@@ -26,3 +25,5 @@ def user_roles(user)
     end
   end
 end
+
+::Spree::Ability.prepend(Spree::AbilityDecorator)

app/models/spree/role_decorator.rb

@@ -1,18 +1,21 @@
-Spree::Role.class_eval do
+module Spree::RoleDecorator
+  def self.prepended(base)
+    base.has_many :roles_permission_sets, dependent: :destroy
+    base.has_many :permission_sets, through: :roles_permission_sets
+    base.has_many :permissions, through: :permission_sets
 
-  has_many :roles_permission_sets, dependent: :destroy
-  has_many :permission_sets, through: :roles_permission_sets
-  has_many :permissions, through: :permission_sets
+    # DEPRECATED: Use permission sets instead. Only here for aiding migration for existing users
+    base.has_and_belongs_to_many :legacy_permissions, join_table: 'spree_roles_permissions', class_name: 'Spree::Permission'
 
-  # DEPRECATED: Use permission sets instead. Only here for aiding migration for existing users
-  has_and_belongs_to_many :legacy_permissions, join_table: 'spree_roles_permissions', class_name: 'Spree::Permission'
+    base.validates :name, uniqueness: true, allow_blank: true
+    base.validates :permission_sets, length: { minimum: 1, too_short: :atleast_one_permission_set_is_required }, on: :update
 
-  validates :name, uniqueness: true, allow_blank: true
-  validates :permission_sets, length: { minimum: 1, too_short: :atleast_one_permission_set_is_required }, on: :update
+    base.scope :default_role, lambda { where(is_default: true) }
+  end
 
   def has_permission?(permission_title)
     permissions.pluck(:title).include?(permission_title)
   end
-
-  scope :default_role, lambda { where(is_default: true) }
 end
+
+::Spree::Role.prepend(Spree::RoleDecorator)

app/models/spree/user_decorator.rb

@@ -1,5 +1,9 @@
 module Spree
-  Spree.user_class.class_eval do
-    alias_attribute :roles, :spree_roles
+  module UserDecorator
+    def self.prepended(base)
+      base.alias_attribute :roles, :spree_roles
+    end
   end
 end
+
+::Spree.user_class.prepend(Spree::UserDecorator)

spree_admin_roles_and_access.gemspec

@@ -14,7 +14,7 @@ Gem::Specification.new do |s|
   s.require_path = 'lib'
   s.requirements << 'none'
 
-  spree_version = '>= 3.2.0', '< 4.0.0'
+  spree_version = '>= 3.2.0', '< 5.0'
 
   s.add_dependency 'spree_core', spree_version
   s.add_dependency 'spree_auth_devise'