money_attribute 0.11.0 → 0.12.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 874e817fa1e157ee49e460c8848dabff3fe525d9849571d21045de2694dd0770
-  data.tar.gz: 933dcdd5adc296d9aa635da79fc181e15652ed884c2e3b1231655a5652a2beca
+  metadata.gz: 306761efbc2f4f645f3e975ea830ff26901bc4ba5fac664346f5d7b5720050b8
+  data.tar.gz: 13dfc1b02f256f22ac1027700164ae57cee60bc77e1b233c28549b20988adaf7
 SHA512:
-  metadata.gz: b5547742974245c1cffd5b690fa4c763e79224e553c90c2e19e88cdccb0d2c7ccdf669f9000e1917abe01fb880c2a50b0b2b475ffebf3fdfe01b787b0f4630ad
-  data.tar.gz: d0b760447e051fb0a1dc15492ba737676af935443ad5dcec4fb18988c898db37bf5980781d52cc64a4c60c651d03646ad77bb63ffdb6cb046c6d19f67cc7d092
+  metadata.gz: 3d39e999c298070baf49359ed226f9582defac099e4da3b342f3b367df81c658575e973095a0c8de4a19d8c7116af778d38160de6dbb263fd050a6a36bb3f0a9
+  data.tar.gz: 02c399705be0e9ac78c3f9dd1bbe9603055d6702df60a43e12e57fb0fa16ecfd4212d129a91294bdb4323f487030def2bc3fe52a7484552b93c4f2e81b6a71ed

data/README.md

@@ -91,7 +91,9 @@ The generator creates `config/initializers/money_attribute.rb`.
 
 ## Migration helpers
 
-MoneyAttribute adds `add_money` / `remove_money` for existing tables and `t.money` / `t.remove_money` for `create_table` / `change_table` blocks:
+MoneyAttribute adds `add_money` / `remove_money` for existing tables and `t.money` / `t.remove_money` for `create_table` / `change_table` blocks.
+
+By default `t.money :price` creates a `decimal(16,4)` amount column and a `string` currency column — both nullable, no default. Pass `amount: { type: :integer }` to store subunits instead, or `currency: false` to skip the currency column entirely.
 
 ```ruby
 class CreateProducts < ActiveRecord::Migration[8.1]
@@ -101,7 +103,7 @@ class CreateProducts < ActiveRecord::Migration[8.1]
       t.money  :price          # price (decimal) + price_currency (string)
       t.money  :price_amount   # price_amount + price_currency (strips _amount suffix)
       t.money  :fee, currency: false             # single column, no currency
-      t.money  :tax, type: :bigint               # bigint amount + currency
+      t.money  :tax, amount: { type: :bigint }   # bigint amount + currency
       t.timestamps
     end
   end
@@ -110,7 +112,7 @@ end
 class AddPriceToProducts < ActiveRecord::Migration[8.1]
   def change
     add_money :products, :price                 # add price + price_currency
-    add_money :products, :discount, type: :integer
+    add_money :products, :discount, amount: { type: :integer }
     remove_money :products, :obsolete_fee       # reversible in change
   end
 end
@@ -123,8 +125,19 @@ end
 | `t.money :price` | `price` decimal + `price_currency` string | `money_attribute :price` |
 | `t.money :price_amount` | `price_amount` decimal + `price_currency` string | `money_attribute :price` |
 | `t.money :price, currency: false` | `price` decimal | `money_attribute :price` |
-| `t.money :price, type: :integer` | `price` integer + `price_currency` string | `money_attribute :price` |
-| `t.money :price, amount: :a, currency: :c` | `a` + `c` | `money_attribute :price, mapping: { amount: :a, currency: :c }` |
+| `t.money :price, amount: { type: :integer }` | `price` integer + `price_currency` string | `money_attribute :price` |
+| `t.money :price, amount: { column: :a }, currency: { column: :c }` | `a` + `c` | `money_attribute :price, mapping: { amount: :a, currency: :c }` |
+| `t.money :price, currency: { limit: 3 }` | `price` decimal + `price_currency` string(3) | `money_attribute :price` |
+| `t.money :price, amount: { precision: 14, scale: 2, null: false }, currency: { limit: 3, default: 'USD' }` | `price` decimal(14,2) NOT NULL + `price_currency` string(3) DEFAULT 'USD' | `money_attribute :price` |
+| `t.remove_money :price` | Removes `price` + `price_currency` | `money_attribute :price` |
+
+Inside `change_table`:
+
+```ruby
+change_table :products do |t|
+  t.remove_money :obsolete_fee   # removes obsolete_fee + obsolete_fee_currency
+end
+```
 
 ## Configuration
 
@@ -224,6 +237,8 @@ offer = Offer.new(price: '12')
 offer.price.currency.code # => "USD"
 ```
 
+Unlike fixed-currency attributes, composite mode does not enforce a specific currency — any registered currency is accepted at assignment.
+
 ## Column type detection
 
 Declare the column as `decimal`, `integer`, or `bigint` — the gem adapts:
@@ -268,7 +283,14 @@ class Invoice < ApplicationRecord
 end
 ```
 
-The mapping keys are `:amount` and `:currency`; values are your database column names.
+The mapping keys are `:amount` and `:currency`; values are your database column names. You can provide only one key — the other falls back to the `<name>_amount` / `<name>_currency` convention:
+
+```ruby
+class Invoice < ApplicationRecord
+  money_attribute :total, mapping: { amount: :total_amount }
+  # currency column inferred as `total_currency`
+end
+```
 
 ## Column resolution
 
@@ -279,8 +301,10 @@ When you declare `money_attribute :name`, the gem resolves which database column
 | 1 | `mapping:` provided | As specified | Explicit composite |
 | 2 | `name_currency` column exists | `name` + `name_currency` | Composite (multi-currency) |
 | 3 | `name == 'amount'` AND `currency` column exists | `amount` + `currency` | Composite (multi-currency) |
-| 4 | `name_amount` + `name_currency` columns exist | `name_amount` + `name_currency` | Composite (multi-currency) |
-| 5 | `name` column exists (no currency partner) | `name` alone | Single-column (fixed-currency) |
+| 4 | `name` column exists (no currency partner) | `name` alone | Single-column (fixed-currency) |
+| 5 | None of the above (name column missing) | `name_amount` + `name_currency` (convention) | Composite (multi-currency) |
+
+Step 5 raises `ArgumentError` if the convention columns don't exist in the table.
 
 **Example**
 

data/lib/money_attribute/migration_extensions/helper.rb

@@ -8,42 +8,48 @@ module MoneyAttribute
       def parse_money_args(accessor, options = {})
         name = accessor.to_s
 
-        amount_col = options.key?(:amount) ? options[:amount].to_s : name
-
-        if options.key?(:currency)
-          currency_col = if options[:currency] == false
-                           nil
-                         else
-                           options[:currency].to_s
-                         end
+        if options.key?(:amount) && options[:amount].is_a?(Hash)
+          amount_col = options[:amount][:column]&.to_s || name
+          opts = options[:amount]
+          amount_opts = {
+            type: opts[:type],
+            null: opts[:null],
+            default: opts[:default],
+            precision: opts[:precision],
+            scale: opts[:scale]
+          }.compact
         else
-          stripped = name.end_with?('_amount') ? name.sub(/_amount$/, '') : name
-          currency_col = "#{stripped}_currency"
+          amount_col = name
+          amount_opts = {}
         end
 
-        col_type = options[:type] || :decimal
-
-        amount_opts = {}
-        amount_opts[:type] = col_type
+        amount_opts[:type] ||= :decimal
 
-        if options.key?(:amount) && options[:amount].is_a?(Hash)
-          amount_opts[:null]      = options[:amount][:null]      if options[:amount].key?(:null)
-          amount_opts[:default]   = options[:amount][:default]   if options[:amount].key?(:default)
-          amount_opts[:precision] = options[:amount][:precision] if options[:amount].key?(:precision)
-          amount_opts[:scale]     = options[:amount][:scale]     if options[:amount].key?(:scale)
-        end
-
-        if col_type == :decimal && !amount_opts.key?(:precision) && !amount_opts.key?(:scale)
+        if amount_opts[:type] == :decimal && !amount_opts.key?(:precision) && !amount_opts.key?(:scale)
           amount_opts[:precision] = 16
           amount_opts[:scale]     = 4
+        elsif amount_opts[:type] != :decimal
+          amount_opts.delete(:precision)
+          amount_opts.delete(:scale)
         end
 
-        currency_opts = {}
-        currency_opts[:limit] = options[:currency_limit] if options[:currency_limit]
-
-        if options[:currency].is_a?(Hash)
-          currency_opts[:null]    = options[:currency][:null]    if options[:currency].key?(:null)
-          currency_opts[:default] = options[:currency][:default] if options[:currency].key?(:default)
+        stripped = name.end_with?('_amount') ? name.sub(/_amount$/, '') : name
+        default_currency_col = "#{stripped}_currency"
+
+        if options.key?(:currency) && options[:currency].is_a?(Hash)
+          currency_col = options[:currency][:column]&.to_s || default_currency_col
+          opts = options[:currency]
+          currency_opts = {
+            limit: opts[:limit],
+            null: opts[:null],
+            default: opts[:default]
+          }.compact
+        elsif options[:currency] == false
+          currency_col = nil
+          currency_opts = {}
+        else
+          currency_col = default_currency_col
+          currency_opts = {}
         end
 
         [amount_col, currency_col, amount_opts, currency_opts]

data/lib/money_attribute/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module MoneyAttribute
-  VERSION = '0.11.0'
+  VERSION = '0.12.0'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: money_attribute
 version: !ruby/object:Gem::Version
-  version: 0.11.0
+  version: 0.12.0
 platform: ruby
 authors:
 - Gilson Ferraz