On 6/7/12 11:15 AM, Ian Bicking wrote:
The reuse of getInstalled this way seems like a pun, not a sensible extension of the API. getInstalled currently means "apps that this origin installed". It has nothing to do with any local installation status. mgmt.getLocallyInstalled() or some other new API is more sensible. Also there needs to be a way to do something with those results, such as an .install() method on app objects, or on some app objects.
Sure, that's what it means right now; doesn't mean we can't change what it means :-)
I'm just trying to arrive at the simplest possible API to give us what we need. We can afford to do this at this stage because we're not really going to break anyone using the API, as the only users right now are us (Mozilla).
And yes, I was thinking that calling mozApps.install() with a manifest URI extracted from an app object returned by getAll() would result in that app now appearing in the getInstalled() set - if the installation succeeded, of course.
RIght now getAll() is used to get 2 (even if somewhat inaccurately), which is why I proposed a new method be added for 1 (getAllRemote()).
Same as above, I perceive this to be more of a redefinition of the method rather than trying to hack in extra functionality to a previously frozen interface (which it isn't).
-Anant _______________________________________________ dev-webapps mailing list [email protected] https://lists.mozilla.org/listinfo/dev-webapps
