brick 1.0.231 → 1.0.232

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6b36099e75ec27d9fa4522da84f319af2fb1c98997c629a5246c88a1e6ddcbe3
-  data.tar.gz: '089a28701ee39956f0447801c70f9f044aefd06638ab575f3a6307690746eca4'
+  metadata.gz: ec8b0390d538c25946d68d51b1394875a02203694bc2d28928df1fd63987f40f
+  data.tar.gz: de49f7332e2a08a21a22e92cbac5c9668079551de6ba36ff45fd02dcac9102db
 SHA512:
-  metadata.gz: 0b8ed54764ce53a403f47d0a42ff578445e9a4d03e60f973264bbafb07a8c6a7af8daf15736ceb172472d2919817d14673163b40fdfffbdd87d4511e759c3747
-  data.tar.gz: '0326081b5c6e69ab4f7a6fe94784c73996117db6c715e3b158bc0cde744245e213e53a50e5b5d01bcd44b6cb24d55022548c2a1c83a53de5ccbca04f23c66b65'
+  metadata.gz: 7e77492d4d72836a829f9e073eaa14cb1f2c676aefbd053b0466e43b0d0f79141ec661bb38a377391d96711338f9bab5535ee52d51fc90964013ea042219cddc
+  data.tar.gz: '099332256ecadb0948c329dc8a3b06c1b460da5de41e8f183fc5dd937651edd57895b2d3c57399455af5732bab7172e6e88435560cd2d6b1d2e6f42eafefe71d'

data/lib/brick/version_number.rb

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

data/lib/generators/brick/airtable_api_caller.rb

@@ -15,6 +15,7 @@ You need only #{usage == :migrations ? 'this scope:' : "these three scopes:
 
 Please provide your Airtable PAT:"
         pat = gets_password
+        pat = ENV['AIRTABLE_PAT'] if pat.blank?
         require 'net/http'
         # Generate a list of bases that can be chosen
         bases = https_get('https://api.airtable.com/v0/meta/bases', pat)
@@ -52,12 +53,12 @@ Please provide your Airtable PAT:"
                               # Queue up to build associative table with two foreign keys
                               camelized = (assoc_name = "#{tbl_name}_#{col_name}_#{frn_tbl}").camelize
                               if associatives.keys.any? { |a| a.camelize == camelized }
-                                puts "Strangely have found two columns in \"#{table.name}\" with a name similar to \"#{col_name}\".  Skipping this to avoid a conflict."
+                                puts "Strangely have found two columns in \"#{table.name}\" with a name similar to \"#{col_name}\".  Skipping one to avoid a conflict."
                                 next
 
                               end
                               associatives[assoc_name] = [col_name, frn_tbl, tbl_name]
-                              fks << [assoc_name, frn_tbl, frn_tbl, col_name.underscore, tbl_name]
+                              fks << [assoc_name, frn_tbl, tbl_name, col_name.underscore, tbl_name]
                             end
                           end
                         else
@@ -94,18 +95,15 @@ Please provide your Airtable PAT:"
                       }
                     end
         associatives.each do |k, v|
-          pri_pk_col = relations[v[1]][:pkey]&.first&.last&.first
-          frn_pk_col = relations[v[2]][:pkey]&.first&.last&.first
           pri_fk_name = "#{v[1]}_id"
-          frn_fk_name = (frn_fk_name == pri_fk_name) ?
-            "#{v[2]}_2_id" # Self-referencing N:M
-            : "#{v[2]}_id" # Standard N:M
+          frn_fk_base = (v[1] == v[2]) ? "#{v[2]}_2" : v[2] # Accommodates self-referencing N:Ms
+          frn_fk_name = "#{frn_fk_base}_id"
           relations[k] = {
             pkey: { "#{k}_pkey" => ['id'] },
             cols: { 'id' => ['integer', nil, false, false] }
           }
-          fks << [v[1], pri_fk_name, k, pri_fk_name.underscore]
-          fks << [v[2], frn_fk_name, k, frn_fk_name.underscore]
+          fks << [v[1], pri_fk_name, k, v[0].underscore]
+          fks << [v[2], frn_fk_name, k, frn_fk_base.underscore]
         end
         fk_idx = 0
         fks.each do |pri_tbl, fk_col, frn_tbl, airtable_col, assoc_tbl|

data/lib/generators/brick/seeds_builder.rb

@@ -161,14 +161,15 @@ module Brick
                   if relation[:cols].keys.include?(col_name = ::Brick::AirtableApiCaller.sane_name(field.first))
                     s[col_name] = obj['fields'][field.first]
                   else # Consider N:M fks
-                    nm_fk = relation[:fks].find do |_k, fk1|
-                      relations[fk1[:assoc_tbl]]&.fetch(:fks, nil)&.find { |_k, fk2| fk2[:assoc_name] == col_name }
-                    end&.last
-                    if (t_table = nm_fk&.fetch(:inverse_table, nil))
+                    hm_fk = relation[:fks].find { |_k, fk1| !fk1[:is_bt] && fk1[:assoc_name] == ::Brick::AirtableApiCaller.sane_name(field.first) }&.last
+                    if (t_table = hm_fk&.fetch(:inverse_table, nil))
+                      associative_fks = relations[hm_fk&.fetch(:inverse_table, nil)][:fks]
+                      # near_side_fk = associative_fks.find { |_k, fk1| fk1[:is_bt] && fk1[:assoc_name] == ::Brick::AirtableApiCaller.sane_name(field.first) }&.last
+                      far_side_fk = associative_fks.find { |_k, fk1| fk1[:is_bt] && fk1[:assoc_name] != ::Brick::AirtableApiCaller.sane_name(field.first) }&.last
                       field.last.each do |nm_rec|
-                        nm_fk_col = nm_fk[:assoc_tbl]
-                        airtable_assoc_recids[t_table] << "#{nm_fk[:fk]}: #{nm_fk[:fk].singularize}_#{obj['id'][3..-1]}, " \
-                                                          "#{nm_fk_col}: #{nm_fk_col.singularize}_#{nm_rec[3..-1]}"
+                        # Can trade out:  hm_fk[:fk]  for:  near_side_fk[:inverse_table]
+                        airtable_assoc_recids[t_table] << "#{hm_fk[:fk]}: #{hm_fk[:fk].singularize}_#{nm_rec[3..-1]}, " \
+                                                          "#{far_side_fk[:assoc_name]}: #{far_side_fk[:inverse_table].singularize}_#{obj['id'][3..-1]}"
                       end
                     end
                   end
@@ -183,7 +184,7 @@ module Brick
               is_empty = false
               # For Airtable, take off the "rec___" prefix
               pk_val = is_airtable ? airtable_id[3..-1] : brick_escape(obj.attributes_before_type_cast[pkey_cols.first])
-              var_name = "#{tbl.gsub('.', '__')}_#{pk_val}"
+              var_name = "#{tbl.singularize.gsub('.', '__')}_#{pk_val}"
               fk_vals = []
               data = []
               updates = []
@@ -203,6 +204,7 @@ module Brick
                 if fk
                   inv_tbl = fk[:inverse_table].gsub('.', '__')
                   fk_val = if is_airtable
+                    inv_tbl = inv_tbl.singularize
                     # Used to be:  fk[:airtable_col]
                     # Take off the "rec___" prefix
                     obj.attributes_before_type_cast[fk[:assoc_name]]&.first&.[](3..-1)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: brick
 version: !ruby/object:Gem::Version
-  version: 1.0.231
+  version: 1.0.232
 platform: ruby
 authors:
 - Lorin Thwaits
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2025-03-12 00:00:00.000000000 Z
+date: 2025-03-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord