New kdelibs/kutils/kpluginselector

[Matthias Kretz]

On Wednesday 01 November 2006 14:12, Rafael Fernández López wrote:
> * The "Default" button works, I think it didn't work for me in the past

I looked at the code - finally - and I believe the KPluginSelector::default() 
implementation is not correct. The default button is supposed to restore 
the "system defaults" and not revert the changes. That's what the load() 
function is meant for. (And I don't understand the comment: 
KCModuleProxy::defaults() should work - if it doesn't please let me know.)

The KPartPlugins stuff is another thing that could be streamlined for KDE4. 
The plugins should be installed as services (and this way they'll be cached 
in ksycoca) which results in only the non-KParts addPlugins method to remain.

I'd say: commit what you have tomorrow and I'll talk to you on IRC...

-- 
C'ya
        Matthias
________________________________________________________
Matthias Kretz (Germany)                            <><
http://Vir.homelinux.org/
MatthiasKretz at gmx.net, kretz at kde.org,
Matthias.Kretz at urz.uni-heidelberg.de
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20061105/b3ff18ce/attachment.sig>