inline_forms 8.1.3 → 8.1.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 977ef4ea7379846bb2c176ef945cdaddb16816aed800d283b0655d08f0e0cfa8
-  data.tar.gz: 5dc4778948c43efa3b4348f7c906137aae4e25545229727ed83172bab4d776f2
+  metadata.gz: a67a5a2a208fff953c0ffe9145425d30e7ae8fefae955271436033c1b2a99771
+  data.tar.gz: a7b359f732f395f368be30c35277b9eecbd1cbd253e388c8f59165b207297a6c
 SHA512:
-  metadata.gz: 513ca378e912320a20a982210cad7f625788f79858a75310ea04e164de83a6124ca25ce43af7162065efc161e8693ec0464ac8f96e05659b46b53a2564647ba2
-  data.tar.gz: e4ec7479913bdeaa570064e1946884d00481774a07758af7e86170342464a55f2269c029822edecaf4f31130c3e822ca18338e8a34d2ef5d60e8d240e170525f
+  metadata.gz: 49e2f7a7fb54b318959f5c003793e4190c812f527fa5a36ed9160260ee217d0761bf1f5c28bdf1cc241f650dcde82662b084140903551f17aadf7c2f0a539498
+  data.tar.gz: bc33c4b2ee408988f01c166f2fcfc5ac07e684c3b966501db41edec5b7eb0e42e8ca50904c9fa5936005974bf0ca527c9abf2086771fea6e051a8fd255ab169f

data/CHANGELOG.md

@@ -4,6 +4,73 @@ All notable changes to this project are documented in this file.
 
 ## [Unreleased]
 
+## [8.1.5] - 2026-05-27
+
+### Changed (hard-breaking)
+
+- **`inline_forms_attribute_list` rows are now 2-element by default.** The empty 2nd element (the unused human-name string) has been removed from every row across the gem, installer, generators, helpers, views, validators, tests, and docs. Labels have come from `Klass.human_attribute_name(attribute)` via locale files for a long time, so the slot was redundant. The migration in 8.1.5 makes that explicit by dropping the slot entirely.
+
+  Old shape:
+
+  ```ruby
+  [ :name, "name", :text_field ],
+  [ :sex, "sex", :radio_button, { 1 => 'f', 2 => 'm' } ],
+  [ :sex, "sex", :dropdown_with_values, values_hash, options_disabled ],
+  ```
+
+  New shape:
+
+  ```ruby
+  [ :name, :text_field ],
+  [ :sex, :radio_button, { 1 => 'f', 2 => 'm' } ],
+  [ :sex, :dropdown_with_values, values_hash, options_disabled ],
+  ```
+
+  **This is a hard break.** Consumers that read positions inside the row (`attributes.assoc(:foo)[3]` for `attribute_values`, `attributes.assoc(:foo)[4]` for `dropdown_with_values`' `options_disabled`) have shifted to `[2]` / `[3]`. Any model file left in the old 3/4/5-element shape will silently misbehave (the empty string `""` will be returned where the values hash or `options_disabled` array used to be), so every existing `inline_forms_attribute_list` in downstream apps must be rewritten.
+
+  Minimal migration for the common 3-element shape — run this once over `app/models/**/*.rb`:
+
+  ```ruby
+  # In your app's root, with a backup or in a clean git tree:
+  Dir.glob("app/models/**/*.rb").each do |path|
+    src = File.read(path)
+    new = src.gsub(/\[\s*(:\w+)\s*,\s*["'][^"']*["']\s*,\s*(:\w+)\s*\]/) { "[ #{$1}, #{$2} ]" }
+    File.write(path, new) if new != src
+  end
+  ```
+
+  For 4/5-element rows (`dropdown_with_values`, `radio_button`, `scale_*`, `slider_*`, `check_box`, `simple_file_field`, `must_be_a_value` validator targets), drop only the empty 2nd element and keep the trailing `values` / `options_disabled` arguments unchanged.
+
+### Changed (mechanical follow-through)
+
+- **`InlineFormsGenerator`** and **`InlineFormsAddtoGenerator`** emit the new 2-element row shape.
+- **Installer** (`inline_forms_installer/installer_core.rb`): hand-written `User` / `<custom>` model `inline_forms_attribute_list` rewritten; `gsub_file` injections for `Apartment#owner` and `Owner#apartments` updated to the new shape.
+- **Runtime consumers** (`InlineFormsController#create`, `app/views/inline_forms/_show.html.erb`, `_new.html.erb`, `_new_nested.html.erb`, `InlineForms.validate_plain_text_configuration_for!`, `InlineForms.validate_no_archived_form_elements_for!`) updated their row destructuring to `|attribute, form_element|`.
+- **Positional lookups** in `InlineFormsHelper#attribute_values`, `MustBeAValueValidator#attribute_values`, and `DropdownWithValuesHelper#…_edit` shifted from `[3]` / `[4]` to `[2]` / `[3]`.
+- **Tests & docs:** all fixtures, assertions, and code samples (USAGE files, READMEs, `archived/*/README.md`) updated to the new shape.
+
+### Notes
+
+- **Example app gate (recorded):** `inline_forms create MyApp -d sqlite --example` against the freshly built **8.1.5** gem trio: install in ~73s, `bundle check: ok`, **88 runs, 502 assertions, 0 failures, 0 errors, 0 skips**. Gem unit tests: 42 runs, 240 assertions, 0 failures.
+
+## [8.1.4] - 2026-05-26
+
+### Fixed
+
+- **Search box on `/users` (or `/members`, etc.) silently returned the full list.** The installer-generated user model declared `scope :inline_forms_list, -> { order(:name, :id) }` but **no** `inline_forms_search` scope, so `InlineFormsController#index`'s `merge(@Klass.inline_forms_search(params[:search]))` fell through to the `ApplicationRecord` no-op (`scope :inline_forms_search, ->(_q) { all }`) and emitted `SELECT … FROM users ORDER BY name ASC, id ASC LIMIT 7` — no `WHERE`, no filter, regardless of the search query. Installer's `User` / `<custom>` model template now ships `scope :inline_forms_search, ->(q) { where("name LIKE :q OR email LIKE :q", q: "%#{q}%") }`, so `/users?search=ad` now returns rows whose name or email matches `%ad%`. Custom user-model classes via `-U <Class>` get the same scope.
+- **Top-bar "More" dropdown was empty in every generated app** (long-standing latent bug). `lib/generators/inline_forms_generator.rb#add_tab` looked for the marker `ActionView::CompiledTemplates::MODEL_TABS = %w(` in `app/controllers/application_controller.rb`, but the installer wrote the initializer at `config/initializers/inline_forms.rb` with the marker `MODEL_TABS = %w(` and the application controller had no marker at all. Thor's `inject_into_file` silently skipped (no error, no message), so `MODEL_TABS` stayed `%w()` through every `rails g inline_forms` call. The header partial (`app/views/inline_forms/_header.html.erb`) iterated an empty list and the dropdown rendered just the chevron with no children — for years.
+  - `add_tab` now targets `config/initializers/inline_forms.rb` with the simpler marker `MODEL_TABS = %w(`, no-ops if the file or marker is missing (so consumers running `rails g inline_forms` in a non-installer-shaped app still work), and stays idempotent on re-run (skips the token if already present).
+  - Installer's initializer is now created **before** the `Locale` / `Role` `generate "inline_forms"` calls (was created at the end of the run), and is pre-seeded with the user-model's pluralised route — `MODEL_TABS = %w(<plural_route> )` — because the user model is hand-written by the installer (not generated) and would otherwise never be added.
+  - Net effect on a fresh `inline_forms create MyApp -d sqlite --example`: `MODEL_TABS = %w(owners apartments roles locales users )` (with `-U Member`: `… members `). The top-bar "More" dropdown now lists every HTML-reachable model the current `current_user` `can? :update`.
+
+### Changed
+
+- **Test skeleton (`test/inline_forms_generator_test.rb#build_destination_skeleton!`)** writes `config/initializers/inline_forms.rb` with `MODEL_TABS = %w()` (matching the installer) instead of the legacy `ActionView::CompiledTemplates::MODEL_TABS = %w()` in `application_controller.rb`. `test_generates_model_controller_route_migration_and_tab_injection` now asserts the token lands in the initializer.
+
+### Notes
+
+- **Example app gate (recorded):** `inline_forms create MyApp -d sqlite --example` against the freshly built **8.1.4** gem trio: install in ~71s, `bundle check: ok`, `MODEL_TABS = %w(owners apartments roles locales users )` in the initializer, **88 runs, 502 assertions, 0 failures, 0 errors, 0 skips**. Same result with `-U Member`, with `users → members` in `MODEL_TABS`.
+
 ## [8.1.3] - 2026-05-26
 
 ### Changed

data/README.rdoc

@@ -62,7 +62,7 @@ You can install the example application manually if you like:
  rails g migration AddOwnerToApartments owner:references
  # Then in app/models/apartment.rb, add (under has_paper_trail):
  #   belongs_to :owner, optional: true
- # and prepend `[ :owner, "owner", :dropdown ],` to inline_forms_attribute_list.
+ # and prepend `[ :owner, :dropdown ],` to inline_forms_attribute_list.
  bundle exec rake db:migrate
  rails s
 
@@ -208,7 +208,7 @@ Active-tab highlighting is unchanged from upstream (still driven by +set_tab :fo
          tab = ALL_TABS.include?(params[:tab].to_s) ? params[:tab].to_s : ALL_TABS.first
          set_tab tab.to_sym
          @inline_forms_attribute_list = TAB_FIELDS.fetch(tab).map { |a|
-           @object.inline_forms_attribute_list.find { |attr, _, _| attr == a }
+           @object.inline_forms_attribute_list.find { |attr, _| attr == a }
          }
          render "clients/show_with_tabs",
                 layout: turbo_frame_request? ? "turbo_rails/frame" : "inline_forms"

data/app/controllers/inline_forms_controller.rb

@@ -135,7 +135,7 @@ class InlineFormsController < ApplicationController
     @object ||= @Klass.new
     @update_span = params[:update]
     attributes = @inline_forms_attribute_list || @object.inline_forms_attribute_list
-    attributes.each do | attribute, name, form_element |
+    attributes.each do | attribute, form_element |
       InlineForms.assert_plain_text_column!(object: @object, attribute: attribute, form_element: form_element)
       send("#{form_element.to_s}_update", @object, attribute) unless form_element == :associated || (cancan_enabled? && cannot?(:read, @object, attribute))
     end

data/app/helpers/inline_forms_helper.rb

@@ -310,7 +310,7 @@ module InlineFormsHelper
     # if we have a hash { 0=>'a', 2=>'b', 3=>'d' } will result in [[0,'a'],[2,'b'],[3,'d']] (it will keep the index and sort on the index)
     # TODO work this out better!
     # 2012-01-23 Use Cases
-    # [ :sex , "sex", :radio_button, { 1 => 'f', 2 => 'm' } ],
+    # [ :sex, :radio_button, { 1 => 'f', 2 => 'm' } ],
     # in this case we want the attribute in the database to be 1 or 2. From that attribute, we need to find the value.
     # using an array, won't work, since [ 'f', 'm' ][1] would be 'm' in stead of 'f'
     # so values should be a hash. BUT since we don't have sorted hashes (ruby 1,.8.7), the order of the values in the edit screen will be random.
@@ -326,10 +326,13 @@ module InlineFormsHelper
     # In the dropdown (or the slider) we definately need the order preserverd.
     # attribulte_values turns this into
     # [ [0,'???'], [1, '--'] .... [5, '++'] ]
-
+    #
+    # Row shape (since 8.1.x): [ :attr, :form_element, values, options_disabled ].
+    # `values` lives at index 2 (was index 3 before the empty label string was
+    # dropped in 8.1.x).
 
     attributes = @inline_forms_attribute_list || object.inline_forms_attribute_list # if we do this as a form_element, @inline.. is nil!!!
-    values = attributes.assoc(attribute.to_sym)[3]
+    values = attributes.assoc(attribute.to_sym)[2]
     raise t("fatal.no_values_defined_in", @Klass, attribute) if values.nil?
     if values.is_a?(Hash)
       temp = Array.new

data/app/validators/must_be_a_value_validator.rb

@@ -28,7 +28,7 @@ class MustBeAValueValidator < ActiveModel::EachValidator
     # if we have a hash { 0=>'a', 2=>'b', 3=>'d' } will result in [[0,'a'],[2,'b'],[3,'d']] (it will keep the index and sort on the index)
     # TODO work this out better!
     # 2012-01-23 Use Cases
-    # [ :sex , "sex", :radio_button, { 1 => 'f', 2 => 'm' } ],
+    # [ :sex, :radio_button, { 1 => 'f', 2 => 'm' } ],
     # in this case we want the attribute in the database to be 1 or 2. From that attribute, we need to find the value.
     # using an array, won't work, since [ 'v', 'm' ][1] would be 'm' in stead of 'v'
     # so values should be a hash. BUT since we don't have sorted hashes (ruby 1,.8.7), the order of the values in the edit screen will be random.
@@ -40,8 +40,11 @@ class MustBeAValueValidator < ActiveModel::EachValidator
     # like value=values.assoc(attribute_from_database)[1] (the [1] is needed since the result of #assoc = [1,'v'] and we need the 'v')
     # I feel it's ugly but it works.
 
+    # Row shape (since 8.1.x): [ :attr, :form_element, values, options_disabled ].
+    # `values` lives at index 2 (was index 3 before the empty label string was
+    # dropped in 8.1.x).
     attributes = @inline_forms_attribute_list || object.inline_forms_attribute_list # if we do this as a form_element, @inline.. is nil!!!
-    values = attributes.assoc(attribute.to_sym)[3]
+    values = attributes.assoc(attribute.to_sym)[2]
     raise "No Values defined in #{@Klass}, #{attribute}" if values.nil?
     if values.is_a?(Hash)
       temp = Array.new

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

@@ -39,7 +39,7 @@
         parent_id: @parent_id),
       new_form_opts do -%>
     <% attributes = @inline_forms_attribute_list || @object.inline_forms_attribute_list -%>
-    <% attributes.each do | attribute, name, form_element | -%>
+    <% attributes.each do | attribute, form_element | -%>
       <% unless form_element.to_sym == :associated || (cancan_enabled? && cannot?(:read, @object, attribute)) -%>
         <% css_class_id = "attribute_#{attribute}_#{@object.id}" -%>
         <% if form_element && form_element.to_sym == :header %>

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

@@ -14,7 +14,7 @@ This goes in _new.html.erb
     </div>
   </div>
     <% nested_attributes = @nested_object.inline_forms_attribute_list -%>
-    <% nested_attributes.each do | nested_attribute, nested_name, nested_form_element | -%>
+    <% nested_attributes.each do | nested_attribute, nested_form_element | -%>
     <% @nested_form_element = nested_form_element %>
     <% @nested_attribute = nested_attribute %>
       <% unless @nested_form_element.to_sym == :associated -%>

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

@@ -20,7 +20,7 @@
     </div>
     <% end %>
     <% attributes = @inline_forms_attribute_list || @object.inline_forms_attribute_list -%>
-    <% attributes.each do | attribute, name, form_element | -%>
+    <% attributes.each do | attribute, form_element | -%>
       <% if cancan_disabled? || can?(:read, @object, attribute) %>
         <% css_class_id = "#{@object.class.name.underscore}_#{@object.id}_#{attribute}" -%>
         <% if form_element == :header %>

data/archived/form_elements/tree/README.md

@@ -5,7 +5,7 @@
 
 ## What it did
 
-Declare in `inline_forms_attribute_list`, e.g. `[ :children, "Children", :tree ]`:
+Declare in `inline_forms_attribute_list`, e.g. `[ :children, :tree ]`:
 
 - **`_show.html.erb`:** header “Children”, `+` for new child, `<turbo-frame>` wrapping **`_tree.html.erb`**.
 - **`_tree.html.erb`:** lists `parent.children` with Turbo row open, list-frame pagination (`update=…_list`).

data/inline_forms.gemspec

@@ -19,7 +19,7 @@ Gem::Specification.new do |s|
   s.test_files = s.files.grep(%r{^(test|spec|features)/})
   s.require_paths = ["lib"]
 
-  s.add_dependency("validation_hints", ">= 8.1.3", "< 9.0")
+  s.add_dependency("validation_hints", ">= 8.1.4", "< 9.0")
   s.add_dependency("rails", ">= 8.1", "< 8.2")
   s.add_dependency("rails-i18n", ">= 8.1", "< 9.0")
 

data/lib/generators/USAGE

@@ -16,14 +16,14 @@ This will create:
           def _presentation
             #define your presentation here
           end
-          def inline_forms_field_list
-            [
-                [ :name, 'name', :text_field ],
-                [ :description, 'description', :plain_text ],
-                [ :yesno, 'yesno', :check_box ],
-                [ :gender, 'gender', :boolean_with_values ],
+          def inline_forms_attribute_list
+            @inline_forms_attribute_list ||= [
+                [ :name, :text_field ],
+                [ :description, :plain_text ],
+                [ :yesno, :check_box ],
+                [ :gender, :boolean_with_values ],
             ]
-         end
+          end
         end
 
         app/controllers/things_controller.rb

data/lib/generators/inline_forms_addto/USAGE

@@ -0,0 +1,46 @@
+Description:
+    Add fields to an existing inline_forms model. The model file must already
+    exist (from `rails g inline_forms` or the installer). Generates an additive
+    migration (`add_column`, `add_reference` for belongs_to/dropdown, a single
+    :string column for image_field/audio_field; no column for rich_text /
+    has_many / has_one / habtm) and updates the model: associations,
+    `has_rich_text`, `mount_uploader`, and new rows in `inline_forms_attribute_list`.
+
+    Does NOT create routes, a controller, or MODEL_TABS — those stay as-is for
+    the existing model. Use `rails g inline_forms` to scaffold new models.
+
+    Install-time-only names (`_no_model`, `_no_migration`, `_id`, `_enabled`) are
+    rejected. `_presentation`, `_list_order`, `_list_search`, and `_order` are
+    skipped unless you pass `--replace`.
+
+Example:
+    rails generate inline_forms_addto User occupation:text_field birthdate:date organization:dropdown bio:rich_text
+
+This will create:
+      create  db/migrate/20260527120000_inline_forms_add_to_users_occupation_birthdate.rb
+      insert  app/models/user.rb
+
+      db/migrate/..._inline_forms_add_to_users_occupation_birthdate.rb:
+        class InlineFormsAddToUsersOccupationBirthdate < ActiveRecord::Migration[8.1]
+          def change
+            add_column :users, :occupation, :string
+            add_column :users, :birthdate, :date
+            add_reference :users, :organization, foreign_key: true
+          end
+        end
+
+      app/models/user.rb (excerpt):
+        class User < ApplicationRecord
+          belongs_to :organization
+          has_rich_text :bio
+
+          def inline_forms_attribute_list
+            @inline_forms_attribute_list ||= [
+              ...
+              [ :occupation, :text_field ],
+              [ :birthdate, :date ],
+              [ :organization, :dropdown ],
+              [ :bio, :rich_text ],
+            ]
+          end
+        end

data/lib/generators/inline_forms_addto_generator.rb

@@ -39,7 +39,7 @@ class InlineFormsAddtoGenerator < Rails::Generators::NamedBase
     class_option :replace, type: :boolean, default: false,
                  desc: "Replace existing _presentation/_list_order/_list_search instead of skipping."
 
-    source_root File.expand_path("templates", __dir__)
+    source_root File.expand_path("inline_forms_addto/templates", __dir__)
 
     def validate!
       unless File.exist?(File.join(destination_root, model_file_path))
@@ -209,7 +209,7 @@ class InlineFormsAddtoGenerator < Rails::Generators::NamedBase
     # `ApplicationRecord`) and no body.
     def add_attribute_list_row!(attribute)
       content = File.read(File.join(destination_root, model_file_path))
-      row = "      [ :#{attribute.name} , \"#{attribute.name}\", :#{attribute.attribute_type} ],\n"
+      row = "      [ :#{attribute.name}, :#{attribute.attribute_type} ],\n"
 
       if content.match?(/@inline_forms_attribute_list \|\|=\s*\[/)
         if content.match?(/\[\s*:#{Regexp.escape(attribute.name)}\s*,/)
@@ -235,7 +235,7 @@ class InlineFormsAddtoGenerator < Rails::Generators::NamedBase
 
           def inline_forms_attribute_list
             @inline_forms_attribute_list ||= [
-              [ :#{attribute.name} , "#{attribute.name}", :#{attribute.attribute_type} ],
+              [ :#{attribute.name}, :#{attribute.attribute_type} ],
             ]
           end
         RUBY

data/lib/generators/inline_forms_generator.rb

@@ -123,8 +123,7 @@ module InlineForms
             @inline_forms_attribute_list << commenter +
               '     [ :' +
               attribute.name +
-              ' , "' + attribute.name +
-              '", :' + attribute.attribute_type.to_s +
+              ', :' + attribute.attribute_type.to_s +
               " ], \n"
           end
         end
@@ -186,11 +185,22 @@ module InlineForms
     def add_tab
       return if @flag_not_accessible_through_html
 
-      relative_path = "app/controllers/application_controller.rb"
-      marker = "ActionView::CompiledTemplates::MODEL_TABS = %w("
+      # The installer seeds `MODEL_TABS = %w(<user_model_route> )` in
+      # `config/initializers/inline_forms.rb` before any `rails g inline_forms`
+      # runs. Inject each generated model's pluralised route token into that
+      # list so the inline_forms top-bar dropdown (rendered by
+      # `app/views/inline_forms/_header.html.erb`) surfaces every
+      # HTML-reachable model. No-op when the initializer is absent (e.g. a
+      # consumer running `rails g inline_forms` in a non-installer-shaped app)
+      # or when the token is already present (idempotent re-run).
+      relative_path = "config/initializers/inline_forms.rb"
+      marker = "MODEL_TABS = %w("
       tab_token = "#{name.pluralize.underscore} "
       full_path = File.join(destination_root, relative_path)
+      return unless File.exist?(full_path)
+
       content = File.read(full_path)
+      return unless content.include?(marker)
       return if content.include?(tab_token.rstrip)
 
       inject_into_file relative_path, tab_token, after: marker

data/lib/inline_forms/archived_form_elements.rb

@@ -50,7 +50,7 @@ module InlineForms
   def self.validate_no_archived_form_elements_for!(klass)
     return unless klass.instance_methods.include?(:inline_forms_attribute_list)
 
-    klass.new.inline_forms_attribute_list.each do |attribute, _label, form_element|
+    klass.new.inline_forms_attribute_list.each do |attribute, form_element|
       key = form_element.to_sym
       next unless ARCHIVED_FORM_ELEMENTS.key?(key)
 

data/lib/inline_forms/form_elements/dropdown_with_values_helper.rb

@@ -16,7 +16,7 @@ module InlineForms
       values = attribute_values(object, attribute)
     
       attributes = @inline_forms_attribute_list || object.inline_forms_attribute_list
-      options_disabled = attributes.assoc(attribute.to_sym)[4]
+      options_disabled = attributes.assoc(attribute.to_sym)[3]
     
       collection_select(  ('_' + object.class.to_s.underscore).to_sym,
                           attribute.to_sym,

data/lib/inline_forms/form_elements/radio_button_helper.rb

@@ -6,7 +6,7 @@ module InlineForms
     # -*- encoding : utf-8 -*-
     # radio buttons are integers in this version
     # us like this:
-    #   [ :sex , "gender", :radio_button, { 1 => 'male', 2 => 'female' } ],
+    #   [ :sex, :radio_button, { 1 => 'male', 2 => 'female' } ],
     
     def radio_button_show(object, attribute)
       values = attribute_values(object, attribute)

data/lib/inline_forms/version.rb

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

data/lib/inline_forms.rb

@@ -58,10 +58,10 @@ module InlineForms
   #
   #    class Example < ApplicationRecord
   #      def inline_forms_attribute_list
-  #        {
-  #          :name  => [ "name", :text_field ],
-  #          :price => [ "price", :text_field ],
-  #        }
+  #        [
+  #          [ :name,  :text_field ],
+  #          [ :price, :text_field ],
+  #        ]
   #      end
   #    end
   # as you see, both :string and :integer are mapped to a :text_field
@@ -92,7 +92,7 @@ module InlineForms
   #   t.belongs_to :country
   # to the migration. (In fact AR will add t.integer :country_id). And
   # it will add
-  #   :country => [ "country", :dropdown ],
+  #   [ :country, :dropdown ],
   # to the inline_forms_attribute_list in the model.
   #
   SPECIAL_COLUMN_TYPES = {
@@ -127,7 +127,7 @@ module InlineForms
     return unless klass.table_exists?
 
     attributes = klass.new.inline_forms_attribute_list
-    attributes.each do |attribute, _label, form_element|
+    attributes.each do |attribute, form_element|
       next unless plain_text_form_element?(form_element)
       next if klass.column_names.include?(attribute.to_s)
 

data/test/archived_form_elements_test.rb

@@ -6,13 +6,13 @@ require "inline_forms/archived_form_elements"
 class ArchivedFormElementsTest < Minitest::Test
   class ModelWithArchivedElement
     def inline_forms_attribute_list
-      [[:address, "Address", :geo_code_curacao]]
+      [[:address, :geo_code_curacao]]
     end
   end
 
   class ModelWithArchivedTree
     def inline_forms_attribute_list
-      [[:children, "Children", :tree]]
+      [[:children, :tree]]
     end
   end
 

data/test/inline_forms_addto_generator_test.rb

@@ -17,8 +17,8 @@ class InlineFormsAddtoGeneratorTest < Minitest::Test
 
       def inline_forms_attribute_list
         @inline_forms_attribute_list ||= [
-          [ :name , "name", :text_field ],
-          [ :category , "category", :belongs_to ],
+          [ :name, :text_field ],
+          [ :category, :belongs_to ],
         ]
       end
 
@@ -38,13 +38,13 @@ class InlineFormsAddtoGeneratorTest < Minitest::Test
 
       def inline_forms_attribute_list
         @inline_forms_attribute_list ||= [
-          [ :header_user_login,         '', :header ],
-          [ :name,                      '', :text_field ],
-          [ :email,                     '', :text_field ],
-          [ :locale ,                   '', :dropdown ],
-          [ :password,                  '', :devise_password_field ],
-          [ :header_user_roles,         '', :header ],
-          [ :roles,                     '', :check_list ],
+          [ :header_user_login, :header ],
+          [ :name,              :text_field ],
+          [ :email,             :text_field ],
+          [ :locale,            :dropdown ],
+          [ :password,          :devise_password_field ],
+          [ :header_user_roles, :header ],
+          [ :roles,             :check_list ],
         ]
       end
 
@@ -94,13 +94,13 @@ class InlineFormsAddtoGeneratorTest < Minitest::Test
     assert_includes(model, "belongs_to :supplier")
     assert_includes(model, "has_rich_text :bio")
     assert_includes(model, "mount_uploader :avatar, AvatarUploader")
-    assert_includes(model, '[ :occupation , "occupation", :text_field ]')
+    assert_includes(model, '[ :occupation, :text_field ]')
     # :belongs_to is a relation -> no row in inline_forms_attribute_list
     # (matches InlineFormsGenerator semantics). :dropdown is not a relation
     # at lookup time, so it does get a row.
-    refute_includes(model, '[ :organization , "organization", :belongs_to ]')
-    assert_includes(model, '[ :supplier , "supplier", :dropdown ]')
-    assert_includes(model, '[ :avatar , "avatar", :image_field ]')
+    refute_includes(model, '[ :organization, :belongs_to ]')
+    assert_includes(model, '[ :supplier, :dropdown ]')
+    assert_includes(model, '[ :avatar, :image_field ]')
 
     assert_includes(migration, "add_column :widgets, :occupation, :string")
     assert_includes(migration, "add_reference :widgets, :organization, foreign_key: true")
@@ -120,7 +120,7 @@ class InlineFormsAddtoGeneratorTest < Minitest::Test
     model = read("app/models/widget.rb")
 
     assert_equal(1, model.scan("belongs_to :organization").size)
-    assert_equal(1, model.scan('[ :occupation , "occupation", :text_field ]').size)
+    assert_equal(1, model.scan('[ :occupation, :text_field ]').size)
   end
 
   def test_appends_row_to_installer_shaped_user_attribute_list
@@ -130,8 +130,8 @@ class InlineFormsAddtoGeneratorTest < Minitest::Test
 
     model = read("app/models/user.rb")
 
-    assert_includes(model, '[ :occupation , "occupation", :text_field ]')
-    assert_includes(model, '[ :birthdate , "birthdate", :date_select ]')
+    assert_includes(model, '[ :occupation, :text_field ]')
+    assert_includes(model, '[ :birthdate, :date_select ]')
     refute_match(/\]\s*\n\s*\[\s*:occupation/, model)
 
     user_array_section = model[/@inline_forms_attribute_list \|\|=\s*\[(.|\n)*?\n\s*\]/]
@@ -150,7 +150,7 @@ class InlineFormsAddtoGeneratorTest < Minitest::Test
 
     assert_includes(out, "no inline_forms_attribute_list found")
     assert_includes(model, "def inline_forms_attribute_list")
-    assert_includes(model, '[ :occupation , "occupation", :text_field ]')
+    assert_includes(model, '[ :occupation, :text_field ]')
   end
 
   def test_unknown_type_raises_thor_error_by_default
@@ -174,7 +174,7 @@ class InlineFormsAddtoGeneratorTest < Minitest::Test
     model = read("app/models/widget.rb")
     migration = read_single_addto_migration_for("widgets")
 
-    assert_includes(model, '[ :payload , "payload", :unknown ]')
+    assert_includes(model, '[ :payload, :unknown ]')
     assert_includes(migration, "#    add_column :widgets, :payload, :unknown")
   end
 

data/test/inline_forms_generator_test.rb

@@ -32,7 +32,7 @@ class InlineFormsGeneratorTest < Minitest::Test
     model = read("app/models/thing.rb")
     controller = read("app/controllers/things_controller.rb")
     routes = read("config/routes.rb")
-    application_controller = read("app/controllers/application_controller.rb")
+    inline_forms_initializer = read("config/initializers/inline_forms.rb")
     migration = read_single_migration_for("things")
 
     assert_includes(model, "class Thing < ApplicationRecord")
@@ -44,8 +44,8 @@ class InlineFormsGeneratorTest < Minitest::Test
     refute_includes(model, "scope :inline_forms_search")
     assert_includes(model, "belongs_to :category")
     assert_includes(model, "has_many :photos")
-    assert_includes(model, "[ :name , \"name\", :text_field ]")
-    assert_includes(model, "[ :category , \"category\", :dropdown ]")
+    assert_includes(model, "[ :name, :text_field ]")
+    assert_includes(model, "[ :category, :dropdown ]")
 
     assert_includes(controller, "class ThingsController < InlineFormsController")
     assert_includes(controller, "set_tab :thing")
@@ -54,7 +54,7 @@ class InlineFormsGeneratorTest < Minitest::Test
     assert_includes(routes, "post 'revert', :on => :member")
     assert_includes(routes, "get 'list_versions', :on => :member")
 
-    assert_includes(application_controller, "MODEL_TABS = %w(things ")
+    assert_includes(inline_forms_initializer, "MODEL_TABS = %w(things ")
 
     assert_includes(migration, "class InlineFormsCreateThings < ActiveRecord::Migration[8.1]")
     assert_includes(migration, "create_table :things do |t|")
@@ -91,7 +91,7 @@ class InlineFormsGeneratorTest < Minitest::Test
     model = read("app/models/mystery.rb")
     migration = read_single_migration_for("mysteries")
 
-    assert_includes(model, "#     [ :payload , \"payload\", :unknown ]")
+    assert_includes(model, "#     [ :payload, :unknown ]")
     assert_includes(migration, "#     t.unknown :payload")
   end
 
@@ -167,7 +167,7 @@ class InlineFormsGeneratorTest < Minitest::Test
     model = read("app/models/note.rb")
     migration = read_single_migration_for("notes")
 
-    assert_includes(model, "[ :description , \"description\", :plain_text ]")
+    assert_includes(model, "[ :description, :plain_text ]")
     assert_includes(migration, "t.text :description")
   end
 
@@ -175,6 +175,7 @@ class InlineFormsGeneratorTest < Minitest::Test
 
   def build_destination_skeleton!
     mkdir_p("config")
+    mkdir_p("config/initializers")
     mkdir_p("app/controllers")
     mkdir_p("app/models")
     mkdir_p("db/migrate")
@@ -192,7 +193,17 @@ class InlineFormsGeneratorTest < Minitest::Test
       "app/controllers/application_controller.rb",
       <<~RUBY
         class ApplicationController < ActionController::Base
-          ActionView::CompiledTemplates::MODEL_TABS = %w()
+        end
+      RUBY
+    )
+
+    # Matches the file the installer writes; the generator's `add_tab` step
+    # injects `<plural_route> ` tokens after the `MODEL_TABS = %w(` marker.
+    write(
+      "config/initializers/inline_forms.rb",
+      <<~RUBY
+        Rails.application.reloader.to_prepare do
+          MODEL_TABS = %w()
         end
       RUBY
     )

data/test/plain_text_configuration_test.rb

@@ -20,7 +20,7 @@ class PlainTextConfigurationTest < Minitest::Test
 
     def inline_forms_attribute_list
       [
-        [:description, "description", :plain_text]
+        [:description, :plain_text]
       ]
     end
   end
@@ -40,7 +40,7 @@ class PlainTextConfigurationTest < Minitest::Test
 
     def inline_forms_attribute_list
       [
-        [:description, "description", :plain_text]
+        [:description, :plain_text]
       ]
     end
   end
@@ -56,7 +56,7 @@ class PlainTextConfigurationTest < Minitest::Test
 
     def inline_forms_attribute_list
       [
-        [:description, "description", :rich_text]
+        [:description, :rich_text]
       ]
     end
   end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: inline_forms
 version: !ruby/object:Gem::Version
-  version: 8.1.3
+  version: 8.1.5
 platform: ruby
 authors:
 - Ace Suares
@@ -17,7 +17,7 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 8.1.3
+        version: 8.1.4
     - - "<"
       - !ruby/object:Gem::Version
         version: '9.0'
@@ -27,7 +27,7 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 8.1.3
+        version: 8.1.4
     - - "<"
       - !ruby/object:Gem::Version
         version: '9.0'
@@ -495,11 +495,12 @@ files:
 - inline_forms.gemspec
 - lib/generators/USAGE
 - lib/generators/assets/stylesheets/inline_forms_devise.css
+- lib/generators/inline_forms_addto/USAGE
+- lib/generators/inline_forms_addto/templates/add_columns_migration.erb
 - lib/generators/inline_forms_addto_generator.rb
 - lib/generators/inline_forms_attribute_overrides.rb
 - lib/generators/inline_forms_generator.rb
 - lib/generators/templates/_inline_forms_tabs.html.erb
-- lib/generators/templates/add_columns_migration.erb
 - lib/generators/templates/application_record.rb
 - lib/generators/templates/controller.erb
 - lib/generators/templates/migration.erb