this is being shadowed in _onSocketMessage in several branches.

[jeffreydwalter]

In the initial branch, _onSocketMessage refers to this._postsView.addPosts(), which works fine.
In later branches, this._postsView.addPosts() is wrapped in other promises and loses it's scope.

Here's one example in Lesson 4, step 7 (task-clean-db branch):

wittr/public/js/main/IndexController.js

Line 174 in 43e3098

this._postsView.addPosts(messages);

The fix is to assign this to var indexController = this; as is being done in the rest of the code.

[jeffreydwalter]

I saw this in at least one other branch earlier but fixed it without really thinking to report it as a bug.

For clarity and consistency sake, you guys should probably just make that assignment in _onSocketMessage() (all the functions in IndexController really) from the very first lesson, even though it's not strictly needed, since you do that in most of the other functions.

[jeffreydwalter]

I created a PR to address this issue in task-clean-db. #40

[jeffreydwalter]

The error message you see in the console is:

[jeffreydwalter]

I left a post in the Udacity forums about this issue as well.

[jeffreydwalter]

Just realized this issue was caused by me moving this._postsView.addPosts(messages); inside this._dbPromise.then(function(db), which is clearly wrong.

I'd still argue that making the var indexController = this; assignment would be good form.