nl2br fix

twigjs · RobLoach · Dec 16, 2019 · Dec 11, 2019 · Dec 11, 2019 · Dec 11, 2019
commit 3a398c45bce20afa1ff4569dc227c8c9f76dacb5
diff --git a/src/twig.filters.js b/src/twig.filters.js
@@ -504,7 +504,7 @@ module.exports = function (Twig) {
         },
 
         nl2br(value) {
-            if (value === undefined || value === null) {
+            if (value === undefined || value === null || value === '') {
                 return;
             }
 

diff --git a/test/test.filters.js b/test/test.filters.js
@@ -495,6 +495,11 @@ describe('Twig.js Filters ->', function () {
             testTemplate.render().should.equal('');
         });
 
+        it('should not fail when passed str', function () {
+            const testTemplate = twig({data: '{{ myemptystr|nl2br }}'});
+            testTemplate.render({myemptystr: ''}).should.equal('');
+        });
+
         it('should not escape br tags if autoescape is on', function () {
             twig({
                 autoescape: true,