avo 1.18.0 → 1.19.0

data/lib/avo/app.rb

@@ -8,6 +8,7 @@ module Avo
     class_attribute :license, default: nil
     class_attribute :current_user, default: nil
     class_attribute :root_path, default: nil
+    class_attribute :view_context, default: nil
 
     class << self
       def boot
@@ -22,17 +23,22 @@ module Avo
         end
       end
 
-      def init(request:, context:, current_user:, root_path:)
+      def init(request:, context:, current_user:, root_path:, view_context:)
         self.request = request
         self.context = context
         self.current_user = current_user
         self.root_path = root_path
+        self.view_context = view_context
 
         self.license = Licensing::LicenseManager.new(Licensing::HQ.new(request).response).license
 
         # Set the current host for ActiveStorage
         begin
-          ActiveStorage::Current.url_options = request.base_url
+          if Rails::VERSION::MAJOR === 6
+            ActiveStorage::Current.host = request.base_url
+          elsif Rails::VERSION::MAJOR === 7
+            ActiveStorage::Current.url_options = request.base_url
+          end
         rescue => exception
           Rails.logger.debug "[Avo] Failed to set ActiveStorage::Current.url_options, #{exception.inspect}"
         end
@@ -121,8 +127,8 @@ module Avo
 
       def get_available_resources(user = nil)
         resources.select do |resource|
-            Services::AuthorizationService.authorize user, resource.model_class, Avo.configuration.authorization_methods.stringify_keys["index"], raise_exception: false
-          end
+          Services::AuthorizationService.authorize user, resource.model_class, Avo.configuration.authorization_methods.stringify_keys["index"], raise_exception: false
+        end
           .sort_by { |r| r.name }
       end
 
@@ -145,7 +151,7 @@ module Avo
             # remove the leading underscore (_)
             filename[0] = ""
             # remove the extension
-            filename.gsub!('.html.erb', '')
+            filename.gsub!(".html.erb", "")
             filename
           end
       end
@@ -163,13 +169,7 @@ module Avo
               resource.is_a? Class
             end
             .map do |resource|
-              route_key = if resource.model_class.present?
-                resource.model_class.model_name.plural.to_sym
-              else
-                resource.to_s.underscore.gsub("_resource", "").downcase.pluralize.to_sym
-              end
-
-              resources route_key
+              resources resource.new.model_key
             end
         end
       end

data/lib/avo/base_action.rb

@@ -14,6 +14,7 @@ module Avo
     class_attribute :resource
     class_attribute :fields
     class_attribute :standalone, default: false
+    class_attribute :visible
 
     attr_accessor :response
     attr_accessor :model
@@ -27,10 +28,11 @@ module Avo
       self.class.to_s.demodulize.underscore.humanize(keep_id_suffix: true)
     end
 
-    def initialize(model: nil, resource: nil, user: nil)
+    def initialize(model: nil, resource: nil, user: nil, view: nil)
       self.class.model = model if model.present?
       self.class.resource = resource if resource.present?
       self.class.user = user if user.present?
+      self.class.view = view if view.present?
 
       self.class.message ||= I18n.t("avo.are_you_sure_you_want_to_run_this_option")
       self.class.confirm_button_label ||= I18n.t("avo.run")
@@ -96,6 +98,12 @@ module Avo
       self
     end
 
+    def visible_in_view
+      return true unless visible.present?
+
+      instance_exec(resource: self.class.resource, view: view, &visible)
+    end
+
     def param_id
       self.class.to_s.demodulize.underscore.tr "/", "_"
     end

data/lib/avo/base_resource.rb

@@ -4,6 +4,14 @@ module Avo
     extend FieldsCollector
     extend HasContext
 
+    include ActionView::Helpers::UrlHelper
+
+    delegate :view_context, to: 'Avo::App'
+    delegate :main_app, to: :view_context
+    delegate :avo, to: :view_context
+    delegate :resource_path, to: :view_context
+    delegate :resources_path, to: :view_context
+
     attr_accessor :view
     attr_accessor :model
     attr_accessor :user
@@ -74,7 +82,9 @@ module Avo
     end
 
     def initialize
-      self.class.model_class = model_class
+      unless self.class.model_class.present?
+        self.class.model_class = model_class.base_class
+      end
     end
 
     def hydrate(model: nil, view: nil, user: nil, params: nil)
@@ -198,10 +208,13 @@ module Avo
     end
 
     def model_class
+      # get the model class off of the static property
       return self.class.model_class if self.class.model_class.present?
 
-      return @model.class if @model.present?
+      # get the model class off of the model
+      return @model.base_class if @model.present?
 
+      # generate a model class
       class_name_without_resource.safe_constantize
     end
 
@@ -253,10 +266,6 @@ module Avo
       view_types
     end
 
-    def route_key
-      model_class.model_name.route_key
-    end
-
     def context
       self.class.context
     end
@@ -355,8 +364,28 @@ module Avo
       end
     end
 
-    def avo_path
-      "#{Avo::App.root_path}/resources/#{model_class.model_name.route_key}/#{model.id}"
+    def route_key
+      model_class.model_name.route_key
+    end
+
+    # This is used as the model class ID
+    # We use this instead of the route_key to maintain compatibility with uncountable models
+    # With uncountable models route key appends an _index suffix (Fish->fish_index)
+    # Example: User->users, MediaItem->medie_items, Fish->fish
+    def model_key
+      model_class.model_name.plural
+    end
+
+    def singular_model_key
+      model_class.model_name.singular
+    end
+
+    def record_path
+      resource_path(model: model, resource: self)
+    end
+
+    def records_path
+      resources_path(resource: self)
     end
 
     def label_field
@@ -410,5 +439,9 @@ module Avo
     rescue
       nil
     end
+
+    def form_scope
+      model.class.base_class.to_s.downcase
+    end
   end
 end

data/lib/avo/configuration.rb

@@ -25,6 +25,8 @@ module Avo
     attr_accessor :search_debounce
     attr_accessor :view_component_path
     attr_accessor :display_license_request_timeout_error
+    attr_accessor :current_user_resource_name
+    attr_accessor :raise_error_on_missing_policy
 
     def initialize
       @root_path = "/avo"
@@ -62,6 +64,8 @@ module Avo
       @search_debounce = 300
       @view_component_path = "app/components"
       @display_license_request_timeout_error = true
+      @current_user_resource_name = "user"
+      @raise_error_on_missing_policy = false
     end
 
     def locale_tag

data/lib/avo/fields/base_field.rb

@@ -3,8 +3,14 @@ module Avo
     class BaseField
       extend ActiveSupport::DescendantsTracker
       extend Avo::Fields::FieldExtensions::HasFieldName
+
+      include ActionView::Helpers::UrlHelper
       include Avo::Fields::FieldExtensions::VisibleInDifferentViews
 
+      delegate :view_context, to: 'Avo::App'
+      delegate :main_app, to: :view_context
+      delegate :avo, to: :view_context
+
       attr_reader :id
       attr_reader :block
       attr_reader :required
@@ -121,11 +127,11 @@ module Avo
 
         # Run callback block if present
         if computable && block.present?
-          final_value = block.call @model, @resource, @view, self
+          final_value = instance_exec(@model, @resource, @view, self, &block)
         end
 
         # Run the value through resolver if present
-        final_value = @format_using.call final_value if @format_using.present?
+        final_value = instance_exec(final_value, &@format_using) if @format_using.present?
 
         final_value
       end

data/lib/avo/fields/files_field.rb

@@ -25,8 +25,6 @@ module Avo
         value.each do |file|
           # Skip empty values
           next unless file.present?
-          # Keep only String values
-          next unless file.class === String
 
           model.send(key).attach file
         end

data/lib/avo/fields/has_and_belongs_to_many_field.rb

@@ -1,6 +1,6 @@
 module Avo
   module Fields
-    class HasAndBelongsToManyField < BaseField
+    class HasAndBelongsToManyField < HasBaseField
       def initialize(id, **args, &block)
         args[:updatable] = false
 
@@ -13,24 +13,6 @@ module Avo
       def view_component_name
         "HasManyField"
       end
-
-      def turbo_frame
-        "#{self.class.name.demodulize.to_s.underscore}_#{id}"
-      end
-
-      def frame_url
-        "#{Avo::App.root_path}/resources/#{@model.model_name.route_key}/#{@model.id}/#{id}?turbo_frame=#{turbo_frame}"
-      end
-
-      def target_resource
-        if @model._reflections[id.to_s].klass.present?
-          Avo::App.get_resource_by_model_name @model._reflections[id.to_s].klass.to_s
-        elsif @model._reflections[id.to_s].options[:class_name].present?
-          Avo::App.get_resource_by_model_name @model._reflections[id.to_s].options[:class_name]
-        else
-          Avo::App.get_resource_by_name id.to_s
-        end
-      end
     end
   end
 end

data/lib/avo/fields/has_base_field.rb

@@ -0,0 +1,35 @@
+module Avo
+  module Fields
+    class HasBaseField < BaseField
+      attr_accessor :display
+
+      def initialize(id, **args, &block)
+        super(id, **args, &block)
+
+        @display = args[:display].present? ? args[:display] : :show
+      end
+
+      def resource
+        Avo::App.get_resource_by_model_name @model.class
+      end
+
+      def turbo_frame
+        "#{self.class.name.demodulize.to_s.underscore}_#{display}_#{id}"
+      end
+
+      def frame_url
+        "#{@resource.record_path}/#{id}?turbo_frame=#{turbo_frame}"
+      end
+
+      def target_resource
+        if @model._reflections[id.to_s].klass.present?
+          Avo::App.get_resource_by_model_name @model._reflections[id.to_s].klass.to_s
+        elsif @model._reflections[id.to_s].options[:class_name].present?
+          Avo::App.get_resource_by_model_name @model._reflections[id.to_s].options[:class_name]
+        else
+          Avo::App.get_resource_by_name id.to_s
+        end
+      end
+    end
+  end
+end

data/lib/avo/fields/has_many_field.rb

@@ -1,6 +1,6 @@
 module Avo
   module Fields
-    class HasManyField < BaseField
+    class HasManyField < HasBaseField
       def initialize(id, **args, &block)
         args[:updatable] = false
 
@@ -9,24 +9,6 @@ module Avo
         hide_on :all
         show_on :show
       end
-
-      def turbo_frame
-        "#{self.class.name.demodulize.to_s.underscore}_#{id}"
-      end
-
-      def frame_url
-        "#{Avo::App.root_path}/resources/#{@model.model_name.route_key}/#{@model.id}/#{id}?turbo_frame=#{turbo_frame}"
-      end
-
-      def target_resource
-        if @model._reflections[id.to_s].klass.present?
-          Avo::App.get_resource_by_model_name @model._reflections[id.to_s].klass.to_s
-        elsif @model._reflections[id.to_s].options[:class_name].present?
-          Avo::App.get_resource_by_model_name @model._reflections[id.to_s].options[:class_name]
-        else
-          Avo::App.get_resource_by_name id.to_s
-        end
-      end
     end
   end
 end

data/lib/avo/fields/has_one_field.rb

@@ -1,8 +1,7 @@
 module Avo
   module Fields
-    class HasOneField < BaseField
+    class HasOneField < HasBaseField
       attr_accessor :relation_method
-      attr_accessor :display
 
       def initialize(id, **args, &block)
         super(id, **args, &block)
@@ -12,7 +11,6 @@ module Avo
         @placeholder ||= I18n.t "avo.choose_an_option"
 
         @relation_method = name.to_s.parameterize.underscore
-        @display = args[:display].present? ? args[:display] : :show
       end
 
       def label
@@ -20,25 +18,11 @@ module Avo
       end
 
       def resource
-        target_resource
-      end
-
-      def turbo_frame
-        "#{self.class.name.demodulize.to_s.underscore}_#{display}_#{id}"
+        Avo::App.get_resource_by_model_name @model.class
       end
 
       def frame_url
-        "#{Avo::App.root_path}/resources/#{@model.model_name.route_key}/#{@model.id}/#{id}/#{value.id}?turbo_frame=#{turbo_frame}"
-      end
-
-      def target_resource
-        if @model._reflections[id.to_s].klass.present?
-          Avo::App.get_resource_by_model_name @model._reflections[id.to_s].klass.to_s
-        elsif @model._reflections[id.to_s].options[:class_name].present?
-          Avo::App.get_resource_by_model_name @model._reflections[id.to_s].options[:class_name]
-        else
-          Avo::App.get_resource_by_name id.to_s
-        end
+        "#{@resource.record_path}/#{id}/#{value.id}?turbo_frame=#{turbo_frame}"
       end
 
       def fill_field(model, key, value, params)

data/lib/avo/fields/text_field.rb

@@ -2,11 +2,13 @@ module Avo
   module Fields
     class TextField < BaseField
       attr_reader :link_to_resource
+      attr_reader :as_html
 
       def initialize(id, **args, &block)
         super(id, **args, &block)
 
         @link_to_resource = args[:link_to_resource].present? ? args[:link_to_resource] : false
+        @as_html = args[:as_html].present? ? args[:as_html] : false
       end
     end
   end

data/lib/avo/services/authorization_service.rb

@@ -48,8 +48,10 @@ module Avo
             end
 
             true
-          rescue Pundit::NotDefinedError
-            false
+          rescue Pundit::NotDefinedError => e
+            return false unless Avo.configuration.raise_error_on_missing_policy
+
+            raise e
           rescue => error
             if args[:raise_exception] == false
               false
@@ -73,8 +75,10 @@ module Avo
 
           begin
             Pundit.policy_scope! user, model
-          rescue
-            model
+          rescue Pundit::NotDefinedError => e
+            return model unless Avo.configuration.raise_error_on_missing_policy
+
+            raise e
           end
         end
 

data/lib/avo/version.rb

@@ -1,3 +1,3 @@
 module Avo
-  VERSION = "1.18.0"
+  VERSION = "1.19.0"
 end

data/lib/generators/avo/templates/locales/avo.en.yml

@@ -84,3 +84,5 @@ en:
       placeholder: 'Search'
       cancel_button: 'Cancel'
     sign_out: 'Sign out'
+    failed: 'Failed'
+    failed_to_load: 'Failed to load'

data/public/avo-packs/css/{application-f9191617.css → application-c3b50b28.css}

@@ -3258,6 +3258,43 @@ input[type=file][data-direct-upload-url][disabled] {
   display: none;
 }
 
+.button-spinner {
+  width: 24px;
+  height: 24px;
+
+  position: relative;
+  margin: 100px auto;
+}
+
+.button-spinner > .double-bounce1,
+.button-spinner > .double-bounce2 {
+  width: 100%;
+  height: 100%;
+  border-radius: 50%;
+  background-color: #333;
+  opacity: 0.5;
+  position: absolute;
+  top: 0;
+  left: 0;
+  animation: sk-bounce 2s infinite ease-in-out;
+}
+
+.button-spinner > .double-bounce2 {
+  animation-delay: -1s;
+}
+
+@keyframes sk-bounce {
+  0%,
+  100% {
+    transform: scale(0);
+    -webkit-transform: scale(0);
+  }
+  50% {
+    transform: scale(1);
+    -webkit-transform: scale(1);
+  }
+}
+
 .spinner {
   width: 16px;
   height: 16px;
@@ -3265,8 +3302,8 @@ input[type=file][data-direct-upload-url][disabled] {
   position: relative;
 }
 
-
-.double-bounce1, .double-bounce2 {
+.spinner .double-bounce1,
+.spinner .double-bounce2 {
   width: 100%;
   height: 100%;
   border-radius: 50%;
@@ -3275,20 +3312,22 @@ input[type=file][data-direct-upload-url][disabled] {
   position: absolute;
   top: 0;
   left: 0;
-  animation: sk-bounce 2.0s infinite ease-in-out;
+  animation: sk-bounce 2s infinite ease-in-out;
 }
 
-.double-bounce2 {
-  animation-delay: -1.0s;
+.spinner .double-bounce2 {
+  animation-delay: -1s;
 }
 
 @keyframes sk-bounce {
-  0%, 100% {
-    transform: scale(0.0);
-    -webkit-transform: scale(0.0);
-  } 50% {
-    transform: scale(1.0);
-    -webkit-transform: scale(1.0);
+  0%,
+  100% {
+    transform: scale(0);
+    -webkit-transform: scale(0);
+  }
+  50% {
+    transform: scale(1);
+    -webkit-transform: scale(1);
   }
 }
 
@@ -4997,6 +5036,9 @@ progress[value]::-moz-progress-bar {
 .h-8 {
     height: 2rem
 }
+.h-40 {
+    height: 10rem
+}
 .h-full {
     height: 100%
 }
@@ -6284,4 +6326,4 @@ html, body{
 }
 
 
-/*# sourceMappingURL=application-f9191617.css.map*/
+/*# sourceMappingURL=application-c3b50b28.css.map*/