brick 1.0.37 → 1.0.40

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: '0178c8135edf190cf65ae2a4b274bde10cc5410f33adab7337798d98636f810f'
-  data.tar.gz: c4c51e9e22548f9f49a08a7550eb003c0075b65bac70fffea82b919b7aa07131
+  metadata.gz: cf29d4af64b65c01ae68df45762b816403a1ebc0035e1221552f249ff3ba2ce9
+  data.tar.gz: 076cdb672d7c2137cc74421467d72491e9ea89751fab32015f427158fc367b45
 SHA512:
-  metadata.gz: b6ad8492e3b19f526af3cc69f53f30e3a4de0f172f73362d36f6dbac884f7137bb7a3944f99054f61279c2fdc27989e9b840e68cb48c23aa378f6ac4bd4d2cfb
-  data.tar.gz: def025f413aa8c89f8f603f4e3422427f9d71b10b7c57c0db6d5fb750b0e26352dc88f3d85f01b526127d54a789e6b056dc7acee64e9046909a039e576421587
+  metadata.gz: ce17038567ddd9d6d9d9dce6eb71fd50612d167b56351898ef9b3f9441d1a7ef3ee4ec3a57f3f24e613829e6c8a2f59517fc0d4e9266f11166b92821310b0c06
+  data.tar.gz: 6a7bb38b009853594a8bf1ab28489bbc92d6300fcd0f8497c7950e81c6a62a4c6da2bd65840f93bd7afea40153727cb93914529120ecf9b0dceca39b21fef880

data/lib/brick/compatibility.rb

@@ -1,7 +1,6 @@
 # frozen_string_literal: true
 
 require 'active_record/version'
-
 # ActiveRecord before 4.0 didn't have #version
 unless ActiveRecord.respond_to?(:version)
   module ActiveRecord
@@ -11,6 +10,16 @@ unless ActiveRecord.respond_to?(:version)
   end
 end
 
+require 'action_view'
+# Older ActionView didn't have #version
+unless ActionView.respond_to?(:version)
+  module ActionView
+    def self.version
+      ActionPack.version
+    end
+  end
+end
+
 # In ActiveSupport older than 5.0, the duplicable? test tries to new up a BigDecimal,
 # and Ruby 2.6 and later deprecates #new.  This removes the warning from BigDecimal.
 # This compatibility needs to be put into place in the application's "config/boot.rb"

data/lib/brick/extensions.rb

@@ -70,7 +70,11 @@ module ActiveRecord
     def self._brick_primary_key(relation = nil)
       return instance_variable_get(:@_brick_primary_key) if instance_variable_defined?(:@_brick_primary_key)
 
-      pk = primary_key.is_a?(String) ? [primary_key] : primary_key || []
+      pk = begin
+             primary_key.is_a?(String) ? [primary_key] : primary_key || []
+           rescue
+             []
+           end
       # Just return [] if we're missing any part of the primary key.  (PK is usually just "id")
       if relation && pk.present?
         @_brick_primary_key ||= pk.any? { |pk_part| !relation[:cols].key?(pk_part) } ? [] : pk
@@ -204,7 +208,7 @@ module ActiveRecord
       assoc_name = CGI.escapeHTML(assoc_name.to_s)
       model_path = Rails.application.routes.url_helpers.send("#{model_underscore.tr('/', '_').pluralize}_path".to_sym)
       av_class = Class.new.extend(ActionView::Helpers::UrlHelper)
-      av_class.extend(ActionView::Helpers::TagHelper) if ActionView.version < ::Gem::Version.new('6.1')
+      av_class.extend(ActionView::Helpers::TagHelper) if ActionView.version < ::Gem::Version.new('7')
       link = av_class.link_to(name, model_path)
       model_underscore == assoc_name ? link : "#{assoc_name}-#{link}".html_safe
     end
@@ -552,7 +556,7 @@ Module.class_exec do
     #   return my_const
     end
 
-    relations = ::Brick.instance_variable_get(:@relations)[ActiveRecord::Base.connection_pool.object_id] || {}
+    relations = ::Brick.relations
     # puts "ON OBJECT: #{args.inspect}" if self.module_parent == Object
     result = if ::Brick.enable_controllers? && class_name.end_with?('Controller') && (plural_class_name = class_name[0..-11]).length.positive?
                # Otherwise now it's up to us to fill in the gaps
@@ -627,8 +631,8 @@ Module.class_exec do
 #         module_prefixes.unshift('') unless module_prefixes.first.blank?
 #         candidate_file = Rails.root.join('app/models' + module_prefixes.map(&:underscore).join('/') + '.rb')
       self._brick_const_missing(*args)
-    elsif self != Object
-      module_parent.const_missing(*args)
+    # elsif self != Object
+    #   module_parent.const_missing(*args)
     else
       puts "MISSING! #{self.name} #{args.inspect} #{table_name}"
       self._brick_const_missing(*args)
@@ -1188,8 +1192,13 @@ module ActiveRecord::ConnectionHandling
       case ActiveRecord::Base.connection.adapter_name
       when 'PostgreSQL', 'SQLite' # These bring back a hash for each row because the query uses column aliases
         # schema ||= 'public' if ActiveRecord::Base.connection.adapter_name == 'PostgreSQL'
+        ar_smtn = if ActiveRecord::Base.respond_to?(:schema_migrations_table_name)
+                    ActiveRecord::Base.schema_migrations_table_name
+                  else
+                    'schema_migrations'
+                  end
         ar_imtn = ActiveRecord.version >= ::Gem::Version.new('5.0') ? ActiveRecord::Base.internal_metadata_table_name : ''
-        ActiveRecord::Base.execute_sql(sql, ActiveRecord::Base.schema_migrations_table_name, ar_imtn).each do |r|
+        ActiveRecord::Base.execute_sql(sql, ar_smtn, ar_imtn).each do |r|
           # If Apartment gem lists the table as being associated with a non-tenanted model then use whatever it thinks
           # is the default schema, usually 'public'.
           schema_name = if ::Brick.config.schema_behavior[:multitenant]

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

@@ -50,23 +50,26 @@ module Brick
         # ====================================
         if ::Brick.enable_views?
           ActionView::LookupContext.class_exec do
+            # Used by Rails 5.0 and above
             alias :_brick_template_exists? :template_exists?
             def template_exists?(*args, **options)
-              unless (is_template_exists = _brick_template_exists?(*args, **options))
-                # Need to return true if we can fill in the blanks for a missing one
-                # args will be something like:  ["index", ["categories"]]
-                args[1] = args[1].each_with_object([]) { |a, s| s.concat(a.split('/')) }
-                args[1][args[1].length - 1] = args[1].last.singularize # Make sure the last item, defining the class name, is singular
-                model = args[1].map(&:camelize).join('::').constantize
-                if is_template_exists = model && (
-                     ['index', 'show'].include?(args.first) || # Everything has index and show
+              _brick_template_exists?(*args, **options) || set_brick_model(args)
+            end
+
+            def set_brick_model(find_args)
+              # Need to return true if we can fill in the blanks for a missing one
+              # args will be something like:  ["index", ["categories"]]
+              find_args[1] = find_args[1].each_with_object([]) { |a, s| s.concat(a.split('/')) }
+              if (class_name = find_args[1].last&.singularize)
+                find_args[1][find_args[1].length - 1] = class_name # Make sure the last item, defining the class name, is singular
+                if (model = find_args[1].map(&:camelize).join('::').constantize) && (
+                     ['index', 'show'].include?(find_args.first) || # Everything has index and show
                      # Only CUD stuff has create / update / destroy
-                      (!model.is_view? && ['new', 'create', 'edit', 'update', 'destroy'].include?(args.first))
+                     (!model.is_view? && ['new', 'create', 'edit', 'update', 'destroy'].include?(find_args.first))
                    )
                   @_brick_model = model
                 end
               end
-              is_template_exists
             end
 
             def path_keys(hm_assoc, fk_name, obj_name, pk)
@@ -82,9 +85,13 @@ module Brick
 
             alias :_brick_find_template :find_template
             def find_template(*args, **options)
-              return  _brick_find_template(*args, **options) unless @_brick_model
+              unless (model_name = (
+                       @_brick_model ||
+                       (ActionView.version < ::Gem::Version.new('5.0') && args[1].is_a?(Array) ? set_brick_model(args) : nil)
+                     )&.name)
+                return _brick_find_template(*args, **options)
+              end
 
-              model_name = @_brick_model.name
               pk = @_brick_model._brick_primary_key(::Brick.relations.fetch(model_name, nil))
               obj_name = model_name.split('::').last.underscore
               path_obj_name = model_name.underscore.tr('/', '_')
@@ -462,8 +469,8 @@ if (headerTop) {
 <select id=\"schema\">#{schema_options}</select>" if ::Brick.config.schema_behavior[:multitenant] && ::Brick.db_schemas.length > 1}
 <select id=\"tbl\">#{table_options}</select>
 <h1>#{model_plural = model_name.pluralize}</h1>#{template_link}<%
-   if (relation = Brick.relations[#{model_name}.table_name])[:description] %><%=
-     relation.fetch(:description, nil) %><br><%
+   if (description = (relation = Brick.relations[#{model_name}.table_name])&.fetch(:description, nil)) %><%=
+     description %><br><%
    end
    if @_brick_params&.present? %>
   <% if @_brick_params.length == 1 # %%% Does not yet work with composite keys
@@ -488,7 +495,7 @@ if (headerTop) {
                ::Brick.config.metadata_columns.include?(col_name) || poly_cols.include?(col_name)
 
        col_order << col_name
-    %><th<%= \" title = \\\"#\{col.comment}\\\"\".html_safe unless col.comment.blank? %>><%
+    %><th<%= \" title = \\\"#\{col.comment}\\\"\".html_safe if col.respond_to?(:comment) && !col.comment.blank? %>><%
        if (bt = bts[col_name]) %>
          BT <%
          bt[1].each do |bt_pair| %><%=
@@ -522,6 +529,7 @@ if (headerTop) {
                # 0..62 because Postgres column names are limited to 63 characters
                #{obj_name}, (descrips = @_brick_bt_descrip[bt.first][bt_class])[0..-2].map { |z| #{obj_name}.send(z.last[0..62]) }, (bt_id_col = descrips.last)
              )
+             bt_txt ||= \"<< Orphaned ID: #\{val} >>\" if val
              bt_id = #{obj_name}.send(*bt_id_col) if bt_id_col&.present? %>
           <%= bt_id ? link_to(bt_txt, send(\"#\{bt_class.base_class.name.underscore.tr('/', '_')\}_path\".to_sym, bt_id)) : bt_txt %>
           <%#= Previously was:  bt_obj = bt[1].first.first.find_by(bt[2] => val); link_to(bt_obj.brick_descrip, send(\"#\{bt[1].first.first.name.underscore\}_path\".to_sym, bt_obj.send(bt[1].first.first.primary_key.to_sym))) if bt_obj %>
@@ -545,8 +553,8 @@ if (headerTop) {
 <select id=\"schema\">#{schema_options}</select>" if ::Brick.config.schema_behavior[:multitenant] && ::Brick.db_schemas.length > 1}
 <select id=\"tbl\">#{table_options}</select>
 <h1>#{model_name}: <%= (obj = @#{obj_name})&.brick_descrip || controller_name %></h1><%
-if (relation = Brick.relations[#{model_name}.table_name])[:description] %><%=
-  relation.fetch(:description, nil) %><br><%
+if (description = (relation = Brick.relations[#{model_name}.table_name])&.fetch(:description, nil)) %><%=
+  description %><br><%
 end
 %><%= link_to '(See all #{obj_name.pluralize})', #{path_obj_name.pluralize}_path %>
 <% if obj %>
@@ -562,7 +570,7 @@ end
     <tr>
     <% next if (#{(pk || []).inspect}.include?(k) && !bts.key?(k)) ||
                ::Brick.config.metadata_columns.include?(k) %>
-    <th class=\"show-field\"<%= \" title = \\\"#\{col.comment}\\\"\".html_safe unless col.comment.blank? %>>
+    <th class=\"show-field\"<%= \" title = \\\"#\{col.comment}\\\"\".html_safe if col.respond_to?(:comment) && !col.comment.blank? %>>
     <% has_fields = true
       if (bt = bts[k])
         # Add a final member in this array with descriptive options to be used in <select> drop-downs
@@ -603,7 +611,11 @@ end
       html_options = { prompt: \"Select #\{bt_name\}\" }
       html_options[:class] = 'dimmed' unless val %>
       <%= f.select k.to_sym, bt[3], { value: val || '^^^brick_NULL^^^' }, html_options %>
-      <%= bt_obj = bt_class&.find_by(bt_pair[1] => val); link_to('⇛', send(\"#\{bt_class.base_class.name.underscore.tr('/', '_')\}_path\".to_sym, bt_obj.send(bt_class.primary_key.to_sym)), { class: 'show-arrow' }) if bt_obj %>
+      <%= if (bt_obj = bt_class&.find_by(bt_pair[1] => val))
+            link_to('⇛', send(\"#\{bt_class.base_class.name.underscore.tr('/', '_')\}_path\".to_sym, bt_obj.send(bt_class.primary_key.to_sym)), { class: 'show-arrow' })
+          elsif val
+            \"Orphaned ID: #\{val}\"
+          end %>
     <% else case #{model_name}.column_for_attribute(k).type
       when :string, :text %>
         <% if is_bcrypt?(val) # || .readonly? %>

data/lib/brick/tasks/orphans.rake

@@ -0,0 +1,57 @@
+# frozen_string_literal: true
+
+if Object.const_defined?('::Rake::TaskManager')
+  namespace :brick do
+    desc 'Find any seemingly-orphaned records'
+    task orphans: :environment do
+      def class_pk(dotted_name, multitenant)
+        Object.const_get((multitenant ? [dotted_name.split('.').last] : dotted_name.split('.')).map { |nm| "::#{nm.singularize.camelize}" }.join).primary_key
+      end
+
+      schema_list = ((multi = ::Brick.config.schema_behavior[:multitenant]) && ::Brick.db_schemas.keys.sort) || []
+      if schema_list.length > 1
+        require 'fancy_gets'
+        include FancyGets
+        schema = gets_list(
+          list: schema_list,
+          chosen: multi[:schema_to_analyse]
+        )
+      elsif schema_list.length.positive?
+        schema = schema_list.first
+      end
+      ActiveRecord::Base.execute_sql("SET SEARCH_PATH = ?", schema) if schema
+      orphans = +''
+      ::Brick.relations.each do |k, v|
+        next if v.key?(:isView) || ::Brick.config.exclude_tables.include?(k) ||
+                !(pri_pk = v[:pkey].values.first&.first) ||
+                !(pri_pk = class_pk(k, multi))
+        v[:fks].each do |k1, v1|
+          next if v1[:is_bt] ||
+                  !(for_rel = ::Brick.relations.fetch(v1[:inverse_table], nil)) ||
+                  v1[:inverse]&.key?(:polymorphic) ||
+                  !(for_pk = for_rel.fetch(:pkey, nil)&.values&.first&.first) ||
+                  !(for_pk = class_pk(v1[:inverse_table], multi))
+          begin
+            ActiveRecord::Base.execute_sql(
+              "SELECT DISTINCT frn.#{v1[:fk]} AS pri_id, frn.#{for_pk} AS fk_id
+              FROM #{v1[:inverse_table]} AS frn
+                LEFT OUTER JOIN #{k} AS pri ON pri.#{pri_pk} = frn.#{v1[:fk]}
+              WHERE frn.#{v1[:fk]} IS NOT NULL AND pri.#{pri_pk} IS NULL
+              ORDER BY 1, 2"
+            ).each do |o|
+              orphans << "#{v1[:inverse_table]} #{o['fk_id']} refers to non-existant #{k} #{o['pri_id']}\n"
+            end
+          rescue StandardError => err
+            puts "Strange -- #{err.inspect}"
+          end
+        end
+      end
+      puts "For #{schema}:\n#{'=' * (schema.length + 5)}" if schema
+      if orphans.blank?
+        puts "No orphans!"
+      else
+        print orphans
+      end
+    end
+  end
+end

data/lib/brick/util.rb

@@ -5,7 +5,7 @@ module Brick
   module Util
     # ===================================
     # Epic require patch
-    def self._patch_require(module_filename, folder_matcher, search_text, replacement_text, autoload_symbol = nil, is_bundler = false)
+    def self._patch_require(module_filename, folder_matcher, replacements, autoload_symbol = nil, is_bundler = false)
       mod_name_parts = module_filename.split('.')
       extension = case mod_name_parts.last
                   when 'rb', 'so', 'o'
@@ -28,10 +28,13 @@ module Brick
           Dir.mkdir(new_part) unless Dir.exist?(new_part)
           new_part
         end
-        if ::Brick::Util._write_patched(folder_matcher, module_filename, extension, custom_require_dir, nil, search_text, replacement_text) &&
+        if ::Brick::Util._write_patched(folder_matcher, module_filename, extension, custom_require_dir, nil, replacements) &&
            !alp.include?(custom_require_dir)
           alp.unshift(custom_require_dir)
         end
+        # require 'pry-byebug'
+        # binding.pry
+        # z = 10
       elsif is_bundler
         puts "Bundler hack"
         require 'pry-byebug'
@@ -56,13 +59,13 @@ module Brick
             define_method(:require) do |name|
               puts name if name.to_s.include?('cucu')
               if (require_override = ::Brick::Util.instance_variable_get(:@_require_overrides)[name])
-                extension, folder_matcher, search_text, replacement_text, autoload_symbol = require_override
+                extension, folder_matcher, replacements, autoload_symbol = require_override
                 patched_filename = "/patched_#{name.tr('/', '_')}#{extension}"
                 if $LOADED_FEATURES.find { |f| f.end_with?(patched_filename) }
                   false
                 else
                   is_replaced = false
-                  if (replacement_path = ::Brick::Util._write_patched(folder_matcher, name, extension, ::Brick::Util._custom_require_dir, patched_filename, search_text, replacement_text))
+                  if (replacement_path = ::Brick::Util._write_patched(folder_matcher, name, extension, ::Brick::Util._custom_require_dir, patched_filename, replacements))
                     is_replaced = Kernel.send(:orig_require, replacement_path)
                   elsif replacement_path.nil?
                     puts "Couldn't find #{name} to require it!"
@@ -75,7 +78,7 @@ module Brick
             end
           end
         end
-        require_overrides[module_filename] = [extension, folder_matcher, search_text, replacement_text, autoload_symbol]
+        require_overrides[module_filename] = [extension, folder_matcher, replacements, autoload_symbol]
       end
     end
 
@@ -94,7 +97,7 @@ module Brick
 
     # Returns the full path to the replaced filename, or
     # false if the file already exists, and nil if it was unable to write anything.
-    def self._write_patched(folder_matcher, name, extension, dir, patched_filename, search_text, replacement_text)
+    def self._write_patched(folder_matcher, name, extension, dir, patched_filename, replacements)
       # See if our replacement file might already exist for some reason
       name = +"/#{name}" unless name.start_with?('/')
       name << extension unless name.end_with?(extension)
@@ -111,9 +114,13 @@ module Brick
         break if path.include?(folder_matcher) && (orig_as = File.open(orig_path))
       end
       puts [folder_matcher, name].inspect
-      if (orig_text = orig_as&.read)
-        File.open(replacement_path, 'w') do |replacement|
-          num_written = replacement.write(orig_text.gsub(search_text, replacement_text))
+      if (updated_text = orig_as&.read)
+        File.open(replacement_path, 'w') do |replaced_file|
+          replacements = [replacements] unless replacements.first.is_a?(Array)
+          replacements.each do |search_text, replacement_text|
+            updated_text.gsub!(search_text, replacement_text)
+          end
+          num_written = replaced_file.write(updated_text)
         end
         orig_as.close
       end

data/lib/brick/version_number.rb

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

data/lib/brick.rb

@@ -2,10 +2,10 @@
 
 require 'brick/compatibility'
 
-# Allow ActiveRecord 4.0 and 4.1 to work with newer Ruby (>= 2.4) by avoiding a "stack level too deep"
+# Allow ActiveRecord 4.2.7 and older to work with newer Ruby (>= 2.4) by avoiding a "stack level too deep"
 # error when ActiveSupport tries to smarten up Numeric by messing with Fixnum and Bignum at the end of:
 # activesupport-4.0.13/lib/active_support/core_ext/numeric/conversions.rb
-if ActiveRecord.version < ::Gem::Version.new('4.2') &&
+if ActiveRecord.version < ::Gem::Version.new('4.2.8') &&
    ActiveRecord.version > ::Gem::Version.new('3.2') &&
    Object.const_defined?('Integer') && Integer.superclass.name == 'Numeric'
   class OurFixnum < Integer; end
@@ -31,25 +31,78 @@ if (ruby_version = ::Gem::Version.new(RUBY_VERSION)) >= ::Gem::Version.new('2.7'
   # Remove circular reference for "now"
   ::Brick::Util._patch_require(
     'active_support/values/time_zone.rb', '/activesupport',
-    '  def parse(str, now=now)',
-    '  def parse(str, now=now())'
+    ['  def parse(str, now=now)',
+     '  def parse(str, now=now())']
   )
   # Remove circular reference for "reflection" for ActiveRecord 3.1
   if ActiveRecord.version >= ::Gem::Version.new('3.1')
     ::Brick::Util._patch_require(
       'active_record/associations/has_many_association.rb', '/activerecord',
-      'reflection = reflection)',
-      'reflection = reflection())',
+      ['reflection = reflection)',
+       'reflection = reflection())'],
       :HasManyAssociation # Make sure the path for this guy is available to be autoloaded
     )
   end
 end
 
+# Add left_outer_join! to Associations::JoinDependency and Relation::QueryMethods
+if ActiveRecord.version < ::Gem::Version.new('5')
+  is_add_left_outer_join = true
+  ::Brick::Util._patch_require(
+    'active_record/associations/join_dependency.rb', '/activerecord', # /associations
+      ["def join_constraints(outer_joins)
+        joins = join_root.children.flat_map { |child|
+          make_inner_joins join_root, child
+        }",
+      "def join_constraints(outer_joins, join_type)
+        joins = join_root.children.flat_map { |child|
+
+          if join_type == Arel::Nodes::OuterJoin
+            make_left_outer_joins join_root, child
+          else
+            make_inner_joins join_root, child
+          end
+        }"],
+    :JoinDependency # This one is in an "eager_autoload do" -- so how to handle it?
+  )
+
+  # Three changes all in the same file, query_methods.rb:
+  ::Brick::Util._patch_require(
+    'active_record/relation/query_methods.rb', '/activerecord',
+    [
+     # Change 1 - Line 904
+     ['build_joins(arel, joins_values.flatten) unless joins_values.empty?',
+    "build_joins(arel, joins_values.flatten) unless joins_values.empty?
+      build_left_outer_joins(arel, left_outer_joins_values.flatten) unless left_outer_joins_values.empty?"
+    ],
+     # Change 2 - Line 992
+     ["raise 'unknown class: %s' % join.class.name
+        end
+      end",
+   "raise 'unknown class: %s' % join.class.name
+        end
+      end
+
+      build_join_query(manager, buckets, Arel::Nodes::InnerJoin)
+   end
+
+   def build_join_query(manager, buckets, join_type)"
+     ],
+     # Change 3 - Line 1012
+    ['join_infos = join_dependency.join_constraints stashed_association_joins',
+    'join_infos = join_dependency.join_constraints stashed_association_joins, join_type'
+     ]
+    ],
+    :QueryMethods
+  )
+end
+
+
 # puts ::Brick::Util._patch_require(
 #     'cucumber/cli/options.rb', '/cucumber/cli/options', # /cli/options
-#     '  def extract_environment_variables',
-#     "  def extract_environment_variables\n
-#     puts 'Patch test!'"
+#     ['  def extract_environment_variables',
+#      "  def extract_environment_variables\n
+#     puts 'Patch test!'"]
 #   ).inspect
 
 # An ActiveRecord extension that uses INFORMATION_SCHEMA views to reflect on all
@@ -295,10 +348,11 @@ module Brick
 
       relations = ::Brick.relations
       if (ars = ::Brick.config.additional_references) || ::Brick.config.polymorphics
+        is_optional = ActiveRecord.version >= ::Gem::Version.new('5.0')
         if ars
           ars.each do |ar|
             fk = ar.length < 5 ? [nil, +ar[0], ar[1], nil, +ar[2]] : [ar[0], +ar[1], ar[2], ar[3], +ar[4], ar[5]]
-            ::Brick._add_bt_and_hm(fk, relations, false, true)
+            ::Brick._add_bt_and_hm(fk, relations, false, is_optional)
           end
         end
         if (polys = ::Brick.config.polymorphics)
@@ -311,7 +365,7 @@ module Brick
             v ||= ActiveRecord::Base.execute_sql("SELECT DISTINCT #{poly}_type AS typ FROM #{table_name}").each_with_object([]) { |result, s| s << result['typ'] if result['typ'] }
             v.each do |type|
               if relations.key?(primary_table = type.underscore.pluralize)
-                ::Brick._add_bt_and_hm([nil, table_name, poly, nil, primary_table, "(brick) #{table_name}_#{poly}"], relations, true, true)
+                ::Brick._add_bt_and_hm([nil, table_name, poly, nil, primary_table, "(brick) #{table_name}_#{poly}"], relations, true, is_optional)
               else
                 missing_stis[primary_table] = type unless ::Brick.existing_stis.key?(type)
               end
@@ -414,7 +468,24 @@ end
 
 require 'brick/version_number'
 
+# Older versions of ActiveRecord would only show more serious error information from "panic" level, which is
+# a level only available in Postgres 12 and older.  This patch will allow older and newer versions of Postgres
+# to work along with fairly old versions of Rails.
+if Object.const_defined?('PG::VERSION') && ActiveRecord.version < ::Gem::Version.new('4.2.6')
+  ::Brick::Util._patch_require(
+    'active_record/connection_adapters/postgresql_adapter.rb', '/activerecord', ["'panic'", "'error'"]
+  )
+end
+
 require 'active_record'
+require 'active_record/relation'
+require 'active_record/relation/query_methods' if is_add_left_outer_join
+
+# Rake tasks
+class Railtie < Rails::Railtie
+  Dir.glob("#{File.expand_path(__dir__)}/brick/tasks/**/*.rake").each { |task| load task }
+end
+
 # Major compatibility fixes for ActiveRecord < 4.2
 # ================================================
 ActiveSupport.on_load(:active_record) do
@@ -633,6 +704,62 @@ ActiveSupport.on_load(:active_record) do
           end
         end
       end
+
+      if is_add_left_outer_join
+        # Final pieces for left_outer_joins support, which was derived from this commit:
+        # https://github.com/rails/rails/commit/3f46ef1ddab87482b730a3f53987e04308783d8b
+        module Associations
+          class JoinDependency
+            def make_left_outer_joins(parent, child)
+              tables    = child.tables
+              join_type = Arel::Nodes::OuterJoin
+              info      = make_constraints parent, child, tables, join_type
+
+              [info] + child.children.flat_map { |c| make_left_outer_joins(child, c) }
+            end
+          end
+        end
+        module Querying
+          delegate :left_outer_joins, to: :all
+        end
+        class Relation
+          MULTI_VALUE_METHODS = MULTI_VALUE_METHODS + [:left_outer_joins] unless MULTI_VALUE_METHODS.include?(:left_outer_joins)
+        end
+        module QueryMethods
+          attr_writer :left_outer_joins_values
+          def left_outer_joins_values
+            @left_outer_joins_values ||= []
+          end
+
+          def left_outer_joins(*args)
+            check_if_method_has_arguments!(:left_outer_joins, args)
+
+            args.compact!
+            args.flatten!
+
+            spawn.left_outer_joins!(*args)
+          end
+
+          def left_outer_joins!(*args) # :nodoc:
+            self.left_outer_joins_values += args
+            self
+          end
+
+          def build_left_outer_joins(manager, outer_joins)
+            buckets = outer_joins.group_by do |join|
+              case join
+              when Hash, Symbol, Array
+                :association_join
+              else
+                raise ArgumentError, 'only Hash, Symbol and Array are allowed'
+              end
+            end
+
+            build_join_query(manager, buckets, Arel::Nodes::OuterJoin)
+          end
+        end
+      end
+      # (End of left_outer_joins support)
     end
   end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: brick
 version: !ruby/object:Gem::Version
-  version: 1.0.37
+  version: 1.0.40
 platform: ruby
 authors:
 - Lorin Thwaits
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-06-27 00:00:00.000000000 Z
+date: 2022-07-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -16,20 +16,34 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '3.0'
+        version: '4.2'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '7.1'
+        version: '7.2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '3.0'
+        version: '4.2'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '7.1'
+        version: '7.2'
+- !ruby/object:Gem::Dependency
+  name: fancy_gets
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: appraisal
   requirement: !ruby/object:Gem::Requirement
@@ -225,6 +239,7 @@ files:
 - lib/brick/join_array.rb
 - lib/brick/serializers/json.rb
 - lib/brick/serializers/yaml.rb
+- lib/brick/tasks/orphans.rake
 - lib/brick/util.rb
 - lib/brick/version_number.rb
 - lib/generators/brick/USAGE