Re: [Zeitgeist] [Merge] lp:~zeitgeist/zeitgeist/bug580364 into lp:zeitgeist

Review: Disapprove Sorry, big NACK from me. It makes no sense to allow this kind of queries right now, as we have no ways to set the storage state implemented. This is why we decided to throw an exception. -- https://code.launchpad.net/~zeitgeist/zeitgeist/bug580364/+merge/34141 Your team

Re: [Zeitgeist] [Merge] lp:~zeitgeist/zeitgeist/bug580364 into lp:zeitgeist

Review: Needs Fixing Well, I disagree with Markus :-) I think it makes perfect sense to include this - otherwise we have a chicken-and-egg problem. In theory anyone could set the storage field upon item insertion. It just so happens that the DS we ship is not feature complete. On top of that,

Re: [Zeitgeist] [Merge] lp:~zeitgeist/zeitgeist/bug580364 into lp:zeitgeist

Review: Needs Fixing Ok, I cave in, fix the things mentioned by Mikkel, run the testsuite and if everything goes well merge it into lp:zeitgeist (but please don't *bzr pull*) -- https://code.launchpad.net/~zeitgeist/zeitgeist/bug580364/+merge/34141 Your team Zeitgeist Framework Team is