Community
Participate
Working Groups
Testing with 4.16 I notice the text "OK" in several StatusDialogs, where the error message would be if the dialogs had errors. Looks like the fix for bug 560667 is to blame, that removed the check for IStatus.isOK(). Since StatusDialog.updateStatus() does not accept null, I think providing Status.OK_STATUS is how the status message should be cleared, but it has its message set to (the localized equivalent of) "ok", not null, which means it will now be printed.
Could you provide steps to reproduce, or some screen shots? Of if you have a patch in mind, please push to gerrit.
It doesn't show in platform's own StatusDialogs as they all seem to create new StatusInfos (which do have a null message by default), rather than reusing Status.OK_STATUS. Anyway, the attached snippet shows the issue. In previous platform versions it has produced an empty dialog with OK / Cancel buttons. In 4.16 it produces a dialog with the text OK. Not sure about the patch. If compatibility matters, both StatusDialog and SelectionStatusDialog need to keep their original behavior, so either they should not share the same MessageLine implementation, or MessageLine needs some flag to signal which behavior for OK status should be used.
Created attachment 283691 [details] StatusDialog snippet
Created attachment 283882 [details] AddRepository dialog with "OK" (instead of empty error message) Looks like the AddRepository dialog (org.eclipse.equinox.internal.p2.ui.dialogs) uses Status.OK_STATUS to clear the StatusDialog error message and now gets "OK" instead.
Same thing with the DependencyPropertiesDialog of the PDE.
