Permalink
Browse files

Add comment regarding url() error behavior

  • Loading branch information...
1 parent b40b854 commit 4ea178a21684b21f90210d9cad43fef5556d3482 Ryan Eastridge committed Jan 3, 2013
Showing with 2 additions and 2 deletions.
  1. +2 −2 src/model.js
View
4 src/model.js
@@ -19,8 +19,8 @@ Thorax.Model = Backbone.Model.extend({
return keys.length > 1 || (keys.length === 1 && keys[0] !== this.idAttribute);
},
shouldFetch: function(options) {
- // TODO: if https://github.com/documentcloud/backbone/pull/2057 is merged
- // then add {silent: true} to url call
+ // url() will throw if model has no `urlRoot` and no `collection`
+ // or has `collection` and `collection` has no `url`
var url;
try {
url = this.url();

0 comments on commit 4ea178a

Please sign in to comment.