nokogiri-html-ext 1.5.0 → 1.6.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: fe48e23deb7150b7adf50eb5298dfef2dbc9a75898574ed21cd567038728d2ed
-  data.tar.gz: 0f5f378ae1180ab2718b1927e220d092363a8cb013766e4ffaf483050c2f7020
+  metadata.gz: 307b06335c0935cec4304b10c13ef113c514646d5a746f703ccb1567a39120ce
+  data.tar.gz: 26894c4ef4b3a018f2b776f4929035e590a8c9514af529fce581c23fb29e4cf0
 SHA512:
-  metadata.gz: beba250e78b23dfea02fe3cdbca224049f4a63f7f6ab990403eedaab6d29770f09aaa47fa2cadc18304237baa2b9b2d751a2940b6d34e98a9b3abd48f815e02e
-  data.tar.gz: 2971a2ae956f0877aae05a70ea6283ccebb72c935667deda53d7337f4cde74b946950ce0a2f826a7a522c7538ddf84090603a683690baf080238709a705f0dd8
+  metadata.gz: 66057c43dd9cc165dcd12603f8546119f39292305b3d943f5930c138686acbec53d2dfd744565343fe1f2351a8aa9a46b84a6d7d1ba7cafe7f10dd17be19548e
+  data.tar.gz: 52e9543d90ed8dc1790458f660a02cdb7c4574cb8c3fc7a093b7780cbe4627a8badc6ef84d51aae7acdc7ad5aa2e88cdec38e5eb597acffd6f8e8652950b62f5

data/lib/nokogiri/html_ext/document.rb

@@ -3,6 +3,9 @@
 module Nokogiri
   module HTML4
     class Document < Nokogiri::XML::Document
+      # @see https://html.spec.whatwg.org/#image-candidate-string
+      IMAGE_CANDIDATE_STRING_PATTERN = /^(?<url>.+?)(?<descriptor>\s+.+)?$/.freeze
+
       # A map of HTML +srcset+ attributes and their associated element names.
       #
       # @see https://html.spec.whatwg.org/#srcset-attributes
@@ -56,6 +59,37 @@ module Nokogiri
         base["href"] = url
       end
 
+      # Conditionally normalize a URL's empty path component.
+      #
+      # @param [String]
+      #
+      # @return [String]
+      def normalize_empty_path(url)
+        uri = Addressable::URI.parse(url.strip)
+
+        uri.path = uri.normalized_path if uri.path.empty? && uri.path != uri.normalized_path
+
+        uri.to_s
+      rescue Addressable::URI::InvalidURIError
+        url
+      end
+
+      # Normalize the document's URLs whose path components are empty.
+      #
+      # @return [self]
+      def normalize_empty_paths!
+        xpath(*ATTRIBUTES_XPATHS).each do |attr_node|
+          attr_node.value =
+            if SRCSET_ATTRIBUTES_MAP.key?(attr_node.name)
+              normalize_srcset_attribute(attr_node.value)
+            else
+              normalize_empty_path(attr_node.value)
+            end
+        end
+
+        self
+      end
+
       # Resolve a URL based on the current document.
       #
       # @param url [String]
@@ -78,7 +112,7 @@ module Nokogiri
         xpath(*ATTRIBUTES_XPATHS).each do |attr_node|
           attr_node.value =
             if SRCSET_ATTRIBUTES_MAP.key?(attr_node.name)
-              resolve_srcset_attributes(attr_node.value.split(/\s*,\s*/))
+              resolve_srcset_attribute(attr_node.value)
             else
               resolve_url(attr_node.value)
             end
@@ -89,20 +123,40 @@ module Nokogiri
 
       private
 
-      # Resolve a set of +String+s that represent +srcset+ attribute image
-      # candidate strings.
+      # Normalize the URLs in a +srcset+ attribute's image candidate strings.
       #
-      # @param srcset_attributes [Array<String>]
+      # @param value [String]
       #
       # @return [String]
-      def resolve_srcset_attributes(srcset_attributes)
-        srcset_attributes.map! do |candidate_string|
-          # rubocop:disable Style/PerlBackrefs
-          candidate_string.sub(/^(.+?)(\s+.+)?$/) { "#{resolve_url($1)}#{$2}" }
-          # rubocop:enable Style/PerlBackrefs
-        end
+      def normalize_srcset_attribute(value)
+        parse_image_candidate_strings(value)
+          .map! { |captures| "#{normalize_empty_path(captures["url"])}#{captures["descriptor"]}" }
+          .join(", ")
+      end
 
-        srcset_attributes.join(", ")
+      # Parse a +srcset+ attribute's value into a +Hash+ of image candidate
+      # strings.
+      #
+      # @see https://html.spec.whatwg.org/#image-candidate-string
+      #
+      # @param [String]
+      #
+      # @return [Array<Hash{String => String, nil}>]
+      def parse_image_candidate_strings(value)
+        value
+          .split(/\s*,\s*/)
+          .map! { |candidate_string| candidate_string.match(IMAGE_CANDIDATE_STRING_PATTERN).named_captures }
+      end
+
+      # Resolve the URLs in a +srcset+ attribute's image candidate strings.
+      #
+      # @param value [String]
+      #
+      # @return [String]
+      def resolve_srcset_attribute(value)
+        parse_image_candidate_strings(value)
+          .map! { |captures| "#{resolve_url(captures["url"])}#{captures["descriptor"]}" }
+          .join(", ")
       end
     end
   end

data/nokogiri-html-ext.gemspec

@@ -4,7 +4,7 @@ Gem::Specification.new do |spec|
   spec.required_ruby_version = ">= 2.7"
 
   spec.name          = "nokogiri-html-ext"
-  spec.version       = "1.5.0"
+  spec.version       = "1.6.0"
   spec.authors       = ["Jason Garber"]
   spec.email         = ["jason@sixtwothree.org"]
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: nokogiri-html-ext
 version: !ruby/object:Gem::Version
-  version: 1.5.0
+  version: 1.6.0
 platform: ruby
 authors:
 - Jason Garber
@@ -55,11 +55,11 @@ licenses:
 - MIT
 metadata:
   bug_tracker_uri: https://codeberg.org/jgarber/nokogiri-html-ext/issues
-  changelog_uri: https://codeberg.org/jgarber/nokogiri-html-ext/releases/tag/v1.5.0
-  documentation_uri: https://rubydoc.info/gems/nokogiri-html-ext/1.5.0
+  changelog_uri: https://codeberg.org/jgarber/nokogiri-html-ext/releases/tag/v1.6.0
+  documentation_uri: https://rubydoc.info/gems/nokogiri-html-ext/1.6.0
   homepage_uri: https://codeberg.org/jgarber/nokogiri-html-ext
   rubygems_mfa_required: 'true'
-  source_code_uri: https://codeberg.org/jgarber/nokogiri-html-ext/src/tag/v1.5.0
+  source_code_uri: https://codeberg.org/jgarber/nokogiri-html-ext/src/tag/v1.6.0
 rdoc_options: []
 require_paths:
 - lib