Merge pull request #46 from danspam/remove-bom

Option to remove BOM from start of file
esamattis · Oct 12, 2015 · 76bace1 · 76bace1
2 parents 94e765d + dba2122
commit 76bace1
Show file tree

Hide file tree

Showing 3 changed files with 33 additions and 0 deletions.
diff --git a/index.js b/index.js
@@ -101,6 +101,10 @@ function hbsfy(file, opts) {
     var compiled = "// hbsfy compiled Handlebars template\n";
     var parsed = null;
     var partials = null;
+
+    if (opts && (opts.b || opts.bom) && buffer.indexOf('') === 0) {
+      buffer = buffer.substring(1);
+    }
 
     try {
       if (traverse) {

diff --git a/test/bom.hbs b/test/bom.hbs
@@ -0,0 +1 @@
+Hello {{msg}}!
diff --git a/test/bom_test.js b/test/bom_test.js
@@ -0,0 +1,28 @@
+
+var assert = require("assert");
+var concat = require("concat-stream");
+var fs = require("fs");
+var hbsfyBom = require("hbsfy");
+var hbsfyNoBom = require("hbsfy").configure({
+  bom: true
+});
+
+var templatePath = __dirname + "/bom.hbs";
+
+fs.createReadStream(templatePath)
+.pipe(hbsfyBom(templatePath))
+.pipe(concat(function(data) {
+  assert(
+    //.test(data.toString()),
+    "The template should contain a bom"
+  );
+}));
+
+fs.createReadStream(templatePath)
+.pipe(hbsfyNoBom(templatePath))
+.pipe(concat(function(data) {
+  assert(
+    //.test(data.toString()) === false,
+    "The template should not contain a bom"
+  );
+}));