[Code-review] review: Gadget current-activity branch

Dafydd Harries dafydd.harries at collabora.co.uk
Thu Aug 14 10:11:51 EDT 2008


Ar 12/08/2008 am 12:06, ysgrifennodd Guillaume Desmottes:
> Le lundi 11 août 2008 à 17:13 +0100, Dafydd Harries a écrit :
> > >+        # (room, activity id)
> > >+        self.current_activity = ('', '')
> > 
> > Why not make this an activity object? (Or None for no current activity.)
> 
> Then that implies that if Gadget is notified about a current-act change
> involving an activity which is not in Gadget's model, we'll have to set
> buddy's current-act to None.
> Is that something we want? IIRC, we agreed that Gadget should always fw
> notifications and that's client's job to ignore this notification if
> that's an unknown activity.

Well, we could still use an Activity object: it doesn't have to be in the
model.

> > >+        # FIXME: what about buddies who are in activity views as
> > >+        # participants ?
> > 
> > I don't understand this comment. Could you elaborate?
> 
> I rephrased it to:
> # FIXME: We only notify changes to buddy views. But buddies can also
> # be in activity views (as participants). We should send notification
> # messages in that case too.

Much clearer.

-- 
Dafydd


More information about the Code-review mailing list