fixed off-by-one error

flying-sheep · flying-sheep · commit 82b5ae04cd8a · 2017-09-08T12:33:26.000+02:00
diff --git a/dataTables.rowsGroup.js b/dataTables.rowsGroup.js
@@ -208,9 +208,9 @@ class RowsGroup {
             }
         }
         $(columnNodes[newSequenceRow]).show();
-        $(columnNodes[newSequenceRow]).attr('rowspan', (iFinishRow - 1) - newSequenceRow + 1);
+        $(columnNodes[newSequenceRow]).attr('rowspan', iFinishRow - newSequenceRow + 1);
         if (columnsIndexesCopy.length > 0) {
-            this._mergeColumn(newSequenceRow, (iFinishRow - 1), columnsIndexesCopy);
+            this._mergeColumn(newSequenceRow, iFinishRow, columnsIndexesCopy);
         }
     }
     

Original file line number	Diff line number	Diff line change
`@@ -208,9 +208,9 @@ class RowsGroup {`
`208`	`208`	`}`
`209`	`209`	`}`
`210`	`210`	`$(columnNodes[newSequenceRow]).show();`
`211`		`- $(columnNodes[newSequenceRow]).attr('rowspan', (iFinishRow - 1) - newSequenceRow + 1);`
	`211`	`+ $(columnNodes[newSequenceRow]).attr('rowspan', iFinishRow - newSequenceRow + 1);`
`212`	`212`	`if (columnsIndexesCopy.length > 0) {`
`213`		`- this._mergeColumn(newSequenceRow, (iFinishRow - 1), columnsIndexesCopy);`
	`213`	`+ this._mergeColumn(newSequenceRow, iFinishRow, columnsIndexesCopy);`
`214`	`214`	`}`
`215`	`215`	`}`
`216`	`216`