Skip to content

Commit

Permalink
ensure activeRowsPipeline is always initialized in row management pip…
Browse files Browse the repository at this point in the history
…eline
  • Loading branch information
olifolkerd committed Jan 21, 2024
1 parent 46d7cd9 commit 64c1374
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 5 deletions.
3 changes: 1 addition & 2 deletions dist/js/tabulator_esm.js
Original file line number Diff line number Diff line change
Expand Up @@ -23388,7 +23388,7 @@ class RowManager extends CoreFeature{
refreshPipelines(handler, stage, index, renderInPosition){
this.dispatch("data-refreshing");

if(!handler){
if(!handler || !this.activeRowsPipeline[0]){
this.activeRowsPipeline[0] = this.rows.slice(0);
}

Expand All @@ -23398,7 +23398,6 @@ class RowManager extends CoreFeature{
//handle case where all data needs refreshing

case "dataPipeline":

for(let i = index; i < this.dataPipeline.length; i++){
let result = this.dataPipeline[i].handler(this.activeRowsPipeline[i].slice(0));

Expand Down
2 changes: 1 addition & 1 deletion dist/js/tabulator_esm.js.map

Large diffs are not rendered by default.

3 changes: 1 addition & 2 deletions src/js/core/RowManager.js
Original file line number Diff line number Diff line change
Expand Up @@ -714,7 +714,7 @@ export default class RowManager extends CoreFeature{
refreshPipelines(handler, stage, index, renderInPosition){
this.dispatch("data-refreshing");

if(!handler){
if(!handler || !this.activeRowsPipeline[0]){
this.activeRowsPipeline[0] = this.rows.slice(0);
}

Expand All @@ -724,7 +724,6 @@ export default class RowManager extends CoreFeature{
//handle case where all data needs refreshing

case "dataPipeline":

for(let i = index; i < this.dataPipeline.length; i++){
let result = this.dataPipeline[i].handler(this.activeRowsPipeline[i].slice(0));

Expand Down

0 comments on commit 64c1374

Please sign in to comment.