~robru/friends/facebook-stories

Viewing all changes in revision 161.

  • Committer: Robert Bruce Park
  • Date: 2013-03-11 22:40:10 UTC
  • Revision ID: robert.park@canonical.com-20130311224010-fihmrn0ndgdqlhrm
Give up on the idea of deduplication across protocols.

We used to have a *ton* of corner case logic that attempted to merge
messages that had the same content, but were from distinct social
networks. That made some sense back in the old gwibber days but it's
becoming an increasingly irrelevant corner case that is needlessly
complicating a lot of our application logic.

This commit removes that logic, and also greatly simplifies the model
schema accordingly, allowing for much easier filtering and sorting of
the model by it's consumers (such as qml-friends).

There's a big impact on the test suites, as this changes *huge*
portions of our fundamental architectural assumptions. In the process
of updating it, I also discovered that Base._unpublish_all was
poorly/indirectly tested, so I wrote some better, explicit tests for it.

expand all expand all

Show diffs side-by-side

added added

removed removed

Lines of Context: