avo 2.45.0 → 3.0.0.beta1

data/lib/avo/concerns/visible_in_different_views.rb

@@ -0,0 +1,126 @@
+module Avo
+  module Concerns
+    module VisibleInDifferentViews
+      attr_accessor :show_on_index
+      attr_accessor :show_on_show
+      attr_accessor :show_on_new
+      attr_accessor :show_on_edit
+      attr_accessor :show_on_preview
+
+      def post_initialize
+        initialize_views
+      end
+
+      def initialize_views
+        # Set defaults
+        @show_on_index = @show_on_index.nil? ? true : @show_on_index
+        @show_on_show = @show_on_show.nil? ? true : @show_on_show
+        @show_on_new = @show_on_new.nil? ? true : @show_on_new
+        @show_on_edit = @show_on_edit.nil? ? true : @show_on_edit
+        @show_on_preview = @show_on_preview.nil? ? false : @show_on_preview
+
+        if @args.present?
+          # Execute options
+          show_on @args[:show_on] if @args[:show_on].present?
+          hide_on @args[:hide_on] if @args[:hide_on].present?
+          only_on @args[:only_on] if @args[:only_on].present?
+          except_on @args[:except_on] if @args[:except_on].present?
+        end
+      end
+
+      # Validates if the field is visible on certain view
+      def visible_in_view?(view:)
+        raise "No view specified on visibility check." if view.blank?
+
+        send "show_on_#{view}"
+      end
+
+      def show_on(*where)
+        return show_on_all if where.include? :all
+
+        normalize_views(where).flatten.each do |view|
+          show_on_view view
+        end
+      end
+
+      def hide_on(*where)
+        return hide_on_all if where.include? :all
+
+        normalize_views(where).flatten.each do |view|
+          hide_on_view view
+        end
+      end
+
+      def only_on(*where)
+        hide_on_all
+        normalize_views(where).flatten.each do |view|
+          show_on_view view
+        end
+      end
+
+      def except_on(*where)
+        show_on_all
+        normalize_views(where).flatten.each do |view|
+          show_on_view view
+        end
+      end
+
+      # When submitting the form on creation, the new page will be create but we don't have a visibility marker for create so we'll default to new
+      def show_on_create
+        show_on_new
+      end
+
+      # When submitting the form on update, the new page will be create but we don't have a visibility marker for update so we'll default to edit
+      def show_on_update
+        show_on_edit
+      end
+
+      private
+
+      def show_on_view(view)
+        send("show_on_#{view}=", true)
+      end
+
+      def hide_on_view(view)
+        send("show_on_#{view}=", false)
+      end
+
+      def only_on_view(view)
+        hide_on_all
+        show_on_view view
+      end
+
+      def except_on_view(view)
+        show_on_all
+        hide_on_view view
+      end
+
+      def show_on_all
+        @show_on_index = true
+        @show_on_show = true
+        @show_on_edit = true
+        @show_on_new = true
+        @show_on_preview = true
+      end
+
+      def hide_on_all
+        @show_on_index = false
+        @show_on_show = false
+        @show_on_edit = false
+        @show_on_new = false
+        @show_on_preview = false
+      end
+
+      def normalize_views(*views_and_groups)
+        forms = views_and_groups.flatten! & [:forms]
+
+        if forms.present?
+          views_and_groups -= forms
+          views_and_groups += [:new, :edit]
+        end
+
+        views_and_groups.flatten.uniq
+      end
+    end
+  end
+end

data/lib/avo/configuration/branding.rb

@@ -32,40 +32,28 @@ class Avo::Configuration::Branding
   end
 
   def logo
-    return @default_logo if Avo::App.license.lacks_with_trial(:branding)
-
     @logo || @default_logo
   end
 
   def logomark
-    return @default_logomark if Avo::App.license.lacks_with_trial(:branding)
-
     @logomark || @default_logomark
   end
 
   def placeholder
-    return @default_placeholder if Avo::App.license.lacks_with_trial(:branding)
-
     @placeholder || @default_placeholder
   end
 
   def chart_colors
-    return @default_chart_colors if Avo::App.license.lacks_with_trial(:branding)
-
     @chart_colors || @default_chart_colors
   end
 
   def favicon
-    return @default_favicon if Avo::App.license.lacks_with_trial(:branding)
-
     @favicon || @default_favicon
   end
 
   private
 
   def colors
-    return @default_colors if Avo::App.license.lacks_with_trial(:branding)
-
     @default_colors.merge(@colors) || @default_colors
   end
 

data/lib/avo/configuration.rb

@@ -5,7 +5,6 @@ module Avo
     attr_writer :app_name
     attr_writer :branding
     attr_writer :root_path
-    attr_writer :cache_store
     attr_accessor :timezone
     attr_accessor :per_page
     attr_accessor :per_page_steps
@@ -38,7 +37,6 @@ module Avo
     attr_accessor :main_menu
     attr_accessor :profile_menu
     attr_accessor :model_resource_mapping
-    attr_accessor :tabs_style
     attr_accessor :resource_default_view
     attr_accessor :authorization_client
     attr_accessor :field_wrapper_layout
@@ -91,13 +89,10 @@ module Avo
       @main_menu = nil
       @profile_menu = nil
       @model_resource_mapping = {}
-      @tabs_style = :tabs
       @resource_default_view = :show
       @authorization_client = :pundit
       @field_wrapper_layout = :inline
       @resources = nil
-      @prefix_path = nil
-      @cache_store = computed_cache_store
     end
 
     def current_user_method(&block)
@@ -143,38 +138,7 @@ module Avo
     end
 
     def app_name
-      if @app_name.respond_to? :call
-        Avo::Hosts::BaseHost.new(block: @app_name).handle
-      else
-        @app_name
-      end
-    end
-
-    def cache_store
-      Avo::ExecutionContext.new(
-        target: @cache_store,
-        production_rejected_cache_stores: %w[ActiveSupport::Cache::MemoryStore ActiveSupport::Cache::NullStore]
-      ).handle
-    end
-
-    # When not in production or test we'll just use the MemoryStore which is good enough.
-    # When running in production we'll use Rails.cache if it's not ActiveSupport::Cache::MemoryStore or ActiveSupport::Cache::NullStore.
-    # If it's one of rejected cache stores, we'll use the FileStore.
-    # We decided against the MemoryStore in production because it will not be shared between multiple processes (when using Puma).
-    def computed_cache_store
-      -> {
-        if Rails.env.production?
-          if Rails.cache.class.to_s.in?(production_rejected_cache_stores)
-            ActiveSupport::Cache.lookup_store(:file_store, Rails.root.join("tmp", "cache"))
-          else
-            Rails.cache
-          end
-        elsif Rails.env.test?
-          Rails.cache
-        else
-          ActiveSupport::Cache.lookup_store(:memory_store)
-        end
-      }
+      Avo::ExecutionContext.new(target: @app_name).handle
     end
   end
 

data/lib/avo/current.rb

@@ -0,0 +1,39 @@
+class Avo::Current < ActiveSupport::CurrentAttributes
+  # if Rails.env.development?
+  # singleton_class.attr_accessor :previous_attributes
+  # before_reset {
+  #   puts ["before_reset->", self.previous_attributes].inspect
+  #   if attributes.present?
+  #     puts ["has attributes->"].inspect
+  #     self.previous_attributes = attributes
+  #   end
+  #   puts ["before_reset->", self.previous_attributes].inspect
+  # }
+
+  # attr_accessor :previous_attributes
+
+  # def previous_attributes=(value)
+  #   @previous_attributes = value
+  # end
+  # end
+
+  attribute :app
+  attribute :license
+  attribute :context, :user, :view_context
+  attribute :error_manager
+  attribute :resource_manager
+  attribute :tool_manager
+  attribute :plugin_manager
+
+  delegate :params, to: :request
+
+  def request
+    view_context.request || ActionDispatch::Request.empty
+  end
+
+  def current_user
+    Rails.logger.warn "DEPRECATION WARNING: Avo::Current.current_user become deprecated and will become obsolete on futhure versions. Please use Avo::Current.user instead."
+
+    user
+  end
+end

data/lib/avo/dsl/field_parser.rb

@@ -72,7 +72,7 @@ module Avo
       end
 
       def field_class_from_symbol(symbol)
-        matched_field = Avo::App.fields.find do |field|
+        matched_field = Avo.field_manager.all.find do |field|
           field[:name].to_s == symbol.to_s
         end
 

data/lib/avo/dynamic_router.rb

@@ -1,15 +1,26 @@
 module Avo
   class DynamicRouter
+    def self.eager_load(entity)
+      paths = Avo::ENTITIES.fetch entity
+
+      return unless paths.present?
+
+      pathname = Rails.root.join(*paths)
+      if pathname.directory?
+        Rails.autoloaders.main.eager_load_dir(pathname.to_s)
+      end
+    end
+
     def self.routes
       Avo::Engine.routes.draw do
         scope "resources", as: "resources" do
           # Check if the user chose to manually register the resource files.
           # If so, eager_load the resources dir.
           if Avo.configuration.resources.nil?
-            Avo::App.eager_load(:resources) unless Rails.application.config.eager_load
+            Avo::DynamicRouter.eager_load(:resources) unless Rails.application.config.eager_load
           end
 
-          Avo::App.fetch_resources
+          Avo::Resources::ResourceManager.fetch_resources
             .select do |resource|
               resource != :BaseResource
             end
@@ -17,7 +28,11 @@ module Avo
               resource.is_a? Class
             end
             .map do |resource|
-              resources resource.new.route_key
+              resources resource.route_key do
+                member do
+                  get :preview
+                end
+              end
             end
         end
       end

data/lib/avo/engine.rb

@@ -3,6 +3,8 @@ Gem.loaded_specs["avo"].dependencies.each do |d|
   case d.name
   when "activerecord"
     require "active_record/railtie"
+  when "activesupport"
+    require "active_support/railtie"
   when "actionview"
     require "action_view/railtie"
   when "activestorage"
@@ -19,23 +21,40 @@ module Avo
     isolate_namespace Avo
 
     config.after_initialize do
-      # Boot Avo
-      ::Avo::App.boot
-    end
+      # Reset before reloads in development
+      ::Avo.asset_manager.reset
 
-    initializer "avo.autoload" do |app|
-      Avo::ENTITIES.values.each do |path_params|
-        path = Rails.root.join(*path_params)
+      # Boot Avo
+      ::Avo.boot
 
-        if File.directory? path.to_s
-          Rails.autoloaders.main.push_dir path.to_s
+      # After deploy we want to make sure the license response is being cleared.
+      # We need a fresh license response.
+      # This is disabled in development because the initialization process might be triggered more than once.
+      unless Rails.env.development?
+        begin
+          Licensing::HQ.new.clear_response
+        rescue => exception
+          Avo.logger.info "Failed to clear Avo HQ response: #{exception.message}"
         end
       end
     end
 
-    initializer "avo.init_fields" do |app|
-      # Init the fields
-      ::Avo::App.init_fields
+    # Ensure we reboot the app when something changes
+    config.to_prepare do
+      # Boot Avo
+      ::Avo.boot
+    end
+
+    initializer "avo.autoload" do |app|
+      # This undoes Rails' previous nested directories behavior in the `app` dir.
+      # More on this: https://github.com/fxn/zeitwerk/issues/250
+      avo_directory = Rails.root.join("app", "avo").to_s
+      ActiveSupport::Dependencies.autoload_paths.delete(avo_directory)
+
+      if Dir.exist?(avo_directory)
+        Rails.autoloaders.main.push_dir(avo_directory, namespace: Avo)
+        app.config.watchable_dirs[avo_directory] = [:rb]
+      end
     end
 
     initializer "avo.reloader" do |app|
@@ -46,17 +65,16 @@ module Avo
       end
     end
 
-    initializer "debug_exception_response_format" do |app|
-      app.config.debug_exception_response_format = :api
-      # app.config.logger = ::Logger.new(STDOUT)
-    end
-
     initializer "avo.test_buddy" do |app|
       if Avo::IN_DEVELOPMENT
-        Rails.autoloaders.main.push_dir Avo::Engine.root.join("spec", "helpers")
+        Rails.autoloaders.main.push_dir Avo::Engine.root.join("spec", "testing_helpers")
       end
     end
 
+    initializer "debug_exception_response_format" do |app|
+      app.config.debug_exception_response_format = :api
+    end
+
     config.app_middleware.use(
       Rack::Static,
       urls: ["/avo-assets"],
@@ -75,18 +93,5 @@ module Avo
     initializer "avo.locales" do |app|
       I18n.load_path += Dir[Avo::Engine.root.join("lib", "generators", "avo", "templates", "locales", "*.{rb,yml}")]
     end
-
-    # After deploy we want to make sure the license response is being cleared.
-    # We need a fresh license response.
-    # This is disabled in development because the initialization process might be triggered more than once.
-    config.after_initialize do
-      unless Rails.env.development?
-        begin
-          Licensing::HQ.new.clear_response
-        rescue => exception
-          puts "Failed to clear Avo HQ response: #{exception.message}"
-        end
-      end
-    end
   end
 end

data/lib/avo/error_manager.rb

@@ -0,0 +1,25 @@
+module Avo
+  class ErrorManager
+    class << self
+      def build
+        new
+      end
+    end
+
+    attr_reader :errors
+
+    alias_method :all, :errors
+
+    def initialize
+      @errors = []
+    end
+
+    def add(error)
+      errors << error
+    end
+
+    def has_errors?
+      errors.present?
+    end
+  end
+end

data/lib/avo/execution_context.rb

@@ -10,13 +10,6 @@ module Avo
         end
       end
 
-      # If you want this block to behave like a view you can delegate the missing methods to the view_context
-      #
-      # Ex: Avo::ExecutionContext.new(target: ..., delegate_missing_to: :view_context).handle
-      if args[:delegate_missing_to].present?
-        self.class.send(:delegate_missing_to, args[:delegate_missing_to])
-      end
-
       # If target doesn't respond to call, we don't need to initialize the others attr_accessors.
       return unless (@target = args[:target]).respond_to? :call
 
@@ -26,15 +19,18 @@ module Avo
       end
 
       # Set defaults on not initialized accessors
-      @context ||= Avo::App.context
-      @params ||= Avo::App.params
-      @view_context ||= Avo::App.view_context
-      @current_user ||= Avo::App.current_user
-      @request ||= view_context&.request
-      @main_app ||= view_context&.main_app
-      @avo ||= view_context&.avo
+      @context ||= Avo::Current.context
+      @current_user ||= Avo::Current.user
+      @params ||= Avo::Current.params
+      @request ||= Avo::Current.request
+      @view_context ||= Avo::Current.view_context
+      @main_app ||= @view_context.main_app
+      @avo ||= @view_context.avo
     end
 
+    delegate :result, to: :card
+    delegate :authorize, to: Avo::Services::AuthorizationService
+
     # Return target if target is not callable, otherwise, execute target on this instance context
     def handle
       target.respond_to?(:call) ? instance_exec(&target) : target

data/lib/avo/fields/badge_field.rb

@@ -8,7 +8,7 @@ module Avo
 
         hide_on [:edit, :new]
 
-        default_options = {info: :info, success: :success, danger: :danger, warning: :warning, neutral: :neutral}
+        default_options = {info: :info, success: :success, danger: :danger, warning: :warning}
         @options = args[:options].present? ? default_options.merge(args[:options]) : default_options
       end
     end