Opened 5 years ago

Closed 4 years ago

#18207 closed enhancement (fixed)

[cla][patch] Allow escaping $ in string.substitute arguments

Reported by: jsonn Owned by: Adam Peller
Priority: low Milestone: 1.11
Component: String Version: 1.10.0
Keywords: Cc:
Blocked By: Blocking:

Description

dojo.string.substitute currently has no way to escape ${ sequences. This makes it harder i.e. to set queryExpr for FilteringSelect? inside a templated widget. The attached below recognizes ${} as escape for $.

Attachments (1)

substitute-escape.diff​ (576 bytes) - added by jsonn 5 years ago.
Possible fix

Download all attachments as: .zip

Change History (4)

Changed 5 years ago by jsonn

Attachment: substitute-escape.diff​ added

Possible fix

comment:1 Changed 5 years ago by dylan

Component: General → String
Milestone: tbd → 1.11
Owner: set to Adam Peller
Priority: undecided → low

comment:2 Changed 5 years ago by dylan

Summary: Allow escaping $ in string.substitute arguments → [cla][patch] Allow escaping $ in string.substitute arguments

comment:3 Changed 4 years ago by dylan

Resolution: → fixed
Status: new → closed

Fixed in ​https://github.com/dojo/dojo/commit/1209d791b7a351d83e6187dac373f671ef1ef080

Note: See TracTickets for help on using tickets.