chobble-forms 0.7.1 → 0.8.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 771171d582d4bdb09d03fc7ac56a37ca4922d7ce9ee76f392c19c0667e2cd6d3
-  data.tar.gz: e692e2d66579f14eae1099989efddef5a7358620653ac6ab7daaf3ffab151333
+  metadata.gz: 5278667464b4fb04d365ef198a36874fec3f5590680f705a18b0d5133776e201
+  data.tar.gz: 565d3f9a541a8e5aa5ce91fe0a2394afe2284c449103664785b5bca2a80123ef
 SHA512:
-  metadata.gz: c7b53bd70bfa1063f24071b476a9789b43d79c0c539e033857b1331caf0456bb324fea113a93471bc0fef9b114c9b9004f52b10c0c3a82ef52c6a85e54b91e34
-  data.tar.gz: f4b9fd0dedb8a2e0d133cf09f0dc0ab5e9e4b390cfeafe510b16424e9ec1037492fdec506db580447c61d6857597bb164d60c2b211fc019f1df58d1472f27202
+  metadata.gz: bc9e9c26ef8f0072faa78015d29783b1dd899269d82cc4a2f33f63311ba0714e7a4a20540a0c7e0d6723858070d722a06e19c939565c406eb6f3b81be27bef16
+  data.tar.gz: 3c869b047f9ae812878c8a7b1be78533a25bef9280fbd9b9860d2dfbc5aca51f01e85d33af9d54b8db750f64754765d6c95cd219f61196ff739246948487be93

data/lib/chobble_forms/helpers.rb

@@ -7,11 +7,11 @@ module ChobbleForms
   module Helpers
     extend T::Sig
 
-    SelectOption = T.type_alias {
+    SelectOption = T.type_alias do
       [String, T.any(String, Integer)]
-    }
+    end
 
-    LocalAssignValue = T.type_alias {
+    LocalAssignValue = T.type_alias do
       T.any(
         String,
         Symbol,
@@ -21,19 +21,32 @@ module ChobbleForms
         T::Array[SelectOption],
         T::Hash[Symbol, T.untyped]
       )
-    }
+    end
 
-    FieldSetupResult = T.type_alias {
+    FieldValue = T.type_alias do
+      T.nilable(T.any(
+        String,
+        Integer,
+        Float,
+        Date,
+        DateTime,
+        Time,
+        T::Boolean,
+        T.untyped
+      ))
+    end
+
+    FieldSetupResult = T.type_alias do
       {
         form_object: T.untyped,
         i18n_base: String,
-        value: T.untyped,
+        value: FieldValue,
         prefilled: T::Boolean,
         field_label: String,
         field_hint: T.nilable(String),
         field_placeholder: T.nilable(String)
       }
-    }
+    end
 
     sig { params(field: Symbol, local_assigns: T::Hash[Symbol, LocalAssignValue]).returns(FieldSetupResult) }
     def form_field_setup(field, local_assigns)
@@ -47,7 +60,7 @@ module ChobbleForms
       build_field_setup_result(field_translations, value, prefilled)
     end
 
-    sig { params(form_object: T.untyped, field: Symbol).returns([T.untyped, T::Boolean]) }
+    sig { params(form_object: T.untyped, field: Symbol).returns([FieldValue, T::Boolean]) }
     def get_field_value_and_prefilled_status(form_object, field)
       return [nil, false] unless form_object&.object
 
@@ -91,7 +104,10 @@ module ChobbleForms
       }
     end
 
-    sig { params(prefilled: T::Boolean, checked_value: T.untyped, expected_value: T.untyped).returns(T::Hash[Symbol, T::Boolean]) }
+    sig do
+      params(prefilled: T::Boolean, checked_value: T.untyped,
+        expected_value: T.untyped).returns(T::Hash[Symbol, T::Boolean])
+    end
     def radio_button_options(prefilled, checked_value, expected_value)
       (prefilled && checked_value == expected_value) ? {checked: true} : {}
     end
@@ -136,7 +152,11 @@ module ChobbleForms
       i18n_base = T.unsafe(instance_variable_get(:@_current_i18n_base))
 
       # Only strip _pass suffix for pass/fail fields, not _comment fields
-      lookup_field = field.to_s.end_with?("_pass") ? FieldUtils.strip_field_suffix(field) : field
+      lookup_field = if field.to_s.end_with?("_pass")
+        FieldUtils.strip_field_suffix(field)
+      else
+        field
+      end
       fields_key = "#{i18n_base}.fields.#{lookup_field}"
       field_label = t(fields_key, raise: true)
 
@@ -152,7 +172,10 @@ module ChobbleForms
       }
     end
 
-    sig { params(field_translations: T::Hash[Symbol, T.nilable(String)], value: T.untyped, prefilled: T::Boolean).returns(FieldSetupResult) }
+    sig do
+      params(field_translations: T::Hash[Symbol, T.nilable(String)], value: T.untyped,
+        prefilled: T::Boolean).returns(FieldSetupResult)
+    end
     def build_field_setup_result(field_translations, value, prefilled)
       form_obj = T.unsafe(instance_variable_get(:@_current_form))
       i18n_base = T.unsafe(instance_variable_get(:@_current_i18n_base))
@@ -168,21 +191,28 @@ module ChobbleForms
       )
     end
 
-    sig { params(model: T.untyped, field: Symbol).returns({value: T.untyped, prefilled: T::Boolean}) }
-    def resolve_field_value(model, field)
-      field_str = field.to_s
+    sig { params(model: T.untyped, field: Symbol, raise_if_missing: T::Boolean).returns(T.untyped) }
+    def get_field_value_from_model(model, field, raise_if_missing: false)
+      return nil unless model
+
+      # Check for base field first (for fields like step_ramp_size that have a base value)
+      # Only check for _pass suffix if base field doesn't exist (for pass_fail_comment partials)
+      if model.respond_to?(field)
+        model.send(field)
+      elsif model.respond_to?("#{field}_pass")
+        model.send("#{field}_pass")
+      elsif raise_if_missing
+        available = model.attributes.keys.sort.join(", ")
+        raise "Field '#{field}' or '#{field}_pass' not found on " \
+              "#{model.class.name}. Available fields: #{available}"
+      end
+    end
 
-      return {value: nil, prefilled: false} if field_str.include?("password")
+    sig { params(model: T.untyped, field: Symbol).returns({value: FieldValue, prefilled: T::Boolean}) }
+    def resolve_field_value(model, field)
+      return {value: nil, prefilled: false} if field.to_s.include?("password")
 
-      # For composite partials, we receive the base field but need to check for suffixed fields
-      # Check for _pass field first (in case both base and _pass exist)
-      if model.respond_to?("#{field}_pass")
-        current_value = model.send("#{field}_pass")
-      elsif model.respond_to?(field)
-        current_value = model.send(field)
-      else
-        raise "Field '#{field}' or '#{field}_pass' not found on #{model.class.name}. Available fields: #{model.attributes.keys.sort.join(", ")}"
-      end
+      current_value = get_field_value_from_model(model, field, raise_if_missing: true)
 
       # Check if this field should not be prefilled based on excluded fields list
       excluded_fields = T.unsafe(instance_variable_get(:@_excluded_prefill_fields))
@@ -198,11 +228,7 @@ module ChobbleForms
         }
       end
 
-      if field_str.end_with?("_id") && field_str != "id"
-        resolve_association_value(model, field_str)
-      else
-        {value: current_value, prefilled: false}
-      end
+      {value: current_value, prefilled: false}
     end
 
     sig { params(previous_inspection: T.untyped, current_model: T.untyped, field: Symbol).returns(T.untyped) }
@@ -212,7 +238,7 @@ module ChobbleForms
       elsif current_model.class.name.include?("Assessment")
         assessment_type = current_model.class.name.demodulize.underscore
         previous_model = previous_inspection.send(assessment_type)
-        previous_model&.send(field)
+        get_field_value_from_model(previous_model, field, raise_if_missing: false)
       else
         previous_inspection.send(field)
       end

data/lib/chobble_forms/version.rb

@@ -2,5 +2,5 @@
 # frozen_string_literal: true
 
 module ChobbleForms
-  VERSION = "0.7.1"
+  VERSION = "0.8.0"
 end

data/views/chobble_forms/_display_field.html.erb

@@ -5,7 +5,16 @@
   # Get value from model object automatically
   model_object = setup[:form_object].object
   value = model_object.send(field) if model_object.respond_to?(field)
+  
+  # Format dates and times nicely
+  formatted_value = if value.is_a?(DateTime) || value.is_a?(Time)
+    value.strftime("%B %-d, %Y - %H:%M")
+  elsif value.is_a?(Date)
+    value.strftime("%B %-d, %Y")
+  else
+    value
+  end
 %>
 
 <%= setup[:form_object].label field, setup[:field_label] %>
-<p><%= value %></p>
+<p><%= formatted_value %></p>

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: chobble-forms
 version: !ruby/object:Gem::Version
-  version: 0.7.1
+  version: 0.8.0
 platform: ruby
 authors:
 - Chobble.com
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2025-08-16 00:00:00.000000000 Z
+date: 2025-10-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails