brick 1.0.87 → 1.0.88

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b178117c13e6d4feb053c27334b558285765a68a8bdc00c0a1d7a0a708ab3e70
-  data.tar.gz: b4b90b2cef91df1797d3a6534d6658914000b01542fe244d7560d31922c2125e
+  metadata.gz: 9e346878b216d9e54ee0ca15be13ec0c2348f8cdf81d14a1eb4bc24ccb07aadb
+  data.tar.gz: 92b54e7227fad318508104690158b5c756cc37171e7eeaaaba41d7da97c5518f
 SHA512:
-  metadata.gz: a7866357360543e3af339aced9e7ff8c1dff803f3f92eebb12926b5cefb4da18415003b729ad5aca3b7771a145035963bd1e49d8a46096bb943a161e4b33ca79
-  data.tar.gz: b2978cf99f81c2ad59b04f656c189763d230542e9eb9d9eb7d3a922c609e79354926357320967f7de53a1a6cc4ec55b61811395138e632cb0f1a96993a31e926
+  metadata.gz: a8797a5e8924e3a89fd958f05d1bd442cf457428257cd697e438e1e6fe93427bc035177cea79fad47c6484427b30d2e3dc778f7a2166bc0fe27d383ce8659c1c
+  data.tar.gz: 0bd6d3cc1ad92a1ec897b1524fde2e3f6849c783705312908d206460f9060677c93c295f3f3b89ddd514e2b3d5faeae4900472b0f4a1419e57e91046a190b3f7

data/lib/brick/extensions.rb

@@ -841,34 +841,9 @@ if Object.const_defined?('ActionView')
                       args.first) ||
                      @_brick_model
       # If not provided, do a best-effort to automatically determine the resource class or object
-      sti_type = nil
       filter_parts = []
       klass_or_obj ||= begin
-                         res_names = ::Brick.relations.each_with_object({}) do |v, s|
-                           v_parts = v.first.split('.')
-                           v_parts.shift if v_parts.first == 'public'
-                           s[v_parts.join('.')] = v.first
-                         end
-                         c_path_parts = controller_path.split('/')
-                         klass = nil
-                         while c_path_parts.present?
-                           possible_c_path = c_path_parts.join('.')
-                           possible_c_path_singular = c_path_parts[0..-2] + [c_path_parts.last.singularize]
-                           possible_sti = possible_c_path_singular.join('/').camelize
-                           break if (
-                                      res_name = res_names[possible_c_path] ||
-                                                 ((klass = Brick.config.sti_namespace_prefixes.key?("::#{possible_sti}") && possible_sti.constantize) &&
-                                                  (sti_type = possible_sti)) ||
-                                                 # %%% Used to have the more flexible:  (DidYouMean::SpellChecker.new(dictionary: res_names.keys).correct(possible_c_path)).first
-                                                 res_names[possible_c_path] || res_names[possible_c_path_singular.join('.')]
-                                    ) &&
-                                    (
-                                      klass ||
-                                      ((rel = ::Brick.relations.fetch(res_name, nil)) &&
-                                      (klass ||= rel[:class_name]&.constantize))
-                                    )
-                           c_path_parts.shift
-                         end
+                         klass, sti_type = ::Brick.ctrl_to_klass(controller_path)
                          if klass
                            type_col = klass.inheritance_column # Usually 'type'
                            filter_parts << "#{type_col}=#{sti_type}" if sti_type && klass.column_names.include?(type_col)
@@ -1811,7 +1786,11 @@ end.class_exec do
     initializer_loaded = false
     orig_schema = nil
     if (relations = ::Brick.relations).empty?
-      # If there's schema things configured then we only expect our initializer to be named exactly this
+      # Very first thing, load inflections since we'll be using .pluralize and .singularize on table and model names
+      if File.exist?(inflections = ::Rails.root.join('config/initializers/inflections.rb'))
+        load inflections
+      end
+      # Now the Brick initializer since there may be important schema things configured
       if File.exist?(brick_initializer = ::Rails.root.join('config/initializers/brick.rb'))
         initializer_loaded = load brick_initializer
       end

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

@@ -502,6 +502,16 @@ window.addEventListener(\"pageshow\", function() {
   }
 });
 
+// Add \"Are you sure?\" behaviour to any data-confirm buttons out there
+document.querySelectorAll(\"input[type=submit][data-confirm]\").forEach(function (btn) {
+  btn.addEventListener(\"click\", function (evt) {
+    if (!confirm(this.getAttribute(\"data-confirm\"))) {
+      evt.preventDefault();
+      return false;
+    }
+  });
+});
+
 function changeout(href, param, value, trimAfter) {
   var hrefParts = href.split(\"?\");
   var params = hrefParts.length > 1 ? hrefParts[1].split(\"&\") : [];
@@ -556,13 +566,19 @@ if (grid) {
         if (lastHighCell) lastHighCell.classList.remove(\"highlight\");
       }
       var lastHighHeader = gridHighHeader;
-      gridHighHeader = headerCols[gridHighCell.cellIndex];
-      if (lastHighHeader !== gridHighHeader) {
+      if ((gridHighHeader = headerCols[gridHighCell.cellIndex]) && lastHighHeader !== gridHighHeader) {
         if (gridHighHeader) gridHighHeader.classList.add(\"highlight\");
         if (lastHighHeader) lastHighHeader.classList.remove(\"highlight\");
       }
     }
   }
+  // // LESS TOUCHY NAVIGATION BACK OR FORWARD IN HISTORY WHEN USING MOUSE WHEEL
+  // grid.addEventListener(\"wheel\", function (evt) {
+  //   grid.scrollLeft += evt.deltaX;
+  //   document.body.scrollTop += (evt.deltaY * 0.6);
+  //   evt.preventDefault();
+  //   return false;
+  // });
 }
 function setHeaderSizes() {
   // console.log(\"start\");
@@ -1185,7 +1201,7 @@ end
 <%   end %>
 
 #{unless args.first == 'new'
-  confirm_are_you_sure = ActionView.version < ::Gem::Version.new('7.0') ? "data: { confirm: 'Are you sure?' }" : "form: { data: { turbo_confirm: 'Are you sure?' } }"
+  confirm_are_you_sure = ActionView.version < ::Gem::Version.new('7.0') ? "data: { confirm: 'Delete #{model_name} -- Are you sure?' }" : "form: { data: { turbo_confirm: 'Delete #{model_name} -- Are you sure?' } }"
   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
@@ -1250,7 +1266,7 @@ flatpickr(\".timepicker\", {enableTime: true, noCalendar: true});
     if (mermaidCode) return; // Cut it short if we've already rendered the diagram
 
     mermaidCode = document.createElement(\"SCRIPT\");
-    mermaidCode.setAttribute(\"src\", \"https://cdn.jsdelivr.net/npm/mermaid/dist/mermaid.min.js\");
+    mermaidCode.setAttribute(\"src\", \"https://cdn.jsdelivr.net/npm/mermaid@9.1.7/dist/mermaid.min.js\");
     mermaidCode.addEventListener(\"load\", function () {
       mermaid.initialize({
         startOnLoad: true,

data/lib/brick/version_number.rb

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

data/lib/brick.rb

@@ -563,6 +563,41 @@ In config/initializers/brick.rb appropriate entries would look something like:
       end
       puts "\n"
     end
+
+    # Attempt to determine an ActiveRecord::Base class and additional STI information when given a controller's path
+    def ctrl_to_klass(ctrl_path, res_names = {})
+      klass = nil
+      sti_type = nil
+
+      if res_names.empty?
+        ::Brick.relations.each_with_object({}) do |v, s|
+          v_parts = v.first.split('.')
+          v_parts.shift if v_parts.first == 'public'
+          res_names[v_parts.join('.')] = v.first
+        end
+      end
+
+      c_path_parts = ctrl_path.split('/')
+      while c_path_parts.present?
+        possible_c_path = c_path_parts.join('.')
+        possible_c_path_singular = c_path_parts[0..-2] + [c_path_parts.last.singularize]
+        possible_sti = possible_c_path_singular.join('/').camelize
+        break if (
+                   res_name = res_names[possible_c_path] ||
+                              ((klass = Brick.config.sti_namespace_prefixes.key?("::#{possible_sti}") && possible_sti.constantize) &&
+                               (sti_type = possible_sti)) ||
+                              # %%% Used to have the more flexible:  (DidYouMean::SpellChecker.new(dictionary: res_names.keys).correct(possible_c_path)).first
+                              res_names[possible_c_path] || res_names[possible_c_path_singular.join('.')]
+                 ) &&
+                 (
+                   klass ||
+                   ((rel = ::Brick.relations.fetch(res_name, nil)) &&
+                   (klass ||= rel[:class_name]&.constantize))
+                 )
+        c_path_parts.shift
+      end
+      [klass, sti_type]
+    end
   end
 
   module RouteSet

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: brick
 version: !ruby/object:Gem::Version
-  version: 1.0.87
+  version: 1.0.88
 platform: ruby
 authors:
 - Lorin Thwaits
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-11-01 00:00:00.000000000 Z
+date: 2022-11-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord