Plasma/PopupApplet and Plasma::PackageStructure

[Aaron J. Seigo]

On February 27, 2010, Yuen Hoe Lim wrote:
> Is this a bug or is this intentional? If the latter, then is it acceptable
> to simply do

looking at your last commit to plasmate, i see why you asked this. all 
plasmoids should have Plasma/Applet in the ServiceTypes= entry of the .desktop 
file, since all Plasma/PopupApplets are also Plasma/Applets. it means we have 
to constantly search for Plasma/PopupApplets to build a complete list of 
applets, and PopupApplet is just a hint for scripting purposes anyways.

right now Plasma::Applet does search for Plasma/PopupApplet as well as 
Plasma/Applet, so this is worked around pretty thoroughly. but i do wonder if 
that code should be made to only rely on Plasma/Applet. the only place 
Plasma/PopupApplet should be needed is in Applet::loadPlasmoid.

the danger is that scripted plasmoids that used to work might stop being shown 
in the applet listings if they just had ServiceTypes=Plasma/PopupApplet. 
however, the documentation on techbase that mentions this as a possibility 
does also state that it must include Plasma/Applet.

so i'm considering removing the references to Plasma/PopupApplet in 
Plasma::Applet except for where it is used to actually create the correct base 
class. Richard: what do you think?

-- 
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 Development Frameworks