[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