backstage 0.1.15 → 0.1.16

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: da02200425a97de115bbd7a4ff85a2cc907ab221647c3c9d22bb105ce690ff99
-  data.tar.gz: 4f53000b38b952e825cb5e82211162a7ce2149834e4efc671cf560595226593b
+  metadata.gz: c93addd91e80f76d095d7fd56542ebea748d27be3043cad7affcee189143718e
+  data.tar.gz: a9df1684acf09fb155e449e612e1e92c117874b581d4161f4b9c4b3749f610b4
 SHA512:
-  metadata.gz: 83f557e7644fa1e07487d3995854645f71454fee792023e9a344f24639fe222cef1d9c8a6bb130e8673dac4f7c4a942df5bb52a7ce06b0628c0a74461e3b1a59
-  data.tar.gz: dfd0f21995ca2a17e92016e86e6c975633b35b54c40a198e9edd8964116080a574055bd8480aceb589d5391739d87ee51cbdb65fcc600c636b3b8e1d4c31894c
+  metadata.gz: 48c81f6f435f100053c573b886ad40f6df175ae871302bbc381c161ade4f99bbe2062505de73133702d8a7c05e679d721e31d0a95f76b6555d3d1923df0f328a
+  data.tar.gz: 6653ba7832ea040c05081c5f46e9b4f8504503ed7bf117a49c7d111e95d144c56edbf9262f7864eed7d12094c482574e31ad0505a33323ea6141601e9d13d4e3

data/CHANGELOG.md

@@ -7,6 +7,15 @@ This project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.htm
 
 ## [Unreleased]
 
+## [0.1.16] — 2026-06-02
+
+### Fixed
+
+- Nested table now uses a `<template>` element instead of a static new row, so unsaved blank rows are never submitted unintentionally
+- New-row template includes readonly fields as plain inputs so new records can be fully populated via the Add button
+- Readonly nested fields are now permitted in Strong Parameters, so new records with readonly fields save correctly
+- `belongs_to :assoc, readonly: true` now propagates `readonly:` to the generated FK field
+
 ## [0.1.15] — 2026-05-27
 
 ### Added

data/app/controllers/backstage/resources_controller.rb

@@ -99,8 +99,7 @@ module Backstage
         elsif field.has_many?
           [{field.name => []}]
         elsif field.nested?
-          writable = field.nested_fields - field.nested_readonly_fields
-          [{"#{field.name}_attributes": [:id, :_destroy, *writable]}]
+          [{"#{field.name}_attributes": [:id, :_destroy, *field.nested_fields]}]
         else
           [field.name]
         end

data/app/views/backstage/fields/_nested.html.erb

@@ -27,13 +27,29 @@
         </td>
       </tr>
     <% end %>
-    <%= f.fields_for field.name, records.klass.new, child_index: records.size do |nf| %>
-      <tr data-nested-new-row>
-        <% field.nested_fields.reject { |col| field.nested_readonly_fields.include?(col) }.each do |col| %>
-          <td><%= nf.text_field col %></td>
+    <%# Template is not part of the DOM — never submitted until the user clicks Add %>
+    <template data-nested-template data-next-index="<%= records.size %>">
+      <tr>
+        <% field.nested_fields.each do |col| %>
+          <td>
+            <input type="text"
+                   name="<%= "#{f.object_name}[#{field.name}_attributes][__INDEX__][#{col}]" %>"
+                   value="" />
+          </td>
         <% end %>
-        <td></td>
+        <td>
+          <input type="hidden"
+                 name="<%= "#{f.object_name}[#{field.name}_attributes][__INDEX__][_destroy]" %>"
+                 value="0"
+                 data-nested-destroy-flag="true" />
+          <button type="button" data-nested-destroy aria-label="Remove">&times;</button>
+        </td>
       </tr>
-    <% end %>
+    </template>
+    <tr>
+      <td colspan="<%= field.nested_fields.size + 1 %>">
+        <button type="button" data-nested-add>+ Add</button>
+      </td>
+    </tr>
   </tbody>
 </table>

data/app/views/layouts/backstage/backstage.html.erb

@@ -26,12 +26,23 @@
                  })
           })
         })
-        document.querySelectorAll("[data-nested-destroy]").forEach(function(btn) {
-          btn.addEventListener("click", function() {
-            var row = btn.closest("tr")
-            row.querySelector("[data-nested-destroy-flag]").value = "1"
+        document.addEventListener("click", function(e) {
+          var destroyBtn = e.target.closest("[data-nested-destroy]")
+          if (destroyBtn) {
+            var row = destroyBtn.closest("tr")
+            var flag = row.querySelector("[data-nested-destroy-flag]")
+            if (flag) flag.value = "1"
             row.hidden = true
-          })
+          }
+          var addBtn = e.target.closest("[data-nested-add]")
+          if (addBtn) {
+            var table = addBtn.closest("table")
+            var tmpl = table.querySelector("[data-nested-template]")
+            var index = parseInt(tmpl.dataset.nextIndex, 10)
+            tmpl.dataset.nextIndex = index + 1
+            var html = tmpl.innerHTML.replace(/__INDEX__/g, index)
+            addBtn.closest("tr").insertAdjacentHTML("beforebegin", html)
+          }
         })
       })
     </script>

data/lib/backstage/resource_config.rb

@@ -63,7 +63,7 @@ module Backstage
     def belongs_to(name, **opts)
       assoc = AssociationConfig.new(name, :belongs_to, opts)
       @associations << assoc
-      fk_field = Field.new(assoc.foreign_key, :belongs_to, association: assoc)
+      fk_field = Field.new(assoc.foreign_key, :belongs_to, association: assoc, readonly: opts[:readonly])
       index_field = Field.new(name.to_sym, :belongs_to, association: assoc)
       @edit_fields.reject! { |f| f.name == fk_field.name }
       @index_fields.reject! { |f| f.name == fk_field.name || f.name == index_field.name }

data/lib/backstage/version.rb

@@ -1,3 +1,3 @@
 module Backstage
-  VERSION = "0.1.15"
+  VERSION = "0.1.16"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: backstage
 version: !ruby/object:Gem::Version
-  version: 0.1.15
+  version: 0.1.16
 platform: ruby
 authors:
 - Gareth James
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2026-05-27 00:00:00.000000000 Z
+date: 2026-06-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: railties