fix some issues with make_dirty

on test:
 - make_dirty([[1,2,3]], dirty={(0,0):0, (0,1):0}) with cellzero=0
	the expected result is [[3]] (but was [[2]] because it starts
	removing 1 in [1,2,3], then 3 in [2,3], instead of removing
	1 and 2 in [1,2,3]
 - same test with non zero cell

basically, this doesn't assume the key traversal is ordered in dirty

additionaly, it use a default cellzero value (to `None`)

Author: nheir

sage_widget_adapters/generic_grid_view_adapter.py

@@ -207,20 +207,25 @@ def make_dirty(self, l, dirty={}):
             [[1, 2, 5, 6], [3, 7, 42], [4]]
             sage: ga.make_dirty(t.to_list(), {(2,0):0})
             [[1, 2, 5, 6], [3, 7]]
+            sage: ga.make_dirty(t.to_list(), {(0,2):0, (0,3):0})
+            [[1, 2], [3, 7], [4]]
         """
+        cellzero = None
+        if hasattr(self, 'cellzero'):
+            cellzero = self.cellzero
         for p in dirty:
             if p[0] < len(l):
                 if p[1] < len(l[p[0]]):
-                    if dirty[p] == self.cellzero:
-                        del l[p[0]][p[1]]
-                    else:
-                        l[p[0]][p[1]] = dirty[p]
-                elif len(l[p[0]]) == p[1] and dirty[p] != self.cellzero:
+                    l[p[0]][p[1]] = dirty[p]
+                elif len(l[p[0]]) <= p[1] and dirty[p] != cellzero:
+                    # padding with cellzero
+                    l[p[0]] += [cellzero] * (p[1] - len(l[p[0]]))
                     l[p[0]].append(dirty[p])
             else:
                 for i in range(p[0] - len(l)):
                     l.append([])
-                l.append([dirty[p]])
+                l.append([cellzero] * p[1] + [dirty[p]])
+        l = [[ val for val in row if val != cellzero ] for row in l]
         return [val for val in l if val]
 
     def set_cell(self, obj, pos, val, dirty={}, constructorname=''):