Remove support for process_attributes

CHANGELOG.md

@@ -2,6 +2,9 @@
 
 All notable changes to this project will be documented in this file. The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/), and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [Unreleased]
+- [Breaking] Removed support for `process_attributes`. Instead, override `__attributes__`, calling `super` with your processed attributes. `__attributes__` receives a single positional argument which is the original attributes hash.
+
 ## [1.9.0] 2024-11-24
 
 - Improved documentation

lib/phlex/elements.rb

@@ -39,11 +39,11 @@ def #{method_name}(**attributes, &block)
 
 				if attributes.length > 0 # with attributes
 					if block # with content block
-						target << "<#{tag}" << (Phlex::ATTRIBUTE_CACHE[attributes.hash] ||= __attributes__(**attributes)) << ">"
+						target << "<#{tag}" << __attributes__(attributes) << ">"
 						yield_content(&block)
 						target << "</#{tag}>"
 					else # without content block
-						target << "<#{tag}" << (Phlex::ATTRIBUTE_CACHE[attributes.hash] ||= __attributes__(**attributes)) << "></#{tag}>"
+						target << "<#{tag}" << __attributes__(attributes) << "></#{tag}>"
 					end
 				else # without attributes
 					if block # with content block
@@ -77,7 +77,7 @@ def #{method_name}(**attributes)
 				target = @_context.target
 
 				if attributes.length > 0 # with attributes
-					target << "<#{tag}" << (Phlex::ATTRIBUTE_CACHE[attributes.hash] ||= __attributes__(**attributes)) << ">"
+					target << "<#{tag}" << __attributes__(attributes) << ">"
 				else # without attributes
 					target << "<#{tag}>"
 				end

lib/phlex/sgml.rb

@@ -348,23 +348,21 @@ def __text__(content)
 		end
 
 		# @api private
-		def __attributes__(**attributes)
-			if respond_to?(:process_attributes)
-				attributes = process_attributes(**attributes)
-			end
-
-			if attributes[:href]&.start_with?(/\s*javascript:/)
-				attributes.delete(:href)
-			end
+		def __attributes__(attributes)
+			Phlex::ATTRIBUTE_CACHE[attributes.hash] ||= (
+				if attributes[:href]&.start_with?(/\s*javascript:/)
+					attributes.delete(:href)
+				end
 
-			if attributes["href"]&.start_with?(/\s*javascript:/)
-				attributes.delete("href")
-			end
+				if attributes["href"]&.start_with?(/\s*javascript:/)
+					attributes.delete("href")
+				end
 
-			buffer = +""
-			__build_attributes__(attributes, buffer: buffer)
+				buffer = +""
+				__build_attributes__(attributes, buffer: buffer)
 
-			buffer
+				buffer
+			)
 		end
 
 		# @api private