Fixes significant performance degradation introduced in r24094 by removing the CSS selectors from scrubbers (#43446).

Patch by Takashi Kato (user:tohosaku).


git-svn-id: https://svn.redmine.org/redmine/trunk@24139 e93f8b46-1217-0410-a6f0-8f06a7374b81

lib/redmine/wiki_formatting/common_mark/alerts_icons_scrubber.rb

@@ -31,22 +31,22 @@ module Redmine
       }.freeze
 
       class AlertsIconsScrubber < Loofah::Scrubber
-        def scrub(doc)
-          doc.search("p.markdown-alert-title").each do |node|
+        def scrub(node)
+          if node.name == 'p' && node['class'] == 'markdown-alert-title'
             parent_node = node.parent
             parent_class_attr = parent_node['class'] # e.g., "markdown-alert markdown-alert-note"
-            next unless parent_class_attr
+            return unless parent_class_attr
 
             # Extract the specific alert type (e.g., "note", "tip", "warning")
             # from the parent div's classes.
             match_data = parent_class_attr.match(/markdown-alert-(\w+)/)
-            next unless match_data && match_data[1] # Ensure a type is found
+            return unless match_data && match_data[1] # Ensure a type is found
 
             alert_type = match_data[1]
 
             # Get the corresponding icon name from our map.
             icon_name = ALERT_TYPE_TO_ICON_NAME[alert_type]
-            next unless icon_name # Skip if no specific icon is defined for this alert type
+            return unless icon_name # Skip if no specific icon is defined for this alert type
 
             # Translate the alert title only if it matches a known alert type
             # (i.e., the title has not been overridden)
@@ -61,7 +61,6 @@ module Redmine
               node.children.first.replace(icon_html)
             end
           end
-          doc
         end
       end
     end

lib/redmine/wiki_formatting/common_mark/syntax_highlight_scrubber.rb

@@ -24,26 +24,27 @@ module Redmine
       # blocks as generated by commonmarker
       class SyntaxHighlightScrubber < Loofah::Scrubber
         def scrub(node)
-          if node.matches?("pre > code")
-            return unless lang = node["class"].presence
-            return unless lang =~ /\Alanguage-(\S+)\z/
+          # Equivalent to the CSS selector "pre > code". Implemented for performance
+          return unless node.name == 'code' && node.parent.name == 'pre'
 
-            lang = $1
-            text = node.inner_text
+          return unless lang = node["class"].presence
+          return unless lang =~ /\Alanguage-(\S+)\z/
 
-            # original language for extension development
-            node["data-language"] = lang unless node["data-language"]
+          lang = $1
+          text = node.inner_text
 
-            if Redmine::SyntaxHighlighting.language_supported?(lang)
-              html = Redmine::SyntaxHighlighting.highlight_by_language(text, lang)
-              return if html.nil?
+          # original language for extension development
+          node["data-language"] = lang unless node["data-language"]
 
-              node.inner_html = html
-              node["class"] = "#{lang} syntaxhl"
-            else
-              # unsupported language, remove the class attribute
-              node.remove_attribute("class")
-            end
+          if Redmine::SyntaxHighlighting.language_supported?(lang)
+            html = Redmine::SyntaxHighlighting.highlight_by_language(text, lang)
+            return if html.nil?
+
+            node.inner_html = html
+            node["class"] = "#{lang} syntaxhl"
+          else
+            # unsupported language, remove the class attribute
+            node.remove_attribute("class")
           end
         end
       end