Spot counter only counts what's on the display

Bug #492225 reported by Ken McLean
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Retromod Traffic
Fix Committed
High
Tudor Holton

Bug Description

Spot counter only counts what's on the display.

ie. if you move the nav arrows to move some of the selected boxes off the page, the counter total changes.

Tags: ui
Ken McLean (kenmclean)
Changed in retromod-traffic:
milestone: sprint-3 → sprint-4
Revision history for this message
Tudor Holton (tudor) wrote : Re: [Bug 492225] Re: Spot counter only counts what's on the display

ok. I need a little bit more information here.

The spot counter currently counts only what's on the display.

We have two options:
1) Count all the spots in the campaign in total, perhaps this could be
connected to the number of spots allocated o the campaign.
2) Count all the spots added/removed in the current session.

(2) is obviously harder than (1), but as to how much harder I don't know.

Ken McLean wrote:
> ** Changed in: retromod-traffic
> Milestone: sprint-3 => sprint-4
>
>

Revision history for this message
Ken McLean (kenmclean) wrote :

This is linked to bug #492213.

Yeah this is tricky because the spotsAllotted value is ascribed to the campaign, not the audio.

Firstly, the spotsAllotted value for the parent campaign needs to be retrieved.

Secondly, the total spots for each audio in the campaign needs to be retrieved, and then the sum of all of those needs to be calculated.

Finally we need two counters.
* One for the audio object that counts up (eg. 25 spots scheduled)
* and a total which counts down starting from spotsAllotted minus total campaign spots. (eg. 75/100 spots allotted)

Bottom line is it needs to count more than just what's on the display.

Revision history for this message
Ken McLean (kenmclean) wrote :

Sorry, I should have mentioned, the reason we need to retrieve the sum of spots for each piece of audio in the campaign (as well as their spotID) is because we will need to colour code the grid to show where other spots in our campaign are already scheduled. I'll add that as another bug.

Revision history for this message
Ken McLean (kenmclean) wrote :

I have fixed this but it depends on a new but incomplete javascript function for bug #492213, so not committed.

Changed in retromod-traffic:
status: New → In Progress
Ken McLean (kenmclean)
Changed in retromod-traffic:
status: In Progress → Fix Committed
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.