inline_forms 7.13.2 → 7.13.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 51e2a1ebb89510ad07098cd4bddbae4b312e8dba44deeae17a6594b9a78e7625
-  data.tar.gz: ab154d4f32670f0dc27cd2fb34a55f9799fe0ca6e31f18b669eb278b6295a6c7
+  metadata.gz: ec05aa4e1a1a1f21dcd041e8022943ec84af7c9b1ad4ccd90a6f57721f1732cd
+  data.tar.gz: 05fac9919e1730afa9797d2b343a0c76484cf28c6ac0ca210daed96312699902
 SHA512:
-  metadata.gz: bda0f225b64be4393f4de458a0757c563ec0ca9b769b106c071cfb18f9723965ad9cea96423f3fefa41f0a72104dda4212ae3b503bddf440cc0f16b6a6363f54
-  data.tar.gz: 5d6f660d7c86a30c20a4030ae637f8dcf1fb14075c00cdbd0c3e9b542ef00f1261f231267e16bb1deb4948f020a943dfb3edcf2ee161f853b18b0992c7965c2a
+  metadata.gz: 927830ff6c84fa2642f243df6956725584b990814ffc10a357983b253117cb2c1c99f3dc7b304438becd1a20e962cae72df6714f4b570cac4c1a54fe65dc3134
+  data.tar.gz: ad534d4e02d38e4dc6d607e18a92725cda21334a73b5a41bbea096dd5bb830fc23697a58928345ac32426fe5c3939a46106fb176068e1aa696637f2a7e7dcd6b

data/CHANGELOG.md

@@ -4,6 +4,30 @@ All notable changes to this project are documented in this file.
 
 ## [Unreleased]
 
+## [7.13.3] - 2026-05-19
+
+### Fixed
+
+- **Versions panel — "empty" `update` row whose Restore link did nothing.** Creating a record with a `rich_text` field (e.g. a new Photo with a `description`) produced a parent-side PaperTrail `update` version with `changeset == {}`. Cause: PaperTrail 16 tracks `:touch` by default (`on: [:create, :update, :destroy, :touch]`), and ActionText's `belongs_to :record, polymorphic: true, touch: true` calls `parent.touch` on every rich-text save. The resulting version reified to the same state, so clicking Restore was a visible no-op.
+  - **`lib/generators/templates/model.erb`**: emit `has_paper_trail on: [:create, :update, :destroy]` (drop `:touch`). New `rails g inline_forms …` models no longer create these noise versions. Existing apps need to re-apply the change to their models.
+  - **`lib/inline_forms_installer/installer_core.rb`** (`config/initializers/rich_text_paper_trail.rb`): mirror the opt-out on `ActionText::RichText` for symmetry against any future `touch: true` association pointing at rich-text rows.
+  - **`app/views/inline_forms/_versions_list.html.erb`**: defensive view-side gate — hide the Restore link when `version.changeset` is `nil` or empty after dropping `updated_at`. Covers legacy apps that still track `:touch`, plus any other empty-update source (e.g. CarrierWave callbacks that don't change attributes). The row stays visible in the audit trail; only the dead Restore link is suppressed.
+
+### Added
+
+- **`test/integration/example_app_apartment_versions_turbo_test.rb`** (installer template):
+  - `creating a record with a rich_text body does not append a touch-only parent update` — pins the generator-template change.
+  - `versions list hides Restore link on empty-changeset update rows` — pins the view-side guard against legacy `:touch` tracking.
+
+### Changed
+
+- **`InlineForms::VERSION`** and **`InlineFormsInstaller::VERSION`** → `7.13.3` (installer's `INLINE_FORMS_VERSION = VERSION` writes the `gem "inline_forms", "~> X.Y.Z"` pin into generated `Gemfile`s).
+
+### Verified
+
+- `gem build inline_forms.gemspec` → `inline_forms-7.13.3.gem`; `gem build inline_forms_installer.gemspec` → `inline_forms_installer-7.13.3.gem`.
+- `inline_forms create MyApp -d sqlite --example` → `bundle exec rails test` — **81 runs, 0 failures, 0 errors, 0 skips** (Ruby 4.0.4 / Rails 7.2.3.1).
+
 ## [7.13.2] - 2026-05-19
 
 ### Fixed

data/app/views/inline_forms/_versions_list.html.erb

@@ -55,9 +55,19 @@
           the record — primary records already have a dedicated Destroy
           button (superadmin), and rich_text content can be cleared by
           editing. Omitting the link here keeps the UI consistent and
-          avoids a NoMethodError in the controller's `save!`. -%>
+          avoids a NoMethodError in the controller's `save!`.
+
+          Empty-changeset update rows (e.g. PaperTrail `touch` events from
+          `belongs_to ..., touch: true` on associated ActionText::RichText
+          or CarrierWave callbacks) reify the same state — Restore on them
+          is a visible no-op. New generated models opt out of touch
+          tracking (`has_paper_trail on: [:create, :update, :destroy]`),
+          but legacy apps still have these rows in the audit trail; the
+          link is hidden here so the action is never offered. -%>
       <% if version.event == "create" %>
         &nbsp;
+      <% elsif version.changeset.nil? || version.changeset.except("updated_at").empty? %>
+        &nbsp;
       <% else %>
         <% row_id = inline_forms_row_turbo_frame_id(@object) %>
         <%= link_to t("inline_forms.view.restore"),

data/lib/generators/templates/model.erb

@@ -2,7 +2,14 @@ class <%= name %> < ApplicationRecord
   attr_reader :per_page
   @per_page = 7
   attr_writer :inline_forms_attribute_list
-  has_paper_trail
+  # PaperTrail 16 defaults to `on: [:create, :update, :destroy, :touch]`.
+  # ActionText's `belongs_to :record, polymorphic: true, touch: true` (set by
+  # `has_rich_text`) calls `parent.touch` on every rich-text save, which
+  # produced a parent-side `update` version with an empty changeset — visible
+  # in the inline_forms versions panel as a meaningless "empty" row whose
+  # Restore link reifies the same state (no-op). Excluding `:touch` here
+  # suppresses that noise without affecting real attribute updates.
+  has_paper_trail on: [:create, :update, :destroy]
 <%= @carrierwave_mounters if @carrierwave_mounters.length > 1 -%>
 <%= @belongs_to if @belongs_to.length > 1 -%>
 <%= @has_many if @has_many.length > 1 -%>

data/lib/inline_forms/version.rb

@@ -1,4 +1,4 @@
 # -*- encoding : utf-8 -*-
 module InlineForms
-  VERSION = "7.13.2"
+  VERSION = "7.13.3"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: inline_forms
 version: !ruby/object:Gem::Version
-  version: 7.13.2
+  version: 7.13.3
 platform: ruby
 authors:
 - Ace Suares