avo 1.18.2 → 1.19.0

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.base_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)
@@ -256,10 +266,6 @@ module Avo
       view_types
     end
 
-    def route_key
-      model_class.model_name.route_key
-    end
-
     def context
       self.class.context
     end
@@ -358,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

data/lib/avo/configuration.rb

@@ -26,6 +26,7 @@ module Avo
     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"
@@ -64,6 +65,7 @@ module Avo
       @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/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/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.2"
+  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*/