Skip to content
This repository has been archived by the owner on Nov 16, 2020. It is now read-only.

DocListLoadEvent event causes event loop #278

Open
kahlep opened this issue Apr 15, 2019 · 1 comment
Open

DocListLoadEvent event causes event loop #278

kahlep opened this issue Apr 15, 2019 · 1 comment

Comments

@kahlep
Copy link
Contributor

kahlep commented Apr 15, 2019

The document data is sometimes unnecessarily refreshed several times from the server which causes overhead on the server and slows down GUI actions.

  • TrpMainWidgetStorageListener#handleDocListLoadEvent in the end causes a reload in DocTableWidgetPagination#reloadDocs triggering another DocListLoadEvent.
  • Also, this event type is used when reloading orphaned documents which might also trigger event handlers unnecessarily.
@kahlep
Copy link
Contributor Author

kahlep commented Apr 15, 2019

DocTableWidgetPagination forced an additional document list reload when the state of its internal collection ID field changed, although Storage had the respective collection and document list already loaded.
97bf7f2 removes the respective check.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant