effective_resources 1.5.4 → 1.6.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2b4cbe9d018c7ea4cb40727d321530ba562c0ca873bea3c7e82492cd7d518b64
-  data.tar.gz: f1249c47c44c4095c30441aa636746f298c8591585e14ac948391693ea98cec5
+  metadata.gz: b8f4f8a61fe16b9a92f77b57d2c296375b1a9173de4d6d3e210ca08531911d61
+  data.tar.gz: 467d2e1778f6fe676653f5fee538c5960eaeb971f1503b455fd785b00f0ccb31
 SHA512:
-  metadata.gz: 1b6ae58c5d781ff5ad6378ad6df6f8f5388dd33213fa57c1d2ea902b593d72088e60dbb551fd5185acd442354803c02383b53d7414304ea53740e6c0db5c9d74
-  data.tar.gz: 2df964ebabaa075f388d96ad5e78f9a5d5b27a2071b5320a4f1593fdb882fd717491ddc3ca66a84d027952ed4638f3fa817f48869a2d2d4c4870b831741f3d3f
+  metadata.gz: 06baf45541024e1990157d58aeb88915ce6399534e01dc7d6ca4b430e0d473b3d294c15de8bbd4fe67f065a50ed4f2bc193c6c73b06892b13beb244400a122ec
+  data.tar.gz: 12cdb59f706b3c3fe1d03f0df1f19b2bc758e61e28b256a894fa880d736c03d28fc0067ab84dafe51fc88a00d599232b93033c1555d7d85e21371d1a4baa6f88

data/README.md

@@ -1,7 +1,7 @@
 # Effective Resources
 
 This gem looks at the current `routes.rb`, authorization `ability.rb`, `current_user` and controller context
-to metaprogram an effective CRUD website. 
+to metaprogram an effective CRUD website.
 
 It automates linking to resource edit, show, delete pages, as well as member and collection actions.
 
@@ -256,6 +256,14 @@ end
 
 and include Effective::CrudController in your resource controller.
 
+## Testing
+
+Run tests by:
+
+```ruby
+rails test
+```
+
 ## License
 
 MIT License.  Copyright [Code and Effect Inc.](http://www.codeandeffect.com/)
@@ -268,4 +276,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/concerns/effective/crud_controller.rb

@@ -118,7 +118,9 @@ module Effective
 
       return unless datatable_klass.present?
 
-      datatable_klass.new(resource_datatable_attributes)
+      datatable = datatable_klass.new(resource_datatable_attributes)
+      datatable.effective_resource = effective_resource if datatable.respond_to?(:effective_resource=)
+      datatable
     end
 
     def resource_params_method_name

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

@@ -80,7 +80,7 @@ module Effective
         raise 'expected a proc or block' unless (obj.respond_to?(:call) || block_given?)
 
         instance_exec do
-          prepend_before_action(opts) { @_effective_resource_scope ||= instance_exec(&(block_given? ? block : obj)) }
+          before_action(opts) { @_effective_resource_scope ||= instance_exec(&(block_given? ? block : obj)) }
         end
       end
 

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

@@ -5,7 +5,7 @@ module Effective
 
       # This is only available to models that use the effective_resource do ... end attributes block
       # It will be called last, and only for those resources
-      # params.require(effective_resource.name).permit!
+      # params.require(effective_resource.resource_name).permit!
       def resource_permitted_params
         raise 'expected resource class to have effective_resource do .. end' if effective_resource.model.blank?
 
@@ -13,10 +13,10 @@ module Effective
 
         if Rails.env.development?
           Rails.logger.info "Effective::CrudController#resource_permitted_params:"
-          Rails.logger.info "params.require(:#{effective_resource.name}).permit(#{permitted_params.to_s[1...-1]})"
+          Rails.logger.info "params.require(:#{effective_resource.resource_name}).permit(#{permitted_params.to_s[1...-1]})"
         end
 
-        params.require(effective_resource.name).permit(*permitted_params)
+        params.require(effective_resource.resource_name).permit(*permitted_params)
       end
 
       # If the resource is ActiveModel, just permit all
@@ -24,13 +24,13 @@ module Effective
       def resource_active_model_permitted_params
         if Rails.env.development?
           Rails.logger.info "Effective::CrudController#resource_permitted_params:"
-          Rails.logger.info "params.require(:#{effective_resource.name}).permit!"
+          Rails.logger.info "params.require(:#{effective_resource.resource_name}).permit!"
         end
 
-        if params[effective_resource.name].present?
-          params.require(effective_resource.name).permit!
+        if params[effective_resource.resource_name].present?
+          params.require(effective_resource.resource_name).permit!
         else
-          params.require((effective_resource.namespaces + [effective_resource.name]).join('_')).permit!
+          params.require((effective_resource.namespaces + [effective_resource.resource_name]).join('_')).permit!
         end
       end
 

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

@@ -64,10 +64,14 @@ module Effective
           else
             if template_present?(action)
               format.html { render(action, locals: { action: action }) }
+            elsif request.referer.to_s.end_with?('/edit')
+              format.html { render :edit }
+            elsif request.referer.to_s.end_with?('/new')
+              format.html { render :new }
             else
               format.html do
                 redirect_flash
-                redirect_to(from_path.presence || resource_redirect_path(action))
+                redirect_to(resource_redirect_path(action))
               end
             end
           end

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

@@ -11,24 +11,36 @@ module Effective
       # Effective::Resource.new('admin/posts').routes[:index]
       def routes
         @routes ||= (
-          matches = [[namespace, plural_name].compact.join('/'), [namespace, name].compact.join('/')]
+          matches = [
+            [namespace, plural_name].compact.join('/'),
+            [namespace, name].compact.join('/')
+          ]
+
+          # Check main Rails app
+          routes = Rails.application.routes.routes.select do |route|
+            (matches & [route.defaults[:controller]]).present? && !route.name.to_s.end_with?('root')
+          end
+
+          # Check engine routes
+          if routes.blank?
+            matches = [
+              [namespace, plural_name].compact.join('/'),
+              [namespace, name].compact.join('/'),
+              ['effective', namespace, plural_name].compact.join('/'),
+              ['effective', namespace, name].compact.join('/')
+            ]
+
+            (Rails::Engine.subclasses.reverse + [Rails.application]).each do |engine|
+              routes = engine.routes.routes.select do |route|
+                (matches & [route.defaults[:controller]]).present? && !route.name.to_s.end_with?('root')
+              end
 
-          routes_engine.routes.routes.select do |route|
-            matches.any? { |match| match == route.defaults[:controller] } && !route.name.to_s.end_with?('root')
-          end.inject({}) do |h, route|
-            h[route.defaults[:action].to_sym] = route; h
+              break if routes.present?
+            end
           end
-        )
-      end
 
-      # Effective::Resource.new('effective/order', namespace: :admin)
-      def routes_engine
-        case class_name
-        when 'Effective::Order'
-          EffectiveOrders::Engine
-        else
-          Rails.application
-        end
+          Array(routes).inject({}) { |h, route| h[route.defaults[:action].to_sym] = route; h }
+        )
       end
 
       # Effective::Resource.new('admin/posts').action_path_helper(:edit) => 'edit_admin_posts_path'
@@ -73,13 +85,15 @@ module Effective
                 h[part] = target.public_send(part)
               end
             end
-          else
+          elsif target.respond_to?(:to_param) || target.respond_to?(:id)
             target
+          else
+            {id: target}
           end
         end
 
         # Generate the path
-        path = routes[action].format(formattable || EMPTY_HASH)
+        path = (routes[action].format(formattable || EMPTY_HASH) rescue nil)
 
         if path.present? && opts.present?
           uri = URI.parse(path)

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

@@ -6,8 +6,40 @@ module Effective
 
       def _initialize_input(input, namespace: nil)
         @initialized_name = input
+        @model_klass = _klass_by_input(input)
 
-        @model_klass = case input
+        # Consider namespaces
+        if namespace
+          @namespaces = (namespace.kind_of?(String) ? namespace.split('/') : Array(namespace))
+        end
+
+        if input.kind_of?(Array) && @namespaces.blank?
+          @namespaces = input[0...-1].map { |input| input.to_s.presence }.compact
+        end
+
+        # Consider relation
+        if input.kind_of?(ActiveRecord::Relation)
+          @relation ||= input
+        end
+
+        if input.kind_of?(ActiveRecord::Reflection::MacroReflection) && input.scope
+          @relation ||= klass.where(nil).merge(input.scope)
+        end
+
+        # Consider instance
+        if klass && input.instance_of?(klass)
+          @instance ||= input
+        end
+
+        if klass && input.kind_of?(Array) && input.last.instance_of?(klass)
+          @instance ||= input.last
+        end
+      end
+
+      def _klass_by_input(input)
+        case input
+        when Array
+          _klass_by_input(input.last)
         when String, Symbol
           _klass_by_name(input)
         when Class
@@ -24,22 +56,6 @@ module Effective
         when nil    ; raise 'expected a string or class'
         else        ; _klass_by_name(input.class.name)
         end
-
-        if namespace
-          @namespaces = (namespace.kind_of?(String) ? namespace.split('/') : Array(namespace))
-        end
-
-        if input.kind_of?(ActiveRecord::Relation)
-          @relation = input
-        end
-
-        if input.kind_of?(ActiveRecord::Reflection::MacroReflection) && input.scope
-          @relation = klass.where(nil).merge(input.scope)
-        end
-
-        if klass && input.instance_of?(klass)
-          @instance = input
-        end
       end
 
       def _klass_by_name(input)
@@ -48,6 +64,7 @@ module Effective
 
         names = input.split('/')
 
+        # Crazy classify
         0.upto(names.length-1) do |index|
           class_name = names[index..-1].map { |name| name.classify } * '::'
           klass = class_name.safe_constantize
@@ -64,35 +81,19 @@ module Effective
           end
         end
 
-        nil
-      end
-
-      # Lazy initialized
-      def _initialize_written
-        @written_attributes = []
-        @written_belong_tos = []
-        @written_scopes = []
-
-        return unless File.exists?(model_file)
-
-        Effective::CodeReader.new(model_file) do |reader|
-          first = reader.index { |line| line == '# Attributes' }
-          last = reader.index(from: first) { |line| line.start_with?('#') == false && line.length > 0 } if first
-
-          if first && last
-            @written_attributes = reader.select(from: first+1, to: last-1).map do |line|
-              Effective::Attribute.parse_written(line).presence
-            end.compact
-          end
-
-          @written_belong_tos = reader.select { |line| line.start_with?('belongs_to ') }.map do |line|
-            line.scan(/belongs_to\s+:(\w+)/).flatten.first
-          end
+        # Crazy engine
+        if names[0] == 'admin'
+          class_name = (['effective'] + names[1..-1]).map { |name| name.classify } * '::'
+          klass = class_name.safe_constantize
 
-          @written_scopes = reader.select { |line| line.start_with?('scope ') }.map do |line|
-            line.scan(/scope\s+:(\w+)/).flatten.first
+          if klass.present?
+            @namespaces ||= names[0...-1]
+            @model_klass = klass
+            return klass
           end
         end
+
+        nil
       end
 
     end

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

@@ -9,8 +9,12 @@ module Effective
       def datatable_klass
         if defined?(EffectiveDatatables)
           "#{namespaced_class_name.pluralize}Datatable".safe_constantize ||
+          "#{namespaced_module_name.pluralize}Datatable".safe_constantize ||
+          "#{namespaced_class_name.pluralize.gsub('::', '')}Datatable".safe_constantize ||
           "#{class_name.pluralize.camelize}Datatable".safe_constantize ||
+          "#{class_name.pluralize.camelize.gsub('::', '')}Datatable".safe_constantize ||
           "#{name.pluralize.camelize}Datatable".safe_constantize ||
+          "#{name.pluralize.camelize.gsub('::', '')}Datatable".safe_constantize ||
           "Effective::Datatables::#{namespaced_class_name.pluralize}".safe_constantize ||
           "Effective::Datatables::#{class_name.pluralize.camelize}".safe_constantize ||
           "Effective::Datatables::#{name.pluralize.camelize}".safe_constantize

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

@@ -11,6 +11,10 @@ module Effective
         name.pluralize
       end
 
+      def resource_name # 'effective_post' used by permitted params
+        @resource_name ||= ((klass.present? ? klass.name : initialized_name).to_s.split(SPLIT).join('_') || '').singularize.underscore
+      end
+
       def initialized_name
         @initialized_name
       end
@@ -27,6 +31,10 @@ module Effective
         (Array(namespaces).map { |name| name.to_s.classify } + [class_name]) * '::'
       end
 
+      def namespaced_module_name # 'Admin::EffectivePosts'
+        Array(namespaces).map { |name| name.to_s.classify }.join('::') + '::' + class_name.gsub('::', '')
+      end
+
       def namespace # 'admin/things'
         (namespaces.join('/') if namespaces.present?)
       end

data/lib/effective_resources.rb

@@ -1,4 +1,5 @@
 require 'effective_resources/engine'
+require 'effective_resources/version'
 
 module EffectiveResources
 

data/lib/effective_resources/version.rb

@@ -1,3 +1,3 @@
 module EffectiveResources
-  VERSION = '1.5.4'.freeze
+  VERSION = '1.6.3'.freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: effective_resources
 version: !ruby/object:Gem::Version
-  version: 1.5.4
+  version: 1.6.3
 platform: ruby
 authors:
 - Code and Effect
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-10-07 00:00:00.000000000 Z
+date: 2020-11-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -24,6 +24,76 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: 4.0.0
+- !ruby/object:Gem::Dependency
+  name: sqlite3
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: effective_bootstrap
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: effective_datatables
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: haml
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: pry-byebug
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 description: Make any controller an effective resource controller.
 email:
 - info@codeandeffect.com
@@ -113,7 +183,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.4
+rubygems_version: 3.1.2
 signing_key:
 specification_version: 4
 summary: Make any controller an effective resource controller.