Errata for Maintainable JavaScript

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);

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);

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.");