brick 1.0.189 → 1.0.190

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 382e04c6f26509410cbd0660360620ad0ada6f40492babff8c65af9e53e41b02
-  data.tar.gz: 4a6cb0143dce1953a601cf26e7217d3723fc36c8ab2ba45750ce21761d931e14
+  metadata.gz: 3e7a12940f4ac57eea3c53bb3b437472f90bbaa857bd6611d3e871463add46f2
+  data.tar.gz: 8afb6e47cebd99a9b4172aa82d9447b71dd66e6df750249063b66b9b9ba939a9
 SHA512:
-  metadata.gz: 45cb41aa504aa8171387e3de188437d6a210b7c8769770c161e58cb566c850e2a557fc7a102538df42bfefb4a35c8f6d5d43e5b65b16920a1adcd2cf5b18068c
-  data.tar.gz: e67fcd671d56691628e2164d266c16d014aa78882d43a2e555b65816b347df8dc61c92df1986e716cd613b25dae7a35160b7e3995dd9a842a41e73a75d75540d
+  metadata.gz: d5379ddb6c61675b5606730a771a3f95638829b8359b42fd78c0509836635a93d36b787b09f21a0b9321623c917a149508b643a68ff4474baf7b40739ad5eeed
+  data.tar.gz: 40b6c6dc4fd31b10632fafb572608ec1c6473b6efa1dab346f1c82fc4f9daa1faba6d3675b336a780291595e25ace20358a95205a9284e7c42ddf450efe1d0a4

data/lib/brick/extensions.rb

@@ -98,6 +98,7 @@ module ActiveRecord
       end
 
       def _br_quoted_name(name)
+        name = name.join('.') if name.is_a?(Array)
         if name == '*'
           name
         elsif is_mysql
@@ -2895,24 +2896,32 @@ ORDER BY 1, 2, c.internal_column_id, acc.position"
       #   end
       # end
       # schema = ::Brick.default_schema # Reset back for this next round of fun
+      kcus = nil
       case ActiveRecord::Base.connection.adapter_name
       when 'PostgreSQL', 'Mysql2', 'Trilogy', 'SQLServer'
-        sql = "SELECT kcu1.CONSTRAINT_SCHEMA, kcu1.TABLE_NAME, kcu1.COLUMN_NAME,
-            kcu2.CONSTRAINT_SCHEMA AS primary_schema, kcu2.TABLE_NAME AS primary_table, kcu1.CONSTRAINT_NAME AS CONSTRAINT_SCHEMA_FK
+        # All KCUs -- use this to virtually JOIN against fk_references in Ruby code
+        sql = "SELECT CONSTRAINT_CATALOG, CONSTRAINT_SCHEMA, CONSTRAINT_NAME, ORDINAL_POSITION,
+                      TABLE_NAME, COLUMN_NAME
+              FROM INFORMATION_SCHEMA.KEY_COLUMN_USAGE#{"
+              WHERE CONSTRAINT_SCHEMA = COALESCE(current_setting('SEARCH_PATH'), 'public')" if is_postgres && schema }"
+        kcus = ActiveRecord::Base.execute_sql(sql).each_with_object({}) do |v, s|
+                 key = "#{v['constraint_name']}.#{v['constraint_schema']}.#{v['constraint_catalog']}.#{v['ordinal_position']}"
+                 key << ".#{v['table_name']}.#{v['column_name']}" unless is_postgres || is_mssql
+                 s[key] = [v['constraint_schema'], v['table_name']]
+               end
+
+        sql = "SELECT kcu.CONSTRAINT_SCHEMA, kcu.TABLE_NAME, kcu.COLUMN_NAME,
+            #{# These will get filled in with real values (effectively doing the JOIN in Ruby)
+              is_postgres || is_mssql ? 'NULL as primary_schema, NULL as primary_table' :
+                                        'kcu.REFERENCED_TABLE_NAME, kcu.REFERENCED_COLUMN_NAME'},
+            kcu.CONSTRAINT_NAME AS CONSTRAINT_SCHEMA_FK,
+            rc.UNIQUE_CONSTRAINT_NAME, rc.UNIQUE_CONSTRAINT_SCHEMA, rc.UNIQUE_CONSTRAINT_CATALOG, kcu.ORDINAL_POSITION
           FROM INFORMATION_SCHEMA.REFERENTIAL_CONSTRAINTS AS rc
-            INNER JOIN INFORMATION_SCHEMA.KEY_COLUMN_USAGE AS kcu1
-              ON kcu1.CONSTRAINT_CATALOG = rc.CONSTRAINT_CATALOG
-              AND kcu1.CONSTRAINT_SCHEMA = rc.CONSTRAINT_SCHEMA
-              AND kcu1.CONSTRAINT_NAME = rc.CONSTRAINT_NAME
-            INNER JOIN INFORMATION_SCHEMA.KEY_COLUMN_USAGE AS kcu2
-              ON kcu2.CONSTRAINT_CATALOG = rc.UNIQUE_CONSTRAINT_CATALOG
-              AND kcu2.CONSTRAINT_SCHEMA = rc.UNIQUE_CONSTRAINT_SCHEMA
-              AND kcu2.CONSTRAINT_NAME = rc.UNIQUE_CONSTRAINT_NAME#{"
-              AND kcu2.TABLE_NAME = kcu1.REFERENCED_TABLE_NAME
-              AND kcu2.COLUMN_NAME = kcu1.REFERENCED_COLUMN_NAME" unless is_postgres || is_mssql }
-              AND kcu2.ORDINAL_POSITION = kcu1.ORDINAL_POSITION#{"
-          WHERE kcu1.CONSTRAINT_SCHEMA = COALESCE(current_setting('SEARCH_PATH'), 'public')" if is_postgres && schema }"
-          # AND kcu2.TABLE_NAME = ?;", Apartment::Tenant.current, table_name
+            INNER JOIN INFORMATION_SCHEMA.KEY_COLUMN_USAGE AS kcu
+              ON kcu.CONSTRAINT_CATALOG = rc.CONSTRAINT_CATALOG
+              AND kcu.CONSTRAINT_SCHEMA = rc.CONSTRAINT_SCHEMA
+              AND kcu.CONSTRAINT_NAME = rc.CONSTRAINT_NAME#{"
+          WHERE kcu.CONSTRAINT_SCHEMA = COALESCE(current_setting('SEARCH_PATH'), 'public')" if is_postgres && schema }"
         fk_references = ActiveRecord::Base.execute_sql(sql)
       when 'SQLite'
         sql = "SELECT NULL AS constraint_schema, m.name, fkl.\"from\", NULL AS primary_schema, fkl.\"table\", m.name || '_' || fkl.\"from\" AS constraint_name
@@ -2943,6 +2952,11 @@ ORDER BY 1, 2, c.internal_column_id, acc.position"
       ::Brick.default_schema ||= 'public' if ActiveRecord::Base.connection.adapter_name == 'PostgreSQL'
       fk_references&.each do |fk|
         fk = fk.values unless fk.is_a?(Array)
+        # Virtually JOIN against fk_references in order to change out the primary schema and primary table
+        if (kcu = kcus&.fetch("#{fk[6]}.#{fk[7]}.#{fk[8]}.#{fk[9]}", nil))
+          fk[3] = kcu[0]
+          fk[4] = kcu[1]
+        end
         # Multitenancy makes things a little more general overall, except for non-tenanted tables
         if ::Brick.is_apartment_excluded_table(::Brick.namify(fk[1]))
           fk[0] = ::Brick.apartment_default_tenant
@@ -2967,6 +2981,7 @@ ORDER BY 1, 2, c.internal_column_id, acc.position"
         end
         ::Brick._add_bt_and_hm(fk, relations)
       end
+      kcus = nil # Allow this large item to be garbage collected
     end
 
     table_name_lookup = (::Brick.table_name_lookup ||= {})

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

@@ -738,8 +738,11 @@ window.addEventListener(\"popstate\", linkSchemas);
                       hms_columns << hm_entry
                     end
                   when 'show', 'new', 'update'
+                    predicates = nil
                     hm_stuff << if hm_fk_name
-                                  if hm_assoc.klass.column_names.include?(hm_fk_name.to_s) ||
+                                  if (hm_fk_name.is_a?(Array) && # Composite key?
+                                      hm_fk_name.all? { |hm_fk_part| hm_assoc.klass.column_names.include?(hm_fk_part) }) ||
+                                     hm_assoc.klass.column_names.include?(hm_fk_name.to_s) ||
                                      (hm_fk_name.is_a?(String) && hm_fk_name.include?('.')) # HMT?  (Could do a better check for this)
                                     predicates = path_keys(hm_assoc, hm_fk_name, pk).map do |k, v|
                                                    if v == '[sti_type]'
@@ -1638,7 +1641,7 @@ end
                  end
       s << "<table id=\"#{hm_name}\" class=\"shadow\">
         <tr><th>#{hm[1]}#{' poly' if hm[0].options[:as]} #{hm[3]}
-          <% if respond_to?(:new_#{partial_new_path_name = hm.first.klass._brick_index(:singular)}_path) %>
+          <% if predicates && respond_to?(:new_#{partial_new_path_name = hm.first.klass._brick_index(:singular)}_path) %>
           <span class = \"add-hm-related\"><%=
             pk_val = (obj_pk = model.primary_key).is_a?(String) ? obj.send(obj_pk) : obj_pk.map { |pk_part| obj.send(pk_part) }
             pk_val_arr = [pk_val] unless pk_val.is_a?(Array)

data/lib/brick/version_number.rb

@@ -5,7 +5,7 @@ module Brick
   module VERSION
     MAJOR = 1
     MINOR = 0
-    TINY = 189
+    TINY = 190
 
     # 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.189
+  version: 1.0.190
 platform: ruby
 authors:
 - Lorin Thwaits
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-11-17 00:00:00.000000000 Z
+date: 2023-11-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord