"q" flag for fn:replace() is ignored
Bug #1026379 reported by
Matthias Brantner
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Zorba |
Fix Released
|
Medium
|
Paul J. Lucas | ||
Bug Description
In the following query, the "q" flag should cause the "$" in the replacement string to become legal.
replace("foo", "f", "$", "q")
5.6.3 fn:replace in F&O
Unless the q flag is used, a literal $ character within the replacement string must be written as \$, and a literal \ character must be written as \\.
Related branches
lp:~paul-lucas/zorba/bug-1026379
- Matthias Brantner: Approve
- Paul J. Lucas: Approve
-
Diff: 107 lines (+19/-12)4 files modifiedChangeLog (+1/-0)
src/runtime/strings/strings_impl.cpp (+16/-12)
test/rbkt/ExpQueryResults/zorba/string/ReplaceFunc/replace20.xml.res (+1/-0)
test/rbkt/Queries/zorba/string/ReplaceFunc/replace20.xq (+1/-0)
| affects: | sausalito → zorba |
| Changed in zorba: | |
| milestone: | none → 2.7 |
| visibility: | private → public |
| Changed in zorba: | |
| status: | New → In Progress |
| summary: |
- "q" flag for replace is ignored + "q" flag for fn:replace() is ignored |
Revision history for this message
| Paul J. Lucas (paul-lucas) wrote | #1 |
| Changed in zorba: | |
| status: | In Progress → Fix Committed |
| Changed in zorba: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
Well, it's not ignored entirely. It just doesn't take it into consideration for the replacement string.