The find/replace does escape the user string when generating the regexp. This means that if $, (, ) or any other character it will generate a bad regexp

CommentFileSizeAuthor
0001-Fix-issue-with-find-replace-not-escaping-the-string-.patch996 bytesgordon
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Status: Needs review » Needs work

The last submitted patch, 0001-Fix-issue-with-find-replace-not-escaping-the-string-.patch, failed testing.

twistor’s picture

twistor CreditAttribution: twistor commented
Assigned: gordon » Unassigned
Status: Needs work » Closed (fixed)

This was fixed a long time ago.