View Issue Details

IDProjectCategoryView StatusLast Update
0004258The Dark ModCodingpublic11.01.2016 00:48
Reportergrayman Assigned ToAgent Jones  
PrioritynormalSeveritynormalReproducibilitysometimes
Status resolvedResolutionfixed 
Product VersionTDM 2.03 
Target VersionTDM 2.04Fixed in VersionTDM 2.04 
Summary0004258: Hang in Mission Downloader
DescriptionIf a mission download hangs, and I return to the Main Screen, then--without closing TDM--I return to the Download screen, it's displaying what it was showing when it hung, and I don't have a chance to reload the download queue and try again. I'm forced to quit TDM.
TagsNo tags attached.
Attached Files
TDM_Cancel_Download.zip (22,311 bytes)

Activities

Agent Jones

Agent Jones

05.01.2016 13:37

reporter   ~0007982

There's no need to go back to the menu screen for this bug to occur. It is caused by a CURL request never returning a result.
One solution would be to add a timeout to the request, but this won't make people with unstable connections happy.
Another solution is to allow the player to cancel downloads from the queue even while they're in progress. What was the rationale behind the design decision not to allow this?
grayman

grayman

05.01.2016 14:06

viewer   ~0007983

Don't know.

I'll post a question in the member's forum to see if anyone can remember the initial design decision, and see if it's okay to cancel a download.
grayman

grayman

07.01.2016 19:12

viewer   ~0007995

No one remembers, so you're clear to go ahead with the cancel.
Agent Jones

Agent Jones

10.01.2016 17:50

reporter   ~0008020

The files in the zip will allow you to cancel downloads in progress from the menu queue, even if they're hanging.

Without this, canceling a hanging request will be impossible (unless you exit the game, which is what issue 0003766 was about) since the current way of canceling the request relies on CURLOPT_WRITEFUNCTION, which is not called when your connection hangs and nothing is transferred.

A question: if a download that comprises a l10n pk4 is canceled and the l10n download is finished the pk4 will remain in the directory. Should I make it so the localization file is removed when a download is canceled? They're usually just a few kilobytes.
grayman

grayman

10.01.2016 20:09

viewer   ~0008021

It should be okay if the I10n file remains. The player will most likely retry the download a few minutes later.
grayman

grayman

10.01.2016 21:50

viewer   ~0008022

What's the recommended way of cancelling?

I left-clicked a hung mission and its name went back to the left side of the screen, and the download picked up the next mission in the queue and started downloading it.

Which is fine.

Except I left-clicked a mission that had completed at 100%, and it went back to the left side, but was still there when I went back to the new missions screen.

Shouldn't a mission at 100% complete ignore the mouse action?
Agent Jones

Agent Jones

10.01.2016 22:56

reporter   ~0008023

You're right. I updated DownloadMenu.cpp
grayman

grayman

10.01.2016 23:51

viewer   ~0008024

Works fine.

Issue History

Date Modified Username Field Change
09.12.2015 12:47 grayman New Issue
11.12.2015 17:04 Agent Jones Assigned To => Agent Jones
11.12.2015 17:04 Agent Jones Status new => assigned
05.01.2016 13:37 Agent Jones Note Added: 0007982
05.01.2016 14:06 grayman Note Added: 0007983
07.01.2016 19:12 grayman Note Added: 0007995
10.01.2016 13:39 Agent Jones File Added: TDM_Cancel_Download.zip
10.01.2016 13:55 Agent Jones File Deleted: TDM_Cancel_Download.zip
10.01.2016 17:41 Agent Jones File Added: TDM_Cancel_Download.zip
10.01.2016 17:50 Agent Jones Note Added: 0008020
10.01.2016 20:09 grayman Note Added: 0008021
10.01.2016 21:50 grayman Note Added: 0008022
10.01.2016 22:54 Agent Jones File Deleted: TDM_Cancel_Download.zip
10.01.2016 22:54 Agent Jones File Added: TDM_Cancel_Download.zip
10.01.2016 22:56 Agent Jones Note Added: 0008023
10.01.2016 23:51 grayman Note Added: 0008024
11.01.2016 00:48 grayman Status assigned => resolved
11.01.2016 00:48 grayman Resolution open => fixed
11.01.2016 00:48 grayman Fixed in Version => TDM 2.04
11.01.2016 00:48 grayman Target Version => TDM 2.04