DocListLoadEvent event causes event loop

Transkribus / TranskribusSwtGui

Note: the repo has been moved to https://gitlab.com/readcoop/Transkribus/TranskribusSwtGui

GNU General Public License v3.0

18 stars 4 forks source link

DocListLoadEvent event causes event loop #278

Open kahlep opened 5 years ago

kahlep commented 5 years ago

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 commented 5 years ago

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.