Duplicate row indices handled poorly by spreadsheet delete_rows functions

[cci-moc]

Duplicate row indices handled poorly by spreadsheet delete_rows functions

[Lematsu]

When is the indices [1, 1, 1]?
I am not sure how to form this test example.

[kamfonik]

So, this is a bug in the spreadsheet interface, not the addusers scripts. When you invoke the delete command, you pass it a list of rows - which could theoretically include the same row more than once, depending on how you constructed it. Maybe a row meets more than 1 of the criteria for deletion, or something.

This showed up in the addusers script at one point a while back when the script was accidentally sending a duplicate row. It's the sort of behavior that shouldn't just happen unannounced in a tool like this - ideally it should remove duplicates from the list.

This bug is not super high priority.

[Lematsu]

I see so we want to announce that there's a duplicate and then remove the duplicates in the spreadsheet interface.

[kamfonik]

Yes, that's right.