Wallpaper plugins

[Aaron J. Seigo]

On Saturday 23 August 2008, Petri Damstén wrote:
> On Saturday 23 August 2008 01:05:38 Aaron J. Seigo wrote:
> > > 2. Was wallpaper config save remove intentional?
> >
> > yes; i moved it to Containment::save
>
> But, doesn't that just save the wallpaper config to old containment. This
> code was there to copy wallpaper config to new containment.

what happens in the code is:

* old containment save settings: old->save(containmentConfig);
* new containment reads those settings: c->restore(containmentConfig);
* we switch config objects: containmentConfig = c->config()
* new containment saves its settings: c->save(containmentConfig);
* config sync requested: corona->requestConfigSync();

so by putting the wallpaper saving into Containment::save, not only does it fix 
a potential problem when we export layouts, but it allows the above to work as 
well.

-- 
Aaron J. Seigo
humru othro a kohnu se
GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA  EE75 D6B7 2EB1 A7F1 DB43

KDE core developer sponsored by Trolltech

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: This is a digitally signed message part.
Url : http://mail.kde.org/pipermail/plasma-devel/attachments/20080823/6517983d/attachment.sig