avo 1.19.1.pre.1 → 1.19.1.pre.5

data/app/assets/config/avo_manifest.js

@@ -1 +1,2 @@
-//= link_directory ../stylesheets/avo .css
+//= link_directory ../images/avo
+//= link_tree ../builds

data/app/{packs/entrypoints/application.css → assets/stylesheets/avo.css}

@@ -6,25 +6,26 @@
 @import './../../../node_modules/flatpickr/dist/flatpickr.css';
 @import './../../../node_modules/@algolia/autocomplete-theme-classic/dist/theme.css';
 
-@import './../stylesheets/tailwindcss/base.css';
-
-@import './../stylesheets/fonts.css';
-@import './../stylesheets/typography.css';
-@import './../stylesheets/tooltips.css';
-@import './../stylesheets/alerts.css';
-@import './../stylesheets/loader.css';
-@import './../stylesheets/pagination.css';
-@import './../stylesheets/breadcrumbs.css';
-@import './../stylesheets/search.css';
-@import './../stylesheets/active-storage.css';
-@import './../stylesheets/spinner.css';
-
-@import './../stylesheets/components/status.css';
-@import './../stylesheets/components/code.css';
-@import './../stylesheets/components/progress.css';
-@import './../stylesheets/tailwindcss/components.css';
-
-@import './../stylesheets/tailwindcss/utilities.css';
+@import 'tailwindcss/base';
+
+@import './css/fonts.css';
+@import './css/typography.css';
+@import './css/tooltips.css';
+@import './css/alerts.css';
+@import './css/loader.css';
+@import './css/pagination.css';
+@import './css/breadcrumbs.css';
+@import './css/search.css';
+@import './css/active-storage.css';
+@import './css/spinner.css';
+
+@import './css/components/status.css';
+@import './css/components/code.css';
+@import './css/components/progress.css';
+
+@import 'tailwindcss/components';
+
+@import 'tailwindcss/utilities';
 
 html,
 body {
@@ -67,13 +68,19 @@ body {
   @apply opacity-0 translate-y-1;
 }
 
-.application-sidebar {
-  .active:hover,
-  .active {
-    @apply bg-gray-200;
-  }
+.application-sidebar .active:hover,
+.application-sidebar .active {
+  @apply bg-gray-200;
 }
 
 .turbo-progress-bar {
   @apply bg-green-500;
 }
+
+body.os-mac .mac\:hidden {
+  display: none;
+}
+
+body.os-pc .pc\:hidden {
+  display: none;
+}

data/app/assets/stylesheets/css/fonts.css

@@ -0,0 +1,26 @@
+/* nunito-regular - latin */
+@font-face {
+  font-family: 'Nunito';
+  font-style: normal;
+  font-weight: 400;
+  src: url('/avo-assets/fonts/nunito-v16-latin-regular.eot'); /* IE9 Compat Modes */
+  src: local(''),
+      url('/avo-assets/fonts/nunito-v16-latin-regular.eot?#iefix') format('embedded-opentype'), /* IE6-IE8 */
+      url('/avo-assets/fonts/nunito-v16-latin-regular.woff2') format('woff2'), /* Super Modern Browsers */
+      url('/avo-assets/fonts/nunito-v16-latin-regular.woff') format('woff'), /* Modern Browsers */
+      url('/avo-assets/fonts/nunito-v16-latin-regular.ttf') format('truetype'), /* Safari, Android, iOS */
+      url('/avo-assets/fonts/nunito-v16-latin-regular.svg#Nunito') format('svg'); /* Legacy iOS */
+}
+/* nunito-700 - latin */
+@font-face {
+  font-family: 'Nunito';
+  font-style: normal;
+  font-weight: 700;
+  src: url('/avo-assets/fonts/nunito-v16-latin-700.eot'); /* IE9 Compat Modes */
+  src: local(''),
+      url('/avo-assets/fonts/nunito-v16-latin-700.eot?#iefix') format('embedded-opentype'), /* IE6-IE8 */
+      url('/avo-assets/fonts/nunito-v16-latin-700.woff2') format('woff2'), /* Super Modern Browsers */
+      url('/avo-assets/fonts/nunito-v16-latin-700.woff') format('woff'), /* Modern Browsers */
+      url('/avo-assets/fonts/nunito-v16-latin-700.ttf') format('truetype'), /* Safari, Android, iOS */
+      url('/avo-assets/fonts/nunito-v16-latin-700.svg#Nunito') format('svg'); /* Legacy iOS */
+}

data/app/{packs/stylesheets → assets/stylesheets/css}/search.css

@@ -27,7 +27,7 @@
 
 .aa-DetachedFormContainer,
 .aa-DetachedContainer .aa-Panel {
-  @apply bg-blue-gray-100 border-b-0;
+  @apply bg-slate-100 border-b-0;
 }
 
 .aa-Form {

data/app/components/avo/edit/field_wrapper_component.html.erb

@@ -1,5 +1,5 @@
 <div class="flex items-center py-0 leading-tight <%= @classes %> min-h-16" data-field-id="<%= @field.id %>" data-field-type="<%= @field.type %>">
-  <div class="h-16 flex self-start items-center text-blue-gray-800">
+  <div class="h-16 flex self-start items-center text-slate-800">
     <div class="<% if @displayed_in_modal %> md:w-48 <% else %> md:w-64 <% end %> w-48 px-8 flex" data-slot="label">
       <%= @form.label @field.id, label %> <% if @field.required %> <span class="text-red-600">*</span> <% end %>
     </div>

data/app/components/avo/fields/common/files_list_viewer_component.html.erb

@@ -1,4 +1,4 @@
-<div class="relative p-3 bg-blue-gray-200 grid grid-cols-3 xl:grid-cols-4 gap-3 rounded-xl">
+<div class="relative p-3 bg-slate-200 grid grid-cols-3 xl:grid-cols-4 gap-3 rounded-xl">
   <% @field.value.attachments.each do |file| %>
     <%= render Avo::Fields::Common::MultipleFileViewerComponent.new id: @field.id, file: file, is_image: @field.is_image, button_size: :xs, resource: @resource %>
   <% end %>

data/app/components/avo/fields/common/heading_component.html.erb

@@ -1,4 +1,4 @@
-<div class="flex items-start py-1 leading-tight border-t bg-blue-gray-600 text-white text-xs">
+<div class="flex items-start py-1 leading-tight border-t bg-slate-600 text-white text-xs">
   <div class="p-3 pl-6 h-full w-full">
     <% if @value.present? %>
       <% if @as_html %>

data/app/components/avo/index/field_wrapper_component.html.erb

@@ -1,4 +1,4 @@
-<td class="px-4 py-2 leading-tight whitespace-nowrap h-12 text-blue-gray-800 <%= @classes %>" data-field-id="<%= @field.id %>" data-field-type="<%= @field.type %>">
+<td class="px-4 py-2 leading-tight whitespace-nowrap h-12 text-slate-800 <%= @classes %>" data-field-id="<%= @field.id %>" data-field-type="<%= @field.type %>">
   <% if @field.value.blank? && @dash_if_blank %>
     —
   <% else %>

data/app/components/avo/index/table_row_component.html.erb

@@ -1,5 +1,5 @@
 <tr
-  class="hover:bg-light-blue-50 hover:shadow-row relative z-20 border-b"
+  class="hover:bg-sky-50 hover:shadow-row relative z-20 border-b"
   <%== item_selector_init @resource %>
 >
   <td class="w-10">

data/app/components/avo/resource_component.rb

@@ -25,10 +25,6 @@ class Avo::ResourceComponent < ViewComponent::Base
 
   private
 
-  def simple_relation?
-    @reflection.is_a? ::ActiveRecord::Reflection::HasManyReflection
-  end
-
   def relation_resource
     ::Avo::App.get_resource_by_model_name params[:via_resource_class].safe_constantize
   end

data/app/components/avo/show/field_wrapper_component.html.erb

@@ -1,5 +1,5 @@
 <div class="flex items-start leading-tight <%= @classes %> min-h-16" data-field-id="<%= @field.id %>" data-field-type="<%= @field.type %>">
-  <div class="h-16 flex self-start items-center text-blue-gray-800">
+  <div class="h-16 flex self-start items-center text-slate-800">
     <div class="w-48 md:w-64 px-8" data-slot="label">
       <%= @field.name %> <% if @field.required %> <span class="text-red-600">*</span> <% end %>
     </div>

data/app/components/avo/views/resource_edit_component.html.erb

@@ -10,7 +10,7 @@
               <%= helpers.svg 'arrow-left' %> <%= t('avo.cancel').capitalize %>
             <% end %>
 
-            <% if @resource.authorization.authorize_action :update, raise_exception: false %>
+            <% if can_see_the_save_button? %>
               <%= a_button color: 'green', variant: 'outlined', spinner: true, type: :submit do %>
                 <%= helpers.svg 'save' %> <%= t('avo.save').capitalize %>
               <% end %>

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

@@ -16,6 +16,12 @@ class Avo::Views::ResourceEditComponent < Avo::ResourceComponent
     end
   end
 
+  # The save button is dependent on the edit? policy method.
+  # The update? method should be called only when the user clicks the Save button so the developer gets access to the params from the form.
+  def can_see_the_save_button?
+    @resource.authorization.authorize_action :edit, raise_exception: false
+  end
+
   private
 
   def via_resource?

data/app/components/avo/views/resource_index_component.html.erb

@@ -3,7 +3,7 @@
     <% c.tools do %>
       <%= render 'actions' if @actions.present? %>
 
-      <% if can_create? %>
+      <% if can_see_the_create_button? %>
         <%= a_link create_path do %>
           <%= svg 'plus' %> <%= t('avo.create_new_item', item: singular_resource_name.downcase ) %>
         <% end %>

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

@@ -44,8 +44,10 @@ class Avo::Views::ResourceIndexComponent < Avo::ResourceComponent
     @index_params[:available_view_types]
   end
 
-  def can_create?
-    @resource.authorization.authorize_action(:create, raise_exception: false) && simple_relation? && !has_reflection_and_is_read_only
+  # The Create button is dependent on the new? policy method.
+  # The create? should be called only when the user clicks the Save button so the developers gets access to the params from the form.
+  def can_see_the_create_button?
+    @resource.authorization.authorize_action(:new, raise_exception: false) && !has_reflection_and_is_read_only
   end
 
   def can_attach?
@@ -58,7 +60,7 @@ class Avo::Views::ResourceIndexComponent < Avo::ResourceComponent
   def has_reflection_and_is_read_only
     if @reflection.present? && @reflection.active_record.name && @reflection.name
       fields = ::Avo::App.get_resource_by_model_name(@reflection.active_record.name).get_field_definitions
-      filtered_fields = fields.filter{ |f| f.id == @reflection.name}
+      filtered_fields = fields.filter { |f| f.id == @reflection.name }
     else
       return false
     end
@@ -75,10 +77,22 @@ class Avo::Views::ResourceIndexComponent < Avo::ResourceComponent
 
     if @reflection.present?
       args = {
-        via_relation_class: @parent_model.model_name,
-        via_resource_id: @parent_model.id
+        via_relation_class: reflection_model_class,
+        via_resource_id: @parent_model.id,
       }
 
+      if @reflection.is_a? ActiveRecord::Reflection::ThroughReflection
+        args[:via_relation] = params[:resource_name]
+      end
+
+      if @reflection.is_a? ActiveRecord::Reflection::HasManyReflection
+        args[:via_relation] = @reflection.name
+      end
+
+      if @reflection.parent_reflection.present? && @reflection.parent_reflection.inverse_of.present?
+        args[:via_relation] = @reflection.parent_reflection.inverse_of.name
+      end
+
       if @reflection.inverse_of.present?
         args[:via_relation] = @reflection.inverse_of.name
       end
@@ -101,9 +115,7 @@ class Avo::Views::ResourceIndexComponent < Avo::ResourceComponent
 
   private
 
-  def simple_relation?
-    return @reflection.is_a? ::ActiveRecord::Reflection::HasManyReflection if @reflection.present?
-
-    true
+  def reflection_model_class
+    @reflection.active_record.to_s
   end
 end

data/app/components/avo/views/resource_new_component.html.erb

@@ -1,6 +1,6 @@
 <div>
   <% @resource.panels.each do |resource_panel| %>
-    <%= form_with model: @resource.model, url: helpers.resources_path(resource: @resource, via_relation_class: params[:via_relation_class], via_resource_id: params[:via_resource_id]), local: true, multipart: true do |form| %>
+    <%= form_with model: @resource.model, url: helpers.resources_path(resource: @resource, via_relation_class: params[:via_relation_class], via_relation: params[:via_relation], via_resource_id: params[:via_resource_id]), local: true, multipart: true do |form| %>
       <%= render Avo::PanelComponent.new(title: resource_panel[:name], display_breadcrumbs: true) do |c| %>
         <% c.tools do %>
           <div class="flex justify-end space-x-2">
@@ -8,9 +8,11 @@
               <%= svg 'arrow-left' %> <%= t('avo.cancel').capitalize %>
             <% end %>
 
-            <%= a_button color: 'green', spinner: true, type: :submit do %>
-              <%= svg 'save' %> <%= t('avo.save').capitalize %>
-            <% end if @resource.authorization.authorize_action :create, raise_exception: false %>
+            <% if can_see_the_save_button? %>
+              <%= a_button color: 'green', spinner: true, type: :submit do %>
+                <%= svg 'save' %> <%= t('avo.save').capitalize %>
+              <% end %>
+            <% end %>
           </div>
         <% end %>
 

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

@@ -20,6 +20,12 @@ class Avo::Views::ResourceNewComponent < ViewComponent::Base
     end
   end
 
+  # The create button is dependent on the new? policy method.
+  # The create? should be called only when the user clicks the Save button so the developers gets access to the params from the form.
+  def can_see_the_save_button?
+    @resource.authorization.authorize_action :new, raise_exception: false
+  end
+
   private
 
   def via_resource?

data/app/components/avo/views/resource_show_component.html.erb

@@ -14,7 +14,7 @@
                 <%= svg 'trash' %> <%= t('avo.detach_item', item: @reflection.name.to_s).capitalize %>
               <% end %>
             <% end %>
-            <% if @resource.authorization.authorize_action(:edit, raise_exception: false) %>
+            <% if can_see_the_edit_button? %>
               <%= a_link edit_path, color: 'indigo' do %>
                 <%= svg 'edit' %> <%= t('avo.edit').capitalize %>
               <% end %>
@@ -24,7 +24,7 @@
               <%= svg 'arrow-left' %> <%= t('avo.go_back') %>
             <% end %>
 
-            <% if @resource.authorization.authorize_action(:destroy, raise_exception: false) %>
+            <% if can_see_the_destroy_button? %>
               <%= form_with url: helpers.resource_path(model: @resource.model, resource: @resource), method: :delete, html: { 'data-turbo-frame': params[:turbo_frame] } do |form| %>
                 <%= a_button title: t('avo.delete_item', item: @resource.model.model_name.name.downcase).capitalize,
                   spinner: true,

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

@@ -43,7 +43,15 @@ class Avo::Views::ResourceShowComponent < Avo::ResourceComponent
   end
 
   def can_detach?
-    authorize_association_for('detach')
+    authorize_association_for("detach")
+  end
+
+  def can_see_the_edit_button?
+    @resource.authorization.authorize_action(:edit, raise_exception: false)
+  end
+
+  def can_see_the_destroy_button?
+    @resource.authorization.authorize_action(:destroy, raise_exception: false)
   end
 
   private

data/app/controllers/avo/application_controller.rb

@@ -12,6 +12,8 @@ module Avo
     before_action :_authenticate!
     before_action :set_container_classes
     before_action :add_initial_breadcrumbs
+    before_action :set_view
+    before_action :set_model_to_fill
 
     rescue_from Pundit::NotAuthorizedError, with: :render_unauthorized
     rescue_from ActiveRecord::RecordInvalid, with: :exception_logger
@@ -99,6 +101,19 @@ module Avo
       @related_model = eager_load_files(@related_resource, @related_resource.class.find_scope).find params[:related_id]
     end
 
+    def set_view
+      @view = action_name.to_sym
+    end
+
+    def set_model_to_fill
+      @model_to_fill = @resource.model_class.new if @view == :create
+      @model_to_fill = @model if @view == :update
+    end
+
+    def fill_model
+      @model = @resource.fill_model(@model_to_fill, cast_nullable(model_params))
+    end
+
     def hydrate_resource
       @resource.hydrate(view: action_name.to_sym, user: _current_user)
     end