avo 2.51.0 → 3.0.0.beta1

data/lib/avo/base_action.rb

@@ -1,53 +1,57 @@
 module Avo
   class BaseAction
-    include Avo::Concerns::HasFields
-    include Avo::Concerns::HasActionStimulusControllers
+    include Avo::Concerns::HasItems
 
     class_attribute :name, default: nil
     class_attribute :message
     class_attribute :confirm_button_label
     class_attribute :cancel_button_label
     class_attribute :no_confirmation, default: false
+    class_attribute :record
+    class_attribute :view
+    class_attribute :user
+    class_attribute :resource
     class_attribute :standalone, default: false
     class_attribute :visible
     class_attribute :may_download_file, default: false
     class_attribute :turbo
 
-    attr_accessor :view
     attr_accessor :response
-    attr_accessor :model
+    attr_accessor :record
     attr_accessor :resource
     attr_accessor :user
     attr_reader :arguments
 
-    delegate :context, to: ::Avo::App
-    delegate :current_user, to: ::Avo::App
-    delegate :params, to: ::Avo::App
-    delegate :view_context, to: ::Avo::App
+    delegate :view, to: :class
+    # TODO: find a differnet way to delegate this to the uninitialized Current variable
+    delegate :context, to: Avo::Current
+    def curent_user
+      Avo::Current.user
+    end
+    delegate :params, to: Avo::Current
+    delegate :view_context, to: Avo::Current
     delegate :avo, to: :view_context
     delegate :main_app, to: :view_context
 
     class << self
-      delegate :context, to: ::Avo::App
+      delegate :context, to: ::Avo::Current
 
       def form_data_attributes
         # We can't respond with a file download from Turbo se we disable it on the form
         if may_download_file
-          {turbo: turbo || false, remote: false}
+          {turbo: turbo || false, remote: false, action_target: :form}
         else
-          {turbo: turbo, turbo_frame: :_top}.compact
+          {turbo: turbo, turbo_frame: :_top, action_target: :form}.compact
         end
       end
 
       # We can't respond with a file download from Turbo se we disable close the modal manually after a while (it's a hack, we know)
       def submit_button_data_attributes
-        attributes = { action_target: "submit" }
-
         if may_download_file
-          attributes[:action] = "click->modal#delayedClose"
+          {action: "click->modal#delayedClose"}
+        else
+          {}
         end
-
-        attributes
       end
     end
 
@@ -57,31 +61,47 @@ module Avo
       self.class.to_s.demodulize.underscore.humanize(keep_id_suffix: true)
     end
 
-    def initialize(model: nil, resource: nil, user: nil, view: nil, arguments: {})
-      @model = model
-      @resource = resource
-      @user = user
-      @view = view
+    def initialize(record: nil, resource: nil, user: nil, view: nil, arguments: {})
+      self.class.record = record
+      self.class.resource = resource
+      self.class.user = user
+      self.class.view = view
       @arguments = arguments
 
       self.class.message ||= I18n.t("avo.are_you_sure_you_want_to_run_this_option")
       self.class.confirm_button_label ||= I18n.t("avo.run")
       self.class.cancel_button_label ||= I18n.t("avo.cancel")
 
+      self.items_holder = Avo::Resources::Items::Holder.new
+      fields
+
       @response ||= {}
       @response[:messages] = []
     end
 
+    # Blank method
+    def fields
+    end
+
     def get_message
-      if self.class.message.respond_to? :call
-        Avo::Hosts::ResourceRecordHost.new(block: self.class.message, record: @model, resource: @resource).handle
-      else
-        self.class.message
-      end
+      Avo::ExecutionContext.new(target: self.class.message, record: self.class.record, resource: self.class.resource).handle
+    end
+
+    def get_attributes_for_action
+      get_fields.map do |field|
+        value = field.value || Avo::ExecutionContext.new(
+          target: field.default,
+          record: self.class.record,
+          resource: self.class.resource,
+          view: view
+        ).handle
+
+        [field.id, value]
+      end.to_h
     end
 
     def handle_action(**args)
-      models, fields, current_user, resource = args.values_at(:models, :fields, :current_user, :resource)
+      records, fields, current_user, resource = args.values_at(:records, :fields, :current_user, :resource)
       # Fetching the field definitions and not the actual fields (get_fields) because they will break if the user uses a `visible` block and adds a condition using the `params` variable. The params are different in the show method and the handle method.
       action_fields = get_field_definitions.map { |field| [field.id, field] }.to_h
 
@@ -111,7 +131,7 @@ module Avo
         resource: resource
       }
 
-      args[:models] = models unless standalone
+      args[:records] = records
 
       handle(**args)
 
@@ -128,12 +148,12 @@ module Avo
       end
 
       # Run the visible block if available
-      Avo::Hosts::VisibilityHost.new(
-        block: visible,
+      Avo::ExecutionContext.new(
+        target: visible,
         params: params,
         parent_resource: parent_resource,
-        resource: @resource,
-        view: @view,
+        resource: self.class.resource,
+        view: self.class.view,
         arguments: arguments
       ).handle
     end
@@ -148,12 +168,6 @@ module Avo
       self
     end
 
-    def fail(text)
-      Rails.logger.warn "DEPRECATION WARNING: Action fail method is deprecated in favor of error method and will be removed from Avo version 3.0.0"
-
-      error text
-    end
-
     def error(text)
       add_message text, :error
 
@@ -185,9 +199,10 @@ module Avo
       self
     end
 
-    def redirect_to(path = nil, **args, &block)
+    def redirect_to(path = nil, allow_other_host: nil, status: nil, &block)
       response[:type] = :redirect
-      response[:redirect_args] = args
+      response[:allow_other_host] = allow_other_host
+      response[:status] = status
       response[:path] = if block.present?
         block
       else
@@ -212,9 +227,9 @@ module Avo
     end
 
     # We're overriding this method to hydrate with the proper resource attribute.
-    def hydrate_fields(model: nil, view: nil)
+    def hydrate_fields
       fields.map do |field|
-        field.hydrate(model: @model, view: @view, resource: resource)
+        field.hydrate(record: @record, view: @view, resource: resource)
       end
 
       self