pxs-forms 0.0.3 → 0.0.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a8da1ac34353aaecca98a9314d669053b8bd4212681349d2cb5a05b5cc555207
-  data.tar.gz: bb24505ddf539187c82ab89baff8a81213351e7ea8f3bdeae907e709e303f4de
+  metadata.gz: b78b1ae5381a69dd4f9b75d4c90878f8eaadc09c3a336c5ded134d92e529ab6b
+  data.tar.gz: 8325584e7c5158a52ffb7f5f8618b2001b897e83f15c41731b55332ac14292ae
 SHA512:
-  metadata.gz: e5174781bfef2632e8544bb3ac51ef58861c3f71a35bbb666bcdaab46a3ef2957ebf67d650ba33439ef1fe98e15ca7792a13a6217c54f0d9d56fc4165af6099e
-  data.tar.gz: 92f020826fb32a28db8ee1a89aa16562195f026e56a9b70af3e234ef79b7e1901f074f762ed0d5c7ecf1ce1605b967a8ed3c5e714fed4ba5668b96cafc070344
+  metadata.gz: f6de19d9021c51c308c53640a1eea5570da58149f0353d552b26ca9b0dab9657ef6a1acab747a98dfcc13ad84036c509636ff43805a70a71ba240e3e70b96e41
+  data.tar.gz: 5c9cefb22209e754a9cc32141bc6b4308c95f48a3cd937952e7ddfc2cd75e50788905e79823c72d9d33bea13e300ee554714af313a696d7af4c4291a8452c9d9

data/lib/pxs/forms/model_form_builder.rb

@@ -2,22 +2,30 @@ class ModelFormBuilder < ActionView::Helpers::FormBuilder
 
     delegate :tag, :safe_join, to: :@template
 
+    # define a model field
+    # the format is deduced from the column type or attribute name
     def field(attribute, options = {})
         @form_options = options
+
+        # extract the object type from the attribute
         object_type = object_type_for_attribute(attribute)
 
+        # set the input type depending on the attribute type
         input_type = case object_type
             when :date then :string
             when :integer then :string
             else object_type
         end
 
+        # if as: :input_type is set, use it to set input type
         override_input_type = if options[:as]
             options[:as]
+        # for collections, use a Select
         elsif options[:collection]
             :select
         end
 
+        # return result of [input_type]_input method
         send("#{override_input_type || input_type}_input", attribute, options)
     end
 
@@ -85,8 +93,10 @@ class ModelFormBuilder < ActionView::Helpers::FormBuilder
 
     def select_input(attribute, options = {})
 
-        value_method = options[:value_method] || :to_s
-        text_method = options[:text_method] || :to_s
+        # default value method to :id
+        value_method = options[:value_method] || :id
+        # default text method to :name
+        text_method = options[:text_method] || :name
         input_options = options[:input_html] || {}
 
         multiple = input_options[:multiple]
@@ -166,7 +176,7 @@ class ModelFormBuilder < ActionView::Helpers::FormBuilder
     end
 
     def field_block(attribute, options = {}, &block)
-        tag.div class: "field #{attribute}" do
+        tag.div class: "field #{attribute}", id: options[:field_id] do
             safe_join [
                 block.call,
                 hint_text(options[:hint]),

data/lib/pxs/forms/model_helpers.rb

@@ -47,7 +47,7 @@ module Pxs
 
             def model_form_with(model: nil, scope: nil, url: nil, format: nil, **options, &block)
                 options = options.reverse_merge(builder: ModelFormBuilder)
-                form_with(model: model, scope: scope, url: url, format: format, class: "form#{options.has_key? :class ? " #{options[:class]}" : ""}", **options, &block)
+                form_with(model: model, scope: scope, url: url, format: format, class: "form#{(options.has_key? :class) ? " #{options[:class]}" : ""}", **options, &block)
             end
         end
     end

data/lib/pxs/forms/version.rb

@@ -2,6 +2,6 @@
 
 module Pxs
   module Forms
-    VERSION = "0.0.3"
+    VERSION = "0.0.4"
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: pxs-forms
 version: !ruby/object:Gem::Version
-  version: 0.0.3
+  version: 0.0.4
 platform: ruby
 authors:
 - Poubelle