Reset the notes collection html when rendering

crowdint · Oct 27, 2011 · 2a02af1 · 2a02af1
1 parent 66e6b6f
commit 2a02af1
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/public/javascripts/views/story_view.js b/public/javascripts/views/story_view.js
@@ -23,7 +23,7 @@ var StoryView = FormView.extend({
     this.model.bind("change:estimate", this.setClassName);
     this.model.bind("change:state", this.setClassName);
 
-    this.model.bind("change:notes", this.renderNotes);
+    this.model.bind("change:notes", this.renderNotesCollection);
 
     this.model.bind("render", this.hoverBox());
     // Supply the model with a reference to it's own view object, so it can
@@ -375,6 +375,7 @@ var StoryView = FormView.extend({
 
   renderNotesCollection: function() {
     var notelist = this.$('div.notelist');
+    notelist.html('');
     this.model.notes.each(function(note) {
       var view;
       if (note.isNew()) {

diff --git a/spec/javascripts/views/story_view.spec.js b/spec/javascripts/views/story_view.spec.js
@@ -344,10 +344,10 @@ describe('StoryView', function() {
 
   describe("notes", function() {
 
-    it("binds change:notes to renderNotes()", function() {
+    it("binds change:notes to renderNotesCollection()", function() {
       var spy = sinon.spy(this.story, 'bind');
       var view = new StoryView({model: this.story});
-      expect(spy).toHaveBeenCalledWith('change:notes', view.renderNotes);
+      expect(spy).toHaveBeenCalledWith('change:notes', view.renderNotesCollection);
     });
 
   });