[rfc][icedtea--web] fix itweb dual config mechanism broken by recent PathsAndFiles changes

Jiri Vanek jvanek at redhat.com
Thu Apr 9 15:17:47 UTC 2015


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.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: madeITWEBSettingsToWorkAgain.patch
Type: text/x-patch
Size: 15159 bytes
Desc: not available
URL: <http://mail.openjdk.java.net/pipermail/distro-pkg-dev/attachments/20150409/f06656b2/madeITWEBSettingsToWorkAgain.patch>


More information about the distro-pkg-dev mailing list