domino-ui icon indicating copy to clipboard operation
domino-ui copied to clipboard

LocalListScrollingDataStore does not invoke 'load()' at the end of its setData method.

Open NickGaens opened this issue 2 years ago • 0 comments

Describe the bug The method setData(List<T> data) of LocalListScrollingDataStore does not invoke load() at the end of its body.

Expected behavior The invocation of the load() method should be present.

Additional context This is the setData method of LocalListDataStore:

/**
   * sets new data list overriding the existing one, and clears all filters, then loads it in the
   * data table
   *
   * @param data the new {@link List} of records
   */
  public void setData(List<T> data) {
    this.original.clear();
    this.original.addAll(data);
    this.filtered.clear();
    this.filtered.addAll(original);
    load();
  }

and this is the one from LocalListScrollingDataStore:

/**
   * sets new data list overriding the existing one, and clears all filters, then loads it in the
   * data table
   *
   * @param data the new {@link List} of records
   */
  public void setData(List<T> data) {
    this.original.clear();
    this.original.addAll(data);
    this.filtered.clear();
    this.filtered.addAll(original);
  }

NickGaens avatar May 11 '22 12:05 NickGaens