Errata

The final line of example code on the page says:

console.log(count); // 15

It should say:

console.log(number); // 15

Inside `function addItem(url, text)`, I believe
`result = sprintf(template, url, text);` was intended to be
`result = sprintf(templateText, url, text);`.

...or line 3: `templateText = mylist.firstChild.nodeValue,` should have been `template = mylist.firstChild.nodeValue,`

Also it seems Line 6: `div.innerHTML = result;` should be omitted. div is undefined, and the result is being inserted properly in line 7: `mylist.insertAdjacentHTML("beforeend", result);`.

Note from the Author or Editor:
Now on page 62. Please remove this line:

div.innerHTML = result;

Inside `function addItem(url, text)`, I believe
`result = sprintf(template, url, text);` was intended to be
`result = sprintf(templateText, url, text);`.

Also:
I believe `list.appendChild(div.firstChild);` was intended to be `mylist.appendChild(div.firstChild);`.

It's also unclear as to why this snippet is using appendChild() whereas the first snippet used insertAdjacentHTML()... I believe they have the same effect, and I would probably have used appendChild() myself. But I am uncertain if there is a difference that I don't recognize. Since there is no discussion of this in the text, I thought I'd point it out and ask.

Note from the Author or Editor:
Now on page 62 -

Please change:

result = sprintf(template, url, text);

To:

result = sprintf(templateText, url, text);

at the very bottom is the same type:

"templateText" vs "template"

Note from the Author or Editor:
In last code example, change:

result = sprintf(template, url, text);

to

result = sprintf(templateText, url, text);

I believe `list.appendChild(div.firstChild);` was intended to be `mylist.appendChild(div.firstChild);`

Note from the Author or Editor:
In the code, please change:

list.appendChild(div.firstChild);

To:

mylist.appendChild(div.firstChild);

In the book it is written: "All you need to do is pass in an object containing the properties name and url"

it should be "... containing the properties text and url"

Note from the Author or Editor:
Please make the changes specified by the reader.

the namespace methods is not "nondestructively", it will overwritten some property;

code:

if(!object[parts[i]])

when object[parts[i]] is false or 0 it will be overwritten

maybe this will be better:

if(object[parts[i]] === undefined)

Note from the Author or Editor:
Replace the line:

if (!object[parts[i]]) {

With

if (typeof object[parts[i]] === "undefined") {

Original text:

� For strings, typeof returns �string.�
� For numbers, typeof returns �number.�
� For booleans, typeof returns �boolean.�
� For undefined, typeof returns �undefined.�

Actually the "typeof" operator will return type name without a point, so that it will be "string", "number", "boolean" and "undefined" respectively.

Note from the Author or Editor:
In each of the bullets mentioned by the reader, please move the period after the closing parentheses. For example, "string." should be "string".

I think this line is missing "this.".
element.className += " " + className;
should be
this.element.className += " " + className;

Note from the Author or Editor:
Please change:

element.className += " " + className;

To:

this.element.className += " " + className;

The //target[@name=version] is implied, but not defined in the first snippet. It's implied when showing the example output after the sentence: ""When this Ant script is executed, the output is:"

The first snippet should probably be something like this:

<?xml version="1.0" encoding="UTF-8"?>
<project name="maintainablejs" default="version">
<property name="version" value="0.1.0" />
<target name="version">
<echo>Version is ${version}</echo>
</target>
</project>

Note from the Author or Editor:
(Now on page 136)

Please replace first code snippet with (indents should be four spaces each):

<?xml version="1.0" encoding="UTF-8"?>

<project name="maintainablejs" default="version">

<property name="version" value="0.1.0" />

<target name="version">

<echo>Version is ${version}</echo>

</target>

</project>

The first code snippet, the directory for tests is incorrect. This line:

yuitestselenium.tests = ${lib.dir}/tests.xml

Should be changed to:

yuitestselenium.tests = ${tests.dir}/tests.xml

"There should be one space between the closing parentheses and the right brace. The right brace should be on the same line as the function keyword."

In each sentence I believe you meant 'left' brace.

Note from the Author or Editor:
Please change:

"There should be one space between the closing parentheses and the right brace. The right brace should be on the same line as the function keyword."

To:

"There should be one space between the closing parentheses and the left brace. The left brace should be on the same line as the function keyword."

In the code example at the top of page 203, there is an unmatched double quote. This is the code block for the switch statement in Appendix A. The default case in this code block reads:

throw new Error("This shouldn't happen.);

There needs to be a closing double quote after the period:

throw new Error("This shouldn't happen.");

Note from the Author or Editor:
Please replace:

throw new Error("This shouldn't happen.);

With:

throw new Error("This shouldn't happen.");