tracking down the issues with how I'm appending table rows

abelmartin · May 3, 2012 · d09aaed · d09aaed
1 parent 58f4d5b
commit d09aaed
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 7 deletions.
diff --git a/.gitignore b/.gitignore
@@ -4,3 +4,4 @@
 *.DS_Store
 my_api_key.rb
 Session.vim
+.idea/
diff --git a/public/javascripts/app/collections/movies.js b/public/javascripts/app/collections/movies.js
@@ -9,8 +9,7 @@ var Movies = Backbone.Collection.extend({
     this.sort_man = new SortManager();
     this.comparator = function(film){
       if (film){
-        var tt = that.sort_man.comp_value(film);
-        return tt;
+        return that.sort_man.comp_value(film);
       }
     }
   },

diff --git a/public/javascripts/app/models/movie.js b/public/javascripts/app/models/movie.js
@@ -1,7 +1,8 @@
 var Movie = Backbone.Model.extend({
   defaults: {
     comment_count: 0,
-    show_details: false
+    show_details: false,
+    v_counter: 0
   },
 
   freshness_class: function(){

diff --git a/public/javascripts/app/views/movie_table_row.js b/public/javascripts/app/views/movie_table_row.js
@@ -6,8 +6,8 @@ var MovieTableRowView = Backbone.View.extend({
   template: $("#TMPMovieRows").html(),
 
   initialize: function(){
-    _.bindAll(this, 
-              'render', 
+    _.bindAll(this,
+              'render',
               'setMovieDetailState',
               'openMovieDetails');
     // this.model.bind('all', this.render);
@@ -16,12 +16,14 @@ var MovieTableRowView = Backbone.View.extend({
     return this;
   },
 
-  events:{ 
+  events:{
     "click a.display_comments" : "setMovieDetailState"
   },
 
   openMovieDetails: function(){
-    COTM.logEvent("Would have opened the detail view");
+    var vc = this.model.get("v_counter");
+    COTM.logEvent("Would have opened the detail view. VC:" + vc);
+    this.model.set({show_details:false, v_counter: vc + 1}, {silent:true});
   },
 
   setMovieDetailState: function(e){