eitil 1.3.12 → 2.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 33223cb7e4d6699f037753eb436ebaefc1c50be4d3e3cb046b98c159606f964d
-  data.tar.gz: fdfc5a0394c7844e12e5e9d20b9fd7ba492a750c9f36c528d46b1afe62df5bc4
+  metadata.gz: 340d9c03c3202df1622b826be81e2855b873a03dde4855e3b17597d9d4ca0916
+  data.tar.gz: c1aff01a4c23ffe4b9b1fce2e1e6a2e146e932ad21c89da7e619fcbde7774ef5
 SHA512:
-  metadata.gz: 5eed402cd2037e081c51e883e7c753ecb4ce62f65e213890c82cc1b774dd10dd5bd4c8eb0a1079955bbdd8a164330bc718e605fad6befd3229470fac81233f3e
-  data.tar.gz: 2735fc3a2d11d735e0ef94e59adcc45f29916d9ddf12a832ecb5fdd31be909aef286e01b939185ef79bfc34a79b0f750bc7f5b1d28efb7a1931e65c08ecc6c95
+  metadata.gz: 5481779145b3b57fbc781ecb02cd3e6348bdd1b3318376a2328d2def32a67406dab91cdfceb19c6788f1c9e1096d47cba8294dfd2886e60e36fac2273065cf8b
+  data.tar.gz: 18d72653d077ba48e93e01e5e62e5678d8b64bc751297d8b329678a3d644b710d33f6d9ac7a9bbe8eeceab4633ab13e728753148d6ed2c8b86bef3fe29345122

data/eitil_integrate/lib/eitil_integrate/application_exporter/lookups.rb

@@ -10,7 +10,7 @@ module EitilIntegrate::RubyXL
     class << self
 
       def exporter_paths
-        EitilSupport::Directory.files('app/exporters/exporters').select { |path| path.end_with? '_exporter.rb' }
+        EitilSupport::Directory.files('app/views/exporters/exporters').select { |path| path.end_with? '_exporter.rb' }
       end
 
       def exporter_names

data/eitil_wrapper/README.md

@@ -37,44 +37,6 @@ Callback helper methods are created for use within a model's callbacks. You can
 
 
 
-## EitilWrapper::Decorators
-
-```ruby
-
-require "eitil_wrapper/decorators"
-
-```
-
-The Eitil decorator wrappers help you to standardize the calling of the right decorator method from within your controller action. Basically it provides you with a decorate macro in each controller.
-
-```ruby
-decorate(dec_item, dec_method: nil, dec_class: nil, **dec_kwargs)
-```
-
-- dec_item is the instance that will be decorated
-- dec_method enabled you to set the desired decorator method. If not provided, it will look into the request params: if params["isMobile"] is present it will call .app, if params["isWeb"] is present it will call :app. If neither is provided in the params, it will call the default method :base.
-- dec_class enables you to overwrite the decorator class that will be called. If not provided, the decorator class will be inferred from the instance model's classname (User => UserDecorator).
-- dec_kwargs enables you to provide additional arguments, which will be passed to your decorator method.
-
-
-### Configuration
-
-1. Your decorator classes should inherit from EitilWrapper::ApplicationDecorator.
-2. Your controllers should inherit the module EitilWrapper::ControllerDecorator, through inclusion in a superclass.
-3. If you set controller ivars for each request, you can make them available in your decorators by providing Eitil with the names of your ivars as an array of symbols:
-
-```ruby
-# initializers/eitil.rb
-
-Eitil.set_config do |config|
-  config.controller_ivars = [:user, :env]
-end
-```
-
-
-
-
-
 ## EitilWrapper::Jobs
 
 ```ruby

data/eitil_wrapper/lib/eitil_wrapper/railtie.rb

@@ -36,6 +36,10 @@ module EitilWrapper
         # works for non database supported instanes, such as an Exporter instance.
 
         module ::EitilWrapper
+          
+          class ::ApplicationJob < ActiveJob::Base
+          end
+
           class SingleMethodJob < ::ApplicationJob
             def perform(*args, _class:, _method:, id: nil, _self: nil, **kwargs)      
               object = 
@@ -85,5 +89,6 @@ module EitilWrapper
       end; end; end; end; end
 
     end
+
   end
 end

data/eitil_wrapper/lib/eitil_wrapper.rb

@@ -5,5 +5,4 @@ require "eitil_wrapper/jobs"
 require "eitil_wrapper/records"
 require "eitil_wrapper/routes"
 require "eitil_wrapper/callbacks"
-require "eitil_wrapper/decorators"
 require "eitil_wrapper/request_logger"

data/lib/eitil/version.rb

@@ -1,5 +1,5 @@
 module Eitil
   
-  VERSION = '1.3.12'
+  VERSION = '2.0.1'
   
 end

data/spec/dummy_app/app/controllers/application_controller.rb

@@ -1,5 +1,3 @@
 class ApplicationController < ActionController::Base
 
-  include EitilWrapper::ControllerDecorator
-
 end

data/spec/dummy_app/app/controllers/users_controller.rb

@@ -3,37 +3,6 @@ class UsersController < ApplicationController
   before_action :set_user, only: [:show, :edit, :update, :destroy]
 
 
-  # ---
-  # Tests for EitilWrapper::Decorator
-
-  before_action :set_user, only: [:decorator_a, :decorator_b, :decorator_c, :decorator_d]
-
-  # default
-  def decorator_a
-    hash = decorate(@user)
-    render json: hash
-  end
-
-  # specific dec_method
-  def decorator_b
-    hash = decorate(@user, dec_method: :first_and_last_name)
-    render json: hash
-  end
-
-  # post request, with 'isWeb' and 'isMobile' params
-  def decorator_c
-    hash = decorate(@user)
-    render json: hash
-  end
-
-  # specific dec_class
-  def decorator_d
-    hash = decorate(@user, dec_class: :Application, dec_method: :timestamps)
-    render json: hash
-  end
-
-  # ---
-
 
 
   # GET /users

data/spec/dummy_app/config/routes.rb

@@ -4,9 +4,4 @@ Rails.application.routes.draw do
   resources :addresses
   resources :users
 
-  get  'users/:id/decorator_a' => 'users#decorator_a'
-  get  'users/:id/decorator_b' => 'users#decorator_b'
-  post 'users/:id/decorator_c' => 'users#decorator_c'
-  get  'users/:id/decorator_d' => 'users#decorator_d'
-
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: eitil
 version: !ruby/object:Gem::Version
-  version: 1.3.12
+  version: 2.0.1
 platform: ruby
 authors:
 - Jurriaan Schrofer
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-10-08 00:00:00.000000000 Z
+date: 2021-10-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -275,9 +275,6 @@ files:
 - eitil_wrapper/lib/eitil_wrapper.rb
 - eitil_wrapper/lib/eitil_wrapper/callbacks.rb
 - eitil_wrapper/lib/eitil_wrapper/callbacks/helper_methods.rb
-- eitil_wrapper/lib/eitil_wrapper/decorators.rb
-- eitil_wrapper/lib/eitil_wrapper/decorators/application_decorator.rb
-- eitil_wrapper/lib/eitil_wrapper/decorators/controller_decorator.rb
 - eitil_wrapper/lib/eitil_wrapper/jobs.rb
 - eitil_wrapper/lib/eitil_wrapper/jobs/new_job.rb
 - eitil_wrapper/lib/eitil_wrapper/jobs/new_job_now.rb
@@ -309,9 +306,6 @@ files:
 - spec/dummy_app/app/controllers/addresses_controller.rb
 - spec/dummy_app/app/controllers/application_controller.rb
 - spec/dummy_app/app/controllers/users_controller.rb
-- spec/dummy_app/app/decorators/address_decorator.rb
-- spec/dummy_app/app/decorators/application_decorator.rb
-- spec/dummy_app/app/decorators/user_decorator.rb
 - spec/dummy_app/app/helpers/addresses_helper.rb
 - spec/dummy_app/app/helpers/application_helper.rb
 - spec/dummy_app/app/helpers/users_helper.rb
@@ -444,7 +438,6 @@ files:
 - spec/eitil_integrate/application_exporter/auto_sum_spec.rb
 - spec/eitil_integrate/application_exporter/initialize_spec.rb
 - spec/eitil_wrapper/callbacks/helper_methods_spec.rb
-- spec/eitil_wrapper/decorators/decorators_spec.rb
 - spec/eitil_wrapper/jobs/single_method_job_spec.rb
 - spec/eitil_wrapper/routes/extended_resources_spec.rb
 - spec/eitil_wrapper/scopes/default_scopes_spec.rb

data/eitil_wrapper/lib/eitil_wrapper/decorators/application_decorator.rb

@@ -1,22 +0,0 @@
-
-require "active_model/railtie"
-
-module EitilWrapper
-  class ApplicationDecorator
-
-    include ActiveModel::Model
-
-    def initialize(attributes={}, **kwargs)      
-      super attributes
-      all_kwargs_to_ivars binding
-    end
-
-    def self.method_missing(method_name, *args, **kwargs)
-      if %i$ web app $.include?(method_name.to_sym)
-        kwargs.any? ? send(:base, *args, **kwargs) : send(:base, *args)
-      end
-      super
-    end
-
-  end
-end

data/eitil_wrapper/lib/eitil_wrapper/decorators/controller_decorator.rb

@@ -1,72 +0,0 @@
-module EitilWrapper
-  module ControllerDecorator
-
-    private
-
-    def decorate(dec_item, dec_method: nil, dec_class: nil, **dec_kwargs)
-      all_args_to_ivars binding
-      set_ivars :dec_class, :decorator, :dec_method
-      send_to_decorator
-
-      rescue NameError => e
-        puts "EitilWrapper::ControllerDecorator => rescuing uninitialised constant #{e.name}, returning the object's default JSON instead."
-        return dec_item.as_json
-    end
-
-    def send_to_decorator
-       @decorator.send(@dec_method, @dec_item)
-     
-      rescue NoMethodError => e       
-        inform_no_method_for_decorator_error
-        @dec_item
-    end
-
-    def inform_no_method_for_decorator_error
-      message = "Warning: NoMethodError for #{@dec_class}##{@dec_method}, returned @dec_item instead."
-      Logger.new("#{Rails.root}/log/decorator_log.log").warn message
-      warn message
-    end
-
-    def set_dec_method
-      @dec_method = @dec_method || derived_dec_method || :base
-    end
-
-    def derived_dec_method
-      return unless respond_to? :params      
-      return :app if @decorator.respond_to?(:app) && params["isMobile"]
-      return :web if @decorator.respond_to?(:web) && params["isWeb"]
-    end
-
-    def set_dec_class
-      @dec_class = @dec_class ? manual_set_dec_class : derived_dec_class
-    end
-
-    def manual_set_dec_class
-      "#{@dec_class}Decorator".constantize
-    end
-
-    def derived_dec_class
-      "#{@dec_item.class.name}Decorator".constantize
-    end
-
-    def set_decorator
-      @dec_class.new **ivars
-    end
-
-    # Converts the controller_ivars, given in your main application's configuration,
-    # into an hash. If no controller_ivars, continue with converting, in order to avoid
-    # issues due to differing data types.
-    def controller_ivars
-      return {} unless Eitil.controller_ivars.any?
-
-      Eitil.controller_ivars.map do |ivar|
-        { ivar => instance_variable_get("@#{ivar.to_s}") }
-      end.inject &:merge
-    end
-
-    def ivars
-       @dec_kwargs.any? ? controller_ivars.merge(@dec_kwargs) : controller_ivars
-    end
-
-  end
-end

data/eitil_wrapper/lib/eitil_wrapper/decorators.rb

@@ -1,3 +0,0 @@
-
-require "eitil_wrapper/decorators/application_decorator"
-require "eitil_wrapper/decorators/controller_decorator"

data/spec/dummy_app/app/decorators/address_decorator.rb

@@ -1,3 +0,0 @@
-class AddressDecorator < ApplicationDecorator
-
-end

data/spec/dummy_app/app/decorators/application_decorator.rb

@@ -1,12 +0,0 @@
-class ApplicationDecorator < EitilWrapper::ApplicationDecorator
-
-  # ---
-  # Tests for EitilWrapper::Decorator
-
-  def timestamps(record)
-    record.slice(:created_at, :updated_at)
-  end
-
-  # ---
-
-end

data/spec/dummy_app/app/decorators/user_decorator.rb

@@ -1,19 +0,0 @@
-class UserDecorator < ApplicationDecorator
-
-  def first_and_last_name(user)
-    user.slice(:first_name, :last_name)
-  end
-
-  def app(user)
-    user.slice(:phone, :email)
-  end
-
-  def web(user)
-    user.slice(:age, :wage)
-  end
-
-  def base(user)
-    user.as_json
-  end
-
-end

data/spec/eitil_wrapper/decorators/decorators_spec.rb

@@ -1,49 +0,0 @@
-
-RSpec.describe "EitilWrapper::Decorators" do
-
-  include Rack::Test::Methods
-
-  let(:user) { User.create(first_name: 'slim', last_name: 'shady') }
-
-  def body(endpoint)
-    @body ||= JSON.parse(get("/users/#{user.id}/#{endpoint}").body)
-  end
-
-  it "should respond to a request" do   
-    response = get "/users/#{user.id}/decorator_a"
-    expect(response).to be_a Rack::MockResponse
-  end
-
-  it "should, by default, return all of a records' columns" do
-    response_keys = body(:decorator_a).keys.map(&:to_sym)
-    table_columns = [:id, :created_at, :updated_at, :first_name, :last_name, :email, :birthday, :phone, :confirmed, :last_sign_in, :age, :wage]
-    expect(table_columns.all? { |column| response_keys.include?(column) }).to be_truthy
-  end
-
-  it "should, if a specific decorator method is passed, return that method's value" do
-    response_keys  = body(:decorator_b).keys.map(&:to_sym)
-    decorator_keys = [:first_name, :last_name]
-    expect(response_keys).to eq decorator_keys
-  end
-
-  it "should, if no decorator method is passed, but param['isMobile'] is present, return the :app method's value" do
-    response       = post "/users/#{user.id}/decorator_c", { "isMobile" => true }
-    response_keys  = JSON.parse(response.body).keys.map(&:to_sym)
-    decorator_keys = [:phone, :email]
-    expect(response_keys).to eq decorator_keys
-  end
-
-  it "should, if no decorator method is passed, but param['isWeb'] is present, return the :web method's value" do
-    response       = post "/users/#{user.id}/decorator_c", { "isWeb" => true }
-    response_keys  = JSON.parse(response.body).keys.map(&:to_sym)
-    decorator_keys = [:age, :wage]
-    expect(response_keys).to eq decorator_keys
-  end
-
-  it "should, if a decorator class is passed, return the value of that class' method" do
-    response_keys  = body(:decorator_d).keys.map(&:to_sym)
-    decorator_keys = [:created_at, :updated_at]
-    expect(response_keys).to eq decorator_keys
-  end
-  
-end