The removeSelectedRows method seems to be broken

[guest]

This method of _grid.js seems to be broken and do not do what is expected. The last item in store is removed from grid insted of selection.

	removeSelectedRows: function(){
			// summary:
			//		Remove the selected rows from the grid.
			this.updateRowCount(Math.max(0, this.rowCount - this.selection.getSelected().length));
			this.selection.clear();
		}

Compare insted the code in Nightly Build 28/05/2008

	removeSelectedRows: function(){
		this.edit.apply();
		var s = this.selection.getSelected();
		if(s.length){
			this.model.remove(s);
			this.selection.clear();
		}
	},

The items are not removed from store

thanks

Eduardo

[guest]

See replication instructions, please

[guest]

Replication instructions.

[1] Add four, five nodes. [2] Select any item except last. [2] Choose remove selected button

Result: Last will be deleted insted item you chose.

Note: In my real application I saw sometimes, a slight different behavior if several alternatively nodes are chosen (example item1,item3, item5 tigether in one selection) . I think that probably there is an additional bug there, but I cannot be sure of that.

thanks !!

Eduardo

[guest]

The current removeSelectedRows function do:

[1] Update the number of rows

[2] Clear selection

No removing of rows (if I do not missed something).

Ps: the selected list may be not a continuous list, ex: rows 1,4,7 and 10

thanks !

Eduardo

[Bryan Forbes]

(In [13997]) fixes #6853 !strict

• Overrode removeSelectedRows in DataGrid? to correctly remove the rows if it's an editable grid.
• Added custom avatar.

[guest]

Hi Bryan

I reopen this PR, because something is not working. May be the problem is not in the dataGrid. My program falls inside the removeSelected when you call the dojo.forEach(items, this.store.deleteItem) line.

I have two grids: gridAssigned and gridUsers when and I transfers selected rows from gridAssigned to gridUsers. In gridUsers I recreate a equivalent line and in gridAssigned I remove each line.

See please the code above , the first work, the second falls, I used a code very similar to your removeSelected.

[1]

   var selRows =  new Array();
    selRows=gridAssigned.selection.getSelected();
    noassigned=selRows.length;

    if(noassigned < 1){
      return;
    }
    for(i=0;i<noassigned;i++){
        name = selRows[i].unam;
        grid = selRows[i].grid;
        id   ="\'"+selRows[i].id+"\'";
        modelUsers.newItem({id:id,unam:name,grid:grid}); 
    }

    /*gridAssigned.removeSelectedRows();**

    gridAssigned.edit.apply();
    for (i=0;i<noassigned;i++){
        gridAssigned.store.deleteItem(selRows[i]);
    }

    
    noassigned=0;
    gridAssigned.selection.clear();
    gridAssigned.render();
    gridUsers.setSortIndex(0,true);

[2]

   var selRows =  new Array();
    selRows=gridAssigned.selection.getSelected();
    noassigned=selRows.length;

    if(noassigned < 1){
      return;
    }
    for(i=0;i<noassigned;i++){
        name = selRows[i].unam;
        grid = selRows[i].grid;
        id   ="\'"+selRows[i].id+"\'";
        modelUsers.newItem({id:id,unam:name,grid:grid}); 
    }

    gridAssigned.edit.apply();
    gridAssigned.removeSelectedRows();   <---- Falls here
    
    noassigned=0;
    gridAssigned.selection.clear();
    gridAssigned.render();
    gridUsers.setSortIndex(0,true);

The error message I received in my FF2 error console is:

this._assert is not a function Line 219 in dojo.js

[Bryan Forbes]

(In [14087]) fixes #6853 !strict

• Changed internal structures of DataGrid? to make indexing more accurate and fix removeSelectedRows.

[guest]

Thanks Bryan !!

Eduardo

[bill]

marking tickets closed in the last three months w/blank milestone to milestone 1.2.