Comment 4 for bug 678090

Revision history for this message
Martin Pool (mbp) wrote :

This is a special case of bug 343110, but perhaps easy to fix separately.

> This looks like we show the wrong attribute (users_affected, not users_affected_count_with_dupes) in templates - so a shallow thing to change /code wise/

I looked at this in the code, and there is indeed a possible snag. users_affected_count is stored on in a column on the bug, but users_affected_count_with_dupes is calculated by finding all the users and counting them, so there's at least a possible performance problem there. We can either see if it is fast enough to query it, or we could add another cache in the database.

Adding them up, whether done at update time or query time, ought to take into account that if person A is affected by bugs X and Y that end up both being marked as dupes of Z, that's only one person.