brick 1.0.187 → 1.0.189

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 72ed0bc43d9ca9ae52dd6641bd5ba400bcb9a4fdb4e13953f931f2b7e1f0d46a
-  data.tar.gz: e5e56338e6e980ccfd733697e4cc9efaa8929e38b7f154c8d668ddc784bd55bc
+  metadata.gz: 382e04c6f26509410cbd0660360620ad0ada6f40492babff8c65af9e53e41b02
+  data.tar.gz: 4a6cb0143dce1953a601cf26e7217d3723fc36c8ab2ba45750ce21761d931e14
 SHA512:
-  metadata.gz: 06321d9f6ca977150dcc5458d85cb3a329955bbe961b26a273581392e8219bebbc669ad3373d2c57030b0c9b6df07e5c8cef0bdb6bcb7f14379e3ddf73ddb23e
-  data.tar.gz: d3432a3ba5983e1ac69d9dde0c5fb6a186d838e32007e0db5ca0af2497f4e918c7f58d9c10db23218c2c7fd605a64e86b50ab8002130845f993fae3a6232a08c
+  metadata.gz: 45cb41aa504aa8171387e3de188437d6a210b7c8769770c161e58cb566c850e2a557fc7a102538df42bfefb4a35c8f6d5d43e5b65b16920a1adcd2cf5b18068c
+  data.tar.gz: e67fcd671d56691628e2164d266c16d014aa78882d43a2e555b65816b347df8dc61c92df1986e716cd613b25dae7a35160b7e3995dd9a842a41e73a75d75540d

data/lib/brick/extensions.rb

@@ -451,7 +451,7 @@ module ActiveRecord
                        ord_expr = ord_part.gsub('^^^', _br_quoted_name(table_name))
                        s << Arel.sql(ord_expr)
                      else
-                      s << Arel.sql(_br_quoted_name(ord_expr))
+                       s << Arel.sql(_br_quoted_name(ord_part))
                      end
                      order_by_txt&.<<("Arel.sql(#{ord_expr.inspect})")
                    else # Expecting only Symbol
@@ -729,12 +729,12 @@ module ActiveRecord
               cc_part_idx -= 1
             end
           end
-          selects << "#{tbl_name}.#{cc_part.last} AS #{col_alias}"
+          selects << "#{_br_quoted_name(tbl_name)}.#{_br_quoted_name(cc_part.last)} AS #{_br_quoted_name(col_alias)}"
           cc_part << col_alias
         end
         # Add a key column unless we've already got it
         if key_alias && !used_col_aliases.key?(key_alias)
-          selects << "#{key_tbl_name}.#{dest_pk} AS #{key_alias}"
+          selects << "#{_br_quoted_name(key_tbl_name)}.#{_br_quoted_name(dest_pk)} AS #{_br_quoted_name(key_alias)}"
           used_col_aliases[key_alias] = nil
         end
         cc[2] = key_alias ? [key_klass, key_alias] : nil
@@ -1018,7 +1018,7 @@ JOIN (SELECT #{hm_selects.map { |s| _br_quoted_name("#{'br_t0.' if from_clause}#
         selects, where_values_hash, nil, translations: translations, join_array: join_array,
         cust_col_override: { '_br' => (descrip_cols = [pieces, my_dsl]) }
       )
-      order_values = "#{klass.table_name}.#{klass.primary_key}"
+      order_values = "#{_br_quoted_name(klass.table_name)}.#{_br_quoted_name(klass.primary_key)}"
       [self.select(selects), descrip_cols]
     end
 
@@ -2158,7 +2158,7 @@ class Object
 
             # %%% Allow params to define which columns to use for order_by
             # Overriding the default by providing a querystring param?
-            order_by = params['_brick_order']&.split(',')&.map(&:to_sym) || Object.send(:default_ordering, table_name, pk)
+            order_by = params['_brick_order']&.split(',')&.map(&:to_s) || Object.send(:default_ordering, table_name, pk)
 
             ar_relation = ActiveRecord.version < Gem::Version.new('4') ? real_model.preload : real_model.all
             params['_brick_is_api'] = true if (is_api = request.format == :js || current_api_root)

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

@@ -467,7 +467,7 @@ function onImagesLoaded(event) {
           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(Arel.sql("#{bt_class.table_name}.#{obj_pk = bt_class.primary_key}"))&.brick_list
+            collection, descrip_cols = bt_class&.order(Arel.sql("#{bt_class._br_quoted_name(bt_class.table_name)}.#{bt_class._br_quoted_name(obj_pk = bt_class.primary_key)}"))&.brick_list
             collection&.brick_(:each) do |obj|
               option_detail << [
                 obj.brick_descrip(

data/lib/brick/version_number.rb

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

data/lib/generators/brick/migrations_generator.rb

@@ -10,6 +10,7 @@ module Brick
   # Auto-generates migration files
   class MigrationsGenerator < ::Rails::Generators::Base
     include ::Brick::MigrationBuilder
+    include FancyGets
 
     desc 'Auto-generates migration files for an existing database.'
 

metadata

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