effective_resources 1.5.5 → 1.6.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f9fae70e2bf9e0d17c6185c5e0431d216edb470593cae928a5c2dac1ff618546
-  data.tar.gz: 8282dbb19a27f4b8634e62413ad0ce6ef2685bac2dc2a314bf88759fe89d7920
+  metadata.gz: 2318bd8d0a93c63c8287583dd80a6b6e067bb5677f330b930d58588677d2cb55
+  data.tar.gz: 82af0a3fdbb5c27bd2498bf82017c398485b755d729674e8963796b5b72f85c9
 SHA512:
-  metadata.gz: 99b8791ca2d3c50e9e2bebfadf4d66a5ba3a26ecf5dd974baf1deca28deb1f1b571bb85b07d4df9a77cd6ce57c673ea4437d87a14fc551be8df644a0e614669d
-  data.tar.gz: 5db56888ffad740845a5258514297e9a556c19a25593fa8a4f889fe424a99872d6cc0501505ea491a5f14699335f6b3fa5e41286ab0cead752fb10f706d0e00a
+  metadata.gz: 5bd4c614b11beea8c848996a541b2e729092164543d9b69727ae346b6146783b0024c8ebeb428615c5806c87107146b10a7055800cf7ad0a7f42a80fea0f54bf
+  data.tar.gz: d942166674ede76968161547a5eef54c1f3e9db015c2878d67ebaaa930117ad53f19beeaeea5632e4d69d39282cadf848c0da5c2cf151b3079d6288d4daeadab

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
 
@@ -96,7 +96,7 @@ module Effective
 
           if (nested_params = permitted_params_for(nested.klass, namespaces)).present?
             nested_params.insert(nested_params.rindex { |obj| !obj.kind_of?(Hash)} + 1, :_destroy)
-            permitted_params << { "#{nested.name}_attributes".to_sym => nested_params }
+            permitted_params << { "#{nested.resource_name}_attributes".to_sym => nested_params }
           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.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'

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

@@ -11,6 +11,9 @@ module Effective
           "#{namespaced_class_name.pluralize}Datatable".safe_constantize ||
           "#{class_name.pluralize.camelize}Datatable".safe_constantize ||
           "#{name.pluralize.camelize}Datatable".safe_constantize ||
+          "#{namespaced_class_name.pluralize.gsub('::', '')}Datatable".safe_constantize ||
+          "#{class_name.pluralize.camelize.gsub('::', '')}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

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.5'.freeze
+  VERSION = '1.6.0'.freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: effective_resources
 version: !ruby/object:Gem::Version
-  version: 1.5.5
+  version: 1.6.0
 platform: ruby
 authors:
 - Code and Effect
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-11-17 00:00:00.000000000 Z
+date: 2020-11-20 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_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: effective_developer
+  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