this.model not a function

Question

EDITED I'm trying destroy or set or another function use but my error is

Uncaught TypeError: this.model.destroy is not a function

Looks like there are no errors on code

my code

define(['jquery',
    'underscore',
    'backbone',
    'handlebars',
    '/resources/app/models/TravelModel.js',
    ], function ($, _, Backbone, Handlebars, Travel) {

     var addAdminPanelView = Backbone.View.extend({

            el:$(".page"),
            model:Travel,
            events:{
                'click #editButton':'editEvent',
            },

            deleteEvent:function(){

                this.model.destroy(),
                });

            },

"Uncaught TypeError: Cannot read property 'destroy' of undefined" @AlivetoDie — curious_, Apr 24 '17 at 11:27
similar error :( why all example on internet working but not me @AlivetoDie — curious_, Apr 24 '17 at 11:34
The uppercase `Travel` would lead me to believe this is a model Class, not Instance. Where do you instantiate it? An *instantiated* model has the `destroy` method, but a class will not. — mikeapr4, Apr 24 '17 at 12:39
Also, you were correct with `this.model.destroy` assuming you assign a model instance to `this.model` — mikeapr4, Apr 24 '17 at 12:40
The uppercase "Travel" is correct using."Travel" is pointing correct path believe me.I added @mikeapr4 — curious_, Apr 24 '17 at 13:26

score 2 · Accepted Answer · edited May 23 '17 at 12:02

2

mikeapr4's assumption is correct.

Your Travel model is probably (and should be) something like this:

define(['Backbone'], function(Backbone) {
    return Backbone.Model({
        // ...
    });
});

So in your view, when your calling this.model.destroy(), it's equivalent to calling Travel.destroy() and it won't work since Travel is not an instance, but a constructor.

Backbone Collection do take a model constructor for its model property, but a Backbone View expects an instance.

You either need to initialize the view and create a Travel instance.

// Uppercase since it's a type, not an instance.
var AddAdminPanelView = Backbone.View.extend({
    el: $(".page"),
    events: {
        'click #editButton': 'editEvent',
    },

    // initialize the view with a Travel instance
    initialize: function() {
        this.model = new Travel();
    },

    deleteEvent: function() {
        this.model.destroy(),
    }
});

Or pass a Travel instance as an option on the view's construction.

var travelInstance = new Travel();

var view = new AddAdminPanelView({
    model: travelInstance
});

edited May 23 '17 at 12:02

Community

1
1

answered Apr 24 '17 at 13:47

Emile Bergeron

17,074
5
83
129

Thanks that's work.So can i ask you something about this question? – curious_ Apr 24 '17 at 15:16
How to i getting table data's id? i tried this.$el.find(".location").data("id") but console print "undefined" – curious_ Apr 24 '17 at 15:19
@marshall12 the syntax looks good, so it's probably that `.location` isn't present in the view, or that it doesn't have a `data-id` attribute on the element. – Emile Bergeron Apr 24 '17 at 15:22
{{location}} i added data-id attribute then this.$el.find("data-id").val() return undefined – curious_ Apr 24 '17 at 15:35
@marshall12 It looks suspiciously like [this other question](http://stackoverflow.com/q/43577375/1218980)... – Emile Bergeron Apr 24 '17 at 15:39
i looked this question.But always return 1 not my id – curious_ Apr 24 '17 at 16:25
@marshall12 if you can't find the answer you're looking for, ask a new question. – Emile Bergeron Apr 24 '17 at 16:27

this.model not a function

1 Answers1