inline_forms 8.1.9 → 8.1.11

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bba19f36ffc279e71b2fe0c0fa9a8ec9501424f5fddab372d50941c1e4905dc3
-  data.tar.gz: 6a0473bcf4539bddf580dabaae2a5ecfc9decc36c0e691cd63aa79a3127a8478
+  metadata.gz: 0e33b151ebeda5706914feaf6ac97bd11b0b379ac8b03435cf9d948c40dbdd9b
+  data.tar.gz: 975d48a428e045905e0fd29acf6b2d136ec7cc5f4a7825db9c951eb0ab3bba3d
 SHA512:
-  metadata.gz: 4f33889510601ad516280f0727f73a2e33a023d82cae3bf0bf28f46747ecb3e0a53f4b89554b5c016fd04657ecab69bdf171091e252e2efd3abaaa274166f4fe
-  data.tar.gz: 3c452681ce7d64bcf4d5716e1e41c2ad4d9d830e6a68cdd2a294d256b20308703d73811ef514f7940b634794cf4521dd18cdd4ee9e8e9e68613692129ca0691b
+  metadata.gz: 37d86c5be4e09df199b6d23b8ee91c2488bb98b178b352213dce075234152cc9ef03d119cd2ccd0eb41632d4e6ec5724918d6f2cfdbd70889ed5a0e4b7a959d1
+  data.tar.gz: 3b40a5c9b0de4707715d43d2186f3d1e4570b7c0cfa17f55f6aa7aa2c151b325bf91a670897ebc3379a1ba086081953f4a0d2f3806a3cd83a13c4fd6149443bb

data/CHANGELOG.md

@@ -4,6 +4,38 @@ All notable changes to this project are documented in this file.
 
 ## [Unreleased]
 
+## [8.1.11] - 2026-05-28
+
+### Fixed
+
+- **Installer's prerelease gem discovery now looks in `pkg/` too.** `Bundler::GemHelper.install_tasks` (the standard Rakefile boilerplate) puts `rake build` output in `pkg/`, not in the checkout root where `gem build` writes. 8.1.10's `dev_checkout_with_gems` + `install_prerelease_gems_from_roots!` only globbed the checkout root, so the moment a maintainer ran `rake build` the freshly-built gem was invisible — the installer fell back to whatever stale `<name>-*.gem` was still sitting in the root from a previous `gem build`. This is the exact shape that caused 8.1.10 to silently use 8.1.6 templates today (default RVM gemset had `inline_forms_installer 8.1.6` as its highest, the checkout root had only the stale `inline_forms-8.1.7.gem`, and the 8.1.8/8.1.9/8.1.10 builds were sitting in `/home/code/inline_forms/pkg/` where nothing was looking). Both helpers now glob `<root>/<name>-*.gem` and `<root>/pkg/<name>-*.gem`.
+
+### Changed
+
+- **Example-app gate caps test parallelism at `PARALLEL_WORKERS=2` by default.** Rails' minitest parallelizer defaults to `workers: number_of_processors`, which on a 20-core box forks 20 full Rails processes — each ~200-300 MB resident with the example app's gem stack (CarrierWave + Devise + PaperTrail + Foundation + tabs_on_rails + money-rails). On a memory-pressured host that's 4-6 GB of workers alone, on top of the parent installer/bundler state, and `systemd-oomd` can (and did, 2026-05-28 07:50:35) kill the whole terminal session before the gate finishes, with no useful signal back to the user. `PARALLEL_WORKERS=2` keeps the worker footprint to ~600 MB and roughly doubles wall-clock vs 20 workers — fine for a one-shot gate. The new `INLINE_FORMS_TEST_WORKERS` env override lets machines with RAM headroom crank it back up (`0` = Rails' default, any positive integer = pin to that count).
+
+### Lockstep
+
+- `inline_forms`, `inline_forms_installer`, and `validation_hints` bumped from 8.1.10 to 8.1.11 in lockstep, even though `inline_forms` (the engine) and `validation_hints` have no behavior change in this release — both are only here so the version trio stays in step.
+
+## [8.1.10] - 2026-05-27
+
+### Changed
+
+- **`:decimal_field` is now a real `decimal(p, s)` column instead of a `varchar`.** The FormElementRegistry mapping moves from `:decimal_field => :string` to `:decimal_field => :decimal`, so `rails g inline_forms Foo price:decimal_field` now emits `t.decimal :price, precision: 10, scale: 2` (the sensible default — invoice/balance up to 99,999,999.99). This closes a long-standing inconsistency: `decimal_field` was the only "numeric" Tier 1 helper that stored its value as text, so `"ace"` would happily round-trip and `Foo.sum(:price)` was a runtime cast error. Existing apps generated before 8.1.10 keep their varchar columns until they migrate; only newly-generated tables get `:decimal`.
+- **New CLI syntax: `name:decimal_field{p,s}` overrides precision/scale.** Mirrors Rails' built-in `name:decimal{p,s}` grammar. Examples: `latitude:decimal_field{9,6}` -> `decimal(9, 6)` (~11 cm GPS accuracy), `longitude:decimal_field{10,6}` -> `decimal(10, 6)`, `dose:decimal_field{6,3}` -> `decimal(6, 3)`. Rails' own `Rails::Generators::GeneratedAttribute.parse` only honors `{…}` on a hardcoded list of built-in types and glues the braces onto custom types as part of the type symbol; `inline_forms_attribute_overrides.rb` now pre-processes the suffix off `:decimal_field` and stores precision/scale in `attr_options` for the migration emitter to read.
+- **`decimal_field_show` / `_edit` render `BigDecimal` as fixed-point.** Once the column became a real `:decimal`, `object[attribute]` started returning `BigDecimal`, whose `to_s` defaults to scientific notation (`"0.1234e2"`). Both helpers now call `value.to_s("F")` for BigDecimal and fall through to the generic `to_s` for legacy varchar values, so existing apps display correctly with either column shape.
+
+### Added
+
+- **`FormElementShowcase` demos the new `{p,s}` syntax.** Two new attributes (`latitude:decimal_field{9,6}`, `longitude:decimal_field{10,6}`) sit alongside `price:decimal_field` in the "Numbers" header, exercising both the default `(10, 2)` and explicit overrides. Seed values are Curaçao's Willemstad coordinates (`12.123456 / -68.987654`) — exactly representable in the chosen scale so the round-trip test is bit-identical rather than float-fuzzy.
+- **`validates :price, latitude, longitude, numericality: …` in the showcase model.** Without an explicit validation, ActiveRecord silently casts `"ace"` to `BigDecimal("0")` on a `:decimal` column. The validation keeps the round-trip honest (the new integration test asserts a 422 on `"ace"`), and the `:latitude` / `:longitude` validations also pin the lat/lon ranges (±90, ±180).
+- **Model + integration tests** cover the precision/scale schema (`columns_hash["latitude"].precision == 9`), the BigDecimal round-trip at full scale, the rejection of out-of-range GPS values, and the rejection of non-numeric input.
+
+### Lockstep
+
+- `inline_forms`, `inline_forms_installer`, and `validation_hints` bumped from 8.1.9 to 8.1.10 in lockstep, even though `validation_hints` has no behavior change in this release.
+
 ## [8.1.9] - 2026-05-27
 
 ### Fixed

data/lib/generators/inline_forms_attribute_overrides.rb

@@ -22,6 +22,37 @@ Rails::Generators::GeneratedAttribute.class_eval do
     true
   end
 
+  # Recognize `name:decimal_field{p,s}` on the command line and hoist the
+  # precision/scale into `attr_options`. Rails' own
+  # `Rails::Generators::GeneratedAttribute.parse` only honors the `{…}`
+  # suffix for a hardcoded list of built-in types (`:string`, `:integer`,
+  # `:primary_key`, `:decimal`, ...); for our custom `:decimal_field` it
+  # just glues the braces onto the type symbol (`:"decimal_field{10,2}"`)
+  # and `column_type` falls through to `:unknown`. Strip the braces here,
+  # delegate to the original parse with a clean `name:decimal_field`, and
+  # then attach the parsed numbers to attr_options so the migration
+  # emitter can render `t.decimal :name, precision: P, scale: S`.
+  #
+  # Bare `name:decimal_field` (no braces) parses unchanged — the
+  # migration emitter applies precision: 10, scale: 2 as the default.
+  class << self
+    unless method_defined?(:parse_with_inline_forms_decimal) || private_method_defined?(:parse_with_inline_forms_decimal)
+      alias_method :parse_without_inline_forms_decimal, :parse
+    end
+
+    def parse(column_definition)
+      if column_definition.is_a?(String) &&
+         (m = column_definition.match(/\A(\w+):decimal_field\{(\d+),(\d+)\}\z/))
+        attr = parse_without_inline_forms_decimal("#{m[1]}:decimal_field")
+        attr.attr_options[:precision] = m[2].to_i
+        attr.attr_options[:scale]     = m[3].to_i
+        attr
+      else
+        parse_without_inline_forms_decimal(column_definition)
+      end
+    end
+  end
+
   # Deducts the column_type for migrations from the type.
   #
   # We first merge the Special Column Types with the Default Column Types,

data/lib/generators/inline_forms_generator.rb

@@ -164,11 +164,29 @@ module InlineForms
           else
             if attribute.migration?
               attribute.attribute_type == :unknown ? commenter = '#' : commenter = ' '
+              # Render precision/scale for real :decimal columns. The override
+              # in lib/generators/inline_forms_attribute_overrides.rb teaches
+              # `Rails::Generators::GeneratedAttribute.parse` to extract
+              # `{p,s}` off the type for :decimal_field (Rails' own parser
+              # only honors `{…}` for a hardcoded list of built-in types
+              # and otherwise glues the braces onto the type symbol).
+              # Apply sensible defaults — precision: 10, scale: 2 — so a
+              # bare `name:decimal_field` becomes a real decimal column
+              # instead of the legacy varchar that 8.1.9 and earlier
+              # emitted. Existing apps keep their varchar columns until
+              # they migrate; only newly-generated tables get :decimal.
+              column_opts = ""
+              if attribute.column_type == :decimal
+                precision = attribute.attr_options[:precision] || 10
+                scale     = attribute.attr_options[:scale]     || 2
+                column_opts = ", precision: #{precision}, scale: #{scale}"
+              end
               @columns << commenter +
                 '     t.' +
                 attribute.column_type.to_s +
                 " :" +
                 attribute.name +
+                column_opts +
                 " \n"
             end
           end

data/lib/inline_forms/form_element_registry.rb

@@ -8,7 +8,7 @@ module InlineForms
       :check_list => :no_migration,
       :ckeditor => :text,
       :date_select => :date,
-      :decimal_field => :string,
+      :decimal_field => :decimal,
       :devise_password_field => :string,
       :dropdown => :belongs_to,
       :dropdown_with_integers => :integer,

data/lib/inline_forms/form_elements/decimal_field_helper.rb

@@ -6,13 +6,33 @@ module InlineForms
     # -*- encoding : utf-8 -*-
     
     def decimal_field_show(object, attribute)
-      link_to_inline_edit object, attribute, object[attribute].nil? ? "<i class='fi-plus'></i>".html_safe : object[attribute], from_callee: __callee__
+      # `:decimal_field` columns are real `:decimal(p, s)` since 8.1.10,
+      # which means `object[attribute]` returns a `BigDecimal`. Calling
+      # `to_s` on a BigDecimal without arguments returns scientific
+      # notation (`"0.1234e2"`), not the `"12.34"` users expect — render
+      # fixed-point. Strings (legacy varchar columns on apps generated
+      # before 8.1.10) and integers/floats also work with `to_s("F")` via
+      # their generic `to_s` fallback.
+      value = object[attribute]
+      label = if value.nil?
+                "<i class='fi-plus'></i>".html_safe
+              elsif value.is_a?(BigDecimal)
+                value.to_s("F")
+              else
+                value.to_s
+              end
+      link_to_inline_edit object, attribute, label, from_callee: __callee__
     end
-    
+
     def decimal_field_edit(object, attribute)
-      text_field_tag attribute, (object.send attribute.to_sym), :class => 'input_decimal_field'  # for abide: , :required => true
+      # Render with fixed-point in the edit textbox for the same reason
+      # decimal_field_show does — otherwise the user sees "0.1234e2" in
+      # the edit field on an existing value, and round-trips it back.
+      value = object.send(attribute.to_sym)
+      value = value.to_s("F") if value.is_a?(BigDecimal)
+      text_field_tag attribute, value, :class => 'input_decimal_field'  # for abide: , :required => true
     end
-    
+
     def decimal_field_update(object, attribute)
       object.send :write_attribute, attribute.to_sym, params[attribute.to_sym]
     end

data/lib/inline_forms/version.rb

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

metadata

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