effective_resources 1.9.1 → 1.9.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d4e5791f81c07c0f264f6c4d91d4cb3e01c0194577457ae38239cfafd99ae6a6
-  data.tar.gz: 5e82bcba8a50e8b9beceffa977241c95000805690f02343aad7ffef7a66d0f97
+  metadata.gz: 83e99f559d60204f03679abaeacb94f0406596ca751221002c69adcb2cea52ee
+  data.tar.gz: 7ff7732c4b04ffeff143812216af2e03a81008830fba3681614c44d59552959b
 SHA512:
-  metadata.gz: 14a3dce0969cf7487a9f950a11b8fc8cd696b5dd01253c1d89792ed1c804af08b8ba886ce640e118e5f75e1ab47a6ff500c4dd3feca75ecacc1b3f5e9646e2ec
-  data.tar.gz: 72546fd49b52e4da7c713bcadf0179438d2f91fae66d9a99d09f08a35d96a71eb149d107add609b9232dc666af8866482e7f7a5d365d00b0af06af03e06cb2a5
+  metadata.gz: 1884f93d5785dde2523add135e534d86a18880fe9cc1ded09d73a915ed1ce8c842907e63a0a9e1ac7aa829489b55d3f1eaf0de2be4add65cfaa806e0701fd422
+  data.tar.gz: 7f88e77db260d68330f44868d79f8aebfd7458e19a3b4a4a205811071d2e3abc32f69777dd2ac2c6b964c0df4e1efde54e7b0233705ee7fd6d906e95facf9d1f

data/app/controllers/concerns/effective/crud_controller/actions.rb

@@ -9,7 +9,7 @@ module Effective
         @page_title ||= resource_plural_name.titleize
 
         self.resources ||= resource_scope.all if resource_scope.respond_to?(:all)
-        @datatable = resource_datatable(:index)
+        @datatable = resource_datatable()
 
         run_callbacks(:resource_render)
       end
@@ -214,7 +214,7 @@ module Effective
         @page_title ||= "#{action.to_s.titleize} #{resource_plural_name.titleize}"
 
         if request.get?
-          @datatable = resource_datatable(action)
+          @datatable = resource_datatable()
           run_callbacks(:resource_render)
 
           view = lookup_context.template_exists?(action, _prefixes) ? action : :index

data/app/controllers/concerns/effective/crud_controller/dsl.rb

@@ -61,7 +61,20 @@ module Effective
         raise 'expected a label or block' unless (label || block_given?)
 
         instance_exec do
-          before_action(opts) { @page_title ||= (block_given? ? instance_exec(&block) : label).to_s }
+          before_action(opts) do
+            @page_title ||= (block_given? ? instance_exec(&block) : label).to_s
+          end
+        end
+      end
+
+      # datatable -> { MyDatatable.new }, only: [:index]
+      def datatable(obj = nil, opts = {}, &block)
+        raise 'expected a proc or block' unless (obj.respond_to?(:call) || block_given?)
+
+        instance_exec do
+          before_action(opts) do
+            @datatable ||= (block_given? ? instance_exec(&block) : obj.call)
+          end
         end
       end
 
@@ -84,7 +97,6 @@ module Effective
         else
           define_method(:resource_scope_relation) { return obj }
         end
-
       end
 
     end

data/app/controllers/concerns/effective/crud_controller.rb

@@ -108,15 +108,16 @@ module Effective
       resource_scope.where_values_hash.symbolize_keys
     end
 
-    def resource_datatable(action)
-      datatable_klass = if action == :index
-        effective_resource.datatable_klass
-      else # Admin::ActionDatatable.new
-        "#{[effective_resource.namespace.to_s.classify.presence, action.to_s.classify].compact.join('::')}Datatable".safe_constantize ||
-        "#{[effective_resource.namespace.to_s.classify.presence, action.to_s.pluralize.classify].compact.join('::')}Datatable".safe_constantize ||
-        "#{[effective_resource.namespace.to_s.classify.presence, action.to_s.singularize.classify].compact.join('::')}Datatable".safe_constantize
+    def resource_datatable
+      # This might have been done from a before action or dsl method
+      unless @datatable.nil?
+        raise('expected @datatable to be an Effective::Datatable') unless @datatable.kind_of?(Effective::Datatable)
+
+        @datatable.effective_resource = effective_resource
+        return @datatable
       end
 
+      datatable_klass = effective_resource.datatable_klass
       return unless datatable_klass.present?
 
       datatable = datatable_klass.new(resource_datatable_attributes)

data/app/models/concerns/acts_as_wizard.rb

@@ -76,7 +76,11 @@ module ActsAsWizard
     end
 
     def next_step
-      required_steps.reverse.find { |step| can_visit_step?(step) } || required_steps.first
+      first_uncompleted_step ||
+      last_completed_step ||
+      required_steps.reverse.find { |step| can_visit_step?(step) } ||
+      required_steps.first ||
+      :start
     end
 
     def previous_step(step)
@@ -89,6 +93,13 @@ module ActsAsWizard
       previous.blank? || has_completed_step?(previous)
     end
 
+    def has_completed_all_previous_steps?(step)
+      index = required_steps.index(step).to_i
+      previous = required_steps[0...index]
+
+      previous.blank? || previous.all? { |step| has_completed_step?(step) }
+    end
+
     def has_completed_last_step?
       has_completed_step?(required_steps.last)
     end
@@ -97,12 +108,12 @@ module ActsAsWizard
 
     def can_revisit_completed_steps(step)
       return (step == required_steps.last) if has_completed_last_step?
-      has_completed_previous_step?(step)
+      has_completed_all_previous_steps?(step)
     end
 
     def cannot_revisit_completed_steps(step)
       return (step == required_steps.last) if has_completed_last_step?
-      has_completed_previous_step?(step) && !has_completed_step?(step)
+      has_completed_all_previous_steps?(step) && !has_completed_step?(step)
     end
 
   end

data/app/models/effective/resource.rb

@@ -14,7 +14,7 @@ module Effective
     include Effective::Resources::Paths
     include Effective::Resources::Relation
     include Effective::Resources::Sql
-
+    include Effective::Resources::Tenants
 
     # In practice, this is initialized two ways
     # With a klass and a namespace from effective_datatables

data/app/models/effective/resources/generator.rb

@@ -30,19 +30,19 @@ module Effective
 
       # Tenants
       def tenant_controller_path
-        (Tenant.module_name.downcase + '/' + controller_path) if defined?(Tenant)
+        (Tenant.module_name.downcase + '/' + controller_path) if tenant?
       end
 
       def tenant_namespaced_class_name
-        (Tenant.module_name + '::' + namespaced_class_name) if defined?(Tenant)
+        (Tenant.module_name + '::' + namespaced_class_name) if tenant?
       end
 
       def tenant_namespaced_module_name
-        (Tenant.module_name + '::' + namespaced_module_name) if defined?(Tenant)
+        (Tenant.module_name + '::' + namespaced_module_name) if tenant?
       end
 
       def tenant_class_name
-        (Tenant.module_name + '::' + class_name) if defined?(Tenant)
+        (Tenant.module_name + '::' + class_name) if tenant?
       end
 
     end

data/app/models/effective/resources/init.rb

@@ -8,7 +8,10 @@ module Effective
 
       def _initialize_input(input, namespace: nil, relation: nil)
         @initialized_name = input
-        @model_klass = (relation ? _klass_by_input(relation) : _klass_by_input(input))
+
+        # Sets the class but also namespaces
+        @model_klass = _klass_by_input(input)
+        @model_klass = _klass_by_input(relation) unless relation.nil?
 
         # Consider controller_name
         if @model_klass && input.kind_of?(String) && namespace.blank?

data/app/models/effective/resources/klass.rb

@@ -40,7 +40,7 @@ module Effective
       end
 
       def find_tenant_datatable_klass
-        return unless defined?(Tenant)
+        return unless tenant?
 
         "::#{tenant_controller_path.classify.pluralize}Datatable".safe_constantize ||
         "::#{tenant_controller_path.classify}Datatable".safe_constantize ||

data/app/models/effective/resources/naming.rb

@@ -65,16 +65,6 @@ module Effective
       def human_plural_name
         name.pluralize.gsub('::', ' ').underscore.gsub('_', ' ')
       end
-
-      def tenant
-        return nil unless defined?(Tenant)
-        return nil unless klass.present?
-        return nil unless class_name.include?('::')
-
-        name = class_name.split('::').first.downcase.to_sym
-        name if Rails.application.config.tenants[name].present?
-      end
-
     end
   end
 end

data/app/models/effective/resources/paths.rb

@@ -5,8 +5,7 @@ module Effective
     module Paths
 
       def tenant_path
-        return unless tenant.present?
-        Tenant.engine_path(tenant).sub("#{Rails.root}/", '')
+        Tenant.engine_path(tenant).sub("#{Rails.root}/", '') if tenant?
       end
 
       def model_file

data/app/models/effective/resources/tenants.rb

@@ -0,0 +1,20 @@
+module Effective
+  module Resources
+    module Tenants
+
+      def tenant?
+        defined?(::Tenant)
+      end
+
+      def tenant
+        return unless tenant?
+        return nil unless klass.present?
+        return nil unless class_name.include?('::')
+
+        name = class_name.split('::').first.downcase.to_sym
+        name if Rails.application.config.tenants[name].present?
+      end
+
+    end
+  end
+end

data/lib/effective_resources/version.rb

@@ -1,3 +1,3 @@
 module EffectiveResources
-  VERSION = '1.9.1'.freeze
+  VERSION = '1.9.5'.freeze
 end

data/lib/effective_resources.rb

@@ -33,6 +33,24 @@ module EffectiveResources
 
   # Utilities
 
+  # This looks up the best class give the name
+  # If the Tenant is present, use those classes first.
+  def self.best(name)
+    klass = if defined?(Tenant)
+      ('::' + Tenant.module_name + '::' + name).safe_constantize ||
+      ('::' + Tenant.module_name + '::Effective::' + name).safe_constantize
+    end
+
+    klass ||= begin
+      ('::' + name).safe_constantize ||
+      ('::Effective::' + name).safe_constantize
+    end
+
+    raise("unable to find best #{name}") if klass.blank?
+
+    klass
+  end
+
   def self.truthy?(value)
     if defined?(::ActiveRecord::ConnectionAdapters::Column::TRUE_VALUES)  # Rails <5
       ::ActiveRecord::ConnectionAdapters::Column::TRUE_VALUES.include?(value)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: effective_resources
 version: !ruby/object:Gem::Version
-  version: 1.9.1
+  version: 1.9.5
 platform: ruby
 authors:
 - Code and Effect
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-11-04 00:00:00.000000000 Z
+date: 2021-11-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -184,6 +184,7 @@ files:
 - app/models/effective/resources/paths.rb
 - app/models/effective/resources/relation.rb
 - app/models/effective/resources/sql.rb
+- app/models/effective/resources/tenants.rb
 - app/views/application/_flash.html.haml
 - app/views/application/create.js.erb
 - app/views/application/destroy.js.erb