Collection namespaces use Superform::Field instead of ApplicationForm::Field

[dhnaranjo]

class PageForm < ApplicationForm
  def template
    div do
      field(:id).input
      collection(:responses).each do |response_fields|
        response_fields.field(:option_id).input
      end
    end
  end
end

The call to field(:id) returns an instance of ApplicationForm::Field, while the one inside the collection returns Superform::Field, so input and all my custom fields are unavailable.

My stupid hack is to monkeypatch Superform::Namespace.

module Superform
  class Namespace < Node
    def initialize(key, parent:, object: nil, field_class: ApplicationForm::Field)
      super(key, parent:)
      @object = object
      @field_class = field_class
      @children = {}
      yield self if block_given?
    end
  end
end

You got a better idea of how to handle this?

[bradgessler]

It looks like the collection method is passing a NamespaceCollection constant into the create_child method, which doesn't pass the reference to the class you've extended.

Here's the code that I think has the bug:

superform/lib/superform.rb

Lines 140 to 144 in 150985c

	# The object within the block is a `Namespace` object that maps each object within the enumerable
	# to another `Namespace` or `Field`.
	def collection(key, &)
	create_child(key, NamespaceCollection, &)
	end

[dhnaranjo]

module NamespacePatch
  def collection(key, &)
    create_child(key, Superform::NamespaceCollection, field_class: @field_class, &)
  end
end

module NamespaceCollectionPatch
  def initialize(*, field_class:, **, &)
    @field_class = field_class
    super(*, **, &)
  end

  def build_namespace(index, **)
    parent.class.new(index, parent: self, field_class: @field_class, **, &@template)
  end
end

This gets it where it needs to be. Want a PR?

[bradgessler]

Want a PR?

Heck yeah!