feat: Bring back process_attributes callback for attribute processing

Author: joelmoss

lib/phlex/html.rb

@@ -55,6 +55,8 @@ def tag(name, **attributes, &)
 			raise Phlex::ArgumentError.new("Expected the tag name to be a Symbol.")
 		end
 
+		attributes = process_attributes(attributes) if respond_to?(:process_attributes)
+
 		if (tag = StandardElements.__registered_elements__[name]) || (tag = name.name.tr("_", "-")).include?("-")
 			if attributes.length > 0 # with attributes
 				if block_given # with content block

lib/phlex/sgml.rb

@@ -462,6 +462,8 @@ def json_escape(string)
 	private def __render_attributes__(attributes)
 		state = @_state
 		return unless state.should_render?
+
+		attributes = process_attributes(attributes) if respond_to?(:process_attributes)
 		state.buffer << (Phlex::ATTRIBUTE_CACHE[attributes] ||= Phlex::SGML::Attributes.generate_attributes(attributes))
 	end
 

lib/phlex/sgml/attributes.rb

@@ -6,7 +6,9 @@ module Phlex::SGML::Attributes
 	UNSAFE_ATTRIBUTES = Set.new(%w[srcdoc sandbox http-equiv]).freeze
 	REF_ATTRIBUTES = Set.new(%w[href src action formaction lowsrc dynsrc background ping]).freeze
 
-	def generate_attributes(attributes, buffer = +"")
+	def generate_attributes(attributes, buffer = +"", callback: nil)
+		callback&.(attributes)
+
 		attributes.each do |k, v|
 			next unless v
 

lib/phlex/sgml/elements.rb

@@ -54,6 +54,8 @@ def #{method_name}(**attributes)
 					return nil
 				end
 
+				attributes = process_attributes(attributes) if respond_to?(:process_attributes)
+
 				if attributes.length > 0 # with attributes
 					if block_given # with content block
 						buffer << "<#{tag}"
@@ -147,6 +149,7 @@ def #{method_name}(**attributes)
 				return unless state.should_render?
 
 				buffer = state.buffer
+				attributes = process_attributes(attributes) if respond_to?(:process_attributes)
 
 				if attributes.length > 0 # with attributes
 					buffer << "<#{tag}"

quickdraw/sgml/attributes.test.rb

@@ -4,6 +4,28 @@
 
 include SGMLHelper
 
+test 'proscess_attributes callback' do
+	component = Class.new(Phlex::HTML) do
+		def initialize(name)
+			@name = name
+			@count = 0
+		end
+
+		def process_attributes(attrs)
+			attrs.tap do
+				it[:age] = 48 if it[:name] == 'Joel'
+			end
+		end
+
+		def view_template
+			i(class: "foo", name: @name) { "hello" }
+		end
+	end
+
+	assert_equal_html component.call('Joel'), %(<i class="foo" name="Joel" age="48">hello</i>)
+	assert_equal_html component.call('Sam'), %(<i class="foo" name="Sam">hello</i>)
+end
+
 test "id attributes must be lower case symbols" do
 	assert_raises(Phlex::ArgumentError) { phlex { div("id" => "abc") } }
 	assert_raises(Phlex::ArgumentError) { phlex { div("ID" => "abc") } }