1119986 - Repeated events in Trending in sidebar

Status: RESOLVED FIXED

(Webtools Graveyard :: Air Mozilla, defect, P1)

Description

[Peter Bengtsson [:peterbe]]

Attachment: Screenshot 2015-01-09 15.03.16.png

See attached screenshot. 

That's an event that belongs to two channels: Main and Mozilla Reps. 
It's currently trending (because it's fresh) and it's showing once when you're on the home page (channel Main).

It should only appear once only.

Comment 1

[Peter Bengtsson [:peterbe]]

Leonard, 
Perhaps you can figure this one out?

Comment 2

[Leonard Camacho [:lcamacho]]

I reviewed the query but can't find anything wrong, can you check if there's two EventHitStats that point to that event?

Comment 3

[Peter Bengtsson [:peterbe]]

It totally is! Now I wonder how that could ever have happened.

Comment 4

[Peter Bengtsson [:peterbe]]

I take it back. They aren't repeated. They can't be. I just got confused (hence my previous comment). 
There's a uniqueness constrain that prevents it https://github.com/mozilla/airmozilla/blob/e66fc056710e0fd524b4277c82dc1d1be1caa259/airmozilla/main/models.py#L698

Comment 5

[Peter Bengtsson [:peterbe]]

I can reproduce this locally now.

Comment 6

[Peter Bengtsson [:peterbe]]

Attachment: Link to Github pull-request: https://github.com/mozilla/airmozilla/pull/331

Comment 7

[[github robot]]

Commits pushed to master at https://github.com/mozilla/airmozilla

https://github.com/mozilla/airmozilla/commit/6d02cf5adf6699be805ad37f78d069a71429748c
fixes bug 1119986 - Repeated events in Trending in sidebar

https://github.com/mozilla/airmozilla/commit/2efc979905f7e114be5cab45d25d3c6983e33336
Merge pull request #331 from peterbe/bug-1119986-repeated-events-in-trending-in-sidebar

fixes bug 1119986 - Repeated events in Trending in sidebar