html-proofer 5.2.0 → 5.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 03f01340b8aebd3ccad6d82ab5aef056991f91bae04498df5091b3c7e9e6641f
-  data.tar.gz: 3dda407dc70dc4d6d65ac88f6b3474e091832c3b37bc70d3bc74e8fdc4d136db
+  metadata.gz: 9883d35b171b2912cb327e9360f7ff63fcecaaff8cafa44c04eb7ba38c2ea226
+  data.tar.gz: '009a17164398c5bfd6456d6c0235e8606cd9bc98602ac392e11269e49574867e'
 SHA512:
-  metadata.gz: f652269525122a2199b373be5f19a3a93541f14e14539621f207605f5e346cf2cd34fe68cea4ed6e86236f37e69e5f1db709c68fab71816847b20248acb487a3
-  data.tar.gz: 487828db630f2916ab417d8d9d3cecbd8aa96a5140ee3695d2c49b589f3d8f286bcb9fef964bf517f08286e35500dfc6c0a1119f2f83d22d9f0a6653ad3a3c8d
+  metadata.gz: 02ba6094a5a0516d6452b72d898be194cc1c2ace2779e6c21b688116b2335dd55704c78ba614cac7d5aab56fa0557b18e82fb0c1d0f084954bc012de584f52e3
+  data.tar.gz: 0cf8697a7bb36c0213ee62596323e21679d0bc679315aee33ab521bea8675f13d8a2adf3a9f8be7284fd488686629b9e18b2fbcbafc7b59b88d424daa063b50e

data/lib/html_proofer/runner.rb

@@ -200,7 +200,7 @@ module HTMLProofer
     end
 
     def failed_checks
-      @reporter.failures.flatten.select { |f| f.is_a?(Failure) }
+      @reporter.failures.flatten.grep(Failure)
     end
 
     def report_failed_checks

data/lib/html_proofer/url_validator/internal.rb

@@ -116,10 +116,13 @@ module HTMLProofer
         return true if blank?(href_hash)
         return true unless @runner.options[:check_internal_hash]
 
-        # prevents searching files we didn't ask about
-        return false unless url.known_extension?
+        # For hash links, we need to defer to file-based checking
+        # (even if URL doesn't have extension, it may resolve to index.html)
         return false unless url.has_hash?
 
+        # If URL has no known extension, defer to file-based checking
+        return false unless url.known_extension?
+
         decoded_href_hash = Addressable::URI.unescape(href_hash)
         fragment_ids = [href_hash, decoded_href_hash]
         # https://www.w3.org/TR/html5/single-page.html#scroll-to-fragid
@@ -134,10 +137,18 @@ module HTMLProofer
 
       private def find_fragments(fragment_ids, html)
         xpaths = fragment_ids.uniq.flat_map do |frag_id|
-          escaped_frag_id = "'#{frag_id.split("'").join("', \"'\", '")}', ''"
+          # Build XPath string argument, handling single quotes
+          if frag_id.include?("'")
+            # Use concat() to handle single quotes: concat('part1', "'", 'part2')
+            escaped_frag_id = frag_id.split("'").map { |part| "'#{part}'" }.join(", \"'\", ")
+            xpath_arg = "concat(#{escaped_frag_id})"
+          else
+            # No single quotes, just use quoted string
+            xpath_arg = "'#{frag_id}'"
+          end
           [
-            "//*[case_sensitive_equals(@id, concat(#{escaped_frag_id}))]",
-            "//*[case_sensitive_equals(@name, concat(#{escaped_frag_id}))]",
+            "//*[case_sensitive_equals(@id, #{xpath_arg})]",
+            "//*[case_sensitive_equals(@name, #{xpath_arg})]",
           ]
         end
         xpaths << XpathFunctions.new

data/lib/html_proofer/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module HTMLProofer
-  VERSION = "5.2.0"
+  VERSION = "5.2.1"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: html-proofer
 version: !ruby/object:Gem::Version
-  version: 5.2.0
+  version: 5.2.1
 platform: ruby
 authors:
 - Garen Torikian
@@ -234,7 +234,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 4.0.3
+rubygems_version: 4.0.6
 specification_version: 4
 summary: A set of tests to validate your HTML output. These tests check if your image
   references are legitimate, if they have alt tags, if your internal links are working,