← Back to team overview

zeitgeist team mailing list archive

[Bug 489186] Re: GetHighestTimestampForActor is superfluous

 

** Changed in: zeitgeist
       Status: New => Triaged

** Changed in: zeitgeist
     Assignee: (unassigned) => Mikkel Kamstrup Erlandsen (kamstrup)

** Changed in: zeitgeist
   Importance: Undecided => Medium

** Changed in: zeitgeist
    Milestone: None => 0.3.0

-- 
GetHighestTimestampForActor is superfluous
https://bugs.launchpad.net/bugs/489186
You received this bug notification because you are a member of
Zeitgeist-Engine, which is the registrant for Zeitgeist Framework.

Status in Zeitgeist Framework: Triaged

Bug description:
The method GetHighestTimestampForActor is superfluous as far as I can tell. It can be simply done fully client side by using FindEventIds like so:

template = Event.new_for_values(actor="gedit")
ids = FindEventIds(TimeRange.until_now(), [template], StorageState.Any, 1, ResultType.MostRecentEvents)
highest_timestamp_for_gedit = GetEvents(ids)[0].timestamp

If I am right about this then I think we should remove it from the DBus API an simply implement a convenience method in ZeitgeistClient.



References