Re: [Zeitgeist] [Merge] lp:~rainct/zeitgeist/fts-current-uri into lp:zeitgeist

2012-03-16 Thread Michal Hruby
Review: Needs Fixing The boolean terms we're adding to the document should also use current_uri. As for CompileEventFilterQuery, since the indexer no longer cares about logged uris, there's basically no distinction between them - therefore it should look at uri field and if that is unset try

Re: [Zeitgeist] [Merge] lp:~rainct/zeitgeist/fts-current-uri into lp:zeitgeist

2012-03-16 Thread Michal Hruby
Review: Approve Awesome! -- https://code.launchpad.net/~rainct/zeitgeist/fts-current-uri/+merge/97655 Your team Zeitgeist Framework Team is subscribed to branch lp:zeitgeist. ___ Mailing list: https://launchpad.net/~zeitgeist Post to :

[Zeitgeist] [Merge] lp:~rainct/zeitgeist/fts-current-uri into lp:zeitgeist

2012-03-16 Thread noreply
The proposal to merge lp:~rainct/zeitgeist/fts-current-uri into lp:zeitgeist has been updated. Status: Approved = Merged For more details, see: https://code.launchpad.net/~rainct/zeitgeist/fts-current-uri/+merge/97655 -- https://code.launchpad.net/~rainct/zeitgeist/fts-current-uri/+merge

[Zeitgeist] [Merge] lp:~rainct/zeitgeist/fts-current-uri into lp:zeitgeist

2012-03-15 Thread Siegfried Gevatter
Siegfried Gevatter has proposed merging lp:~rainct/zeitgeist/fts-current-uri into lp:zeitgeist. Requested reviews: Michal Hruby (mhr3) For more details, see: https://code.launchpad.net/~rainct/zeitgeist/fts-current-uri/+merge/97655 --

Re: [Zeitgeist] [Merge] lp:~rainct/zeitgeist/fts-current-uri into lp:zeitgeist

2012-03-15 Thread Siegfried Gevatter
Outstanding issue: Should CompileEventFilterQuery also be changed to use current_uri instead (or in addition to?) uri? -- https://code.launchpad.net/~rainct/zeitgeist/fts-current-uri/+merge/97655 Your team Zeitgeist Framework Team is subscribed to branch lp:zeitgeist.