inline_forms 8.1.21 → 8.1.40

data/lib/inline_forms/form_element_registry.rb

@@ -3,38 +3,38 @@
 module InlineForms
   module FormElementRegistry
     ENTRIES = {
-      :audio_field => :string,
-      :check_box => :boolean,
-      :check_list => :no_migration,
-      :date_select => :date,
-      :decimal_field => :decimal,
-      :devise_password_field => :string,
-      :dropdown => :belongs_to,
-      :dropdown_with_integers => :integer,
-      :dropdown_with_other => :belongs_to,
-      :dropdown_with_values => :integer,
-      :dropdown_with_values_with_stars => :integer,
-      :file_field => :string,
-      :header => :string,
-      :image_field => :string,
-      :info => :string,
-      :integer_field => :integer,
-      :money_field => :integer,
-      :month_select => :integer,
-      :month_year_picker => :date,
-      :multi_image_field => :string,
-      :plain_text => :text,
-      :plain_text_area => :text,
-      :question_list => :no_migration,
-      :radio_button => :integer,
-      :rich_text => :no_migration,
-      :scale_with_integers => :integer,
-      :scale_with_values => :integer,
-      :simple_file_field => :string,
-      :slider_with_values => :integer,
-      :text_area => :text,
-      :text_field => :string,
-      :time_select => :time,
+      audio_field: :string,
+      check_box: :boolean,
+      check_list: :no_migration,
+      color_field: :string,
+      date_select: :date,
+      decimal_field: :decimal,
+      devise_password_field: :string,
+      dropdown: :belongs_to,
+      dropdown_with_integers: :integer,
+      dropdown_with_other: :belongs_to,
+      dropdown_with_values: :integer,
+      dropdown_with_values_with_stars: :integer,
+      file_field: :string,
+      header: :string,
+      image_field: :string,
+      info: :string,
+      integer_field: :integer,
+      money_field: :integer,
+      month_select: :integer,
+      month_year_picker: :date,
+      multi_image_field: :string,
+      plain_text: :text,
+      plain_text_area: :text,
+      radio_button: :integer,
+      rich_text: :no_migration,
+      scale_with_integers: :integer,
+      scale_with_values: :integer,
+      simple_file_field: :string,
+      slider_with_values: :integer,
+      text_area: :text,
+      text_field: :string,
+      time_select: :time
     }.freeze
 
     def self.apply!

data/lib/inline_forms/form_elements/color_field_helper.rb

@@ -0,0 +1,44 @@
+# -*- encoding : utf-8 -*-
+module InlineForms
+  module FormElements
+    module ColorFieldHelper
+      module_eval(<<~'INLINE_FORMS_FORM_ELEMENT', __FILE__, __LINE__ + 1)
+    # -*- encoding : utf-8 -*-
+    
+    # color_field (8.1.32): hex color stored as "#rrggbb" in a string column,
+    # edited with the native <input type="color"> picker. Introduced for the
+    # per-user theme color overrides (Pattern 2), usable anywhere.
+    def color_field_show(object, attribute)
+      value = object.send(attribute).to_s
+      display =
+        if value.match?(/\A#\h{6}\z/)
+          # Swatch + hex text. The value is validated above, so the inline
+          # style cannot carry anything but the color.
+          ("<span class='color_swatch' style='display:inline-block;width:1em;height:1em;" \
+           "vertical-align:-0.15em;border:1px solid #8888;background-color:#{value};'></span> " \
+           "#{value}").html_safe
+        else
+          "<i class='fi-plus'></i>".html_safe
+        end
+      link_to_inline_edit object, attribute, display, from_callee: __callee__
+    end
+    
+    def color_field_edit(object, attribute)
+      css_id = 'color_field_' + object.class.to_s.underscore + '_' + object.id.to_s + '_' + attribute.to_s
+      current = object.send(attribute).to_s
+      current = "#000000" unless current.match?(/\A#\h{6}\z/)
+      color_field_tag attribute, current, :id => css_id, :class => 'color_field'
+    end
+    
+    def color_field_update(object, attribute)
+      raw = params[attribute.to_sym].to_s.strip.downcase
+      object[attribute.to_sym] = raw.match?(/\A#\h{6}\z/) ? raw : nil
+    end
+    
+    def color_field_info(object, attribute)
+      object.send(attribute).to_s
+    end
+      INLINE_FORMS_FORM_ELEMENT
+    end
+  end
+end

data/lib/inline_forms/form_elements/date_helper.rb

@@ -10,9 +10,12 @@ module InlineForms
       link_to_inline_edit object, attribute, object.send(attribute).nil? ? "<i class='fi-plus'></i>".html_safe : object.send(attribute).to_date.strftime("%d-%m-%Y"), from_callee: __callee__
     end
     
+    # Native <input type="date"> (8.1.25; replaces jQuery UI datepicker).
+    # The browser shows a locale-formatted date but always submits ISO 8601
+    # (YYYY-MM-DD), which Active Record casts directly.
     def date_select_edit(object, attribute)
       css_id = 'datepicker_' + object.class.to_s.underscore + '_' + object.id.to_s + '_' + attribute.to_s
-      out = text_field_tag attribute, ( object.send(attribute).nil? ? "" : object.send(attribute).to_date.strftime("%d-%m-%Y") ), :id => css_id, :class =>'datepicker'
+      date_field_tag attribute, ( object.send(attribute).nil? ? "" : object.send(attribute).to_date.strftime("%F") ), :id => css_id, :class => 'date_select'
     end
     
     def date_select_update(object, attribute)

data/lib/inline_forms/form_elements/dropdown_with_other_helper.rb

@@ -19,124 +19,38 @@ module InlineForms
       link_to_inline_edit object, attribute, attribute_value, from_callee: __callee__
     end
     
+    # Native combobox (8.1.26; replaces the jQuery UI autocomplete widget and
+    # its inline <script>): a text input backed by a <datalist> of the existing
+    # records. Typing a listed presentation picks that record; any other text
+    # becomes the free-form "other" value. The old hidden <select> is gone --
+    # dropdown_with_other_update never read the posted foreign key anyway (it
+    # re-derives it by name lookup from the submitted text).
     def dropdown_with_other_edit(object, attribute)
       attribute = attribute.to_s
       foreign_key = object.class.reflect_on_association(attribute.to_sym).options[:foreign_key] || attribute.foreign_key.to_sym
       o = attribute.camelcase.constantize
       values = o.all
       values = o.accessible_by(current_ability) if cancan_enabled?
-      values.each do |v|
-        v.name = v._presentation
-      end
-      # values.sort_by(&:name)
     
-      collection = values.map {|v|[v.name, v.id]}
-      collection << [object[attribute + '_other'], 0] unless object[attribute + '_other'].nil? || object[attribute + '_other'].empty?
-      out = '<div class="ui-widget">'
-      out << select('_' + object.class.to_s.underscore, foreign_key.to_sym, collection, {selected: object[foreign_key.to_sym]}, {id: '_' + object.class.to_s.underscore + '_' + object.id.to_s + '_'  + foreign_key.to_s})
-      out << '</div>
-      <script>
-        (function( $ ) {
-        $.widget( "custom.combobox", {
-          _create: function() {
-            this.wrapper = $( "<span>" )
-              .addClass( "custom-combobox" )
-              .insertAfter( this.element );
-     
-            this.element.hide();
-            this._createAutocomplete();
-            this._createShowAllButton();
-          },
-     
-          _createAutocomplete: function() {
-            var selected = this.element.children( ":selected" ),
-              value = selected.val() ? selected.text() : "";
-     
-            this.input = $( "<input name=\''
-            
-        out << '_' + object.class.to_s.underscore + '[' + attribute + '_other]' 
-            
-            out << '\'>" )
-              .appendTo( this.wrapper )
-              .val( value )
-              .attr( "title", "" )
-              .addClass( "custom-combobox-input ui-widget ui-widget-content ui-state-default ui-corner-left" )
-              .autocomplete({
-                delay: 0,
-                minLength: 0,
-                source: $.proxy( this, "_source" )
-              })
-              .tooltip({
-                tooltipClass: "ui-state-highlight"
-              });
-     
-            this._on( this.input, {
-              autocompleteselect: function( event, ui ) {
-                ui.item.option.selected = true;
-                this._trigger( "select", event, {
-                  item: ui.item.option
-                });
-              }
-            });
-          },
-     
-          _createShowAllButton: function() {
-            var input = this.input,
-              wasOpen = false;
-     
-            $( "<a>" )
-              .attr( "tabIndex", -1 )
-              .attr( "title", "Show All Items" )
-              .tooltip()
-              .appendTo( this.wrapper )
-              .button({
-                icons: {
-                  primary: "ui-icon-triangle-1-s"
-                },
-                text: false
-              })
-              .removeClass( "ui-corner-all" )
-              .addClass( "custom-combobox-toggle ui-corner-right" )
-              .mousedown(function() {
-                wasOpen = input.autocomplete( "widget" ).is( ":visible" );
-              })
-              .click(function() {
-                input.focus();
-     
-                // Close if already visible
-                if ( wasOpen ) {
-                  return;
-                }
-     
-                // Pass empty string as value to search for, displaying all results
-                input.autocomplete( "search", "" );
-              });
-          },
-     
-          _source: function( request, response ) {
-            var matcher = new RegExp( $.ui.autocomplete.escapeRegex(request.term), "i" );
-            response( this.element.children( "option" ).map(function() {
-              var text = $( this ).text();
-              if ( this.value && ( !request.term || matcher.test(text) ) )
-                return {
-                  label: text,
-                  value: text,
-                  option: this
-                };
-            }) );
-          },
-        });
-      })( jQuery );
-     
-      $(function() {
-        $( "'
-        out << '#_' + object.class.to_s.underscore + '_' + object.id.to_s + '_' + attribute.foreign_key.to_s 
-        out << '" ).combobox();
-      });
-      </script>'
-      out.html_safe
-      
-      
+      current =
+        if object[foreign_key].to_i == 0
+          object[attribute + '_other'].to_s
+        else
+          (object.send(attribute)._presentation rescue '')
+        end
+    
+      input_name  = '_' + object.class.to_s.underscore + '[' + attribute + '_other]'
+      input_id    = '_' + object.class.to_s.underscore + '_' + object.id.to_s + '_' + foreign_key.to_s
+      datalist_id = input_id + '_options'
+    
+      options = values.map { |v| content_tag(:option, nil, value: v._presentation) }.join.html_safe
+    
+      text_field_tag(input_name, current,
+                     id: input_id,
+                     list: datalist_id,
+                     class: 'dropdown_with_other',
+                     autocomplete: 'off') +
+        content_tag(:datalist, options, id: datalist_id)
     end
     
     def dropdown_with_other_update(object, attribute)
@@ -148,9 +62,7 @@ module InlineForms
       lookup_model = attribute.camelcase.constantize
       name_field = 'name_' + I18n.locale.to_s
       name_field = 'name' unless lookup_model.new.respond_to? name_field
-      puts 'XXXXXXXXXXXXXXXXXXXX ' + name_field.inspect
       match = lookup_model.where(name_field.to_sym => other).first # problem if there are dupes!
-      puts 'XXXXXXXXXXXXXXXXXXXX ' + match.inspect
       match.nil? ? object[foreign_key] = 0 : object[foreign_key] = match.id # problem if there is a record with id: 0 !
       match.nil? ? object[attribute + '_other'] = other : object[attribute + '_other'] = nil  
     end

data/lib/inline_forms/form_elements/month_year_picker_helper.rb

@@ -10,14 +10,25 @@ module InlineForms
       link_to_inline_edit object, attribute, object.send(attribute).nil? ? "<i class='fi-plus'></i>".html_safe : object.send(attribute).strftime("%B %Y"), from_callee: __callee__
     end
     
+    # Native <input type="month"> (8.1.25; replaces the jQuery UI month/year
+    # datepicker hack). Submits ISO "YYYY-MM"; stored as the first of month.
     def month_year_picker_edit(object, attribute)
       css_id = 'datepicker_' + object.class.to_s.underscore + '_' + object.id.to_s + '_' + attribute.to_s
-      out = text_field_tag attribute, ( object.send(attribute).nil? ? "" : object.send(attribute).strftime("%B %Y") ), :id => css_id, :class =>'datepicker datepicker-month-year'
+      month_field_tag attribute, ( object.send(attribute).nil? ? "" : object.send(attribute).strftime("%Y-%m") ), :id => css_id, :class => 'month_year_picker'
     end
     
     def month_year_picker_update(object, attribute)
       raw = params[attribute.to_sym].to_s
-      object[attribute.to_sym] = raw.empty? ? nil : Date.parse(raw).strftime("%F").to_s
+      object[attribute.to_sym] =
+        if raw.empty?
+          nil
+        elsif raw.match?(/\A\d{4}-\d{2}\z/)
+          # <input type="month"> value; Date.parse would raise on it.
+          Date.strptime(raw, "%Y-%m").strftime("%F")
+        else
+          # Legacy "September 2026" style (pre-8.1.25 clients / tests).
+          Date.parse(raw).strftime("%F")
+        end
     rescue Date::Error
       object[attribute.to_sym] = nil
     end

data/lib/inline_forms/form_elements/multi_image_field_helper.rb

@@ -5,25 +5,37 @@ module InlineForms
       module_eval(<<~'INLINE_FORMS_FORM_ELEMENT', __FILE__, __LINE__ + 1)
     # -*- encoding : utf-8 -*-
     
+    # A `mount_uploaders` (plural) column returns an *array* of uploaders; the
+    # pre-8.1.28 code called `.url` on that array and raised NoMethodError the
+    # moment a multi_image_field actually held images. Render every image
+    # (using the `palm` thumb version when the uploader defines one), and keep
+    # accepting a bare single uploader for backward compatibility.
     def multi_image_field_show(object, attribute)
       o = object.send(attribute)
       msg = "<i class='fi-plus'></i>".html_safe
       if o.send(:present?)
-        if o.respond_to? :palm
-          msg = image_tag(o.send(:palm).send(:url))
-        else
-          msg = image_tag(o.send(:url))
+        uploads = o.is_a?(Array) ? o : [o]
+        rendered = uploads.filter_map do |u|
+          next unless u.respond_to?(:url) && u.url.present?
+          u.respond_to?(:palm) ? image_tag(u.palm.url) : image_tag(u.url)
         end
+        msg = safe_join(rendered) unless rendered.empty?
       end
       link_to_inline_edit object, attribute, msg, from_callee: __callee__
     end
     
+    # `name="gallery[]"`: file_field_tag with `multiple:` does not add the []
+    # itself, and without it Rack keeps only the last file — the update then
+    # stored a single image. With [] the params value is an array, which is
+    # what a mount_uploaders (plural) setter expects.
     def multi_image_field_edit(object, attribute)
-      file_field_tag attribute, multiple: true, class: 'input_text_field multi_image_field'
+      file_field_tag "#{attribute}[]", multiple: true, class: 'input_text_field multi_image_field'
     end
     
     def multi_image_field_update(object, attribute)
-      object.send(attribute.to_s + '=', params[attribute.to_sym])
+      files = params[attribute.to_sym]
+      return if files.blank?
+      object.send(attribute.to_s + '=', Array(files))
     end
       INLINE_FORMS_FORM_ELEMENT
     end

data/lib/inline_forms/form_elements/slider_with_values_helper.rb

@@ -5,35 +5,34 @@ module InlineForms
       module_eval(<<~'INLINE_FORMS_FORM_ELEMENT', __FILE__, __LINE__ + 1)
     # -*- encoding : utf-8 -*-
     
-    # slider_with_values
+    # slider_with_values -- native <input type="range"> since 8.1.26 (replaces
+    # the jQuery UI slider and its inline <script>). The label text next to the
+    # slider is kept in sync by initInlineFormsWidgets (class hook
+    # `slider_with_values`, labels in data-slider-values, target <output> id in
+    # data-slider-output); helpers emit no per-field scripts.
     def slider_with_values_show(object, attribute)
       values = attribute_values(object, attribute)
       value = object.send(attribute).to_i         # should be an int
-      display_value = values.assoc(value)[1]      # values should be [ [ 0, value ], [ 3, value2 ] .... ] and we lookup the key, not the place in the array!
+      # values should be [ [ 0, value ], [ 3, value2 ] .... ] and we lookup the
+      # key, not the place in the array! A value with no entry (e.g. 0 when the
+      # values hash starts at 1) falls back to the bare number instead of
+      # crashing (pre-8.1.26 code did values.assoc(value)[1] unguarded).
+      display_value = (pair = values.assoc(value)) ? pair[1] : value.to_s
       css_id = "#{object.class.to_s.underscore}_#{object.id}_#{attribute}"
       if value == 0
         out = "?"   # we use this as the 'unknown' value. So in the data, 0 should always be the unknown value. This gives problems with sliders where the real value is 0.
       else
+        # <progress> (not a disabled <input type=range>): the show panel is
+        # wrapped in the inline-edit link, and a disabled input would swallow
+        # the click; progress is non-interactive so the click-to-edit works.
         out = "".html_safe
-        out << "<div class='slider slider_#{attribute.to_s}' id='slider_#{css_id}'></div>".html_safe
-        out << "<div class='slider_value' id='value_#{css_id}'>#{display_value}</div>".html_safe
-        out << "<div style='clear: both' />".html_safe
-        out << "<input type='hidden' name='_#{object.class.to_s.underscore}[#{attribute}]' value='0' id='input_#{css_id}' />".html_safe
-        out << ('<script>
-                  $(function() {
-                    var displayvalues = ' + values.collect {|x| x[1]}.inspect + ';
-                    $( "#slider_' + css_id + '" ).slider(
-                      {
-                        value:' + value.to_s + ',
-                        range: "min",
-                        disabled: true,
-                        min: 0,
-                        max: ' + values.length.to_s + ',
-                        step: 1,
-                      }
-                    );
-                  });
-                 </script>').html_safe
+        out << content_tag(:progress, nil,
+                           value: value,
+                           max: values.collect(&:first).max.to_i,
+                           class: "slider slider_#{attribute}",
+                           id: "slider_#{css_id}")
+        out << content_tag(:div, display_value, class: 'slider_value', id: "value_#{css_id}")
+        out << "<div style='clear: both'></div>".html_safe
       end
       link_to_inline_edit object, attribute, out, from_callee: __callee__
     end
@@ -43,56 +42,24 @@ module InlineForms
       values = attribute_values(object, attribute)
       value = object.send(attribute).to_i         # should be an int, will be 0 if nil
       css_id = "#{object.class.to_s.underscore}_#{object.id}_#{attribute}"
-      display_value = values.assoc(value)[1]      # values should be [ [ 0, value ], [ 3, value2 ] .... ] and we lookup the key, not the place in the array!
+      display_value = (pair = values.assoc(value)) ? pair[1] : value.to_s
+      labels = values.collect { |x| x[1] }
       out = "".html_safe
-      out << "<div class='slider slider_#{attribute.to_s}' id='slider_#{css_id}'></div>".html_safe
-      out << "<div class='slider_value' id='value_#{css_id}'>#{display_value}</div>".html_safe
-      out << "<div style='clear: both' />".html_safe
-      out << "<input type='hidden' name='_#{object.class.to_s.underscore}[#{attribute}]' value='0' id='input_#{css_id}' />".html_safe
-      out << ('<script>
-    	$(function() {
-        var displayvalues = ' + values.collect {|x| x[1]}.inspect + ';
-    		$( "#slider_' + css_id + '" ).slider(
-          {
-            value:' + value.to_s + ',
-            range: "min",
-            min: 0,
-            max: 5,
-            step: 1,
-            slide: function( event, ui ) {
-              $( "#input_' + css_id + '" ).val( ui.value );
-              $( "#value_' + css_id + '" ).html( displayvalues[ui.value] );
-            }
-      		}
-        );').html_safe
-      out << ('$( "#value_' + css_id + '" ).html(displayvalues[' + value.to_s + ']);').html_safe
-    	out << ('$( "#input_' + css_id + '" ).val(' + value.to_s + ');').html_safe
-    	out << ('});
-    	</script>').html_safe
+      out << range_field_tag("_#{object.class.to_s.underscore}[#{attribute}]", value,
+                             id: "input_#{css_id}",
+                             class: "slider slider_#{attribute} slider_with_values",
+                             in: 0..(values.collect(&:first).max.to_i),
+                             step: 1,
+                             data: { slider_values: labels.to_json,
+                                     slider_output: "value_#{css_id}" })
+      out << content_tag(:output, display_value, class: 'slider_value', id: "value_#{css_id}")
+      out << "<div style='clear: both'></div>".html_safe
       out
     end
     
     def slider_with_values_update(object, attribute)
       object[attribute.to_sym] = params[('_' + object.class.to_s.underscore).to_sym][attribute.to_sym]
     end
-    
-    #<style>
-    #	#demo-frame > div.demo { padding: 10px !important; };
-    #	</style>
-    #
-    #
-    #
-    #<div class="demo">
-    #
-    #<p>
-    #	<label for="amount">Donation amount ($50 increments):</label>
-    #	<input type="text" id="amount" style="border:0; color:#f6931f; font-weight:bold;" />
-    #</p>
-    #
-    #<div id="slider"></div>
-    #
-    #</div><!-- End demo -->
-    #
       INLINE_FORMS_FORM_ELEMENT
     end
   end

data/lib/inline_forms/form_elements/time_helper.rb

@@ -10,9 +10,11 @@ module InlineForms
         link_to_inline_edit object, attribute, object.send(attribute).nil? ? "<i class='fi-plus'></i>".html_safe : object.send(attribute).to_datetime.strftime("%l:%M%P"), from_callee: __callee__
       end
     
+      # Native <input type="time"> (8.1.25; replaces the jQuery timepicker).
+      # Submits 24h "HH:MM", which Active Record casts directly to a time column.
       def time_select_edit(object, attribute)
         css_id = 'timepicker_' + object.class.to_s.underscore + '_' + object.id.to_s + '_' + attribute.to_s
-        out = text_field_tag attribute, ( object.send(attribute).nil? ? "" : object.send(attribute).to_datetime.strftime("%l:%M%P") ), :id => css_id, :class =>'timepicker'
+        time_field_tag attribute, ( object.send(attribute).nil? ? "" : object.send(attribute).to_datetime.strftime("%H:%M") ), :id => css_id, :class => 'time_select'
       end
     
       def time_select_update(object, attribute)

data/lib/inline_forms/gem_files.rb

@@ -1,4 +1,5 @@
 # -*- encoding: utf-8 -*-
+
 module InlineFormsGemFiles
   INSTALLER_FILE_PREFIXES = %w[
     bin/inline_forms