Remove the "retrieved items" stuff from UpdateItemStatesThread and VersionControlObserver.
Review Request #114992 - Created Jan. 12, 2014 and submitted
Remove the "retrieved items" stuff from UpdateItemStatesThread and VersionControlObserver. Showing an error message, makes no sense in this case, because the user can see it, when the update of version information failed. The plugins still have the ability to show error/warning messages on real errors. (where it makes sense ;)
Works. No random "Update of version infromation failed." errors in version controlled folders anymore. With enabled dolphin-dropbox-plugin: No "Update of version infromation failed." when dropbox is not running.
Thanks for that patch! I'm not extremely familiar with this code, and now I'm not really sure what the purpose of the member m_retrievedItems actually is. After going through the code, my understanding is that this member will always be set to true by the thread's run() method, unless m_itemStates is empty (because the body of the foreach loop will never be executed in this case). Therefore, it looks to me like the reason why this error message is shown by VersionControlObserver::slotThreadFinished() can only be that m_itemStates is empty (assuming that the thread's run() method is always run before the thread emits its finished() signal). Or am I overlooking something? I mean, I agree that it probably makes sense to remove this member and the error message. I'm just trying to figure out what the root cause of the random error messages that you mention might be. Just to make sure that there is no other problem that we haven't seen yet, and which might get hidden by this patch.