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

Leo Marco Corpuz
Leo Marco Corpuz
18,975 Points

Quiz after the binding values tutorial

Is anything missing in my code? I'm still not clear about how to use the % placeholders in my code.

index.js
var utilities = require("./utilities");

var mailValues = {};

mailValues.first_name = "Janet";

var emailTemplate = "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(var key in values){
    content.replace(key,values[key])

    };
  return content;
}


module.exports.merge = merge;

1 Answer

Steven Parker
Steven Parker
218,691 Points

You're pretty close. But remember that you don't just want to replace the key, but the key and the percent signs surrounding it.

Also, the "replace" method doesn't change the string you apply it to, it returns a new string with the changes. So you'll want to save that somewhere.

Leo Marco Corpuz
Leo Marco Corpuz
18,975 Points

I think I've fixed it. function merge(content, values) { for(var key in values){ var newContent= content.replace("%" + key + "%",values[key]) }; return newContent; } module.exports.merge = merge;

Steven Parker
Steven Parker
218,691 Points

Apparently the challenge only tests using an object with a single property, and this does pass. :+1:

But to be able to handle objects with more properties, you'd probably want to re-use the "content" variable instead of creating a new one.