avo 1.19.1.pre.3 → 1.19.1.pre.4

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,11 +68,9 @@ 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 {

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/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/views/resource_index_component.rb

@@ -47,7 +47,7 @@ class Avo::Views::ResourceIndexComponent < Avo::ResourceComponent
   # 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) && simple_relation? && !has_reflection_and_is_read_only
+    @resource.authorization.authorize_action(:new, raise_exception: false) && !has_reflection_and_is_read_only
   end
 
   def can_attach?
@@ -60,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
@@ -77,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
@@ -103,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">

data/app/controllers/avo/base_controller.rb

@@ -7,7 +7,6 @@ module Avo
     before_action :hydrate_resource
     before_action :set_model, only: [:show, :edit, :destroy, :update]
     before_action :set_model_to_fill
-    before_action :fill_model, only: [:create, :update]
     before_action :authorize_action
     before_action :reset_pagination_if_filters_changed, only: :index
     before_action :cache_applied_filters, only: :index
@@ -25,8 +24,8 @@ module Avo
         @query = @resource.class.query_scope
       end
 
-      # Remove default_scope for index view
-      if @resource.unscoped_queries_on_index
+      # Remove default_scope for index view if no parent_resource present
+      if @resource.unscoped_queries_on_index && @parent_resource.blank?
         @query = @query.unscoped
       end
 
@@ -112,17 +111,47 @@ module Avo
     end
 
     def create
-      # model gets instantiated and filled in the fill_model before_action
+      # model gets instantiated and filled in the fill_model method
+      fill_model
       saved = @model.save
       @resource.hydrate(model: @model, view: :new, user: _current_user)
 
+      # This means that the record has been created through another parent record and we need to attach it somehow.
+      if params[:via_resource_id].present?
+        @reflection = @model._reflections[params[:via_relation]]
+        # Figure out what kind of association does the record have with the parent record
+
+        # belongs_to
+        # has_many
+        # Get the foreign key and set it to the id we received in the params
+        if @reflection.is_a?(ActiveRecord::Reflection::BelongsToReflection) || @reflection.is_a?(ActiveRecord::Reflection::HasManyReflection)
+          foreign_key = @reflection.foreign_key
+          @model.send("#{foreign_key}=", params[:via_resource_id])
+          @model.save
+        end
+
+        # has_one
+        # has_one_through
+
+        # has_many_through
+        # has_and_belongs_to_many
+        # polymorphic
+        if @reflection.is_a? ActiveRecord::Reflection::ThroughReflection
+          # find the record
+          via_resource = ::Avo::App.get_resource_by_model_name params[:via_relation_class]
+          @related_record = via_resource.model_class.find params[:via_resource_id]
+
+          @model.send(params[:via_relation]) << @related_record
+        end
+      end
+
       respond_to do |format|
         if saved
-          redirect_path = if params[:via_relation_class].present? && params[:via_resource_id].present?
+          redirect_path = resource_path(model: @model, resource: @resource)
+
+          if params[:via_relation_class].present? && params[:via_resource_id].present?
             parent_resource = ::Avo::App.get_resource_by_model_name params[:via_relation_class].safe_constantize
-            resource_path(model: params[:via_relation_class].safe_constantize, resource: parent_resource, resource_id: params[:via_resource_id])
-          else
-            resource_path(model: @model, resource: @resource)
+            redirect_path = resource_path(model: params[:via_relation_class].safe_constantize, resource: parent_resource, resource_id: params[:via_resource_id])
           end
 
           format.html { redirect_to redirect_path, notice: "#{@model.class.name} #{t("avo.was_successfully_created")}." }
@@ -136,7 +165,8 @@ module Avo
     end
 
     def update
-      # model gets instantiated and filled in the fill_model before_action
+      # model gets instantiated and filled in the fill_model method
+      fill_model
       saved = @model.save
       @resource = @resource.hydrate(model: @model, view: :edit, user: _current_user)
 

data/app/helpers/avo/application_helper.rb

@@ -1,18 +1,7 @@
 module Avo
   module ApplicationHelper
-    include ::Manifester::Helper
     include ::Pagy::Frontend
 
-    def current_webpacker_instance
-      return Avo.webpacker if Avo::IN_DEVELOPMENT
-
-      super
-    end
-
-    def current_manifester_instance
-      Avo.manifester
-    end
-
     def render_license_warnings
       render partial: "avo/sidebar/license_warnings", locals: {
         license: Avo::App.license.properties
@@ -116,7 +105,7 @@ module Avo
       options[:class] += args[:extra_class].present? ? " #{args[:extra_class]}" : ""
 
       # Create the path to the svgs directory
-      file_path = "#{Avo::Engine.root}/app/packs/svgs/#{file_name}"
+      file_path = "#{Avo::Engine.root}/app/assets/svgs/#{file_name}"
       file_path = "#{file_path}.svg" unless file_path.end_with? ".svg"
 
       # Create a cache hash

data/app/{packs/entrypoints/application.js → javascript/avo.js}

@@ -4,15 +4,14 @@ import 'core-js/stable'
 import 'regenerator-runtime/runtime'
 import * as ActiveStorage from '@rails/activestorage'
 import * as Mousetrap from 'mousetrap'
-import { Application } from 'stimulus'
 import { Turbo } from '@hotwired/turbo-rails'
-import { definitionsFromContext } from 'stimulus/webpack-helpers'
 import Rails from '@rails/ujs'
 import tippy from 'tippy.js'
 
 // Toastr alerts
-import '../js/active-storage'
-import '../js/toastr'
+import './js/active-storage'
+import './js/controllers'
+import './js/toastr'
 
 Rails.start()
 
@@ -33,15 +32,8 @@ function initTippy() {
 }
 window.initTippy = initTippy
 
-const application = Application.start()
 ActiveStorage.start()
 
-const context = require.context('./../js/controllers', true, /\.js$/)
-application.load(definitionsFromContext(context))
-
-const fieldsContext = require.context('./../js/controllers/fields', true, /\.js$/)
-application.load(definitionsFromContext(fieldsContext))
-
 document.addEventListener('turbo:load', () => {
   document.body.classList.remove('turbo-loading')
   initTippy()
@@ -58,10 +50,3 @@ document.addEventListener('turbo:submit-start', () => document.body.classList.ad
 document.addEventListener('turbo:before-cache', () => {
   document.querySelectorAll('[data-turbo-remove-before-cache]').forEach((element) => element.remove())
 })
-
-// Uncomment to copy all static images under ../images to the output folder and reference
-// them with the image_pack_tag helper in views (e.g <%= image_pack_tag 'rails.png' %>)
-// or the `imagePath` JavaScript helper below.
-//
-const images = require.context('../images', true)
-const imagePath = (name) => images(name, true)

data/app/javascript/js/application.js

@@ -0,0 +1,9 @@
+import { Application } from 'stimulus'
+
+const application = Application.start()
+
+// Configure Stimulus development experience
+application.debug = false
+window.Stimulus = application
+
+export { application }

data/app/{packs → javascript}/js/controllers/action_controller.js

@@ -1,5 +1,5 @@
 import { Controller } from 'stimulus'
-import { castBoolean } from '@/js/helpers/cast_boolean'
+import { castBoolean } from '../helpers/cast_boolean'
 
 export default class extends Controller {
   static targets = ['controllerDiv', 'resourceIds', 'form']

data/app/{packs → javascript}/js/controllers/fields/code_field_controller.js

@@ -15,9 +15,10 @@ import 'codemirror/mode/xml/xml'
 import 'codemirror/mode/yaml/yaml'
 
 import { Controller } from 'stimulus'
-import { castBoolean } from '@/js/helpers/cast_boolean'
 import CodeMirror from 'codemirror'
 
+import { castBoolean } from '../../helpers/cast_boolean'
+
 export default class extends Controller {
   static targets = ['element']
 

data/app/{packs → javascript}/js/controllers/fields/date_field_controller.js

@@ -1,6 +1,6 @@
 import { Controller } from 'stimulus'
 import { DateTime } from 'luxon'
-import { castBoolean } from '@/js/helpers/cast_boolean'
+import { castBoolean } from '../../helpers/cast_boolean'
 import flatpickr from 'flatpickr'
 
 export default class extends Controller {

data/app/{packs → javascript}/js/controllers/fields/key_value_controller.js

@@ -1,6 +1,6 @@
 /* eslint-disable max-len */
 import { Controller } from 'stimulus'
-import { castBoolean } from '@/js/helpers/cast_boolean'
+import { castBoolean } from '../../helpers/cast_boolean'
 
 export default class extends Controller {
   static targets = ['input', 'controller', 'rows']

data/app/{packs → javascript}/js/controllers/fields/trix_field_controller.js

@@ -1,6 +1,6 @@
 import 'trix'
 import { Controller } from 'stimulus'
-import { castBoolean } from '@/js/helpers/cast_boolean'
+import { castBoolean } from '../../helpers/cast_boolean'
 
 export default class extends Controller {
   static targets = ['editor', 'controller']

data/app/{packs → javascript}/js/controllers/search_controller.js

@@ -3,7 +3,7 @@ import * as Mousetrap from 'mousetrap'
 import { Controller } from 'stimulus'
 import { Turbo } from '@hotwired/turbo-rails'
 import { autocomplete } from '@algolia/autocomplete-js'
-import debouncePromise from '@/js/helpers/debounce_promise'
+import debouncePromise from '../helpers/debounce_promise'
 
 export default class extends Controller {
   static targets = ['autocomplete', 'button']