template cache

1.js

TemplateCache = {
  get: function(selector){
    if (!this.templates){ this.templates = {}; }

    var template = this.templates[selector];
    if (!template){
      var template = $(selector).html();
      this.templates[selector] = template;
    }

    return template;
  }
}

2.js

TemplateCache = {
  get: function(selector){
    if (!this.templates){ this.templates = {}; }

    var template = this.templates[selector];
    if (!template){
      template = $(selector).html();

      // precompile the template, for underscore.js templates
      template = _.template(template);

      this.templates[selector] = template;
    }

    return template;
  }
}

is 'template' re-declared inside the if for a reason?

the "var" shouldn't be there on line 7. my intention was to re-use the same template variable (which this does... its just bad form)

fixed