Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

Uncaught TypeError: Cannot call method 'on' of undefined - Backbone.js

I'm having some error in my app and don't know why.

Uncaught TypeError: Cannot call method 'on' of undefined 

It happens on my collection view, follow the code:

App.WorkoutsView = Backbone.View.extend({
    initialize: function(){
        this.collection.on('add', this.addOne, this);
        this.collection.on('reset', this.addAll, this);
    },

    render: function() {
        this.addAll();
        return this;
    },

    addAll: function() {
        this.$el.empty();
        this.collection.forEach(this.addOne, this);
    },

    addOne: function(Workout) {
        var workoutView = new App.WorkoutView({model: App.Workout});
        this.$el.append(workoutView.render().el);
    }
});

the problem is on:

this.collection.on('add', this.addOne, this);

Anyone knows why?

** Edit: Collection code **

App.WorkoutItems = Backbone.Collection.extend({
    model: App.Workout,
    url: '/workouts',

    localStorage: function() {new Backbone.LocalStorage('Workout')},

    initialize: function() {
        this.on('remove', this.hideWorkout, this);
    },

    hideWorkout: function() {
        model.trigger('hide');
    },

    focusOnWorkoutItem: function() {
        var modelsToRemove = this.filter(function(workoutItem) {
            return workoutItem.id != id;
        });
    this.remove(modelsToRemove);
    }
});

Edit: my Router code where i'm instantiating WorkoutsView:

App.Router = Backbone.Router.extend({
    routes: {
        '':'index'
    },

    initialize: function() {
        this.workoutItems = new App.WorkoutItems();
        this.workoutsView = new App.WorkoutsView();
        this.workoutsView.render();
    },

    index: function() {
        $('#workouts').html(this.workoutsView.el);
        this.workoutsItem.fetch();
    }
});

new App.Router;
Backbone.history.start();
like image 461
Guilherme Carlos Avatar asked May 31 '13 15:05

Guilherme Carlos


1 Answers

 this.workoutsView = new App.WorkoutsView();

is supposed to pass in a collection

 this.workoutsView = new App.WorkoutsView({collection : this.workoutItems});

Because you view when being initialized is expecting to have a collection available to it as you are binding a event to it in the Initialize method of the view.

like image 66
Sushanth -- Avatar answered Oct 05 '22 17:10

Sushanth --