Update plugins doesn't remember skipping

[Daeron]

I update my josm and plugins manually from svn. I tried to select the remember checkbox, and then skip update, but the dialog still appear every time i have updated my josm.

[stoecker]

HelpAwareOptionPane does not care for the dontShowAgain message at all, so it is no wonder it does not work.

@Gubaer:
You need to copy the dont show functionality from ExtendedDialog to HelpAwareOptionPane instead of having a special UpdatePluginPanel which anway does not work, as it does not affect behaviour of the dialog.

BTW: using pluginmanager.version instead of message.pluginmanger.version also kills the version check.

[Gubaer]

If you already know *how* to fix it, then please fix it instead of providing a lenghty analysis. You have commit rights. Don't explain how to fix things, if nobody asks for it, just fix them.

[stoecker]

Between finding the reason and the actual fix usually is a longer time. I investigated the reason, found it to be a non-trivial cause and assigned it to the one who produced the bug.

[Gubaer]

(In [2866]) fixed #4376: Update plugins doesn't remember skipping

[stoecker]

This fix removes the previously existing possibility to select what you really want: update or cancel. Adding new functionality is nice, but removimng stuff which has been requested in many Trac tickets is not really the way to go.

[stoecker]

I still do not understand why ExtendedDialog needs to be replaced by HelpAwareOptionPane at all.

[Gubaer]

I still do not understand why ExtendedDialog needs to be replaced by HelpAwareOptionPane at

all.

Let me explain.

Remember the former situation: when the ExtendedDialog was launched, the splash screen was hidden (by some automagic trigger in the SplasScreen class I've removed too). There is no need to hide it and I didn't want to hide it anymore because it now displays progress information during startup (which wasn't the case before). Next step was to lauch the ExtendedDialog with the splash screen as parent which didn't work either, first because on Linux it was displayed behind the splash screen, and second because it was launched with the wrong size (collapsed to mininum size). I could have tried to fix the size issue in ExtendedDialog, instead I opted to replace it with HelpAwareOptonPane. It works and the user experience is at least as good as with ExtendedDialog (did you notice the tooltip texts on the buttons, which you didn't have before?). In the meantime the splash screen turned into an undecorated JFrame which would probably allow to use ExtendedDialog again, but there's no need switch back. HelpAwareOptionPane works just fine.

[stoecker]

Well, there are several issues here, which are hidden:

• Don't show again now working as expected
• Splash screen will hide any messages (the plugin message is not the only one).

The "splash screen disappears" was a solution, which fixed a larger sets of startup troubles caused by hidden dialogs which could not be modified, seen or handled. When you disabled that feature, than we have all these troubles back. Go back to the time when splash was introduced and check Trac there. Many issues have been related to this until the disappear was added.

I don't think removing this feature is really a good idea.

[Gubaer]

The "splash screen disappears" was a solution, which fixed a larger sets of startup
troubles caused by hidden dialogs which could not be modified, seen or handled.
When you disabled that feature, than we have all these troubles back.

Why? Which troubles are back? Why do you think the splash screen will hide any message? Pardon me, but this sound like FUD to me.

As I explained above, hiding the splash screen was suboptimal. I tried to find a better solution and apparently I did, see #4365. It's tested on Linux and Windows, user report it to work and it's closed.

If you want to continue this, please reopen #4365 and continue there. This tickets deals with another issue. Let's just fix it and close it.

[stoecker]

Why do you think the splash screen will hide any message?

You said yourself, that "I could have tried to fix the size issue in ExtendedDialog, instead I". This means ExtendedDialog as well as probably all other dialogs still are hidden behind splash screen. This includes all error dialogs as well as any other messages we have.

[Gubaer]

This means ExtendedDialog as well as probably all other dialogs still are hidden behind splash screen.

"Probably" is not good enough. The result of recent tests of myself and other users indicate: no they aren't. Again, if something is broken there please reopen #4365. Here it's off topic.

[Gubaer]

Fixed in r2924.