effective_trash 0.3.3 → 0.4.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: c0d2f4720fc5c9eab98d6f37af76dd186969899b
-  data.tar.gz: 1c24401c5cb30fbdd9954e48c84b65a74ff61b96
+SHA256:
+  metadata.gz: a74c78184cb3ec900ce2336bd3cc00b2f4db31df0743aae8f0c0e81345cc262e
+  data.tar.gz: 1e79f5bfacebcda566c36b5f11ce271be53e5f975294fb49babd82e349c98f0a
 SHA512:
-  metadata.gz: 322a4d6d987aaa52a181afcc16225b4f01537d430194f8ab81a6b8a22c557fbb8db86213044ea06e3217c153f0e1197926e4926aad228a40975adbf73137ffd9
-  data.tar.gz: 5d03c55908dad4827371a2edb199a2aea6c8724637bcfc86decbc57f3bf0a2adc1553468b1d9e0561365a6460be1108284ae80026b4869917508bea46a47722d
+  metadata.gz: 85d54f6faddf75ef480bd44c6a81c12013aca4ca3f63c81a64f228be7cda2cf8720a6743d2871c0f7d191c70431837ebe07086e3aff5fb0c56f70459219b365b
+  data.tar.gz: 3bc56877b4ec4abd95d7958aa945375c64ff16463b244514c0fef3dd9c2c2210e46a67c14e2a25921c1ad8871e278972fdfac1e8d87fd9aa0aedfa1966f55ccd

data/README.md

@@ -46,7 +46,7 @@ and to your contoller:
 
 ```ruby
 class ApplicationController < ActionController::Base
-  before_action :set_effective_trash_current_user
+  around_action :set_effective_trash_current_user
 end
 ```
 
@@ -114,4 +114,3 @@ MIT License.  Copyright [Code and Effect Inc.](http://www.codeandeffect.com/)
 4. Push to the branch (`git push origin my-new-feature`)
 5. Bonus points for test coverage
 6. Create new Pull Request
-

data/app/controllers/admin/trash_controller.rb

@@ -11,16 +11,16 @@ module Admin
 
       @page_title = 'Trash'
 
-      EffectiveTrash.authorize!(self, :index, Effective::Trash)
-      EffectiveTrash.authorize!(self, :admin, :effective_trash)
+      EffectiveResources.authorize!(self, :index, Effective::Trash)
+      EffectiveResources.authorize!(self, :admin, :effective_trash)
     end
 
     def show
       @trash = Effective::Trash.all.find(params[:id])
       @page_title = "Trash item - #{@trash.trashed_to_s}"
 
-      EffectiveTrash.authorize!(self, :show, @trash)
-      EffectiveTrash.authorize!(self, :admin, :effective_trash)
+      EffectiveResources.authorize!(self, :show, @trash)
+      EffectiveResources.authorize!(self, :admin, :effective_trash)
     end
   end
 end

data/app/controllers/effective/trash_controller.rb

@@ -8,7 +8,7 @@ module Effective
 
       @page_title = 'Trash'
 
-      EffectiveTrash.authorize!(self, :index, Effective::Trash.new(user_id: current_user.id))
+      EffectiveResources.authorize!(self, :index, Effective::Trash.new(user_id: current_user.id))
     end
 
     # This is the User show event
@@ -16,12 +16,12 @@ module Effective
       @trash = Effective::Trash.where(user_id: current_user.id).find(params[:id])
       @page_title = "Trash item - #{@trash.to_s}"
 
-      EffectiveTrash.authorize!(self, :show, @trash)
+      EffectiveResources.authorize!(self, :show, @trash)
     end
 
     def restore
       @trash = Effective::Trash.find(params[:id])
-      EffectiveTrash.authorize!(self, :update, @trash)
+      EffectiveResources.authorize!(self, :update, @trash)
 
       Effective::Trash.transaction do
         begin

data/app/models/concerns/acts_as_trashable.rb

@@ -34,9 +34,9 @@ module ActsAsTrashable
   end
 
   module ClassMethods
+    def acts_as_trashable?; true; end
   end
 
   # Regular instance methods
 
 end
-

data/config/effective_trash.rb

@@ -2,27 +2,6 @@ EffectiveTrash.setup do |config|
   # Configure Database Table
   config.trash_table_name = :trash
 
-  # Authorization Method
-  #
-  # This method is called by all controller actions with the appropriate action and resource
-  # If the method returns false, an Effective::AccessDenied Error will be raised (see README.md for complete info)
-  #
-  # Use via Proc (and with CanCan):
-  # config.authorization_method = Proc.new { |controller, action, resource| can?(action, resource) }
-  #
-  # Use via custom method:
-  # config.authorization_method = :my_authorization_method
-  #
-  # And then in your application_controller.rb:
-  #
-  # def my_authorization_method(action, resource)
-  #   current_user.is?(:admin)
-  # end
-  #
-  # Or disable the check completely:
-  # config.authorization_method = false
-  config.authorization_method = Proc.new { |controller, action, resource| authorize!(action, resource) } # CanCanCan
-
   # Admin Screens Layout Settings
   config.layout = 'application'   # All EffectiveTrash controllers will use this layout
 

data/lib/effective_trash/set_current_user.rb

@@ -6,10 +6,14 @@ module EffectiveTrash
       # before_action :set_effective_trash_current_user
 
       def set_effective_trash_current_user
-        EffectiveTrash.current_user = current_user
+        begin
+          EffectiveTrash.current_user = current_user
+          yield
+        ensure
+          EffectiveTrash.current_user = nil
+        end
       end
 
     end
   end
 end
-

data/lib/effective_trash/version.rb

@@ -1,3 +1,3 @@
 module EffectiveTrash
-  VERSION = '0.3.3'.freeze
+  VERSION = '0.4.1'.freeze
 end

data/lib/effective_trash.rb

@@ -4,33 +4,13 @@ require 'effective_trash/version'
 
 module EffectiveTrash
 
-  # The following are all valid config keys
-  mattr_accessor :trash_table_name
-
-  mattr_accessor :authorization_method
-  mattr_accessor :layout
-  mattr_accessor :routes_enabled
-
-  def self.setup
-    yield self
+  def self.config_keys
+    [
+      :trash_table_name, :layout, :routes_enabled
+    ]
   end
 
-  def self.authorized?(controller, action, resource)
-    @_exceptions ||= [Effective::AccessDenied, (CanCan::AccessDenied if defined?(CanCan)), (Pundit::NotAuthorizedError if defined?(Pundit))].compact
-
-    return !!authorization_method unless authorization_method.respond_to?(:call)
-    controller = controller.controller if controller.respond_to?(:controller)
-
-    begin
-      !!(controller || self).instance_exec((controller || self), action, resource, &authorization_method)
-    rescue *@_exceptions
-      false
-    end
-  end
-
-  def self.authorize!(controller, action, resource)
-    raise Effective::AccessDenied unless authorized?(controller, action, resource)
-  end
+  include EffectiveGem
 
   # This is set by the "set_effective_trash_current_user" before_filter.
   def self.current_user=(user)
@@ -45,13 +25,14 @@ module EffectiveTrash
   # This is run in a before_destroy, or through a script.
   def self.trash!(obj)
     args = (obj.respond_to?(:acts_as_trashable_options) ? obj.acts_as_trashable_options : {})
+    details = Effective::Resource.new(obj).instance_attributes(only: args[:only], except: args[:except])
 
     trash = Effective::Trash.new(
       trashed: obj,
       user: EffectiveTrash.current_user,
       trashed_to_s: obj.to_s,
-      trashed_extra: (trashed_extra if obj.respond_to?(:trashed_extra)),
-      details: Effective::Resource.new(obj).instance_attributes(args)
+      trashed_extra: obj.try(:trashed_extra),
+      details: details
     ).save!
   end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: effective_trash
 version: !ruby/object:Gem::Version
-  version: 0.3.3
+  version: 0.4.1
 platform: ruby
 authors:
 - Code and Effect
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2019-04-02 00:00:00.000000000 Z
+date: 2023-05-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -80,7 +80,6 @@ files:
 - app/datatables/effective_trash_datatable.rb
 - app/helpers/effective_trash_helper.rb
 - app/models/concerns/acts_as_trashable.rb
-- app/models/effective/access_denied.rb
 - app/models/effective/trash.rb
 - app/views/admin/trash/_actions.html.haml
 - app/views/admin/trash/index.html.haml
@@ -102,7 +101,7 @@ homepage: https://github.com/code-and-effect/effective_trash
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -117,9 +116,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.4.5.1
-signing_key: 
+rubygems_version: 3.1.2
+signing_key:
 specification_version: 4
 summary: Trash and Restore any Active Record object.
 test_files: []

data/app/models/effective/access_denied.rb

@@ -1,17 +0,0 @@
-unless defined?(Effective::AccessDenied)
-  module Effective
-    class AccessDenied < StandardError
-      attr_reader :action, :subject
-
-      def initialize(message = nil, action = nil, subject = nil)
-        @message = message
-        @action = action
-        @subject = subject
-      end
-
-      def to_s
-        @message || I18n.t(:'unauthorized.default', :default => 'Access Denied')
-      end
-    end
-  end
-end