[rfc][icedtea--web] fix itweb dual config mechanism broken by recent PathsAndFiles changes
Jie Kang
jkang at redhat.com
Thu Apr 9 19:21:28 UTC 2015
Hello,
This patch works for me. Doesn't look the cleanest but a fix for this is nice all the same.
Regards,
----- Original Message -----
> Hi!
>
> Today Iwonted to add "reset to default" button to temporary internet files
> pane in itweb settings
> (as it is heavily missing) and found, that I broke the system of
> change/(cancel x ok/apply)
> mechanism in itweb-settings.
>
> This patch is adding the "reset button" and fixing this behaviour.
>
>
> The issue is, that itweb-settings (then and only them) are working with
> *two* config instances. So
> they canallow those cancel/apply schema. By moving to PathsAndFiles +
> getter/setter the second
> config was lost.
>
> So I have reverted all those constructor(config) -> constructor() changes and
> added stub to
> PathsAndFiles which allows to set/get from those secondar prperties. Not
> nicest:-/
>
>
> Maybe some abstraction like ItwebSettings configurable file ? Maybe I will
> find something tomorow,
> maybe reviwer will.
>
> Still this is important to fix:(
>
>
> J.
>
--
Jie Kang
OpenJDK Team - Software Engineering Intern
More information about the distro-pkg-dev
mailing list