Welcome to the Treehouse Community

The Treehouse Community is a meeting place for developers, designers, and programmers of all backgrounds and skill levels to get support. Collaborate here on code errors or bugs that you need feedback on, or asking for an extra set of eyes on your latest project. Join thousands of Treehouse students and alumni in the community today. (Note: Only Treehouse students can comment or ask questions, but non-students are welcome to browse our conversations.)

Looking to learn something new?

Treehouse offers a seven day free trial for new students. Get access to thousands of hours of content and a supportive community. Start your free trial today.

JavaScript Build a Simple Dynamic Site with Node.js Creating a Basic Template Engine in Node.js A Simple Merge Utility

Tried using the replace method but totally failed to work

its as if the replace object its not recognised
index.js
var utilities = require("./utilities");
var fs = require("fs");

var mailValues = {};

mailValues.first_name = "Janet";

var emailTemplate = fs.readFile("Hi %first_name%! Thanks for completing this code challenge :)");

var mergedContent = utilities.merge(emailTemplate, mailValues);

//mergedContent === "Hi Janet! Thanks for completing this code challenge :)";
utilities.js
function merge(content, values) {
  for(let key in values){
    content =  content.replace("{{"+ key +"}}",values[key]);
  }
  return content
}


module.exports.merge = merge;

2 Answers

Steven Parker
Steven Parker
218,596 Points

Your content argument may be undefined or null.

I notice you're passing the return value from fs.readFile to your merge method, but I don't think it actually returns anything. I would also expect it to need different arguments.

Please take a look at the documentation page covering fs.readFile for proper usage information. I might also be that you intended to do something else there entirely.

Mark Warren
Mark Warren
19,250 Points

It's pretty similar to the function in renderer.js, except in this case the value you're replacing is wrapped with %..% instead of {{...}}

function merge(content, values) {
  for (let key in values) {
    content = content.replace("%" + key + "%", values[key]);
  }
  return content;
}

module.exports.merge = merge;