Top contributors list broken

Thanks Moritz for reporting this problem.
I will look at what happened and will try to fix it... most probably is my fault.

Adi, btw, you should probably use slicing to return up to 20 items from a SQL result object (it will be most efficient way). If some get removed because they are merged, it still is not too bad to return eg. 17 items, compared to serializing all 1300 Person objects for some languages (like Spanish or French).

To do it, simply do something like:

  top_translators = self.context.translators[:20]
  returned_translators = []
  for translator in top_translators:
    ...

Ok. I will do it. I didn't use the slicing to avoid creating a top 20 that only contains 17 items.

If you say that returning 17 items is ok, then I will use slicing and maybe I can change the text from "Top 20 contributors" to "Top contributors".

I will change the code and will create a MP right away.

Fixed in stable r10765 <http://bazaar.launchpad.net/~launchpad-pqm/launchpad/stable/revision/10765>