By Developers, For Developers
| PDF Pg | Paper Pg | Type | Description | Fixed on | Comments |
|---|---|---|---|---|---|
| 33 | 17 | ERROR | In the code snippets variables ‘queryStr’ and ‘selectUser’ should be the same | ||
| 51 | 36 | ERROR | var showPromise = $flash.show(); I think they should be: var showPromise = $flash.show().promise(); | ||
| 70 | 55 | SUGGEST | I think it’s better to mention that “to invoke the callback function in the worker is a must”, or the queue cannot work properly. | ||
| 127 | TYPO | (ePub) Last paragraph, first sentence: “Node’s error conventions may not ideal for Async.js’s collection methods.” Should be “Node’s error conventions may not be ideal for Async.js’s collection methods.” | |||
| 23 | TYPO | This is from the LeanPub edition, so the error possibly doesn’t exist (presuming it’s an error). You list “…under jQuery 1.5+…” I believe that the last .fail should be .always. If so, you might have caught this before its current printing. | |||
| 17 | TYPO | In the checkPassword function, the query variable is defined as queryStr but used as selectUser. | |||
| 51 | TYPO | In Asyncjs/forEachSeries.js code sample, “functio isFilename” has a typo. “function” is missing last ‘n’ (functio). | |||
| 34 | ERROR | The jsFiddle referenced is missing the alway() callback used in the text, so it does not log “A choice was made” as the text claims it will. | |||
| 38 | ERROR | The book states “So, to recap, a Promise takes three kinds of callbacks: done, fail, and progress.” I believe you are overlooking the fourth callback type, always. | |||
| 27 | TYPO | $container.one(‘newTooltip’, function() { | |||
| 4 | TYPO | A loop like this will run forever: | |||
| 14 | TYPO | whomever should be whoever around the end of the page | |||
| 43 | 27 | TYPO | At the bottom of the page: “$container.one(‘newToolTip’, …” should be “$container.on(’newToolTip, …)”. |