brick 1.0.103 → 1.0.105

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 74743982216a9827513b4aaa68fe0887c9ba3e7b78c45ff52f0a2f2b99dc7b88
-  data.tar.gz: 3857a95e891e7561d285dd3ecbdeb6c4c16cc9a8205e7b733b55867ed7a69f64
+  metadata.gz: 48fc69654d7cbdf5c7feb9ba7803b7a953c6c0ccd3c176614fe9a61994bb39e1
+  data.tar.gz: f3a4837d334cf97cb491cacd658cb6e6d8e3f3a5c640f80c151a083bf6811b7a
 SHA512:
-  metadata.gz: 8b71aa5b1d4da8c4aa1ce88af6d0767c12bdf72b2457594575795d4b9b3f36d1bfb2adef82413a3bc80e041813e7bc6c05768016c5e4cfc197e3bbb730113934
-  data.tar.gz: 6399e040cb4b54eac4d7a09b907cfba11037a4b3391dd3a86b4845e79bd0c4d4e169586c095fdaaaa8506f346628072c9c4c9b6d2ed60c3306a58866e6f9d030
+  metadata.gz: e51134710bae78f033b9dcf7827cd7c087cfd642ef8bc6244e2429f22158a80e9f804022bc703f55fa55f85cbc009ff25440289205ade1b415f5ba0af3d44c77
+  data.tar.gz: d6069676ad70e8401f7597b92b641f0f95e3c28fffa54747f630932500c778d7b4ac967c5bbd70059647d11d398598362d9c0bb5c82ef0500b1025371104a578

data/lib/brick/config.rb

@@ -94,13 +94,13 @@ module Brick
       @mutex.synchronize { @enable_api = enable }
     end
 
-    def api_root
+    def api_roots
       ver = api_version
-      @mutex.synchronize { @api_root || "/api/#{ver}/" }
+      @mutex.synchronize { @api_roots || ["/api/#{ver}/"] }
     end
 
-    def api_root=(path)
-      @mutex.synchronize { @api_root = path }
+    def api_roots=(path)
+      @mutex.synchronize { @api_roots = path }
     end
 
     def api_version

data/lib/brick/extensions.rb

@@ -257,11 +257,11 @@ module ActiveRecord
       assoc_html_name ? "#{assoc_name}-#{link}".html_safe : link
     end
 
-    def self._brick_index(mode = nil)
+    def self._brick_index(mode = nil, separator = '_')
       tbl_parts = ((mode == :singular) ? table_name.singularize : table_name).split('.')
       tbl_parts.shift if ::Brick.apartment_multitenant && tbl_parts.length > 1 && tbl_parts.first == ::Brick.apartment_default_tenant
       tbl_parts.unshift(::Brick.config.path_prefix) if ::Brick.config.path_prefix
-      index = tbl_parts.map(&:underscore).join('_')
+      index = tbl_parts.map(&:underscore).join(separator)
       # Rails applies an _index suffix to that route when the resource name is singular
       index << '_index' if mode != :singular && index == index.singularize
       index
@@ -396,6 +396,8 @@ module ActiveRecord
   end
 
   class Relation
+    attr_accessor :_brick_page_num
+
     # Links from ActiveRecord association pathing names over to real table correlation names
     # that get chosen when the AREL AST tree is walked.
     def brick_links
@@ -785,8 +787,21 @@ JOIN (SELECT #{hm_selects.map { |s| "#{'br_t0.' if from_clause}#{s}" }.join(', '
         end
         self.order_values |= final_order_by # Same as:  order!(*final_order_by)
       end
-      # Don't want to get too carried away just yet
-      self.limit_value = 1000 # Same as:  limit!(1000)
+      if (page = params['_brick_page']&.to_i)
+        page = 1 if page < 1
+        limit = params['_brick_page_size'] || 1000
+        offset = (page - 1) * limit.to_i
+      else
+        offset = params['_brick_offset']
+        limit = params['_brick_limit']
+      end
+      if offset.is_a?(Numeric) || offset&.present?
+        offset = offset.to_i
+        self.offset_value = offset unless offset == 0
+        @_brick_page_num = (offset / limit.to_i) + 1 if limit&.!= 0 && (offset % limit.to_i) == 0
+      end
+      # By default just 1000 rows  (Like doing:  limit!(1000)  but this way is compatible with AR <= 4.2)
+      self.limit_value = limit&.to_i || 1000 unless limit.is_a?(String) && limit.empty?
       wheres unless wheres.empty? # Return the specific parameters that we did use
     end
 
@@ -1511,7 +1526,11 @@ class Object
         self.protect_from_forgery unless: -> { self.request.format.js? }
         unless is_avo
           self.define_method :index do
-            if (is_openapi || request.env['REQUEST_PATH'].start_with?(::Brick.api_root)) &&
+            current_api_root = ::Brick.config.api_roots.find do |ar|
+              request.path.start_with?(ar) || # Exact match?
+              request.path.split('/')[-2] == ar.split('/').last # Version at least matches?
+            end
+            if (current_api_root || is_openapi) &&
                !params&.key?('_brick_schema') &&
                (referrer_params = request.env['HTTP_REFERER']&.split('?')&.last&.split('&')&.map { |x| x.split('=') }).present?
               if params
@@ -1523,6 +1542,7 @@ class Object
             _schema, @_is_show_schema_list = ::Brick.set_db_schema(params || api_params)
 
             if is_openapi
+              current_api_ver = current_api_root.split('/').last&.[](1..-1).to_i
               json = { 'openapi': '3.0.1', 'info': { 'title': Rswag::Ui.config.config_object[:urls].last&.fetch(:name, 'API documentation'), 'version': ::Brick.config.api_version },
                        'servers': [
                          { 'url': '{scheme}://{defaultHost}', 'variables': {
@@ -1531,15 +1551,19 @@ class Object
                          } }
                        ]
                      }
-              json['paths'] = relations.inject({}) do |s, relation|
+              json['paths'] = relations.each_with_object({}) do |relation, s|
                 unless ::Brick.config.enable_api == false
+                  next if (api_vers = relation.last.fetch(:api, nil)) &&
+                          !(api_ver_path = api_vers[current_api_ver])
+
+                  relation_name = api_ver_path || relation.first.tr('.', '/')
                   table_description = relation.last[:description]
-                  s["#{::Brick.config.api_root}#{relation.first.tr('.', '/')}"] = {
+                  s["#{current_api_root}#{relation_name}"] = {
                     'get': {
                       'summary': "list #{relation.first}",
                       'description': table_description,
                       'parameters': relation.last[:cols].map do |k, v|
-                                      param = { 'name' => k, 'schema': { 'type': v.first } }
+                                      param = { in: 'query', 'name' => k, 'schema': { 'type': v.first } }
                                       if (col_descrip = relation.last.fetch(:col_descrips, nil)&.fetch(k, nil))
                                         param['description'] = col_descrip
                                       end
@@ -1549,7 +1573,7 @@ class Object
                     }
                   }
 
-                  s["#{::Brick.config.api_root}#{relation.first.tr('.', '/')}/{id}"] = {
+                  s["#{current_api_root}#{relation_name}/{id}"] = {
                     'patch': {
                       'summary': "update a #{relation.first.singularize}",
                       'description': table_description,
@@ -1563,7 +1587,6 @@ class Object
                       'responses': { '200': { 'description': 'successful' } }
                     }
                   } unless relation.last.fetch(:isView, nil)
-                  s
                 end
               end
               render inline: json.to_json, content_type: request.format
@@ -1577,9 +1600,10 @@ class Object
               end
               render inline: exported_csv, content_type: request.format
               return
-            elsif request.format == :js || request.path.start_with?('/api/') # Asking for JSON?
+            elsif request.format == :js || current_api_root # Asking for JSON?
+              # %%% Add:  where, order, page, page_size, offset, limit
               data = (model.is_view? || !Object.const_defined?('DutyFree')) ? model.limit(1000) : model.df_export(model.brick_import_template)
-              render inline: data.to_json, content_type: request.format == '*/*' ? 'application/json' : request.format
+              render inline: { data: data }.to_json, content_type: request.format == '*/*' ? 'application/json' : request.format
               return
             end
 

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

@@ -228,8 +228,8 @@ window.addEventListener(\"popstate\", linkSchemas);
             # When available, add a clickable brick icon to go to the Brick version of the page
             PanelComponent.class_exec do
               alias _brick_init initialize
-              def initialize(*args)
-                _brick_init(*args)
+              def initialize(*args, **kwargs)
+                _brick_init(*args, **kwargs)
                 @name = BrickTitle.new(@name, self)
               end
             end
@@ -263,6 +263,16 @@ window.addEventListener(\"popstate\", linkSchemas);
                 _brick_resource_view_path
               end
             end
+
+            module Concerns::HasFields
+              class_methods do
+                alias _brick_field field
+                def field(name, *args, **kwargs, &block)
+                  kwargs.merge!(args.pop) if args.last.is_a?(Hash)
+                  _brick_field(name, **kwargs, &block)
+                end
+              end
+            end
           end # module Avo
 
           # Steer any Avo-related controller/action based URL lookups to the Avo RouteSet
@@ -454,6 +464,12 @@ window.addEventListener(\"popstate\", linkSchemas);
               table_options << "<option value=\"#{prefix}brick_orphans\">(Orphans)</option>".html_safe if is_orphans
               table_options << "<option value=\"#{prefix}brick_orphans\">(Crosstab)</option>".html_safe if is_crosstab
               css = +"<style>
+#titleSticky {
+  position: sticky;
+  display: inline-block;
+  left: 0;
+}
+
 h1, h3 {
   margin-bottom: 0;
 }
@@ -465,7 +481,6 @@ h1, h3 {
   cursor: pointer;
 }
 #mermaidErd {
-  position: relative;
   display: none;
 }
 #mermaidErd .exclude {
@@ -866,6 +881,7 @@ if (grid) {
   // });
 }
 function setHeaderSizes() {
+  document.getElementById(\"titleBox\").style.width = grid.clientWidth;
   // console.log(\"start\");
   // See if the headerTop is already populated
   // %%% Grab the TRs from headerTop, clear it out, do this stuff, add them back
@@ -1112,13 +1128,16 @@ erDiagram
 %></title>
 </head>
 <body>
+<div id=\"titleBox\"><div id=\"titleSticky\">
 <p style=\"color: green\"><%= notice %></p>#{"
 #{schema_options}" if schema_options}
 <select id=\"tbl\">#{table_options}</select>
 <table id=\"resourceName\"><tr>
-  <td><h1><%= model.name %></h1></td>
+  <td><h1><%= td_count = 2
+              model.name %></h1></td>
   <td id=\"imgErd\" title=\"Show ERD\"></td>
-  <% if Object.const_defined?('Avo') && ::Avo.respond_to?(:railtie_namespace) %>
+  <% if Object.const_defined?('Avo') && ::Avo.respond_to?(:railtie_namespace)
+       td_count += 1 %>
     <td><%= link_to_brick(
         avo_svg,
         { index_proc: Proc.new do |avo_model|
@@ -1127,7 +1146,9 @@ erDiagram
           title: \"#\{model.name} in Avo\" }
       ) %></td>
   <% end %>
-</tr></table>#{template_link}<%
+</tr><%= if (page_num = @#{table_name}._brick_page_num)
+           \"<tr><td colspan=\\\"#\{td_count}\\\">Page #\{page_num}</td></tr>\".html_safe
+         end %></table>#{template_link}<%
    if description.present? %><%=
      description %><br><%
    end
@@ -1162,6 +1183,7 @@ erDiagram
     });
   </script>
 <% end %>
+</div></div>
 #{erd_markup}
 
 <%= # Consider getting the name from the association -- hm.first.name -- if a more \"friendly\" alias should be used for a screwy table name
@@ -1327,7 +1349,7 @@ end
       options[:url] = send(\"#\{#{model_name}._brick_index(:singular)}_path\".to_sym, obj) if ::Brick.config.path_prefix
  %>
   <br><br>
-  <%= form_for(obj.becomes(#{model_name}), options) do |f| %>
+<%= form_for(obj.becomes(#{model_name}), options) do |f| %>
   <table class=\"shadow\">
   <% has_fields = false
     @#{obj_name}.attributes.each do |k, val|
@@ -1432,7 +1454,7 @@ end
            is_revert = false %>
       <% else %>
         <%= is_revert = false
-           display_value(col_type, val) %>
+           display_value(col_type, val).html_safe %>
       <% end
        end
        if is_revert
@@ -1449,7 +1471,7 @@ end
     <tr><td colspan=\"2\">(No displayable fields)</td></tr>
   <% end %>
   </table>
-<%   end %>
+<%  end %>
 
 #{unless args.first == 'new'
   # Was:  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?' } }"
@@ -1538,7 +1560,7 @@ flatpickr(\".timepicker\", {enableTime: true, noCalendar: true});
   if (imgErd) imgErd.addEventListener(\"click\", showErd);
   function showErd() {
     imgErd.style.display = \"none\";
-    mermaidErd.style.display = \"inline-block\";
+    mermaidErd.style.display = \"block\";
     if (mermaidCode) return; // Cut it short if we've already rendered the diagram
 
     mermaidCode = document.createElement(\"SCRIPT\");

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

@@ -56,85 +56,85 @@ module Brick::Rails::FormTags
          end
     out << "</tr></thead>
   <tbody>"
-  # %%% Have once gotten this error with MSSQL referring to http://localhost:3000/warehouse/cold_room_temperatures__archive
-  #     ActiveRecord::StatementTimeout in Warehouse::ColdRoomTemperatures_Archive#index
-  #     TinyTds::Error: Adaptive Server connection timed out
-  #     (After restarting the server it worked fine again.)
-  relation.each do |obj|
-    out << "<tr>\n"
-    out << "<td>#{link_to('⇛', send("#{klass._brick_index(:singular)}_path".to_sym,
-                                    pk.map { |pk_part| obj.send(pk_part.to_sym) }), { class: 'big-arrow' })}</td>\n" if pk.present?
-    sequence.each do |col_name|
-      val = obj.attributes[col_name]
-      out << '<td'
-      out << ' class=\"dimmed\"' unless cols.key?(col_name) || (cust_col = cust_cols[col_name]) || 
-                                                 (col_name.is_a?(Symbol) && bts.key?(col_name)) # HOT
-      out << '>'
-      if (bt = bts[col_name])
-        if bt[2] # Polymorphic?
-          bt_class = obj.send("#{bt.first}_type")
-          base_class_underscored = (::Brick.existing_stis[bt_class] || bt_class).constantize.base_class._brick_index(:singular)
-          poly_id = obj.send("#{bt.first}_id")
-          out << link_to("#{bt_class} ##{poly_id}", send("#{base_class_underscored}_path".to_sym, poly_id)) if poly_id
-        else # BT or HOT
-          bt_class = bt[1].first.first
-          descrips = bt_descrip[bt.first][bt_class]
-          bt_id_col = if descrips.nil?
-                        puts "Caught it in the act for obj / #{col_name}!"
-                      elsif descrips.length == 1
-                        [obj.class.reflect_on_association(bt.first)&.foreign_key]
-                      else
-                        descrips.last
-                      end
-          bt_txt = bt_class.brick_descrip(
-            # 0..62 because Postgres column names are limited to 63 characters
-            obj, descrips[0..-2].map { |id| obj.send(id.last[0..62]) }, bt_id_col
-          )
-          bt_txt = display_binary(bt_txt).html_safe if bt_txt&.encoding&.name == 'ASCII-8BIT'
-          bt_txt ||= "<span class=\"orphan\">&lt;&lt; Orphaned ID: #{val} >></span>" if val
-          bt_id = bt_id_col&.map { |id_col| obj.respond_to?(id_sym = id_col.to_sym) ? obj.send(id_sym) : id_col }
-          out << (bt_id&.first ? link_to(bt_txt, send("#{bt_class.base_class._brick_index(:singular)}_path".to_sym, bt_id)) : bt_txt || '')
-        end
-      elsif (hms_col = hms_cols[col_name])
-        if hms_col.length == 1
-          out << hms_col.first
-        else
-          hm_klass = (col = cols[col_name])[1]
-          if col[2] == 'HO'
-            descrips = bt_descrip[col_name.to_sym][hm_klass]
-            if (ho_id = (ho_id_col = descrips.last).map { |id_col| obj.send(id_col.to_sym) })&.first
-              ho_txt = hm_klass.brick_descrip(obj, descrips[0..-2].map { |id| obj.send(id.last[0..62]) }, ho_id_col)
-              out << link_to(ho_txt, send("#{hm_klass.base_class._brick_index(:singular)}_path".to_sym, ho_id))
-            end
+    # %%% Have once gotten this error with MSSQL referring to http://localhost:3000/warehouse/cold_room_temperatures__archive
+    #     ActiveRecord::StatementTimeout in Warehouse::ColdRoomTemperatures_Archive#index
+    #     TinyTds::Error: Adaptive Server connection timed out
+    #     (After restarting the server it worked fine again.)
+    relation.each do |obj|
+      out << "<tr>\n"
+      out << "<td>#{link_to('⇛', send("#{klass._brick_index(:singular)}_path".to_sym,
+                                      pk.map { |pk_part| obj.send(pk_part.to_sym) }), { class: 'big-arrow' })}</td>\n" if pk.present?
+      sequence.each do |col_name|
+        val = obj.attributes[col_name]
+        out << '<td'
+        out << ' class=\"dimmed\"' unless cols.key?(col_name) || (cust_col = cust_cols[col_name]) ||
+                                          (col_name.is_a?(Symbol) && bts.key?(col_name)) # HOT
+        out << '>'
+        if (bt = bts[col_name])
+          if bt[2] # Polymorphic?
+            bt_class = obj.send("#{bt.first}_type")
+            base_class_underscored = (::Brick.existing_stis[bt_class] || bt_class).constantize.base_class._brick_index(:singular)
+            poly_id = obj.send("#{bt.first}_id")
+            out << link_to("#{bt_class} ##{poly_id}", send("#{base_class_underscored}_path".to_sym, poly_id)) if poly_id
+          else # BT or HOT
+            bt_class = bt[1].first.first
+            descrips = bt_descrip[bt.first][bt_class]
+            bt_id_col = if descrips.nil?
+                          puts "Caught it in the act for obj / #{col_name}!"
+                        elsif descrips.length == 1
+                          [obj.class.reflect_on_association(bt.first)&.foreign_key]
+                        else
+                          descrips.last
+                        end
+            bt_txt = bt_class.brick_descrip(
+              # 0..62 because Postgres column names are limited to 63 characters
+              obj, descrips[0..-2].map { |id| obj.send(id.last[0..62]) }, bt_id_col
+            )
+            bt_txt = display_binary(bt_txt).html_safe if bt_txt&.encoding&.name == 'ASCII-8BIT'
+            bt_txt ||= "<span class=\"orphan\">&lt;&lt; Orphaned ID: #{val} >></span>" if val
+            bt_id = bt_id_col&.map { |id_col| obj.respond_to?(id_sym = id_col.to_sym) ? obj.send(id_sym) : id_col }
+            out << (bt_id&.first ? link_to(bt_txt, send("#{bt_class.base_class._brick_index(:singular)}_path".to_sym, bt_id)) : bt_txt || '')
+          end
+        elsif (hms_col = hms_cols[col_name])
+          if hms_col.length == 1
+            out << hms_col.first
           else
-            if (ct = obj.send(hms_col[1].to_sym)&.to_i)&.positive?
-              out << "#{link_to("#{ct || 'View'} #{hms_col.first}",
-                                   send("#{hm_klass._brick_index}_path".to_sym,
-                                        hms_col[2].each_with_object({}) { |v, s| s[v.first] = v.last.is_a?(String) ? v.last : obj.send(v.last) })
-                               )}\n"
+            hm_klass = (col = cols[col_name])[1]
+            if col[2] == 'HO'
+              descrips = bt_descrip[col_name.to_sym][hm_klass]
+              if (ho_id = (ho_id_col = descrips.last).map { |id_col| obj.send(id_col.to_sym) })&.first
+                ho_txt = hm_klass.brick_descrip(obj, descrips[0..-2].map { |id| obj.send(id.last[0..62]) }, ho_id_col)
+                out << link_to(ho_txt, send("#{hm_klass.base_class._brick_index(:singular)}_path".to_sym, ho_id))
+              end
+            else
+              if (ct = obj.send(hms_col[1].to_sym)&.to_i)&.positive?
+                out << "#{link_to("#{ct || 'View'} #{hms_col.first}",
+                                  send("#{hm_klass._brick_index}_path".to_sym,
+                                       hms_col[2].each_with_object({}) { |v, s| s[v.first] = v.last.is_a?(String) ? v.last : obj.send(v.last) })
+                                 )}\n"
+              end
             end
           end
+        elsif (col = cols[col_name]).is_a?(ActiveRecord::ConnectionAdapters::Column)
+          binding.pry if col.is_a?(Array)
+          col_type = col&.sql_type == 'geography' ? col.sql_type : col&.type
+          out << display_value(col_type || col&.sql_type, val).to_s
+        elsif cust_col
+          data = cust_col.first.map { |cc_part| obj.send(cc_part.last) }
+          cust_txt = klass.brick_descrip(cust_col[-2], data)
+          if (link_id = obj.send(cust_col.last[1]) if cust_col.last)
+            out << link_to(cust_txt, send("#{cust_col.last.first._brick_index(:singular)}_path", link_id))
+          else
+            out << (cust_txt || '')
+          end
+        else # Bad column name!
+          out << '?'
         end
-      elsif (col = cols[col_name]).is_a?(ActiveRecord::ConnectionAdapters::Column)
-        binding.pry if col.is_a?(Array)
-        col_type = col&.sql_type == 'geography' ? col.sql_type : col&.type
-        out << display_value(col_type || col&.sql_type, val).to_s
-      elsif cust_col
-        data = cust_col.first.map { |cc_part| obj.send(cc_part.last) }
-        cust_txt = klass.brick_descrip(cust_col[-2], data)
-        if (link_id = obj.send(cust_col.last[1]) if cust_col.last)
-          out << link_to(cust_txt, send("#{cust_col.last.first._brick_index(:singular)}_path", link_id))
-        else
-          out << (cust_txt || '')
-        end
-      else # Bad column name!
-        out << '?'
+        out << '</td>'
       end
-      out << '</td>'
+      out << '</tr>'
     end
-  out << '</tr>'
-  end
-  out << "  </tbody>
+    out << "  </tbody>
 </table>
 "
     out.html_safe
@@ -143,6 +143,16 @@ module Brick::Rails::FormTags
   def link_to_brick(*args, **kwargs)
     return unless ::Brick.config.mode == :on
 
+    kwargs.merge!(args.pop) if args.last.is_a?(Hash)
+    # Avoid infinite recursion
+    if (visited = kwargs.fetch(:visited, nil))
+      return if visited.key?(object_id)
+
+      kwargs[:visited][object_id] = nil
+    else
+      kwargs[:visited] = {}
+    end
+
     text = ((args.first.is_a?(String) || args.first.is_a?(Proc)) && args.shift) || args[1]
     text = text.call if text.is_a?(Proc)
     klass_or_obj = ((args.first.is_a?(ActiveRecord::Relation) ||
@@ -192,13 +202,14 @@ module Brick::Rails::FormTags
       app_routes = Rails.application.routes # In case we're operating in another engine, reference the application since Brick routes are placed there.
       if (klass_or_obj&.is_a?(Class) && klass_or_obj < ActiveRecord::Base) ||
          (klass_or_obj&.is_a?(ActiveRecord::Base) && klass_or_obj.new_record? && (klass_or_obj = klass_or_obj.class))
-        path = (proc = kwargs[:index_proc]) ? proc.call(klass_or_obj) : "#{app_routes.path_for(controller: klass_or_obj.base_class._brick_index, action: :index)}#{filter}"
+        path = (proc = kwargs[:index_proc]) ? proc.call(klass_or_obj) : "#{app_routes.path_for(controller: klass_or_obj.base_class._brick_index(nil, '/'), action: :index)}#{filter}"
         lt_args = [text || "Index for #{klass_or_obj.name.pluralize}", path]
       else
         # If there are multiple incoming parameters then last one is probably the actual ID, and first few might be some nested tree of stuff leading up to it
-        path = (proc = kwargs[:show_proc]) ? proc.call(klass_or_obj) : "#{app_routes.path_for(controller: klass_or_obj.class.base_class._brick_index, action: :show, id: klass_or_obj)}#{filter}"
+        path = (proc = kwargs[:show_proc]) ? proc.call(klass_or_obj) : "#{app_routes.path_for(controller: klass_or_obj.class.base_class._brick_index(nil, '/'), action: :show, id: klass_or_obj)}#{filter}"
         lt_args = [text || "Show this #{klass_or_obj.class.name}", path]
       end
+      kwargs.delete(:visited)
       link_to(*lt_args, **kwargs)
     else
       # puts "Warning:  link_to_brick could not find a class for \"#{controller_path}\" -- consider setting @_brick_model within that controller."
@@ -215,7 +226,7 @@ module Brick::Rails::FormTags
       if links.length == 1 # If there's only one match then use any text that was supplied
         link_to_brick(text || links.first.last.join('/'), links.first.first, **kwargs)
       else
-        links.map { |k, v| link_to_brick(v.join('/'), v, **kwargs) }.join(' &nbsp; ').html_safe
+        links.each_with_object([]) { |v, s| s << link if link = link_to_brick(v.join('/'), v, **kwargs) }.join(' &nbsp; ').html_safe
       end
     end
   end # link_to_brick

data/lib/brick/version_number.rb

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

data/lib/brick.rb

@@ -334,12 +334,17 @@ module Brick
 
     # @api public
     def api_root=(path)
-      Brick.config.api_root = path
+      Brick.config.api_roots = [path]
     end
 
     # @api public
-    def api_root
-      Brick.config.api_root
+    def api_roots=(paths)
+      Brick.config.api_roots = paths
+    end
+
+    # @api public
+    def api_roots
+      Brick.config.api_roots
     end
 
     # @api public
@@ -657,19 +662,65 @@ In config/initializers/brick.rb appropriate entries would look something like:
                              # Turn something like {"::Spouse"=>"Person", "::Friend"=>"Person"} into {"Person"=>["Spouse", "Friend"]}
                              s[v.last] << v.first[2..-1] unless v.first.end_with?('::')
                            end
+          versioned_views = {} # Track which views have already been done for each api_root
           ::Brick.relations.each do |k, v|
             next if !(controller_name = v.fetch(:resource, nil)&.pluralize) || existing_controllers.key?(controller_name)
 
+            schema_name = v.fetch(:schema, nil)
             options = {}
             options[:only] = [:index, :show] if v.key?(:isView)
-            # First do the API routes
+            # First do the API routes if necessary
             full_resource = nil
-            if (schema_name = v.fetch(:schema, nil))
-              full_resource = "#{schema_name}/#{v[:resource]}"
-              send(:get, "#{::Brick.api_root}#{full_resource}", { to: "#{controller_prefix}#{schema_name}/#{controller_name}#index" }) if Object.const_defined?('Rswag::Ui')
-            else
-              # Normally goes to something like:  /api/v1/employees
-              send(:get, "#{::Brick.api_root}#{v[:resource]}", { to: "#{controller_prefix}#{controller_name}#index" }) if Object.const_defined?('Rswag::Ui')
+            ::Brick.api_roots&.each do |api_root|
+              api_done_views = (versioned_views[api_root] ||= {})
+              found = nil
+              view_relation = nil
+              # If it's a view then see if there's a versioned one available by searching for resource names
+              # versioned with the closest number (equal to or less than) compared with our API version number.
+              if v.key?(:isView) && (ver = k.match(/^v([\d_]*)/).captures.first)[-1] == '_'
+                next if api_done_views.key?(unversioned = k[ver.length + 1..-1])
+
+                # # if ().length.positive? # Does it have a version number?
+                # try_num = (ver_num = (ver = ver[1..-1].gsub('_', '.')).to_d)
+
+                # Expect that the last item in the path generally holds versioning information
+                api_ver = api_root.split('/')[-1]&.gsub('_', '.')
+                vn_idx = api_ver.rindex(/[^\d._]/) # Position of the first numeric digit at the end of the version number
+                # Was:  .to_d
+                api_ver_num = api_ver[vn_idx + 1..-1].gsub('_', '.').to_i # Attempt to turn something like "v3" into the decimal value 3
+                # puts [api_ver, vn_idx, api_ver_num, unversioned].inspect
+
+                next if ver.to_i > api_ver_num # Don't surface any newer views in an older API
+
+                api_ver_num -= 1 until api_ver_num.zero? ||
+                                       (view_relation = ::Brick.relations.fetch(
+                                         found = "v#{api_ver_num}_#{k[ver.length + 1..-1]}", nil
+                                       ))
+                api_done_views[unversioned] = nil # Mark that for this API version this view is done
+
+                # puts "Found #{found}" if view_relation
+                # If we haven't found "v3_view_name" or "v2_view_name" or so forth, at the last
+                # fall back to simply looking for "v_view_name", and then finally  "view_name".
+                unversioned = "v_#{unversioned}"
+                view_relation ||= ::Brick.relations.fetch(found = unversioned,
+                                                          ::Brick.relations.fetch(found = unversioned, nil)
+                                                         )
+                if found && view_relation && k != (found = unversioned)
+                  view_relation[:api][api_ver_num] = found
+                end
+              end
+
+              # view_ver_num = if (first_part = k.split('_').first) =~ /^v[\d_]+/
+              #                  first_part[1..-1].gsub('_', '.').to_i
+              #                end
+              controller_name = view_relation.fetch(:resource, nil)&.pluralize if view_relation
+              if schema_name
+                full_resource = "#{schema_name}/#{found || v[:resource]}"
+                send(:get, "#{api_root}#{full_resource}", { to: "#{controller_prefix}#{schema_name}/#{controller_name}#index" })
+              else
+                # Normally goes to something like:  /api/v1/employees
+                send(:get, "#{api_root}#{found || v[:resource]}", { to: "#{controller_prefix}#{controller_name}#index" })
+              end
             end
 
             # Track routes being built
@@ -716,15 +767,19 @@ In config/initializers/brick.rb appropriate entries would look something like:
           unless ::Brick.routes_done
             if Object.const_defined?('Rswag::Ui')
               rswag_path = ::Rails.application.routes.routes.find { |r| r.app.app == Rswag::Ui::Engine }&.instance_variable_get(:@path_formatter)&.instance_variable_get(:@parts)&.join
-              if (doc_endpoint = Rswag::Ui.config.config_object[:urls]&.last)
+              first_endpoint_parts = nil
+              (doc_endpoints = Rswag::Ui.config.config_object[:urls]&.uniq!)&.each do |doc_endpoint|
                 puts "Mounting OpenApi 3.0 documentation endpoint for \"#{doc_endpoint[:name]}\" on #{doc_endpoint[:url]}"
                 send(:get, doc_endpoint[:url], { to: 'brick_openapi#index' })
                 endpoint_parts = doc_endpoint[:url]&.split('/')
-                if rswag_path && endpoint_parts
-                  puts "API documentation now available when navigating to:  /#{endpoint_parts&.find(&:present?)}/index.html"
+                first_endpoint_parts ||= endpoint_parts
+              end
+              if doc_endpoints.present?
+                if rswag_path && first_endpoint_parts
+                  puts "API documentation now available when navigating to:  /#{first_endpoint_parts&.find(&:present?)}/index.html"
                 else
                   puts "In order to make documentation available you can put this into your routes.rb:"
-                  puts "  mount Rswag::Ui::Engine => '/#{endpoint_parts&.find(&:present?) || 'api-docs'}'"
+                  puts "  mount Rswag::Ui::Engine => '/#{first_endpoint_parts&.find(&:present?) || 'api-docs'}'"
                 end
               else
                 sample_path = rswag_path || '/api-docs'

data/lib/generators/brick/install_generator.rb

@@ -159,8 +159,9 @@ if ActiveRecord::Base.respond_to?(:brick_select)
   # Brick.enable_views = true # Setting this to \"false\" will disable views in development
 
   # # If The Brick sees that RSwag gem is present, it allows for API resources to be automatically served out.
-  # # You can configure the root path for these resources:
-  # ::Brick.api_root = '/api/v1/'
+  # # You can configure one or more root path(s) for these resources, and when there are multiple then an attempt
+  # # is made to return data from that version of the view or table name, or the most recent prior to that version:
+  # ::Brick.api_roots = ['/api/v1/']
   # # You may also want to add an OpenAPI 3.0 documentation endpoint using Rswag::Ui:
   # Rswag::Ui.configure do |config|
   #   config.swagger_endpoint '/api-docs/v1/swagger.json', 'API V1 Docs'
@@ -288,16 +289,18 @@ if ActiveRecord::Base.respond_to?(:brick_select)
 
   # # POLYMORPHIC ASSOCIATIONS
 
-  # # 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.
+  # # Polymorphic associations are set up by providing a model name and polymorphic association name#{poly}
+
+  # # For multi-tenant databases that use a separate schema for each tenant, a single representative database schema
+  # # can be analysed to determine the range of polymorphic classes that can be used for each association.  Hopefully
+  # # the schema chosen is one loaded with existing data that is representative of all possible polymorphic
+  # # associations.
   # Brick.schema_behavior = :namespaced
 #{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'"
 } } }
 
-  # # Polymorphic associations are set up by providing a model name and polymorphic association name#{poly}
-
   # # DEFAULT ROOT ROUTE
 
   # # If a default route is not supplied, Brick attempts to find the most \"central\" table and wires up the default

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: brick
 version: !ruby/object:Gem::Version
-  version: 1.0.103
+  version: 1.0.105
 platform: ruby
 authors:
 - Lorin Thwaits
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-01-09 00:00:00.000000000 Z
+date: 2023-01-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord