Agree about clunky but I want it to always return Boolean and not a string.

On Wednesday, January 2, 2013, Kevin Decker wrote:

> In src/collection.js:
>
> > @@ -19,6 +19,9 @@ Thorax.Collection = Backbone.Collection.extend({
> >    isPopulated: function() {
> >      return this._fetched || this.length > 0 || (!this.length && !_.result(this, 'url'));
> >    },
> > +  shouldFetch: function(options) {
> > +    return options.fetch && !!_.result(this, 'url') && !this.isPopulated();
>
> nit: !! isn't necessary and !!_. feels clunky.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/walmartlabs/thorax/pull/90/files#r2532063>.
>
>