brick 1.0.114 → 1.0.115

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5006a6dabf5c9e86965e57a40ae344f9d67c2fc5e76caec8a00e978117089062
-  data.tar.gz: 5bd0fef75e305514b4ef22e4930660cdbf9385c22b07ee92d6eb77a626a13943
+  metadata.gz: 6e92e1445b603e8611ad927b8864b6ff04fd819d2f2243051d109d0894838e0d
+  data.tar.gz: b76bc7adf85927a5ff4988027a0d7ff1d40b84d4a80531d23390867bcca0fc5e
 SHA512:
-  metadata.gz: b0621761d15baec575a9cd16fdb1b22e9a1d4784308c071d9878121ac841f6cf7419cb6fbcc210afad2ce98a69814c27279dc951a5d84750e7573e203d2f806a
-  data.tar.gz: ed4b5c992076b4f28c85ab8af48bab847ab80105f2999371475529219cc6b0053444dc722a700d686fc7ed7ea39d3d939cb448c43d798bde18a0b5cfd80be23e
+  metadata.gz: 9caa3d9fdbc5229123058ec526482bf4c6926c42a44012d974a26d4f94f6f0e2f1dd2040ee76b1fbec21e431ea479694030365a1982878c6100245a607803260
+  data.tar.gz: 0e09bb3d9556bc8b4a442eccc9f19a374684c901dc0f9769030313ebdc57248f31a994e5deb7dbde394f066187b961eab169545e3466ab0851cb6df7636c62f1

data/lib/brick/extensions.rb

@@ -972,7 +972,10 @@ Module.class_exec do
                   end
     # puts "#{self.name} - #{args.first}"
     desired_classname = (self == Object || !name) ? requested : "#{name}::#{requested}"
-    if ((is_defined = self.const_defined?(args.first)) && (possible = self.const_get(args.first)) && possible.name == desired_classname) ||
+    if ((is_defined = self.const_defined?(args.first)) && (possible = self.const_get(args.first)) &&
+        # Reset `possible` if it's a controller request that's not a perfect match
+        # Was:  (possible = nil)  but changed to #local_variable_set in order to suppress the "= should be ==" warning
+        (possible.name == desired_classname || (is_controller && binding.local_variable_set(:possible, nil)))) ||
        # Try to require the respective Ruby file
        ((filename = ActiveSupport::Dependencies.search_for_file(desired_classname.underscore) ||
                     (self != Object && ActiveSupport::Dependencies.search_for_file((desired_classname = requested).underscore))
@@ -983,7 +986,7 @@ Module.class_exec do
        # then return what we've found.
        (is_defined && !::Brick.is_eager_loading) # Used to also have:   && possible != self
       if (!brick_root && (filename || possible.instance_of?(Class))) ||
-         (possible.instance_of?(Module) && possible.module_parent == self) ||
+         (possible.instance_of?(Module) && possible&.module_parent == self) ||
          (possible.instance_of?(Class) && possible == self) # Are we simply searching for ourselves?
         return possible
       end

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

@@ -82,6 +82,15 @@ function linkSchemas() {
   }
 };
 "
+      BRICK_SVG = "<svg version=\"1.1\" style=\"display: inline; padding-left: 0.5em;\" xmlns=\"http://www.w3.org/2000/svg\" xmlns:xlink=\"http://www.w3.org/1999/xlink\"
+  viewBox=\"0 0 58 58\" height=\"1.4em\" xml:space=\"preserve\">
+<g>
+  <polygon style=\"fill:#C2615F;\" points=\"58,15.831 19.106,35.492 0,26.644 40,6\"/>
+  <polygon style=\"fill:#6D4646;\" points=\"19,52 0,43.356 0,26.644 19,35\"/>
+  <polygon style=\"fill:#894747;\" points=\"58,31.559 19,52 19,35 58,15.831\"/>
+</g>
+</svg>
+".html_safe
 
       # paths['app/models'] << 'lib/brick/frameworks/active_record/models'
       config.brick = ActiveSupport::OrderedOptions.new
@@ -246,15 +255,7 @@ window.addEventListener(\"popstate\", linkSchemas);
               end
               def to_s
                 @_name.to_s.html_safe + @vc.instance_variable_get(:@__vc_helpers)&.link_to_brick(nil,
-                  "<svg version=\"1.1\" style=\"display: inline; padding-left: 0.5em;\" xmlns=\"http://www.w3.org/2000/svg\" xmlns:xlink=\"http://www.w3.org/1999/xlink\"
-  viewBox=\"0 0 58 58\" height=\"1.4em\" xml:space=\"preserve\">
-<g>
-  <polygon style=\"fill:#C2615F;\" points=\"58,15.831 19.106,35.492 0,26.644 40,6\"/>
-  <polygon style=\"fill:#6D4646;\" points=\"19,52 0,43.356 0,26.644 19,35\"/>
-  <polygon style=\"fill:#894747;\" points=\"58,31.559 19,52 19,35 58,15.831\"/>
-</g>
-</svg>
-".html_safe,
+                  BRICK_SVG,
                   { title: "#{@_name} in Brick" }
                 )
               end
@@ -298,7 +299,14 @@ window.addEventListener(\"popstate\", linkSchemas);
                     )
                 nil
               else # This is either a non-Avo request or a proper Avo request, so carry on
-                _brick_url_for(options, *args)
+                begin
+                  _brick_url_for(options, *args)
+                rescue
+                  # Last-ditch effort in case we were in yet a different RouteSet
+                  unless (rar = ::Rails.application.routes) == self
+                    rar.url_for(options, *args)
+                  end
+                end
               end
             end
           end
@@ -314,14 +322,34 @@ window.addEventListener(\"popstate\", linkSchemas);
                 ::Brick.relations.each do |k, v|
                   next if k == 'active_admin_comments'
 
-                  if (class_name = Object.const_get(v.fetch(:class_name, nil)))
-                    ::ActiveAdmin.register(class_name) { config.clear_batch_actions! }
+                  begin
+                    if (class_name = Object.const_get(v.fetch(:class_name, nil)))
+                      ::ActiveAdmin.register(class_name) { config.clear_batch_actions! }
+                    end
+                  rescue
                   end
                 end
                 _brick_routes(*args)
               end
             end
           end
+          ::ActiveAdmin::Views::TitleBar.class_exec do
+            alias _brick_build_title_tag build_title_tag
+            def build_title_tag
+              if klass = begin
+                           aa_id = helpers.instance_variable_get(:@current_tab)&.id
+                           ::Brick.relations.fetch(aa_id, nil)&.fetch(:class_name, nil)&.constantize
+                         rescue
+                         end
+                h2((@title + link_to_brick(nil,
+                  BRICK_SVG, # This would do well to be sized a bit smaller
+                  { title: "#{@_name} in Brick" }
+                )).html_safe)
+              else
+                _brick_build_title_tag # Revert to the original
+              end
+            end
+          end
         end
 
         # ====================================

data/lib/brick/version_number.rb

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

data/lib/brick.rb

@@ -660,7 +660,7 @@ In config/initializers/brick.rb appropriate entries would look something like:
   module RouteSet
     def finalize!
       routeset_to_use = ::Rails.application.routes
-      return super unless self == routeset_to_use
+      return super if self != routeset_to_use || ::Brick.routes_done
 
       path_prefix = ::Brick.config.path_prefix
       existing_controllers = routes.each_with_object({}) do |r, s|
@@ -867,9 +867,12 @@ In config/initializers/brick.rb appropriate entries would look something like:
 
           # Trestle compatibility
           if Object.const_defined?('Trestle') && ::Trestle.config.options&.key?(:site_title) &&
-             !Object.const_defined?("#{resource_name.camelize}Admin")
-            ::Trestle.resource(res_sym = k.to_sym) do
-              menu { item res_sym, icon: "fa fa-star" }
+             !Object.const_defined?("#{(res_name = resource_name.tr('/', '_')).camelize}Admin")
+            begin
+              ::Trestle.resource(res_sym = res_name.to_sym, model: class_name&.constantize) do
+                menu { item res_sym, icon: "fa fa-star" }
+              end
+            rescue
             end
           end
         end
@@ -920,7 +923,6 @@ In config/initializers/brick.rb appropriate entries would look something like:
             end
           end
 
-          ::Brick.routes_done = true
           puts "\n" if tables.present? || views.present?
           if tables.present?
             puts "Classes that can be built from tables:#{' ' * (table_class_length - 38)}  Path:"
@@ -934,6 +936,7 @@ In config/initializers/brick.rb appropriate entries would look something like:
           end
         end
       end
+      ::Brick.routes_done = true
       super
     end
   end
@@ -976,6 +979,21 @@ module ::Rails
   end
 end
 
+# # Support for when a table or view is named just 's'
+# ActiveSupport::Inflector.class_eval do
+#   class << self
+#     alias _brick_singularize singularize
+#     def singularize(word, locale = :en)
+#       if word.present? && ((ret = _brick_singularize(word, locale)).empty? || ret[-1] == '/')
+#         # puts word
+#         # binding.pry if ret.empty? || ret[-1] == '/'
+#         ret << 's'
+#       end
+#       ret
+#     end
+#   end
+# end
+
 # Major compatibility fixes for ActiveRecord < 4.2
 # ================================================
 ActiveSupport.on_load(:active_record) do

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: brick
 version: !ruby/object:Gem::Version
-  version: 1.0.114
+  version: 1.0.115
 platform: ruby
 authors:
 - Lorin Thwaits
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-02-19 00:00:00.000000000 Z
+date: 2023-02-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord