Dev warning when can.Model is extended without static properties.

[justinbmeyer]

When can.Model is extended without static properties like:

Todo = can.Model.extend({
  findAll: "GET /todos"
});

This is almost always a mistake. We should warn users with something like:

can/model/model.js: can.Model extended without static properties.

VOTE FOR THIS FEATURE HERE: http://bithub.com/event/25835

[daffl]

Closed via #1016

[daliborfilus]

This helped me discover something unusual. If you do this in canjs 2.2.5:

steal('can/model', 'can/map/attributes', function() {
can.Model.extend('HiThere',
// @Static
{
  attributes: {},
  findAll: function() {
    // doesn't matter
  }
},
// @Prototype
{}
)
})

It will throw this error and the model is empty, without definitions. Incomplete. Dead.
But I'm giving it the name, static props and prototype props.

The cause of this is the stealing of can/map/attributes, which breaks it completely and causes it to break. It is deprecated, but it's removed from docs but not from canjs package, so I didn't even knew it was deprecated :-DThis helped me discover something unusual. If you do this in canjs 2.2.5:

steal('can/model', 'can/map/attributes', function() {
can.Model.extend('HiThere',
// @Static
{
  attributes: {},
  findAll: function() {
    // doesn't matter
  }
},
// @Prototype
{}
)
})

It will throw this error and the model is empty, without definitions. Incomplete. Dead.
But I'm giving it the name, static props and prototype props.

The cause of this is the stealing of can/map/attributes, which breaks it completely and causes it to break. It is deprecated, but it's removed from docs but not from canjs package, so I didn't even knew it was deprecated :-D