add setType to PackageStucture

[Aaron J. Seigo]

On Thursday 30 October 2008, Petri Damstén wrote:
> On Thursday 30 October 2008 12:49:01 Aaron J. Seigo wrote:
> > On Thursday 30 October 2008, Petri Damstén wrote:
> > > On Thursday 30 October 2008 12:30:10 Aaron J. Seigo wrote:
> > > > On Thursday 30 October 2008, Petri Damstén wrote:
> > > > > If we copy Plasma::Applet::packageStructure() to custom package
> > > > > structure type is also copied and we have no setType to correct it.
> > > >
> > > > what is the exact use case for this?
> > >
> > > e.g. in comic package structure constructor:
> >
> > i think that's an incorrect use. you should just subclass
> > PackageStructure directly and create a ComicProviderPackage class.
>
> It's a subclass of PackageStructure, but I want to copy data from
> PlasmoidPackage which is what Applet::packageStructure() returns.

why? (sorry if i'm seeming dense atm, but i'm in the middle of a day-long 
meeting, my 4th in a row...)

> And this
> is actually what you recommended for webapplet package :-)
> http://mail.kde.org/pipermail/plasma-devel/2008-September/001009.html
>
> There it works since webapplet is also a plasmoid type.

yes, which is why i suggested that approach in that case =)

-- 
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 Qt Software

-------------- 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/20081030/3a6f2be1/attachment.sig