brick 1.0.204 → 1.0.205

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9bf1dd73cdb0040884c92622e0a7dab4c36232440b5f70e74c2f9f362843373d
-  data.tar.gz: 19fd7bbfd6088ed1606b6c2a5ad2fa67cb9dd3b6217313bdfaa368b4fc1ca5f0
+  metadata.gz: 3abdf25aeb401676febcc1ea98ce104a9e005ecae652f17daf3736baab63d5ea
+  data.tar.gz: 195eb8e27607f719808954914a409b356c249384cacd0535fab65652b02cd4ec
 SHA512:
-  metadata.gz: eb16cd17c3ad18718164242957e71353ccb5708650136a44517a13255b65d40dc47d6b579defd92d76b9b6a77c696a386880db70f6c2d1495a0f198f64866dbc
-  data.tar.gz: 97e22cd5f88a8008313362558ccb5ab7829772568573f0622e2eba61640d5a59ae44135d4870878f90bd750d496ed8aeef0e22badffe4168b142580911650b11
+  metadata.gz: 187b1740a3f0e54f061c9b293df233c32993eb00ca4a218e1b32b7888c8b72c4fb9d2889ec7a2a469f95d3b39242b9adc8c02167c76065a5dcfd45262714793c
+  data.tar.gz: 132fecbea3bcf8783a0ad2ca540747af4a74e882d0fdba60bf0491c0ee19e52e9ef378d6a39ab3df89cbac3bcd6dee760959f969f45e17587b6a0b643f9e1bb3

data/lib/brick/config.rb

@@ -234,7 +234,7 @@ module Brick
 
     # Polymorphic associations
     def polymorphics
-      @mutex.synchronize { @polymorphics }
+      @mutex.synchronize { @polymorphics ||= {} }
     end
 
     def polymorphics=(polys)
@@ -344,7 +344,7 @@ module Brick
     end
 
     def table_name_prefixes
-      @mutex.synchronize { @table_name_prefixes }
+      @mutex.synchronize { @table_name_prefixes ||= {} }
     end
 
     def table_name_prefixes=(value)
@@ -384,7 +384,7 @@ module Brick
     end
 
     def metadata_columns
-      @mutex.synchronize { @metadata_columns }
+      @mutex.synchronize { @metadata_columns ||= ['created_at', 'updated_at', 'deleted_at'] }
     end
 
     def metadata_columns=(columns)

data/lib/brick/extensions.rb

@@ -1175,34 +1175,6 @@ Might want to add this in your brick.rb:
   end
 end
 
-if ::Brick.enable_routes? && Object.const_defined?('ActionDispatch')
-  require 'brick/route_mapper'
-  ActionDispatch::Routing::RouteSet.class_exec do
-    # In order to defer auto-creation of any routes that already exist, calculate Brick routes only after having loaded all others
-    prepend ::Brick::RouteSet
-  end
-  ActionDispatch::Routing::Mapper.class_exec do
-    include ::Brick::RouteMapper
-  end
-
-  # Do the root route before the Rails Welcome one would otherwise take precedence
-  if (route = ::Brick.config.default_route_fallback).present?
-    action = "#{route}#{'#index' unless route.index('#')}"
-    if ::Brick.config.path_prefix
-      ::Rails.application.routes.append do
-        send(:namespace, ::Brick.config.path_prefix) do
-          send(:root, action)
-        end
-      end
-    elsif ::Rails.application.routes.named_routes.send(:routes)[:root].nil?
-      ::Rails.application.routes.append do
-        send(:root, action)
-      end
-    end
-    ::Brick.established_drf = "/#{::Brick.config.path_prefix}#{action[action.index('#')..-1]}"
-  end
-end
-
 if Object.const_defined?('ActionView')
   require 'brick/frameworks/rails/form_tags'
   require 'brick/frameworks/rails/form_builder'
@@ -1800,6 +1772,18 @@ class Object
             new_model_class.send(:has_many, hmt_name.to_sym, **options)
           end
         end
+
+        # Auto-support Ransack if it's present
+        if self.respond_to?(:ransackable_attributes)
+          def self.ransackable_attributes(auth_object = nil)
+            column_names + _ransackers.keys
+          end
+
+          def self.ransackable_associations(auth_object = nil)
+            reflect_on_all_associations.map { |a| a.name.to_s } + _ransackers.keys
+          end
+        end
+
         code << "end # model #{full_name}\n"
       end # model class definition
       [built_model, code]

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

@@ -209,51 +209,95 @@ function linkSchemas() {
       # paths['app/models'] << 'lib/brick/frameworks/active_record/models'
       config.brick = ActiveSupport::OrderedOptions.new
       ActiveSupport.on_load(:before_initialize) do |app|
-        if ::Rails.application.respond_to?(:reloader)
-          ::Rails.application.reloader.to_prepare { Module.class_exec &::Brick::ADD_CONST_MISSING }
-        else # For Rails < 5.0, just load it once at the start
-          Module.class_exec &::Brick::ADD_CONST_MISSING
+
+        # --------------------------------------------
+        # 1. Load three initializers early
+        #    (inflectsions.rb, brick.rb, apartment.rb)
+        # Very first thing, load inflections since we'll be using .pluralize and .singularize on table and model names
+        if File.exist?(inflections = ::Rails.root&.join('config/initializers/inflections.rb') || '')
+          load inflections
         end
         require 'brick/join_array'
-        is_development = (ENV['RAILS_ENV'] || ENV['RACK_ENV'])  == 'development'
-        ::Brick.enable_models = app.config.brick.fetch(:enable_models, true)
-        ::Brick.enable_controllers = app.config.brick.fetch(:enable_controllers, is_development)
-        ::Brick.enable_views = app.config.brick.fetch(:enable_views, is_development)
-        ::Brick.enable_routes = app.config.brick.fetch(:enable_routes, is_development)
-        ::Brick.skip_database_views = app.config.brick.fetch(:skip_database_views, false)
-
-        # Specific database tables and views to omit when auto-creating models
-        ::Brick.exclude_tables = app.config.brick.fetch(:exclude_tables, [])
-
-        # When table names have specific prefixes, automatically place them in their own module with a table_name_prefix.
-        ::Brick.config.table_name_prefixes ||= app.config.brick.fetch(:table_name_prefixes, {})
-
-        # Columns to treat as being metadata for purposes of identifying associative tables for has_many :through
-        ::Brick.metadata_columns = app.config.brick.fetch(:metadata_columns, ['created_at', 'updated_at', 'deleted_at'])
-
-        # Columns for which to add a validate presence: true even though the database doesn't have them marked as NOT NULL
-        ::Brick.not_nullables = app.config.brick.fetch(:not_nullables, [])
-
-        # Additional references (virtual foreign keys)
-        ::Brick.additional_references = app.config.brick.fetch(:additional_references, nil)
-
-        # Custom columns to add to a table, minimally defined with a name and DSL string
-        ::Brick.custom_columns = app.config.brick.fetch(:custom_columns, nil)
-
-        # When table names have specific prefixes, automatically place them in their own module with a table_name_prefix.
-        ::Brick.order = app.config.brick.fetch(:order, {})
+        # Now the Brick initializer since there may be important schema things configured
+        if !::Brick.initializer_loaded && File.exist?(brick_initializer = ::Rails.root&.join('config/initializers/brick.rb') || '')
+          ::Brick.initializer_loaded = load brick_initializer
+
+          # After loading the initializer, add compatibility for ActiveStorage and ActionText if those haven't already been
+          # defined.  (Further JSON configuration for ActiveStorage metadata happens later in the after_initialize hook.)
+          # begin
+            ['ActiveStorage', 'ActionText'].each do |ar_extension|
+              if Object.const_defined?(ar_extension) &&
+                 (extension = Object.const_get(ar_extension)).respond_to?(:table_name_prefix) &&
+                 !::Brick.config.table_name_prefixes.key?(as_tnp = extension.table_name_prefix)
+                ::Brick.config.table_name_prefixes[as_tnp] = ar_extension
+              end
+            end
+          # rescue # NoMethodError
+          # end
 
-        # Skip creating a has_many association for these
-        ::Brick.exclude_hms = app.config.brick.fetch(:exclude_hms, nil)
+          # Support the followability gem:  https://github.com/nejdetkadir/followability
+          if Object.const_defined?('Followability') && !::Brick.config.table_name_prefixes.key?('followability_')
+            ::Brick.config.table_name_prefixes['followability_'] = 'Followability'
+          end
+        end
+        # Load the initializer for the Apartment gem a little early so that if .excluded_models and
+        # .default_schema are specified then we can work with non-tenanted models more appropriately
+        if (apartment = Object.const_defined?('Apartment')) &&
+           File.exist?(apartment_initializer = ::Rails.root.join('config/initializers/apartment.rb'))
+          require 'apartment/adapters/abstract_adapter'
+          Apartment::Adapters::AbstractAdapter.class_exec do
+            if instance_methods.include?(:process_excluded_models)
+              def process_excluded_models
+                # All other models will share a connection (at Apartment.connection_class) and we can modify at will
+                Apartment.excluded_models.each do |excluded_model|
+                  begin
+                    process_excluded_model(excluded_model)
+                  rescue NameError => e
+                    (@bad_models ||= []) << excluded_model
+                  end
+                end
+              end
+            end
+          end
+          unless @_apartment_loaded
+            load apartment_initializer
+            @_apartment_loaded = true
+          end
+        end
 
-        # Has one relationships
-        ::Brick.has_ones = app.config.brick.fetch(:has_ones, nil)
+        if ::Brick.enable_routes? && Object.const_defined?('ActionDispatch')
+          require 'brick/route_mapper'
+          ActionDispatch::Routing::RouteSet.class_exec do
+            # In order to defer auto-creation of any routes that already exist, calculate Brick routes only after having loaded all others
+            prepend ::Brick::RouteSet
+          end
+          ActionDispatch::Routing::Mapper.class_exec do
+            include ::Brick::RouteMapper
+          end
 
-        # accepts_nested_attributes_for relationships
-        ::Brick.nested_attributes = app.config.brick.fetch(:nested_attributes, nil)
+          # Do the root route before the Rails Welcome one would otherwise take precedence
+          if (route = ::Brick.config.default_route_fallback).present?
+            action = "#{route}#{'#index' unless route.index('#')}"
+            if ::Brick.config.path_prefix
+              ::Rails.application.routes.append do
+                send(:namespace, ::Brick.config.path_prefix) do
+                  send(:root, action)
+                end
+              end
+            elsif ::Rails.application.routes.named_routes.send(:routes)[:root].nil?
+              ::Rails.application.routes.append do
+                send(:root, action)
+              end
+            end
+            ::Brick.established_drf = "/#{::Brick.config.path_prefix}#{action[action.index('#')..-1]}"
+          end
+        end
 
-        # Polymorphic associations
-        ::Brick.polymorphics = app.config.brick.fetch(:polymorphics, nil)
+        if ::Rails.application.respond_to?(:reloader)
+          ::Rails.application.reloader.to_prepare { Module.class_exec &::Brick::ADD_CONST_MISSING }
+        else # For Rails < 5.0, just load it once at the start
+          Module.class_exec &::Brick::ADD_CONST_MISSING
+        end
       end
 
       # After we're initialized and before running the rest of stuff, put our configuration in place
@@ -497,20 +541,43 @@ window.addEventListener(\"popstate\", linkSchemas);
               end
             end
           end
-          ::ActiveAdmin::Views::TitleBar.class_exec do
-            alias _brick_build_title_tag build_title_tag
-            def build_title_tag
-              if klass = begin
-                           aa_id = helpers.instance_variable_get(:@current_tab)&.id
-                           ::Brick.relations.fetch(aa_id, nil)&.fetch(:class_name, nil)&.constantize
-                         rescue
-                         end
-                h2((@title + link_to_brick(nil,
-                  BRICK_SVG.html_safe, # This would do well to be sized a bit smaller
-                  { title: "#{@_name} in Brick" }
-                )).html_safe)
-              else
-                _brick_build_title_tag # Revert to the original
+          if (aav = ::ActiveAdmin::Views).const_defined?('TitleBar') # ActiveAdmin < 4.0
+            aav::TitleBar.class_exec do
+              alias _brick_build_title_tag build_title_tag
+              def build_title_tag
+                if klass = begin
+                             aa_id = helpers.instance_variable_get(:@current_tab)&.id
+                             ::Brick.relations.fetch(aa_id, nil)&.fetch(:class_name, nil)&.constantize
+                           rescue
+                           end
+                  h2((@title + link_to_brick(nil,
+                    BRICK_SVG.html_safe, # This would do well to be sized a bit smaller
+                    { title: "#{@_name} in Brick" }
+                  )).html_safe)
+                else
+                  _brick_build_title_tag # Revert to the original
+                end
+              end
+            end
+          else # ActiveAdmin 4.0 or later
+            is_aa_4x = true
+            ::ActiveAdmin::ResourceController.class_exec do
+              include ActionView::Helpers::UrlHelper
+              include ::Brick::Rails::FormTags
+              alias _brick_default_page_title default_page_title
+              def default_page_title
+                dpt = _brick_default_page_title
+                if klass = begin
+                             ::Brick.relations.fetch(@current_menu_item&.id, nil)&.fetch(:class_name, nil)&.constantize
+                           rescue
+                           end
+                  (dpt + link_to_brick(nil,
+                    BRICK_SVG.html_safe,
+                    { title: "#{dpt} in Brick" }
+                  )).html_safe
+                else
+                  dpt
+                end
               end
             end
           end
@@ -523,9 +590,8 @@ window.addEventListener(\"popstate\", linkSchemas);
                 div class: "blank_slate_container", id: "dashboard_default_message" do
                   span class: "blank_slate" do
                     span I18n.t("active_admin.dashboard_welcome.welcome")
-                    small I18n.t("active_admin.dashboard_welcome.call_to_action")
                   end
-                end
+                end unless is_aa_4x
               end
             end
           end
@@ -1402,7 +1468,7 @@ end %>#{"
   end
   \"<tr><td colspan=\\\"#\{td_count}\\\">Children: #\{child_links.join(' ')}</tr>\".html_safe
 end
-%><%= if (page_num = @#{res_name = table_name.pluralize}._brick_page_num)
+%><%= if (page_num = @#{res_name = table_name.pluralize}&._brick_page_num)
            \"<tr><td colspan=\\\"#\{td_count}\\\">Page #\{page_num}</td></tr>\".html_safe
          end %></table>#{template_link}<%
    if description.present? %><span class=\"__brick\"><%=
@@ -1790,11 +1856,12 @@ flatpickr(\".timepicker\", {enableTime: true, noCalendar: true});
 <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
+<% # Good up until v0.19.0, and then with v0.20.0 of vanilla-jsoneditor started to get:
+   # Uncaught TypeError: Failed to resolve module specifier \"immutable-json-patch\". Relative references must start with either \"/\", \"./\", or \"../\".
    if @_json_fields_present %>
-<link rel=\"stylesheet\" type=\"text/css\" href=\"https://cdn.jsdelivr.net/npm/vanilla-jsoneditor/themes/jse-theme-default.min.css\">
+<link rel=\"stylesheet\" type=\"text/css\" href=\"https://cdn.jsdelivr.net/npm/vanilla-jsoneditor@0.19.0/themes/jse-theme-default.min.css\">
 <script type=\"module\">
-  import { JSONEditor } from \"https://cdn.jsdelivr.net/npm/vanilla-jsoneditor/index.min.js\";
+  import { JSONEditor } from \"https://cdn.jsdelivr.net/npm/vanilla-jsoneditor@0.19.0/index.min.js\";
   document.querySelectorAll(\"input.jsonpicker\").forEach(function (inp) {
     var jsonDiv;
     if (jsonDiv = document.getElementById(\"_br_json_\" + inp.id)) {

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

@@ -69,8 +69,8 @@ module Brick::Rails::FormTags
 
       if show_aa_button != false && Object.const_defined?('ActiveAdmin')
         ActiveAdmin.application.namespaces.names.each do |ns|
-        out << "
-          <td>#{link_to_brick(
+          out << "
+        <td>#{link_to_brick(
             ::Brick::Rails::AA_PNG.html_safe,
             { index_proc: Proc.new do |aa_model, relation|
                             path_helper = "#{ns}_#{relation.fetch(:auto_prefixed_schema, nil)}#{aa_model.model_name.route_key}_path".to_sym

data/lib/brick/reflect_tables.rb

@@ -31,61 +31,8 @@ module Brick
 
       orig_schema = nil
       if (relations = ::Brick.relations).keys == [:db_name]
-        ::Brick.remove_instance_variable(:@_additional_references_loaded) if ::Brick.instance_variable_defined?(:@_additional_references_loaded)
+        # ::Brick.remove_instance_variable(:@_additional_references_loaded) if ::Brick.instance_variable_defined?(:@_additional_references_loaded)
 
-        # --------------------------------------------
-        # 1. Load three initializers early
-        #    (inflectsions.rb, brick.rb, apartment.rb)
-        # Very first thing, load inflections since we'll be using .pluralize and .singularize on table and model names
-        if File.exist?(inflections = ::Rails.root&.join('config/initializers/inflections.rb') || '')
-          load inflections
-        end
-        # Now the Brick initializer since there may be important schema things configured
-        if !::Brick.initializer_loaded && File.exist?(brick_initializer = ::Rails.root&.join('config/initializers/brick.rb') || '')
-          ::Brick.initializer_loaded = load brick_initializer
-
-          # After loading the initializer, add compatibility for ActiveStorage and ActionText if those haven't already been
-          # defined.  (Further JSON configuration for ActiveStorage metadata happens later in the after_initialize hook.)
-          # begin
-            ['ActiveStorage', 'ActionText'].each do |ar_extension|
-              if Object.const_defined?(ar_extension) &&
-                 (extension = Object.const_get(ar_extension)).respond_to?(:table_name_prefix) &&
-                 !::Brick.config.table_name_prefixes.key?(as_tnp = extension.table_name_prefix)
-                ::Brick.config.table_name_prefixes[as_tnp] = ar_extension
-              end
-            end
-          # rescue # NoMethodError
-          # end
-
-          # Support the followability gem:  https://github.com/nejdetkadir/followability
-          if Object.const_defined?('Followability') && !::Brick.config.table_name_prefixes.key?('followability_')
-            ::Brick.config.table_name_prefixes['followability_'] = 'Followability'
-          end
-        end
-        # Load the initializer for the Apartment gem a little early so that if .excluded_models and
-        # .default_schema are specified then we can work with non-tenanted models more appropriately
-        if (apartment = Object.const_defined?('Apartment')) &&
-           File.exist?(apartment_initializer = ::Rails.root.join('config/initializers/apartment.rb'))
-          require 'apartment/adapters/abstract_adapter'
-          Apartment::Adapters::AbstractAdapter.class_exec do
-            if instance_methods.include?(:process_excluded_models)
-              def process_excluded_models
-                # All other models will share a connection (at Apartment.connection_class) and we can modify at will
-                Apartment.excluded_models.each do |excluded_model|
-                  begin
-                    process_excluded_model(excluded_model)
-                  rescue NameError => e
-                    (@bad_models ||= []) << excluded_model
-                  end
-                end
-              end
-            end
-          end
-          unless @_apartment_loaded
-            load apartment_initializer
-            @_apartment_loaded = true
-          end
-        end
         # Only for Postgres  (Doesn't work in sqlite3 or MySQL)
         # puts ActiveRecord::Base.execute_sql("SELECT current_setting('SEARCH_PATH')").to_a.inspect
 
@@ -438,7 +385,7 @@ ORDER BY 1, 2, c.internal_column_id, acc.position"
         loop do
           klass = Object
           proposed_name_parts.each do |part|
-            if klass.const_defined?(part)
+            if klass.const_defined?(part) && klass.name != part
               begin
                 klass = klass.const_get(part)
               rescue NoMethodError => e

data/lib/brick/version_number.rb

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

data/lib/brick.rb

@@ -232,10 +232,6 @@ module Brick
                 )
               )
             )
-        # unless a.polymorphic? || (!a.belongs_to? && (through = a.options[:through])) ||
-        #        (a.klass && ::Brick.config.exclude_tables.exclude?(a.klass.table_name) &&
-        #          (!a.belongs_to? || (same_type = (fk_type = model_cols[a.foreign_key.to_s]&.type) == pk_type))
-        #        )
           if same_type == false # We really do want to test specifically for false here, and not nil!
             puts "WARNING:
   Foreign key column #{a.klass.table_name}.#{a.foreign_key} is #{fk_type}, but the primary key it relates to, #{primary_klass.table_name}.#{a_pk}, is #{pk_type}.

data/lib/generators/brick/install_generator.rb

@@ -171,7 +171,7 @@ if ActiveRecord::Base.respond_to?(:brick_select) && !::Brick.initializer_loaded
 
   # # Custom path prefix to apply to all auto-generated Brick routes.  Also causes auto-generated controllers
   # # to be created inside a module with the same name.
-  # ::Brick.path_prefix = 'admin'
+  # ::Brick.path_prefix = 'brick'
 
   # # Normally all are enabled in development mode, and for security reasons only models are enabled in production
   # # and test.  This allows you to either (a) turn off models entirely, or (b) enable controllers, views, and routes

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: brick
 version: !ruby/object:Gem::Version
-  version: 1.0.204
+  version: 1.0.205
 platform: ruby
 authors:
 - Lorin Thwaits
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-02-11 00:00:00.000000000 Z
+date: 2024-02-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -285,7 +285,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: 1.3.6
 requirements: []
-rubygems_version: 3.2.33
+rubygems_version: 3.1.6
 signing_key:
 specification_version: 4
 summary: Create a Rails app from data alone