Time check happening too early

[David Serrano]

Hi,

I've just found that, when I switched from PI/Windows to PI/Linux, I didn't uninstall the former. I've tried to run it now with the Ctrl key pressed, so I could clean the registry and then delete the folder. But PixInsight decided that I was going to actually use the program and went to check the date and time.

I suggest that the time check be moved after the is-Ctrl-pressed check, so users wanting to uninstall PixInsight (sigh) could clean their registry (or maybe don't do this change to the code, thus punishing those users that want to do bad things :twisted:). Otherwise, they would have to either end up with some useless entries in the registry (doh... they do anyway :^P) or download the current version just to be able to properly uninstall it.

Cheers,

[Juan Conejero]

or maybe don't do this change to the code, thus punishing those users that want to do bad things Twisted Evil

You got it!

Just joking :lol: Seriously, I'll have a look at this. Nevertheless, I'm considering adding a new main menu command to exit the core application erasing the whole settings/registry thing. This sounds cleaner than the Ctrl hack, doesn't it?

Cheers,

[David Serrano]

Sure. But...

Users that have the "Confirm program termination" checked, will get used to click "Yes" on exit, without thinking. If that new option is directly above of below the existent one, users may delete the registry without wanting so.

I'd allow deleting the registry only via the controle, for example "PixInsight --delete-registry".


[Edited: changed the meaningless "exit --delete-registry" to "PixInsight --delete-registry"]