emoacht / SnowyImageCopy

A Windows desktop app to copy images from FlashAir by a wireless connection
https://www.microsoft.com/store/apps/9MTLPNGRW85L
MIT License
33 stars 14 forks source link

When multiple windows are open, an error of one window causes the close of other window #47

Closed dentphoto closed 4 years ago

dentphoto commented 4 years ago

I am very satisfied with the flashair multiple cards version. Pictures from each of the two cards are transferred well. But there seems to be a problem.

When I close the FLASHAIR window during operation, an error message appears. Subsequently, the FLASHAIR2 window is also closed. Conversely, when I close the FLASHAIR2 window, an error message appears as well. And the FLASHAIR window is closed in succession.

I think it is right that not only flashair window and flashair2 window simultaneously, but also flashair window alone or flashair2 window alone must work well.

https://youtu.be/pEjkw-ic5I8 Is it a bug?

emoacht commented 4 years ago

The situation you described is by itself normal. The multiple windows are shown by one instance of this app (only one instance can run at a time) and as a result, if an unmanageable error happens, the instance will be closed along with all windows. In general, I don't recommend to close this app when it is doing some operations. Stopping operations gracefully may take some time. It is especially true in the case where a network connection is involved. Having said that, I am interested in the error message this app left. In such case, a variety of messages may be thrown. If you still have it, could you post it?

dentphoto commented 4 years ago

Thank you for answer.

When two windows are open and each is in AUTO-CHECKING, Closing one window results in an error. And the other window is also closed. exception.log

My findings... When I first STOP auto-checking and then close the window, No error occurs and the remaining window does not close and operate normally. ==> https://www.youtube.com/watch?v=oGMJG8U3L9w

emoacht commented 4 years ago

@dentphoto Thank you for the log. I made some changes to make this app close a window as gracefully as possible. I fixed some other bugs including exceptions after timeout, unnecessary shortcut creation and activation after toast. The store version is now in the pipeline for release.