Groovy
  1. Groovy
  2. GROOVY-3248

String.replaceAll(String, Closure) should quote the replacement string

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.5.7, 1.6-rc-1
    • Fix Version/s: 1.6-rc-2, 1.5.8
    • Component/s: None
    • Labels:
      None

      Description

      String.replaceAll(String, Closure) is passing the stringified closure result directly to Matcher.appendReplacement.

      The consequence is that the resulting string is interpreted as a "replacement string" which treats '$' and '\' specially so that group replacements can be done. For my purposes that definitely undesireable.

      The whole reason I see for the closure API is so that those substitutions are done in the closure and that the closure expects it's result string to be substituted literally.

      An example of the problem is like this:

      groovy> println 'x123z45'.replaceAll(/([^z]*)(z)/, { all, m, d ->  m })
      x12345
      
      groovy> println '$123z45'.replaceAll(/([^z]*)(z)/, { all, m, d ->  m })
      
      $1232345
      

      The fix is that quoteReplacement needs to be called on the result before appendReplacement is called. This is similar to the problem we had with minus.

      http://jira.codehaus.org/browse/GROOVY-1637

      Hmmm, there they avoided the problem that quoteReplacement is JDK 1.5 by not using replaceAll. Looks like we need to implement RegexUtils.quoteReplacement after all.

        Activity

          People

          • Assignee:
            James P. White
            Reporter:
            James P. White
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development