13

I am building a simple todo app using reactFire, Firebase, and reactJS. The problem I am running into is when I try to bulk delete completed entries in the list.

componentWillMount: function() {
        this.ref = Firebase.database().ref("items/");
        this.bindAsArray(this.ref, "items");
        this.ref.on('value', this.handleDataLoaded);
}

for (var i in this.state.items) {
            var key = items[i]['.key'];
            if(items[i].done){
                this.ref.child(key).remove();
            }
}

The loop ends prematurely i.e. before deleting all the completed entries, because the render function is called.

Frank van Puffelen
  • 565,676
  • 79
  • 828
  • 807

1 Answers1

22

You can delete all completed items in one go by using a multi-location update:

var updates = {};
for (var i in this.state.items) {
    var key = items[i]['.key'];
    if(items[i].done){
        updates[key] = null; // setting value to null deletes the key
    }
}
this.ref.update(updates);
Frank van Puffelen
  • 565,676
  • 79
  • 828
  • 807