Labyricorn/liquidjs
1
0
Fork 0
mirror of https://github.com/harttle/liquidjs.git synced 2026-09-15 04:10:40 -07:00
Code Issues Packages Projects Releases Wiki Activity

fix: handle undefined replacement argument in replace filter (#864)

Browse Source
This commit is contained in:
Joe Cottam
2026-03-26 01:18:40 +08:00
committed by GitHub
parent 97d829116c
commit 0ad2b11ab1
2 changed files with 9 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
test/integration/filters/string.spec.ts
+8
View File
@@ -109,6 +109,14 @@ describe('filters/string', function () {
return test('{{ "Take my protein pills and put my helmet on" | replace: "my", "your" }}',
'Take your protein pills and put your helmet on')
})
it('should support replace with undefined replacement', function () {
return test('{{ "Take my protein pills and put my helmet on" | replace: "my" }}',
'Take protein pills and put helmet on')
})
it('should support replace with undefined variable as replacement', function () {
return test('{{ "Take my protein pills and put my helmet on" | replace: "my", missing_variable }}',
'Take protein pills and put helmet on')
})
it('should support replace_first', function () {
return test('{% assign my_string = "Take my protein pills and put my helmet on" %}\n' +
'{{ my_string | replace_first: "my", "your" }}',