Clean up method to close model: don't trigger events in the view

Author: chrisachard

web/app/models/item.coffee

@@ -3,5 +3,8 @@ module.exports = class Item extends Backbone.Model
 
   initialize: -> 
 
+  close: () =>
+    @trigger "closed", @
+
 
 

web/app/views/items/item.coffee

@@ -7,11 +7,10 @@ module.exports = class ItemView extends Backbone.View
     @template = require "templates/items/show"
 
   events: =>
-    "click .close": "closeEvent"
+    "click .close": "closeModel"
 
-  closeEvent: =>
-    # Remove from collection here
-    @model.trigger "closed", @model
+  closeModel: =>
+    @model.close()
 
   render: ->
     @$el.html @template

web/app/views/items/item_collection.coffee

@@ -22,16 +22,15 @@ module.exports = class ItemCollectionView extends Backbone.View
     if e.keyCode == 13 # On Enter
       @addNewItem()
 
-  modClosed: (mod) =>
-    @collection.remove mod
-
   render: =>
     @$el.html @template
 
     item_container = @$el.find(".items_list")
 
     for model in @collection.models
-      model.on "closed", @modClosed
+      model.on "closed", () =>
+        @collection.remove(model)
+
 
       item_view = new ItemView
         model: model