fix(Table): Make sure correct cells are editable by students (#730)

- Set correct row indexes of editable cells in Tabulator
- Update correct row on edit of custom sorted table
- Set correct cells as editable with custom sorting

#727

src/app/services/tabulatorDataService.spec.ts

@@ -32,7 +32,7 @@ function convertTableDataToTabulator() {
       expect(tabulatorData.data.length).toBe(3);
       expect(tabulatorData.data[1]['0']).toBe('10');
       expect(tabulatorData.options.maxHeight).toBe('500px');
-      expect(tabulatorData.editableCells[0]).toEqual(['1','2']);
+      expect(tabulatorData.editableCells[1]).toEqual(['1', '2']);
     });
   });
 }

src/assets/wise5/components/table/table-student/table-student.component.ts

@@ -1169,7 +1169,7 @@ export class TableStudent extends ComponentStudent {
 
   tabulatorCellChanged(cell: Tabulator.CellComponent): void {
     const columnIndex = parseInt(cell.getColumn().getField());
-    const rowIndex = cell.getRow().getPosition() + 1;
+    const rowIndex = cell.getRow().getIndex() + 1;
     this.tableData[rowIndex][columnIndex].text = cell.getValue();
     this.studentDataChanged();
   }

src/assets/wise5/components/table/tabulator-table/tabulator-table.component.ts

@@ -96,7 +96,7 @@ export class TabulatorTableComponent implements OnChanges, AfterViewInit {
   }
 
   private isCellEditable(cell: Tabulator.CellComponent): boolean {
-    const rowIndex = cell.getRow().getPosition();
+    const rowIndex = cell.getRow().getIndex() + 1;
     const field = cell.getColumn().getField();
     const row = this.editableCells[rowIndex];
     return row && row.includes(field);

src/assets/wise5/components/table/tabulatorDataService.ts

@@ -16,7 +16,7 @@ export class TabulatorDataService {
         const rowData = this.getTabulatorRowDataFromTableRow(row, content.columns);
         rowData.values.id = rowIndex;
         content.data.push(rowData.values);
-        content.editableCells[rowIndex] = rowData.editableCells;
+        content.editableCells[index] = rowData.editableCells;
       }
     }
     return content;