Bug 7873 – New AppletManager API

Bug 7873 - New AppletManager API

Summary:

New AppletManager API

Status:	CLOSED FIXED

Product:	SB Controller
Classification:	Unclassified
Component:	Setup
Version:	unspecified
Platform:	PC Windows XP

Importance:	P1 normal (vote)
Target Milestone:	7.3
Assigned To:	Ben Klaas

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2008-04-18 06:25 UTC by Richard Titmuss
Modified:	2008-12-15 12:04 UTC (History)
CC List:	0 users

See Also:
Category:	---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Richard Titmuss 2008-04-18 06:25:25 UTC

Applets should register services in the meta files. The getAppletInstance / loadApplet calls should be removed. If an applet service is needed it should be called like: appletManager:service(name, ...)

This makes the api easier to use, and also hides the applet loading/freeing from the caller.

Comment 1 Richard Titmuss 2008-07-24 05:01:01 UTC

This is partly done in 7.1, and other applet calls now need migrating.

Comment 2 Ben Klaas 2008-07-25 07:06:07 UTC

punting to 7.3

Comment 3 Ben Klaas 2008-07-25 07:06:50 UTC

Richard says "refactoring like this should really get done first in a release cycle"

moving to priority P1

Comment 4 Ben Klaas 2008-10-24 08:14:38 UTC

I think we're in good shape with this bug now.

Richard, SqueezeboxJiveMeta.lua still calls loadApplet(), but I'm thinking that might be correct for that one?

I think we can close this...

Comment 5 James Richardson 2008-12-15 12:04:39 UTC

This bug has been fixed in the 7.3.0 release version of SqueezeCenter!

Please download the new version from http://www.slimdevices.com/su_downloads.html if you haven't already.  

If you are still experiencing this problem, feel free to reopen the bug with your new comments and we'll have another look.