Merge pull request #92 from bookernath/replace-safestring

Allow replace helper to handle safestrings

Author: bookernath

helpers/replace.js

@@ -1,7 +1,10 @@
 'use strict';
+const common = require('./lib/common.js');
 
 const factory = () => {
     return function(needle, haystack) {
+        needle = common.unwrapIfSafeString(needle);
+        haystack = common.unwrapIfSafeString(haystack);
         const options = arguments[arguments.length - 1];
 
         if (typeof needle !== 'string') {

spec/helpers/replace.js

@@ -12,6 +12,7 @@ describe('replace helper', function() {
     const context = {
         content: "Either you run the %%var%% or the  %%var%% runs you",
         price: '$49.99',
+        facet: 'brand',
     };
 
     const runTestCases = testRunner({context, templates});
@@ -52,6 +53,15 @@ describe('replace helper', function() {
         ], done);
     });
 
+    it('should work nicely with other helpers that use safestring', function(done) {
+        runTestCases([
+            {
+                input: "Replace+Concat+Hyphenated: fifth-{{#replace '&' (concat '&' (hyphenate facet)) }}{{/replace}}",
+                output: "Replace+Concat+Hyphenated: fifth-brand",
+            },
+        ], done);
+    });
+
     it('should gracefully handle not strings', function(done) {
         runTestCases([
             {