avo 0.5.0.beta9 → 0.5.0.beta14

data/app/components/avo/views/resource_new_component.rb

@@ -21,7 +21,8 @@ class Avo::Views::ResourceNewComponent < ViewComponent::Base
   end
 
   private
-    def via_resource?
-      params[:via_relation_class].present? and params[:via_resource_id].present?
-    end
+
+  def via_resource?
+    params[:via_relation_class].present? && params[:via_resource_id].present?
+  end
 end

data/app/components/avo/views/resource_show_component.rb

@@ -38,29 +38,29 @@ class Avo::Views::ResourceShowComponent < Avo::ResourceComponent
   end
 
   private
-    def via_resource?
-      params[:via_resource_class].present? and params[:via_resource_id].present?
-    end
 
-    def split_panel_fields
-      @fields_by_panel = {}
-      @has_one_panels = []
-      @has_many_panels = []
-      @has_as_belongs_to_many_panels = []
+  def via_resource?
+    params[:via_resource_class].present? && params[:via_resource_id].present?
+  end
 
-      @resource.get_fields.each do |field|
-        case field.class.to_s
-        when 'Avo::Fields::HasOneField'
-          @has_one_panels << field
-        when 'Avo::Fields::HasManyField'
-          @has_many_panels << field
-        when 'Avo::Fields::HasAndBelongsToManyField'
-          @has_as_belongs_to_many_panels << field
-        else
-          @fields_by_panel[field.panel_name] ||= []
-          @fields_by_panel[field.panel_name] << field
-        end
-      end
+  def split_panel_fields
+    @fields_by_panel = {}
+    @has_one_panels = []
+    @has_many_panels = []
+    @has_as_belongs_to_many_panels = []
 
+    @resource.get_fields.each do |field|
+      case field.class.to_s
+      when "Avo::Fields::HasOneField"
+        @has_one_panels << field
+      when "Avo::Fields::HasManyField"
+        @has_many_panels << field
+      when "Avo::Fields::HasAndBelongsToManyField"
+        @has_as_belongs_to_many_panels << field
+      else
+        @fields_by_panel[field.panel_name] ||= []
+        @fields_by_panel[field.panel_name] << field
+      end
     end
+  end
 end

data/app/controllers/avo/actions_controller.rb

@@ -1,4 +1,4 @@
-require_dependency 'avo/application_controller'
+require_dependency "avo/application_controller"
 
 module Avo
   class ActionsController < ApplicationController
@@ -11,11 +11,11 @@ module Avo
     end
 
     def handle
-      resource_ids = action_params[:fields][:resource_ids].split(',').map(&:to_i)
+      resource_ids = action_params[:fields][:resource_ids].split(",").map(&:to_i)
       models = @resource.model_class.find resource_ids
 
       fields = action_params[:fields].select do |key, value|
-        key != 'resource_ids'
+        key != "resource_ids"
       end
 
       performed_action = @action.handle_action(models: models, fields: fields)
@@ -24,44 +24,45 @@ module Avo
     end
 
     private
-      def action_params
-        params.permit(:resource_name, :action_id, fields: {})
-      end
 
-      def set_action
-        action_class = params[:action_id].gsub('avo_actions_', '').classify.safe_constantize
+    def action_params
+      params.permit(:resource_name, :action_id, fields: {})
+    end
 
-        if params[:id].present?
-          model = @resource.model_class.find params[:id]
-        end
+    def set_action
+      action_class = params[:action_id].gsub("avo_actions_", "").classify.safe_constantize
 
-        @action = action_class.new(model: model, resource: resource, user: _current_user)
+      if params[:id].present?
+        model = @resource.model_class.find params[:id]
       end
 
-      def respond(response)
-        response[:type] ||= :reload
-        response[:message_type] ||= :notice
-        response[:message] ||= I18n.t('avo.action_ran_successfully')
+      @action = action_class.new(model: model, resource: resource, user: _current_user)
+    end
 
-        if response[:type] == :download
-          return send_data response[:path], filename: response[:filename]
-        end
+    def respond(response)
+      response[:type] ||= :reload
+      response[:message_type] ||= :notice
+      response[:message] ||= I18n.t("avo.action_ran_successfully")
 
-        respond_to do |format|
-          format.html do
-            if response[:type] == :redirect
-              path = response[:path]
+      if response[:type] == :download
+        return send_data response[:path], filename: response[:filename]
+      end
 
-              if path.respond_to? :call
-                path = instance_eval &path
-              end
+      respond_to do |format|
+        format.html do
+          if response[:type] == :redirect
+            path = response[:path]
 
-              redirect_to path, "#{response[:message_type]}": response[:message]
-            elsif response[:type] == :reload
-              redirect_back fallback_location: resources_path(@resource.model_class), "#{response[:message_type]}": response[:message]
+            if path.respond_to? :call
+              path = instance_eval(&path)
             end
+
+            redirect_to path, "#{response[:message_type]}": response[:message]
+          elsif response[:type] == :reload
+            redirect_back fallback_location: resources_path(@resource.model_class), "#{response[:message_type]}": response[:message]
           end
         end
       end
+    end
   end
 end

data/app/controllers/avo/application_controller.rb

@@ -23,11 +23,13 @@ module Avo
     def exception_logger(exception)
       respond_to do |format|
         format.html { raise exception }
-        format.json { render json: {
-          errors: exception.respond_to?(:record) && exception.record.present? ? exception.record.errors : [],
-          message: exception.message,
-          traces: exception.backtrace,
-        }, status: ActionDispatch::ExceptionWrapper.status_code_for_exception(exception.class.name) }
+        format.json {
+          render json: {
+            errors: exception.respond_to?(:record) && exception.record.present? ? exception.record.errors : [],
+            message: exception.message,
+            traces: exception.backtrace
+          }, status: ActionDispatch::ExceptionWrapper.status_code_for_exception(exception.class.name)
+        }
       end
     end
 
@@ -48,8 +50,7 @@ module Avo
         if keep_query_params
           existing_params = Addressable::URI.parse(request.fullpath).query_values.symbolize_keys
         end
-      rescue;end
-
+      rescue; end
       send :"resources_#{model.model_name.route_key}_path", **existing_params, **args
     end
 
@@ -62,22 +63,13 @@ module Avo
         if keep_query_params
           existing_params = Addressable::URI.parse(request.fullpath).query_values.symbolize_keys
         end
-      rescue;end
-
+      rescue; end
       Addressable::Template.new("#{Avo.configuration.root_path}/resources/#{@parent_resource.model.model_name.route_key}/#{@parent_resource.model.id}/#{@resource.route_key}{?query*}")
         .expand({query: {**existing_params, **args}})
         .to_str
     end
 
     def resource_path(model = nil, resource_id: nil, keep_query_params: false, **args)
-      existing_params = {}
-
-      begin
-        if keep_query_params
-          existing_params = Addressable::URI.parse(request.fullpath).query_values.symbolize_keys
-        end
-      rescue;end
-
       return send :"resources_#{model.model_name.route_key.singularize}_path", resource_id, **args if resource_id.present?
 
       send :"resources_#{model.model_name.route_key.singularize}_path", model, **args
@@ -108,151 +100,146 @@ module Avo
     end
 
     private
-      def set_resource_name
-        @resource_name = resource_name
-      end
 
-      def set_related_resource_name
-        @related_resource_name = related_resource_name
-      end
+    def set_resource_name
+      @resource_name = resource_name
+    end
 
-      def set_resource
-        @resource = resource.hydrate(params: params)
-      end
+    def set_related_resource_name
+      @related_resource_name = related_resource_name
+    end
 
-      def set_related_resource
-        @related_resource = related_resource.hydrate(params: params)
-      end
+    def set_resource
+      @resource = resource.hydrate(params: params)
+    end
 
-      def set_model
-        @model = eager_load_files(@resource, @resource.model_class).find params[:id]
-      end
+    def set_related_resource
+      @related_resource = related_resource.hydrate(params: params)
+    end
 
-      def set_related_model
-        @related_model = eager_load_files(@related_resource, @related_resource.model_class).find params[:related_id]
-      end
+    def set_model
+      @model = eager_load_files(@resource, @resource.model_class).find params[:id]
+    end
 
-      def hydrate_resource
-        @resource.hydrate(view: action_name.to_sym, user: _current_user)
-      end
+    def set_related_model
+      @related_model = eager_load_files(@related_resource, @related_resource.model_class).find params[:related_id]
+    end
 
-      def hydrate_related_resource
-        @related_resource.hydrate(view: action_name.to_sym, user: _current_user)
-      end
+    def hydrate_resource
+      @resource.hydrate(view: action_name.to_sym, user: _current_user)
+    end
 
-      def authorize_action
-        if @model.present?
-          @authorization.set_record(@model).authorize_action :index
-        else
-          @authorization.set_record(@resource.model_class).authorize_action :index
-        end
-      end
+    def hydrate_related_resource
+      @related_resource.hydrate(view: action_name.to_sym, user: _current_user)
+    end
 
-      # Get the pluralized resource name for this request
-      # Ex: projects, teams, users
-      def resource_name
-        return params[:resource_name] if params[:resource_name].present?
+    def authorize_action
+      if @model.present?
+        @authorization.set_record(@model).authorize_action :index
+      else
+        @authorization.set_record(@resource.model_class).authorize_action :index
+      end
+    end
 
-        return controller_name if controller_name.present?
+    # Get the pluralized resource name for this request
+    # Ex: projects, teams, users
+    def resource_name
+      return params[:resource_name] if params[:resource_name].present?
 
-        begin
-          request.path
-            .match(/\/?#{Avo.configuration.root_path.gsub('/', '')}\/resources\/([a-z1-9\-_]*)\/?/mi)
-            .captures
-            .first
-        rescue => exception
-        end
-      end
+      return controller_name if controller_name.present?
 
-      def related_resource_name
-        params[:related_name]
+      begin
+        request.path
+          .match(/\/?#{Avo.configuration.root_path.delete('/')}\/resources\/([a-z1-9\-_]*)\/?/mi)
+          .captures
+          .first
+      rescue
       end
+    end
 
-      # Gets the Avo resource for this request based on the request from the `resource_name` "param"
-      # Ex: Avo::Resources::Project, Avo::Resources::Team, Avo::Resources::User
-      def resource
+    def related_resource_name
+      params[:related_name]
+    end
 
-        resource = App.get_resource @resource_name.to_s.camelize.singularize
+    # Gets the Avo resource for this request based on the request from the `resource_name` "param"
+    # Ex: Avo::Resources::Project, Avo::Resources::Team, Avo::Resources::User
+    def resource
+      resource = App.get_resource @resource_name.to_s.camelize.singularize
 
-        return resource if resource.present?
+      return resource if resource.present?
 
-        App.get_resource_by_controller_name @resource_name
-      end
+      App.get_resource_by_controller_name @resource_name
+    end
 
-      def related_resource
-        reflection = @model._reflections[params[:related_name]]
+    def related_resource
+      reflection = @model._reflections[params[:related_name]]
 
-        if reflection.is_a? ::ActiveRecord::Reflection::HasOneReflection
-          reflected_model = reflection.klass
-        else
-          reflected_model = reflection.klass
-        end
+      reflected_model = reflection.klass
 
-        App.get_resource_by_model_name reflected_model
-      end
+      App.get_resource_by_model_name reflected_model
+    end
 
-      def eager_load_files(resource, query)
-        if resource.attached_file_fields.present?
-          resource.attached_file_fields.map do |field|
-            # abort "#{field.pluralize}".inspect
-            # abort field.class.inspect
-            attachment = case field.class.to_s
-            when 'Avo::Fields::FileField'
-              'attachment'
-            when 'Avo::Fields::FilesField'
-              'attachments'
-            else
-              'attachment'
-            end
-
-            return query.eager_load "#{field.id}_#{attachment}": :blob
-            # return query.send :"with_attached_#{field}"
+    def eager_load_files(resource, query)
+      if resource.attached_file_fields.present?
+        resource.attached_file_fields.map do |field|
+          # abort "#{field.pluralize}".inspect
+          # abort field.class.inspect
+          attachment = case field.class.to_s
+          when "Avo::Fields::FileField"
+            "attachment"
+          when "Avo::Fields::FilesField"
+            "attachments"
+          else
+            "attachment"
           end
-        end
 
-        query
+          return query.eager_load "#{field.id}_#{attachment}": :blob
+          # return query.send :"with_attached_#{field}"
+        end
       end
 
+      query
+    end
 
-      # def authorize_user
-      #   return if params[:controller] == 'avo/search'
-
-      #   model = record = resource.model
+    # def authorize_user
+    #   return if params[:controller] == 'avo/search'
 
-      #   if ['show', 'edit', 'update'].include?(params[:action]) && params[:controller] == 'avo/resources'
-      #     record = resource
-      #   end
+    #   model = record = resource.model
 
-      #   # AuthorizationService::authorize_action _current_user, record, params[:action] return render_unauthorized unless
-      # end
+    #   if ['show', 'edit', 'update'].include?(params[:action]) && params[:controller] == 'avo/resources'
+    #     record = resource
+    #   end
 
-      def _authenticate!
-        instance_eval(&Avo.configuration.authenticate)
-      end
+    #   # AuthorizationService::authorize_action _current_user, record, params[:action] return render_unauthorized unless
+    # end
 
-      def render_unauthorized(exception)
-        if !exception.is_a? Pundit::NotDefinedError
-          flash[:notice] = t 'avo.not_authorized'
+    def _authenticate!
+      instance_eval(&Avo.configuration.authenticate)
+    end
 
-          redirect_url = if request.referrer.blank? or request.referrer == request.url
-            root_url
-          else
-            request.referrer
-          end
+    def render_unauthorized(exception)
+      if !exception.is_a? Pundit::NotDefinedError
+        flash[:notice] = t "avo.not_authorized"
 
-          redirect_to(redirect_url)
+        redirect_url = if request.referrer.blank? || (request.referrer == request.url)
+          root_url
+        else
+          request.referrer
         end
-      end
 
-      def set_authorization
-        @authorization = Services::AuthorizationService.new _current_user
+        redirect_to(redirect_url)
       end
+    end
 
-      def set_container_classes
-        contain = !Avo.configuration.full_width_container
-        contain = false if Avo.configuration.full_width_index_view && action_name.to_sym == :index && self.class.superclass.to_s == 'Avo::ResourcesController'
+    def set_authorization
+      @authorization = Services::AuthorizationService.new _current_user
+    end
 
-        @container_classes = contain ? '2xl:container 2xl:mx-auto' : ''
-      end
+    def set_container_classes
+      contain = !Avo.configuration.full_width_container
+      contain = false if Avo.configuration.full_width_index_view && action_name.to_sym == :index && self.class.superclass.to_s == "Avo::ResourcesController"
+
+      @container_classes = contain ? "2xl:container 2xl:mx-auto" : ""
+    end
   end
 end