<div dir="ltr">On 1 May 2013 00:27, Daniel Drake <span dir="ltr"><<a href="mailto:dsd@laptop.org" target="_blank">dsd@laptop.org</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br><div class="im">
> -    def get_type(self):<br>
> +    def get_bundle_id(self):<br>
>          """Retrieve the activity bundle id for future reference"""<br>
> -        if not self._windows:<br>
> -            return None<br>
> -        else:<br>
> -            return SugarExt.wm_get_bundle_id(self._windows[0].get_xid())<br>
> +        self._bundle_id<br>
<br>
</div>On Sat, Feb 2, 2013 at 6:18 AM, Daniel Narvaez <<a href="mailto:dwnarvaez@gmail.com">dwnarvaez@gmail.com</a>> wrote:<br>Missing return statement here?<br>
<br></blockquote><div><br></div><div>Ooops, of course! <br></div><div class="im"> <br>
> diff --git a/src/jarabe/view/service.py b/src/jarabe/view/service.py<br>
> index 61b3d59..c7187c7 100644<br>
> --- a/src/jarabe/view/service.py<br>
> +++ b/src/jarabe/view/service.py<br>
> @@ -80,9 +80,9 @@ class UIService(dbus.service.Object):<br>
>          return False<br>
><br>
>      @dbus.service.method(_DBUS_SHELL_IFACE,<br>
> -                         in_signature='ss', out_signature='')<br>
> -    def NotifyLaunch(self, bundle_id, activity_id):<br>
> -        shell.get_model().notify_launch(activity_id, bundle_id)<br>
> +                         in_signature='ssi', out_signature='')<br>
> +    def NotifyLaunch(self, bundle_id, activity_id, pid):<br>
> +        shell.get_model().notify_launch(activity_id, bundle_id, pid)<br>
<br>
</div><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">This is an API change. The toolkit calls this method so it must be<br>
updated as well, otherwise activity launch breaks. And I guess we<br>
might have broken compatibility with some other things like etoys.<br></blockquote><div><br></div><div>It has been a while since I wrote this patch, hopefully I'm not misremembering... I think NotifyLaunch is in practice private to the shell, activities should have no need to use it (and are not supposed to be importing activityfactory.py).<br>
</div></div></div></div>