Merge pull request #77 from plicjo/ruby_3_4

Avoid string mutation for Ruby 3.4

Author: rmm5t

lib/strip_attributes.rb

@@ -63,21 +63,21 @@ def self.strip_string(value, options = {})
     replace_newlines = options[:replace_newlines]
     regex            = options[:regex]
 
-    value.gsub!(regex, "") if regex
+    value = value.gsub(regex, "") if regex
 
-    if MULTIBYTE_SUPPORTED && Encoding.compatible?(value, MULTIBYTE_SPACE)
-      value.gsub!(MULTIBYTE_SPACE_AT_ENDS, "")
+    value = if MULTIBYTE_SUPPORTED && Encoding.compatible?(value, MULTIBYTE_SPACE)
+      value.gsub(MULTIBYTE_SPACE_AT_ENDS, "")
     else
-      value.strip!
+      value.strip
     end
 
-    value.gsub!(NEWLINES, " ") if replace_newlines
+    value = value.gsub(NEWLINES, " ") if replace_newlines
 
     if collapse_spaces
-      if MULTIBYTE_SUPPORTED && Encoding.compatible?(value, MULTIBYTE_BLANK)
-        value.gsub!(MULTIBYTE_BLANK_REPEATED, " ")
+      value = if MULTIBYTE_SUPPORTED && Encoding.compatible?(value, MULTIBYTE_BLANK)
+        value.gsub(MULTIBYTE_BLANK_REPEATED, " ")
       else
-        value.squeeze!(" ")
+        value.squeeze(" ")
       end
     end
 

test/strip_attributes_test.rb

@@ -428,7 +428,7 @@ def test_should_strip_unicode
       skip "multi-byte characters not supported by this version of Ruby" unless StripAttributes::MULTIBYTE_SUPPORTED
 
       assert_equal "foo", StripAttributes.strip("\u200A\u200B foo\u200A\u200B ")
-      assert_equal "foo\u20AC".force_encoding("ASCII-8BIT"), StripAttributes.strip("foo\u20AC ".force_encoding("ASCII-8BIT"))
+      assert_equal "foo\u20AC".b, StripAttributes.strip("foo\u20AC ".b)
     end
   end
 end