brick 1.0.41 → 1.0.44

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4455ebfde80fe8a4bd51f19438171abae2614e0740adba98ab39184952843a01
-  data.tar.gz: d5157748c37564156fc2f823cc3c4753b536491aa2cf81175683c8241fb2a2b9
+  metadata.gz: 0d43e8d88203f6496d0e86f57a3de2638f254a74da070d6a0f0ae107bf596a87
+  data.tar.gz: d5ce2e3814f36fa758762e3c4774032c70ce18a101cdd1e30622dd86c757ab1a
 SHA512:
-  metadata.gz: 2a649c78c3e597004b36d6c33f1cd812a712fcfd561dfe94994de28e489de81508f9fea30b0896b4447c4ce0ffd0877fb8c5d7e20a34011212fd797a3cbd02ca
-  data.tar.gz: 58ecdc4acb12abb77169a8fda8cb8146fceaedeacdf77f1cec992008f98b05412e3a9e2f5b6a9721880caa710ba94792a187d3587248eea4d9946d309c91be56
+  metadata.gz: 36b8bf590c1d980655b8c465e52659fdbcdf3e13b7050e46dcb41c2b3fba44ebc9ff14db823afddbb49b3e6b736a33fa1ef794ed6062aba4e35ade3cd95c5bbd
+  data.tar.gz: d1bd0954d0400068aaf8f7c83cc599f283fbe9c75e347fd4193d1fa3973c0d2bd3198f3265990cd3d24f406cd3b55f3174518bd6a991801c75a06d1c749cc157

data/lib/brick/extensions.rb

@@ -354,6 +354,7 @@ module ActiveRecord
           next if bt[2] # Polymorphic?
 
           # join_array will receive this relation name when calling #brick_parse_dsl
+          # binding.pry if bt.length > 2
           bt_descrip[bt.first] = if bt[1].is_a?(Array)
                                    bt[1].each_with_object({}) { |bt_class, s| s[bt_class] = bt_class.brick_parse_dsl(join_array, bt.first, translations, true) }
                                  else
@@ -364,7 +365,12 @@ module ActiveRecord
         hms.each do |k, hm|
           next if skip_klass_hms.key?(k)
 
-          hm_counts[k] = hm
+          if hm.macro == :has_one
+            # For our purposes a :has_one is similar enough to a :belongs_to that we can just join forces
+            bt_descrip[k] = { hm.klass => hm.klass.brick_parse_dsl(join_array, k, translations) }
+          else # Standard :has_many
+            hm_counts[k] = hm
+          end
         end
       end
 
@@ -650,10 +656,11 @@ class Object
   private
 
     def build_model(schema_name, inheritable_name, model_name, singular_table_name, table_name, relations, matching)
-      full_name = if (::Brick.config.schema_behavior[:multitenant] && Object.const_defined?('Apartment') && schema_name == Apartment.default_schema)
-                    relation = relations["#{schema_name}.#{matching}"]
-                    inheritable_name || model_name
-                  elsif schema_name.blank?
+      if ::Brick.config.schema_behavior[:multitenant] && Object.const_defined?('Apartment') &&
+         schema_name == Apartment.default_schema
+        relation = relations["#{schema_name}.#{matching}"]
+      end
+      full_name = if relation || schema_name.blank?
                     inheritable_name || model_name
                   else # Prefix the schema to the table name + prefix the schema namespace to the class name
                     schema_module = if schema_name.instance_of?(Module) # from an auto-STI namespace?
@@ -749,10 +756,10 @@ class Object
               if assoc[:is_bt]
                 invs = invs.first # Just do the first one of what would be multiple identical polymorphic belongs_to
               else
-                invs.each { |inv| build_bt_or_hm(relations, model_name, relation, hmts, assoc, inverse_assoc_name, inv, code) }
+                invs.each { |inv| build_bt_or_hm(full_name, relations, relation, hmts, assoc, inverse_assoc_name, inv, code) }
               end
             end
-            build_bt_or_hm(relations, model_name, relation, hmts, assoc, inverse_assoc_name, invs, code) unless invs.is_a?(Array)
+            build_bt_or_hm(full_name, relations, relation, hmts, assoc, inverse_assoc_name, invs, code) unless invs.is_a?(Array)
             hmts
           end
           # # Not NULLables
@@ -804,7 +811,7 @@ class Object
       [built_model, code]
     end
 
-    def build_bt_or_hm(relations, model_name, relation, hmts, assoc, inverse_assoc_name, inverse_table, code)
+    def build_bt_or_hm(full_name, relations, relation, hmts, assoc, inverse_assoc_name, inverse_table, code)
       singular_table_name = inverse_table&.singularize
       options = {}
       macro = if assoc[:is_bt]
@@ -826,8 +833,14 @@ class Object
                 end
                 if (inverse = assoc[:inverse])
                   inverse_assoc_name, _x = _brick_get_hm_assoc_name(relations[inverse_table], inverse)
-                  has_ones = ::Brick.config.has_ones&.fetch(inverse[:alternate_name].camelize, nil)
-                  if has_ones&.key?(singular_inv_assoc_name = ActiveSupport::Inflector.singularize(inverse_assoc_name))
+                  # If it's multitenant with something like:  public.____ ...
+                  if (it_parts = inverse_table.split('.')).length > 1 &&
+                     ::Brick.config.schema_behavior[:multitenant] && Object.const_defined?('Apartment') &&
+                     it_parts.first == Apartment.default_schema
+                    it_parts.shift # ... then ditch the generic schema name
+                  end
+                  has_ones = ::Brick.config.has_ones&.fetch(it_parts.join('/').singularize.camelize, nil)
+                  if has_ones&.key?(singular_inv_assoc_name = ActiveSupport::Inflector.singularize(inverse_assoc_name.tr('.', '_')))
                     inverse_assoc_name = if has_ones[singular_inv_assoc_name]
                                            need_inverse_of = true
                                            has_ones[singular_inv_assoc_name]
@@ -838,7 +851,6 @@ class Object
                 end
                 :belongs_to
               else
-                # need_class_name = ActiveSupport::Inflector.singularize(assoc_name) == ActiveSupport::Inflector.singularize(table_name.underscore)
                 # Are there multiple foreign keys out to the same table?
                 assoc_name, need_class_name = _brick_get_hm_assoc_name(relation, assoc)
                 if assoc.key?(:polymorphic)
@@ -846,8 +858,8 @@ class Object
                 else
                   need_fk = "#{ActiveSupport::Inflector.singularize(assoc[:inverse][:inverse_table].split('.').last)}_id" != assoc[:fk]
                 end
-                # fks[table_name].find { |other_assoc| other_assoc.object_id != assoc.object_id && other_assoc[:assoc_name] == assoc[assoc_name] }
-                if (has_ones = ::Brick.config.has_ones&.fetch(model_name, nil))&.key?(singular_assoc_name = ActiveSupport::Inflector.singularize(assoc_name))
+                has_ones = ::Brick.config.has_ones&.fetch(full_name, nil)
+                if has_ones&.key?(singular_assoc_name = ActiveSupport::Inflector.singularize(assoc_name.tr('.', '_')))
                   assoc_name = if (custom_assoc_name = has_ones[singular_assoc_name])
                                  need_class_name = custom_assoc_name != singular_assoc_name
                                  custom_assoc_name
@@ -905,10 +917,8 @@ class Object
         if plural_class_name == 'BrickGem'
           self.define_method :orphans do
             instance_variable_set(:@orphans, ::Brick.find_orphans(::Brick.set_db_schema(params)))
-            puts "BrickGemController #{action_name} #{params.inspect}"
-            # render inline: 'Brick gem!'
           end
-          return [new_controller_class, code + ' # BrickGem controller!']
+          return [new_controller_class, code + ' # BrickGem controller']
         end
 
         unless (is_swagger = plural_class_name == 'BrickSwagger') # && request.format == :json)
@@ -1061,10 +1071,11 @@ class Object
           if is_need_params
             code << "private\n"
             code << "  def #{params_name}\n"
-            code << "    params.require(:#{singular_table_name}).permit(#{model.columns_hash.keys.map { |c| c.to_sym.inspect }.join(', ')})\n"
+            code << "    params.require(:#{require_name = model.name.underscore.tr('/', '_')
+                             }).permit(#{model.columns_hash.keys.map { |c| c.to_sym.inspect }.join(', ')})\n"
             code << "  end\n"
             self.define_method(params_name) do
-              params.require(singular_table_name.to_sym).permit(model.columns_hash.keys)
+              params.require(require_name.to_sym).permit(model.columns_hash.keys)
             end
             private params_name
             # Get column names for params from relations[model.table_name][:cols].keys

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

@@ -112,7 +112,8 @@ module Brick
                               else
                                 hm_assoc.foreign_key
                               end
-                  if args.first == 'index'
+                  case args.first
+                  when 'index'
                     hms_columns << if hm_assoc.macro == :has_many
                                      set_ct = if skip_klass_hms.key?(assoc_name.to_sym)
                                                 'nil'
@@ -124,16 +125,20 @@ module Brick
                                      if hm_fk_name
 "<%= ct = #{set_ct}
      link_to \"#\{ct || 'View'\} #{assoc_name}\", #{hm_assoc.klass.name.underscore.tr('/', '_').pluralize}_path({ #{path_keys(hm_assoc, hm_fk_name, obj_name, pk)} }) unless ct&.zero? %>\n"
-                                     else
+                                     else # %%% Would be able to remove this when multiple foreign keys to same destination becomes bulletproof
 "#{assoc_name}\n"
                                      end
                                    else # has_one
-"<%= obj = #{obj_name}.#{hm.first}; link_to(obj.brick_descrip, obj) if obj %>\n"
+                                     # 0..62 because Postgres column names are limited to 63 characters
+"<%= descrips = @_brick_bt_descrip[#{hm.first.inspect}][ho_class = #{hm[1].klass.name}]
+     ho_txt = ho_class.brick_descrip(#{obj_name}, descrips[0..-2].map { |id| #{obj_name}.send(id.last[0..62]) }, (ho_id_col = descrips.last))
+     ho_id = ho_id_col.map { |id_col| #{obj_name}.send(id_col.to_sym) }
+     ho_id&.first ? link_to(ho_txt, send(\"#\{ho_class.base_class.name.underscore.tr('/', '_')\}_path\".to_sym, ho_id)) : ho_txt %>\n"
                                    end
-                  elsif args.first == 'show'
+                  when 'show', 'update'
                     hm_stuff << if hm_fk_name
                                   "<%= link_to '#{assoc_name}', #{hm_assoc.klass.name.underscore.tr('/', '_').pluralize}_path({ #{path_keys(hm_assoc, hm_fk_name, "@#{obj_name}", pk)} }) %>\n"
-                                else
+                                else # %%% Would be able to remove this when multiple foreign keys to same destination becomes bulletproof
                                   assoc_name
                                 end
                   end
@@ -213,6 +218,9 @@ table tbody tr.active-row {
   color: #009879;
 }
 
+td.val {
+  display: block;
+}
 a.show-arrow {
   font-size: 1.5em;
   text-decoration: none;
@@ -226,11 +234,27 @@ a.big-arrow {
   overflow: hidden;
 }
 .wide-input input[type=text] {
-  width: 100%;
+  display: inline-block;
+  width: 90%;
 }
 .dimmed {
   background-color: #C0C0C0;
 }
+
+#revertTemplate {
+  display: none;
+}
+svg.revert {
+  display: none;
+  margin-left: 0.25em;
+}
+.wide-input > svg.revert {
+  float: right;
+}
+input+svg.revert {
+  top: 0.5em;
+}
+
 input[type=submit] {
   background-color: #004998;
   color: #FFF;
@@ -239,7 +263,9 @@ input[type=submit] {
   text-align: right;
 }
 </style>
-<% def is_bcrypt?(val)
+<% is_includes_dates = nil
+
+def is_bcrypt?(val)
   val.is_a?(String) && val.length == 60 && val.start_with?('$2a$')
 end
 def hide_bcrypt(val, max_len = 200)
@@ -311,7 +337,13 @@ window.addEventListener(\"pageshow\", function() {
   });
 
   if (tblSelect) { // Always present
-    tblSelect.value = changeout(location.href)[schemaSelect ? 1 : 0];
+    var i = schemaSelect ? 1 : 0,
+        changeoutList = changeout(location.href);
+    for (; i < changeoutList.length; ++i) {
+      tblSelect.value = changeoutList[i];
+      if (tblSelect.value !== \"\") break;
+    }
+
     tblSelect.addEventListener(\"change\", function () {
       var lhr = changeout(location.href, null, this.value);
       if (brickSchema)
@@ -328,13 +360,13 @@ function changeout(href, param, value, trimAfter) {
     var pathParts = hrefParts[hrefParts.length - 1].split(\"/\");
     if (value === undefined)
       // A couple possibilities if it's namespaced, starting with two parts in the path -- and then try just one
-      return [pathParts.slice(1, 3).join('/'), pathParts.slice(1, 2)];
+      return [pathParts.slice(1, 3).join('/'), pathParts.slice(1, 2)[0]];
     else
       return hrefParts[0] + \"://\" + pathParts[0] + \"/\" + value;
   }
   if (trimAfter) {
     var pathParts = hrefParts[0].split(\"/\");
-    while (pathParts.lastIndexOf(trimAfter) != pathParts.length - 1) pathParts.pop();
+    while (pathParts.lastIndexOf(trimAfter) !== pathParts.length - 1) pathParts.pop();
     hrefParts[0] = pathParts.join(\"/\");
   }
   var params = hrefParts.length > 1 ? hrefParts[1].split(\"&\") : [];
@@ -478,7 +510,7 @@ if (headerTop) {
                          end
 # %%% Instead of our current "for Janet Leverling (Employee)" kind of link we previously had this code that did a "where x = 123" thing:
 #   (where <%= @_brick_params.each_with_object([]) { |v, s| s << \"#\{v.first\} = #\{v.last.inspect\}\" }.join(', ') %>)
-"#{css}
++"#{css}
 <p style=\"color: green\"><%= notice %></p>#{"
 <select id=\"schema\">#{schema_options}</select>" if ::Brick.config.schema_behavior[:multitenant] && ::Brick.db_schemas.length > 1}
 <select id=\"tbl\">#{table_options}</select>
@@ -504,8 +536,7 @@ if (headerTop) {
   <thead><tr>#{'<th></th>' if pk.present?}<%
      col_order = []
      @#{table_name}.columns.each do |col|
-       col_name = col.name
-       next if (#{(pk || []).inspect}.include?(col_name) && col.type == :integer && !bts.key?(col_name)) ||
+       next if (#{(pk || []).inspect}.include?(col_name = col.name) && col.type == :integer && !bts.key?(col_name)) ||
                ::Brick.config.metadata_columns.include?(col_name) || poly_cols.include?(col_name)
 
        col_order << col_name
@@ -518,12 +549,12 @@ if (headerTop) {
        else %><%=
          col_name %><%
        end
-  %></th><%
+    %></th><%
      end
      # Consider getting the name from the association -- h.first.name -- if a more \"friendly\" alias should be used for a screwy table name
   %>#{hms_headers.map do |h|
         if h.first.options[:through] && !h.first.through_reflection
-    "<th>#{h[1]} #{h[2]} %></th>"
+    "<th>#{h[1]} #{h[2]} %></th>" # %%% Would be able to remove this when multiple foreign keys to same destination becomes bulletproof
         else
     "<th>#{h[1]} <%= link_to('#{h[2]}', #{h.first.klass.name.underscore.tr('/', '_').pluralize}_path) %></th>"
         end
@@ -547,11 +578,11 @@ if (headerTop) {
            else
              bt_txt = (bt_class = bt[1].first.first).brick_descrip(
                # 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)
+               #{obj_name}, (descrips = @_brick_bt_descrip[bt.first][bt_class])[0..-2].map { |id| #{obj_name}.send(id.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 %>
+             bt_id = bt_id_col.map { |id_col| #{obj_name}.send(id_col.to_sym) } if bt_id_col&.present? %>
+          <%= bt_id&.first ? 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 %>
         <% end %>
       <% else %>
@@ -569,7 +600,7 @@ if (headerTop) {
 #{script}"
                        when 'orphans'
                          if is_orphans
-"#{css}
++"#{css}
 <p style=\"color: green\"><%= notice %></p>#{"
 <select id=\"schema\">#{schema_options}</select>" if ::Brick.config.schema_behavior[:multitenant] && ::Brick.db_schemas.length > 1}
 <select id=\"tbl\">#{table_options}</select>
@@ -584,7 +615,14 @@ if (headerTop) {
                          end
 
                        when 'show', 'update'
-"#{css}
++"#{css}
+
+<svg id=\"revertTemplate\" version=\"1.1\" xmlns=\"http://www.w3.org/2000/svg\" xmlns:xlink=\"http://www.w3.org/1999/xlink\"
+  width=\"32px\" height=\"32px\" viewBox=\"0 0 512 512\" xml:space=\"preserve\">
+<path id=\"revertPath\" fill=\"#2020A0\" d=\"M271.844,119.641c-78.531,0-148.031,37.875-191.813,96.188l-80.172-80.188v256h256l-87.094-87.094
+  c23.141-70.188,89.141-120.906,167.063-120.906c97.25,0,176,78.813,176,176C511.828,227.078,404.391,119.641,271.844,119.641z\" />
+</svg>
+
 <p style=\"color: green\"><%= notice %></p>#{"
 <select id=\"schema\">#{schema_options}</select>" if ::Brick.config.schema_behavior[:multitenant] && ::Brick.db_schemas.length > 1}
 <select id=\"tbl\">#{table_options}</select>
@@ -642,30 +680,39 @@ end
       <%= k %>
     <% end %>
     </th>
-    <td>
-    <% if bt
+    <td class=\"val\">
+    <% dt_pickers = { datetime: 'datetimepicker', timestamp: 'datetimepicker', time: 'timepicker', date: 'datepicker' }
+    if bt
       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 %>
       <%= 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
+            \"<span>Orphaned ID: #\{val}</span>\".html_safe
+          end %><svg class=\"revert\" width=\"1.5em\" viewBox=\"0 0 512 512\"><use xlink:href=\"#revertPath\" /></svg>
+    <% else case (col_type = #{model_name}.column_for_attribute(k).type)
       when :string, :text %>
         <% if is_bcrypt?(val) # || .readonly? %>
           <%= hide_bcrypt(val, 1000) %>
         <% else %>
-          <div class=\"wide-input\"><%= f.text_field k.to_sym %></div>
+          <div class=\"wide-input\"><%= f.text_field k.to_sym %><svg class=\"revert\" width=\"1.5em\" viewBox=\"0 0 512 512\"><use xlink:href=\"#revertPath\" /></svg></div>
         <% end %>
       <% when :boolean %>
-        <%= f.check_box k.to_sym %>
-      <% when :integer, :decimal, :float, :date, :datetime, :time, :timestamp
-         # What happens when keys are UUID?
-         # Postgres naturally uses the +uuid_generate_v4()+ function from the uuid-ossp extension
-         # If it's not yet enabled then:  enable_extension 'uuid-ossp'
-         # ActiveUUID gem created a new :uuid type %>
+        <%= f.check_box k.to_sym %><svg class=\"revert\" width=\"1.5em\" viewBox=\"0 0 512 512\"><use xlink:href=\"#revertPath\" /></svg>
+      <% when :integer, :decimal, :float
+        <%= if col_type == :integer
+              f.text_field k.to_sym, { pattern: '\\d*', class: 'check-validity' }
+            else
+              f.number_field k.to_sym
+            end %><svg class=\"revert\" width=\"1.5em\" viewBox=\"0 0 512 512\"><use xlink:href=\"#revertPath\" /></svg>
+      <% when *dt_pickers.keys
+           is_includes_dates = true %>
+        <%= f.text_field k.to_sym, { class: dt_pickers[col_type] } %><svg class=\"revert\" width=\"1.5em\" viewBox=\"0 0 512 512\"><use xlink:href=\"#revertPath\" /></svg>
+      <% when :uuid %>
+        # Postgres naturally uses the +uuid_generate_v4()+ function from the uuid-ossp extension
+        # If it's not yet enabled then:  enable_extension 'uuid-ossp'
+        # ActiveUUID gem created a new :uuid type %>
         <%= val %>
       <% when :binary, :primary_key %>
       <% end %>
@@ -682,6 +729,7 @@ end
   <% end %>
 
   #{hms_headers.each_with_object(+'') do |hm, s|
+    # %%% Would be able to remove this when multiple foreign keys to same destination becomes bulletproof
     next if hm.first.options[:through] && !hm.first.through_reflection
 
     if (pk = hm.first.klass.primary_key)
@@ -707,6 +755,66 @@ end
 #{script}"
 
                        end
+              inline << "
+<% if is_includes_dates %>
+<link rel=\"stylesheet\" href=\"https://cdn.jsdelivr.net/npm/flatpickr/dist/flatpickr.min.css\">
+<style>
+.flatpickr-calendar {
+  background: #A0FFA0;
+}
+</style>
+<script src=\"https://cdn.jsdelivr.net/npm/flatpickr\"></script>
+<script>
+flatpickr(\".datepicker\");
+flatpickr(\".datetimepicker\", {enableTime: true});
+flatpickr(\".timepicker\", {enableTime: true, noCalendar: true});
+</script>
+<% end %>
+<script>
+document.querySelectorAll(\"input, select\").forEach(function (inp) {
+  var origVal = getInpVal(),
+      prevVal = origVal;
+  var revert;
+  if ((revert = ((inp.tagName === \"SELECT\" && inp.nextElementSibling.nextElementSibling) ||
+       inp.nextElementSibling ||
+       inp.parentElement.nextElementSibling)) && revert.tagName.toLowerCase() === \"svg\")
+    revert.addEventListener(\"click\", function (e) {
+      if (inp.type === \"checkbox\")
+        inp.checked = origVal;
+      else
+        inp.value = origVal;
+      revert.style.display = \"none\";
+      if (inp._flatpickr)
+        inp._flatpickr.setDate(origVal);
+      else
+        inp.focus();
+    });
+  inp.addEventListener(inp.type === \"checkbox\" ? \"change\" : \"input\", function (e) {
+    if(inp.className.split(\" \").indexOf(\"check-validity\") > 0) {
+      if (inp.checkValidity()) {
+        prevVal = getInpVal();
+      } else {
+        inp.value = prevVal;
+      }
+    } else {
+      // If this is the result of changing an hour or minute, keep the calendar open.
+      // And if it was the result of selecting a date, the calendar can now close.
+      if (inp._flatpickr &&
+           // Test only for changes in the date portion of a date or datetime
+           ((giv = getInpVal()) && (giv1 = giv.split(' ')[0])) !== (prevVal && prevVal.split(' ')[0]) &&
+           giv1.indexOf(\":\") < 0 // (definitely not any part of a time thing)
+         )
+        inp._flatpickr.close();
+      prevVal = getInpVal();
+    }
+    // Show or hide the revert button
+    if (revert) revert.style.display = getInpVal() === origVal ? \"none\" : \"inline-block\";
+  });
+  function getInpVal() {
+    return inp.type === \"checkbox\" ? inp.checked : inp.value;
+  }
+});
+</script>"
               # As if it were an inline template (see #determine_template in actionview-5.2.6.2/lib/action_view/renderer/template_renderer.rb)
               keys = options.has_key?(:locals) ? options[:locals].keys : []
               handler = ActionView::Template.handler_for_extension(options[:type] || 'erb')

data/lib/brick/version_number.rb

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

data/lib/generators/brick/install_generator.rb

@@ -224,8 +224,8 @@ module Brick
 # # Database schema to use when analysing existing data, such as deriving a list of polymorphic classes in the case that
 # # it wasn't originally specified.
 # Brick.schema_behavior = :namespaced
-#{Brick.config.schema_behavior ? "Brick.schema_behavior = { multitenant: { schema_to_analyse: #{
-  Brick.config.schema_behavior[:multitenant][:schema_to_analyse].inspect}" :
+#{Brick.config.schema_behavior.present? ? "Brick.schema_behavior = { multitenant: { schema_to_analyse: #{
+  Brick.config.schema_behavior[:multitenant]&.fetch(:schema_to_analyse, nil).inspect}" :
 "# Brick.schema_behavior = { multitenant: { schema_to_analyse: 'engineering'"
 } } }
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: brick
 version: !ruby/object:Gem::Version
-  version: 1.0.41
+  version: 1.0.44
 platform: ruby
 authors:
 - Lorin Thwaits
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-07-04 00:00:00.000000000 Z
+date: 2022-07-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord