Draft Page Does Not Correctly Account For Skipped Bans

[salmarnir]

Current Behavior
In Captain's Mode games (and presumably Captain's Draft), if a team lets the time expire during a ban phase, they essentially skip that ban. However in the draft tab on OpenDota, this is not correctly captured. It instead appears to assign whatever the next hero that was picked or banned to that slot, regardless of which team chose it. This will also cause subsequent picks/bans to be misaligned and results in the page having incorrect and incomplete information. Note that this also affects the time used/time remaining shown at each pick.

Expected behavior/code
Rather than assigning each hero picked/banned in the draft to what it expects the next available "slot" to be, should also parse which team picked or banned the hero to take skipped bans into consideration.

Additional context/Screenshots
For an example of this, please see https://www.opendota.com/matches/5293089497/draft

[howardchung]

The data looks like:

"picks_bans": [
{
"is_pick": false,
"hero_id": 57,
"team": 1,
"order": 0
},
{
"is_pick": false,
"hero_id": 89,
"team": 1,
"order": 1
},
{
"is_pick": false,
"hero_id": 82,
"team": 1,
"order": 2
},
{
"is_pick": true,
"hero_id": 68,
"team": 1,
"order": 3
},
{
"is_pick": true,
"hero_id": 87,
"team": 0,
"order": 4
},
{
"is_pick": true,
"hero_id": 7,
"team": 0,
"order": 5
},
{
"is_pick": true,
"hero_id": 71,
"team": 1,
"order": 6
},
"hero_id": 68,
"team": 1,
"order": 3
},

I think we have to update the drafts page to account for the team picking rather than having a set order.