brick 1.0.135 → 1.0.137

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 28451931f3f43343f010a14d83dd3ed2d4c2fc80e61234d5a96775dd16ff4f11
-  data.tar.gz: aeb7327ab42e4252d60ec92bc25d93c778fe15d83744b830aad7673d833c3e75
+  metadata.gz: f25abe6f88b55efadd50c9deb65e7d9aabac31482a77f0f911f84157d4394079
+  data.tar.gz: a437b8efd9116a7c1e5a81fa6d67a66755b15a275db56717ecb7abe76bbbb50a
 SHA512:
-  metadata.gz: bd85040932260d4a6d68eb46a0406cdbce355a3beb793fa8b56fca51228f4227c430b44fdb7852a8280894720220af4e6564775dfd474bbecf722749909779c3
-  data.tar.gz: f4f34d81eab0bf2922c888d618dd1e115f605132578627f7a16c39420d78a574bf63b2fd3e4d3a9278ceab52c4abba6bbc16bf1884dfd4183c0473c8943e42ca
+  metadata.gz: 21ea85675d6e711261c8873c12187d78ecfdb53961a8ba795d35210b2a7cd00940d3522def3f3c8f10d37e3f7a8409f68efefc923798847b602f8d8afe24316f
+  data.tar.gz: b34c4d0396aea262f73e34bc25d4f0e2d9e2d3bddb36aa1293d7e7293edfed9f930318724c8471f11344822b6faa4afa942018148ed59cfa27d50eef6bca0eea

data/lib/brick/extensions.rb

@@ -989,7 +989,7 @@ end
 if Object.const_defined?('ActionView')
   require 'brick/frameworks/rails/form_tags'
   require 'brick/frameworks/rails/form_builder'
-  module ActionView::Helpers
+  module ::ActionView::Helpers
     module FormTagHelper
       include ::Brick::Rails::FormTags
     end
@@ -999,8 +999,8 @@ if Object.const_defined?('ActionView')
   end
 
   # FormBuilder#field_id isn't available in Rails < 7.0.  This is a rudimentary version with no `index`.
-  unless ActionView::Helpers::FormBuilder.methods.include?(:field_id)
-    ActionView::Helpers::FormBuilder.class_exec do
+  unless ::ActionView::Helpers::FormBuilder.methods.include?(:field_id)
+    ::ActionView::Helpers::FormBuilder.class_exec do
       def field_id(method)
         [object_name, method.to_s].join('_')
       end
@@ -1149,7 +1149,7 @@ Module.class_exec do
                                    ::Brick.is_oracle ? class_name.upcase : class_name,
                                    (plural_class_name = class_name.pluralize)].find { |s| Brick.db_schemas&.include?(s) }&.camelize ||
                                   (::Brick.config.sti_namespace_prefixes&.key?("::#{class_name}::") && class_name) ||
-                                  (::Brick.config.table_name_prefixes.values.include?(class_name) && class_name))
+                                  (::Brick.config.table_name_prefixes&.values&.include?(class_name) && class_name))
                return self.const_get(schema_name) if !is_tnp_module && self.const_defined?(schema_name)
 
                # Build out a module for the schema if it's namespaced
@@ -1942,8 +1942,9 @@ class Object
               end
               render json: { result: ::Brick.unexclude_column(table_name, col) }
             else
-              instance_variable_set("@#{singular_table_name}".to_sym,
-                                    model.send(:create, send(params_name_sym)))
+              @_lookup_context.instance_variable_set("@#{singular_table_name}".to_sym,
+                                                     model.send(:create, send(params_name_sym)))
+              @_lookup_context.instance_variable_set(:@_brick_model, model)
               index
               render :index
             end

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

@@ -1430,6 +1430,12 @@ end
                          end
                        end
 
+    # %%% Why in the Canvas LMS app does ActionView::Helpers get cleared / reloaded, or otherwise lose access to #brick_grid ???
+    # Possible fix if somewhere we can implement the #include with:
+    # (ActiveSupport.const_defined?('Reloader') ? ActiveSupport : ActionDispatch)::Reloader.to_prepare do ... end
+    # or
+    # Rails.application.reloader.to_prepare do ... end
+    self.class.class_exec { include ::Brick::Rails::FormTags } unless respond_to?(:brick_grid)
     # Write out the mega-grid
     brick_grid(@#{table_name}, @_brick_bt_descrip, @_brick_sequence, @_brick_incl, @_brick_excl,
                cols, poly_cols, bts, #{hms_keys.inspect}, {#{hms_columns.join(', ')}}) %>
@@ -1625,7 +1631,7 @@ end
         if bt.length < 4
           bt << (option_detail = [[\"(No #\{bt_name\} chosen)\", '^^^brick_NULL^^^']])
           # %%% Accommodate composite keys for obj.pk at the end here
-          collection, descrip_cols = bt_class&.order(obj_pk = bt_class.primary_key).brick_list
+          collection, descrip_cols = bt_class&.order(obj_pk = bt_class.primary_key)&.brick_list
           collection&.each do |obj|
             option_detail << [
               obj.brick_descrip(

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

@@ -36,9 +36,9 @@ module Brick::Rails::FormTags
     end
     sequence.reject! { |nm| exclusions.include?(nm) } if exclusions
     out << sequence.each_with_object(+'') do |col_name, s|
+             s << '<th '
              if (col = cols[col_name]).is_a?(ActiveRecord::ConnectionAdapters::Column)
-               s << '<th '
-               s << "title=\"#{col.comment}\"" if col.respond_to?(:comment) && !col.comment.blank?
+               s << "title=\"#{col.comment}\" " if col.respond_to?(:comment) && !col.comment.blank?
                s << if (bt = bts[col_name])
                       # Allow sorting for any BT except polymorphics
                       "x-order=\"#{bt.first.to_s + '"' unless bt[2]}>BT " +
@@ -49,18 +49,18 @@ module Brick::Rails::FormTags
              elsif col # HM column
                options = {}
                options[col[1].inheritance_column] = col[1].name unless col[1] == col[1].base_class
-               s << "<th x-order=\"#{col_name + '"' if true}>#{col[2]} "
+               s << "x-order=\"#{col_name + '"' if true}>#{col[2]} "
                s << (col.first ? "#{col[3]}" : "#{link_to(col[3], send("#{col[1]._brick_index}_path", options))}")
              elsif cust_cols.key?(col_name) # Custom column
-               s << "<th x-order=\"#{col_name}\">#{col_name}"
+               s << "x-order=\"#{col_name}\">#{col_name}"
              elsif col_name.is_a?(Symbol) && (hot = bts[col_name]) # has_one :through
-               s << "<th x-order=\"#{hot.first.to_s}\">HOT " +
+               s << "x-order=\"#{hot.first.to_s}\">HOT " +
                     hot[1].map { |hot_pair| hot_pair.first.bt_link(col_name) }.join(' ')
              elsif (bt = composite_bt_names[col_name])
-               s << "<th x-order=\"#{bt.first.to_s + '"' unless bt[2]}>BT comp " +
+               s << "x-order=\"#{bt.first.to_s + '"' unless bt[2]}>BT comp " +
                     bt[1].map { |bt_pair| bt_pair.first.bt_link(bt.first) }.join(' ')
              else # Bad column name!
-               s << "<th title=\"<< Unknown column >>\">#{col_name}"
+               s << "title=\"<< Unknown column >>\">#{col_name}"
              end
              s << '</th>'
            end
@@ -93,18 +93,20 @@ module Brick::Rails::FormTags
             end
           else # BT or HOT
             bt_class = bt[1].first.first
-            descrips = bt_descrip[bt.first][bt_class]
-            bt_id_col = if descrips.nil?
-                          puts "Caught it in the act for obj / #{col_name}!"
-                        elsif descrips.length == 1
-                          [obj.class.reflect_on_association(bt.first)&.foreign_key]
-                        else
-                          descrips.last
-                        end
-            bt_txt = bt_class.brick_descrip(
-              # 0..62 because Postgres column names are limited to 63 characters
-              obj, descrips[0..-2].map { |id| obj.send(id.last[0..62]) }, bt_id_col
-            )
+            if bt_descrip
+              descrips = bt_descrip[bt.first][bt_class]
+              bt_id_col = if descrips.nil?
+                            puts "Caught it in the act for obj / #{col_name}!"
+                          elsif descrips.length == 1
+                            [obj.class.reflect_on_association(bt.first)&.foreign_key]
+                          else
+                            descrips.last
+                          end
+            end
+            br_descrip_args = [obj]
+            # 0..62 because Postgres column names are limited to 63 characters
+            br_descrip_args += [descrips[0..-2].map { |id| obj.send(id.last[0..62]) }, bt_id_col] if descrips
+            bt_txt = bt_class.brick_descrip(*br_descrip_args)
             bt_txt = ::Brick::Rails.display_binary(bt_txt).html_safe if bt_txt&.encoding&.name == 'ASCII-8BIT'
             bt_txt ||= "<span class=\"orphan\">&lt;&lt; Orphaned ID: #{val} >></span>" if val
             bt_id = bt_id_col&.map { |id_col| obj.respond_to?(id_sym = id_col.to_sym) ? obj.send(id_sym) : id_col }
@@ -128,13 +130,11 @@ module Brick::Rails::FormTags
                          end
                 out << link_to(ho_txt, send("#{hm_klass.base_class._brick_index(:singular)}_path".to_sym, ho_id))
               end
-            else
-              if (ct = obj.send(hms_col[1].to_sym)&.to_i)&.positive?
-                predicates = hms_col[2].each_with_object({}) { |v, s| s[v.first] = v.last.is_a?(String) ? v.last : obj.send(v.last) }
-                predicates.each { |k, v| predicates[k] = obj.class.name if v == '[sti_type]' }
-                out << "#{link_to("#{ct || 'View'} #{hms_col.first}",
-                                  send("#{hm_klass._brick_index}_path".to_sym, predicates))}\n"
-              end
+            elsif obj.respond_to?(ct_col = hms_col[1].to_sym) && (ct = obj.send(ct_col)&.to_i)&.positive?
+              predicates = hms_col[2].each_with_object({}) { |v, s| s[v.first] = v.last.is_a?(String) ? v.last : obj.send(v.last) }
+              predicates.each { |k, v| predicates[k] = obj.class.name if v == '[sti_type]' }
+              out << "#{link_to("#{ct || 'View'} #{hms_col.first}",
+                                send("#{hm_klass._brick_index}_path".to_sym, predicates))}\n"
             end
           end
         elsif (col = cols[col_name]).is_a?(ActiveRecord::ConnectionAdapters::Column)

data/lib/brick/version_number.rb

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

data/lib/brick.rb

@@ -163,6 +163,8 @@ module Brick
 
     # All tables and views (what Postgres calls "relations" including column and foreign key info)
     def relations
+      return {} if ::ActiveRecord::Base.connection_handler.connection_pool_list.blank?
+
       # Key our list of relations for this connection off of the connection pool's object_id
       (@relations ||= {})[ActiveRecord::Base.connection_pool.object_id] ||= Hash.new { |h, k| h[k] = Hash.new { |h, k| h[k] = {} } }
     end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: brick
 version: !ruby/object:Gem::Version
-  version: 1.0.135
+  version: 1.0.137
 platform: ruby
 authors:
 - Lorin Thwaits
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-04-27 00:00:00.000000000 Z
+date: 2023-05-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord