initialize model returns error

[rendrjs]

initialize model returns error

[spikebrehm]

Hi, thanks for the issue report! Probably the issue is that you need to call the model super's initialize method, because Rendr utilizes initialize to add some important functionality to BaseModel: https://github.com/airbnb/rendr/blob/master/shared/base/model.js#L11

In your model, you could do something like this:

var BaseModel = require('rendr/shared/base/model');
module.exports = BaseModel.extend({
  initialize: function() {
    BaseModel.prototype.initialize.apply(this, arguments);

    // your code here
  }
});
module.exports.id = 'MyModelName';

Give that a try. But this brings up a good point that Rendr should endeavor to not hijack the model's initialize method; instead we should just hijack the constructor. Will leave this open as a reminder to address that.

[grudelsud]

it works!

thank you very much man, really helpful. just so you know, I forked the rendr-app-template about a month ago and without any "super" initialization it worked just fine, the problem started after I synced with the latest updates yesterday, and now fixed calling the initialize function of the base model.

thank you again, closing this issue!

[EvanCarroll]

Where is this idiom of calling the BaseModel.prototype.initialize documented? I've seen lots of examples where this is never done...

• Here is one example where that never happens in the docs, source

# app/models/user

BaseModel = require('rendr/base/model')

module.exports = class User extends BaseModel
  url: '/users/:id'

• other examples