fix feature loading / adding

skalb · Apr 23, 2012 · 48e93a2 · 48e93a2
1 parent 98bd9bd
commit 48e93a2
Show file tree

Hide file tree

Showing 4 changed files with 14 additions and 15 deletions.
diff --git a/app/assets/javascripts/backbone/views/features/index_view.js.coffee b/app/assets/javascripts/backbone/views/features/index_view.js.coffee
@@ -4,7 +4,7 @@ class Trackbone.Views.Features.IndexView extends Backbone.View
   template: JST["backbone/templates/features/index"]
 
   initialize: () ->
-    @options.features.bind('reset', @addAll)
+    @options.features.bind('sync', @render)
 
   addAll: () =>
     @options.features.each(@addOne)
@@ -15,5 +15,6 @@ class Trackbone.Views.Features.IndexView extends Backbone.View
 
   render: =>
     $(@el).html(@template(features: @options.features.toJSON() ))
+    @addAll()
 
     return this
diff --git a/app/assets/javascripts/backbone/views/features/new_view.js.coffee b/app/assets/javascripts/backbone/views/features/new_view.js.coffee
@@ -6,25 +6,16 @@ class Trackbone.Views.Features.NewView extends Backbone.View
   events:
     "submit #new-feature": "save"
 
-  constructor: (options) ->
-    super(options)
-    @model = new @collection.model()
-
-    @model.bind("change:errors", () =>
-      this.render()
-    )
-
   save: (e) ->
     e.preventDefault()
     e.stopPropagation()
 
-    @model.unset("errors")
-
-    @collection.create(@model.toJSON())
+    name = $("#new-feature #name").val()
+    if name
+      $("#new-feature #name").val('')
+      @collection.create(name: name)
 
   render: ->
-    $(@el).html(@template(@model.toJSON() ))
-
-    this.$("form").backboneLink(@model)
+    $(@el).html(@template())
 
     return this
diff --git a/app/assets/javascripts/backbone/views/projects/project_view.js.coffee b/app/assets/javascripts/backbone/views/projects/project_view.js.coffee
@@ -15,7 +15,13 @@ class Trackbone.Views.Projects.ProjectView extends Backbone.View
       @model.features.fetch success: ->
         featuresView = new Trackbone.Views.Features.IndexView(features: @model.features)
         $("#features").html(featuresView.render().el)
+
+        # We should ideally only render this one
+        newFeaturesView = new Trackbone.Views.Features.NewView(collection: @model.features)
+        $("#new-features").html(newFeaturesView.render().el)
+
         $("#bugs").html('')
+        $("#new-bugs").html('')
     @model.features.fetch()
 
   destroy: () ->

diff --git a/app/controllers/features_controller.rb b/app/controllers/features_controller.rb
@@ -17,6 +17,7 @@ def show
 
   def create
     @feature = Feature.new(params[:feature])
+    @feature.project_id = params[:project_id]
 
     respond_to do |format|
       if @feature.save