5. Using Firebase for persistency

[tripy37]

In the codelabs instructions there is a typo in the function app.onFirebaseLogin the property accessor on updateItems needs to be changed from this to app.

From

app.onFirebaseLogin = function(event) {
  this.ref = new Firebase(this.firebaseURL + '/user/' + 
                                                  event.detail.user.uid);
  this.ref.on('value', function(snapshot) {
    this.updateItems(snapshot);
  });
};

To

app.onFirebaseLogin = function(event) {
  this.ref = new Firebase(this.firebaseURL + '/user/' + 
                                                  event.detail.user.uid);
  this.ref.on('value', function(snapshot) {
    app.updateItems(snapshot);
  });
};

[rachelslurs]

👍

Was just about to file this!

[rachelslurs]

I notice the source on Github is correct actually, but the code lab itself is not.

https://github.com/googlecodelabs/polymer-firebase/blob/master/step4/main.js#L43

[oddg]

To be consistent with the style of app.updateItems, it can also be fixed with Function.prototype.bind() as follows

this.ref.on('value', function(snapshot) {
  app.updateItems(snapshot);
}.bind(this) );