Merge pull request #2300 from bdonald25/fixManualEntry

Fix "title already exists" bug
engaging-computing · Dec 4, 2015 · d563de0 · d563de0
2 parents a86910e + d040ee4
commit d563de0
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/app/assets/javascripts/data_sets/editTable.js.coffee b/app/assets/javascripts/data_sets/editTable.js.coffee
@@ -243,6 +243,7 @@ class Grid
         curr
       , []
       row.forEach (pair) ->
+        if pair[1] == null then pair[1] = ""
         unless buckets[pair[0]]? then buckets[pair[0]] = []
         buckets[pair[0]].push pair[1]
 

diff --git a/test/integration/slickgrid_test.rb b/test/integration/slickgrid_test.rb
@@ -127,6 +127,9 @@ def slickgrid_add_data(start, field_map)
     # add data to the rows
     slickgrid_add_data 0, field_map
 
+    # click restrictions but don't select one to see if it breaks things (bug #2298)
+    2.times { all('.slick-cell.l3.r3').last.click }
+
     # save the dataset
     find(:css, '#edit_table_save_1').click