brick 1.0.117 → 1.0.118

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2ed704f637bf6db73f5856be2959ba3490eb721e6de052008914d12cd029a673
-  data.tar.gz: 327a383fc224aebce1c96ba96b02c8a8a8604136bf61355b13b0029c203ff5c2
+  metadata.gz: bc284b8783ca27862ff567079ba680a238690e3779a57d705ee67a54ed016ab2
+  data.tar.gz: 812044a88cff33dbf4fe7dbfa37a80e91b1b1a94980091590f4665a123529e80
 SHA512:
-  metadata.gz: eec983fb53ff9e5f010473e7a7f1b4f1b6d07f4b8d166f4dccb66b701565a61fb91ebf30c84145dffd585e83ce13bb86c7746906b2751bbb6702ee2dc09eb7a4
-  data.tar.gz: 339edb3796933184d06de44def59b3ad5fd7d1c37995a73ffab4d62e7bb5dac0d8b63c28cb387e951e9899524ca8513fa938848449460aa234d901c28532f6c5
+  metadata.gz: 9a5f82a5956b40beeaa8aeca2861672561ea552b3b21cb85ebf9b426a31bdbd4025f4c047131ae962cdefb31378c8845c9bea947a50dfc34cf497cd9b68ae6b9
+  data.tar.gz: 83e1c1399045dc67d6c5dcd98347d2c84a7dec4ac474b5cc09da43b86ce21859d01204844be75d6764a5653c7ea1c220f3887835d07f703c789a82b6311bfad0

data/lib/brick/extensions.rb

@@ -279,7 +279,7 @@ module ActiveRecord
     end
 
     def self.brick_import_template
-      template = constants.include?(:IMPORT_TEMPLATE) ? self::IMPORT_TEMPLATE : suggest_template(false, false, 0)
+      template = constants.include?(:IMPORT_TEMPLATE) ? self::IMPORT_TEMPLATE : suggest_template(0, false, true)
       # Add the primary key to the template as being unique (unless it's already there)
       template[:uniques] = [pk = primary_key.to_sym]
       template[:all].unshift(pk) unless template[:all].include?(pk)
@@ -1034,18 +1034,19 @@ Module.class_exec do
 
              # AVO Resource
              elsif base_module == Object && Object.const_defined?('Avo') && requested.end_with?('Resource') &&
-                   ['MotorResource'].exclude?(requested) # Expect that anything called MotorResource could be from that administrative gem
-               if (model = Object.const_get(requested[0..-9]))
+                   # Expect that anything called MotorResource or SpinaResource could be from those administrative gems
+                   requested.length > 8 && ['MotorResource', 'SpinaResource'].exclude?(requested)
+               if (model = Object.const_get(requested[0..-9])) && model < ActiveRecord::Base
                  require 'generators/avo/resource_generator'
                  field_generator = Generators::Avo::ResourceGenerator.new([''])
                  field_generator.instance_variable_set(:@model, model)
-                 fields = field_generator.send(:generate_fields).split("\n")
-                                         .each_with_object([]) do |f, s|
+                 fields = field_generator.send(:generate_fields)&.split("\n")
+                                         &.each_with_object([]) do |f, s|
                                            if (f = f.strip).start_with?('field ')
                                              f = f[6..-1].split(',')
                                              s << [f.first[1..-1].to_sym, [f[1][1..-1].split(': :').map(&:to_sym)].to_h]
                                            end
-                                         end
+                                         end || []
                  built_resource = Class.new(Avo::BaseResource) do |new_resource_class|
                    self.model_class = model
                    self.title = :brick_descrip
@@ -2385,11 +2386,11 @@ ORDER BY 1, 2, c.internal_column_id, acc.position"
         "NOT IN ('information_schema', 'pg_catalog', 'pg_toast', 'heroku_ext',
                  'INFORMATION_SCHEMA', 'sys')"
         :
-        "= '#{ActiveRecord::Base.connection.current_database.tr("'", "''")}'"}#{"
+        "= '#{ActiveRecord::Base.connection.current_database&.tr("'", "''")}'"}#{"
       AND t.table_schema = COALESCE(current_setting('SEARCH_PATH'), 'public')" if is_postgres && schema }
   --          AND t.table_type IN ('VIEW') -- 'BASE TABLE', 'FOREIGN TABLE'
       AND t.table_name NOT IN ('pg_stat_statements', ?, ?)
-    ORDER BY 1, t.table_type DESC, 2, kcu.ordinal_position"
+    ORDER BY 1, t.table_type DESC, 2, c.ordinal_position"
     ActiveRecord::Base.execute_sql(sql, *ar_tables)
   end
 

data/lib/brick/frameworks/rails/engine.rb

@@ -76,6 +76,11 @@ function linkSchemas() {
         [... ev.target.getElementsByTagName(\"SELECT\")].forEach(function (select) {
           if (select.value === \"^^^brick_NULL^^^\") select.value = null;
         });
+        // Take outer <div> tag off the HTML being returned by any Trix editor
+        [... document.getElementsByTagName(\"TRIX-EDITOR\")].forEach(function (trix) {
+          var trixHidden = trix.inputElement;
+          if (trixHidden) trixHidden.value = trixHidden.value.slice(5, -6);
+        });
         return true;
       });
     });
@@ -215,16 +220,16 @@ function linkSchemas() {
             TurboFrameWrapperComponent.class_exec do
               alias _brick_content content
               def content
-                # Avo's logo partial fails if there is not a URL helper called exactly "root_path"
-                # (Finicky line over there is:  avo/app/views/avo/partials/_logo.html.erb:1)
                 if ::Brick.instance_variable_get(:@_brick_avo_js) == view_renderer.object_id
                   _brick_content
                 else
                   ::Brick.instance_variable_set(:@_brick_avo_js, view_renderer.object_id)
+                  # Avo's logo partial fails if there is not a URL helper called exactly "root_path"
+                  # (Finicky line over there is:  avo/app/views/avo/partials/_logo.html.erb:1)
                   unless ::Rails.application.routes.named_routes.names.include?(:root) || ActionView::Base.respond_to?(:root_path)
                     ActionView::Base.class_exec do
                       def root_path
-                        Avo::App.root_path
+                        Avo.configuration.root_path
                       end
                     end
                   end
@@ -420,6 +425,32 @@ window.addEventListener(\"popstate\", linkSchemas);
           end
         end
 
+        # Unconfigured Mobility gem?
+        if Object.const_defined?('Mobility') && Mobility.respond_to?(:translations_class)
+          # Find the current defaults
+          defs = if Mobility.instance_variable_defined?(:@translations_class)
+                   ::Mobility.translations_class.defaults
+                 else
+                   {}
+                 end
+          # Fill in the blanks for any missing defaults
+          ::Mobility.configure do |config|
+            config.plugins do
+              # Default initializer would also set these:
+              #  :backend_reader=>true
+              #  :query=>:i18n
+              #  :cache=>true
+              #  :presence=>true
+              backend :key_value, type: :string unless defs.key?(:backend)
+              reader unless defs.key?(:reader)
+              writer unless defs.key?(:writer)
+              active_record unless ::Mobility::Plugins.instance_variable_get(:@plugins)&.key?(:active_record)
+              fallbacks false unless defs.key?(:fallbacks)
+              default nil unless defs.key?(:default)
+            end
+          end
+        end
+
         # ====================================
         # Dynamically create generic templates
         # ====================================
@@ -765,6 +796,7 @@ input+svg.revert {
 
 <% is_includes_dates = nil
    is_includes_json = nil
+   is_includes_text = nil
 def is_bcrypt?(val)
   val.is_a?(String) && val.length == 60 && val.start_with?('$2a$')
 end
@@ -1569,32 +1601,36 @@ end
     <td>
     <table><tr><td>
     <% dt_pickers = { datetime: 'datetimepicker', timestamp: 'datetimepicker', time: 'timepicker', date: 'datepicker' }
-    html_options = {}
-    html_options[:class] = 'dimmed' unless val
-    is_revert = true
-    if bt
-      html_options[:prompt] = \"Select #\{bt_name\}\" %>
-      <%= f.select k.to_sym, bt[3], { value: val || '^^^brick_NULL^^^' }, html_options %>
-      <%= if (bt_obj = bt_class&.find_by(bt_pair[1] => val))
-            link_to('⇛', send(\"#\{bt_class.base_class._brick_index(:singular)\}_path\".to_sym, bt_obj.send(bt_class.primary_key.to_sym)), { class: 'show-arrow' })
-          elsif val
-            \"<span class=\\\"orphan\\\">Orphaned ID: #\{val}</span>\".html_safe
-          end %>
+       html_options = {}
+       html_options[:class] = 'dimmed' unless val
+       is_revert = true
+       if bt
+         html_options[:prompt] = \"Select #\{bt_name\}\" %>
+     <%= f.select k.to_sym, bt[3], { value: val || '^^^brick_NULL^^^' }, html_options %>
+     <%= if (bt_obj = bt_class&.find_by(bt_pair[1] => val))
+           link_to('⇛', send(\"#\{bt_class.base_class._brick_index(:singular)\}_path\".to_sym, bt_obj.send(bt_class.primary_key.to_sym)), { class: 'show-arrow' })
+         elsif val
+           \"<span class=\\\"orphan\\\">Orphaned ID: #\{val}</span>\".html_safe
+         end %>
     <% else
-      col_type = if ::Brick.config.json_columns[tbl_name]&.include?(k)
-                   :json
-                 elsif col&.sql_type == 'geography'
-                   col.sql_type
-                 else
-                   col&.type
-                 end
-      case (col_type ||= col&.sql_type)
-      when :string, :text %>
-        <% if is_bcrypt?(val) # || .readonly?
+         col_type = if ::Brick.config.json_columns[tbl_name]&.include?(k)
+                     :json
+                   elsif col&.sql_type == 'geography'
+                     col.sql_type
+                   else
+                     col&.type
+                   end
+         case (col_type ||= col&.sql_type)
+         when :string, :text
+           if is_bcrypt?(val) # || .readonly?
              is_revert = false %>
           <%= hide_bcrypt(val, nil, 1000) %>
-        <% else %>
+        <% elsif col_type == :string %>
           <%= f.text_field(k.to_sym, html_options) %>
+        <% else
+             is_includes_text = true %>
+          <%= f.hidden_field(k.to_sym, html_options) %>
+          <trix-editor input=\"<%= f.field_id(k) %>\"></trix-editor>
         <% end %>
       <% when :boolean %>
         <%= f.check_box k.to_sym %>
@@ -1739,6 +1775,11 @@ flatpickr(\".timepicker\", {enableTime: true, noCalendar: true});
 <link rel=\"stylesheet\" type=\"text/css\" href=\"https://cdnjs.cloudflare.com/ajax/libs/slim-select/1.27.1/slimselect.min.css\">
 <% end %>
 
+<% if is_includes_text %>
+<script src=\"https://cdn.jsdelivr.net/npm/trix@2.0/dist/trix.umd.min.js\"></script>
+<link rel=\"stylesheet\" type=\"text/css\" href=\"https://cdn.jsdelivr.net/npm/trix@2.0/dist/trix.min.css\">
+<% end %>
+
 <% # Started with v0.14.4 of vanilla-jsoneditor
    if is_includes_json %>
 <link rel=\"stylesheet\" type=\"text/css\" href=\"https://cdn.jsdelivr.net/npm/vanilla-jsoneditor/themes/jse-theme-default.min.css\">

data/lib/brick/version_number.rb

@@ -5,7 +5,7 @@ module Brick
   module VERSION
     MAJOR = 1
     MINOR = 0
-    TINY = 117
+    TINY = 118
 
     # PRE is nil unless it's a pre-release (beta, RC, etc.)
     PRE = nil

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: brick
 version: !ruby/object:Gem::Version
-  version: 1.0.117
+  version: 1.0.118
 platform: ruby
 authors:
 - Lorin Thwaits
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-02-26 00:00:00.000000000 Z
+date: 2023-03-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord