View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0004291 | The Dark Mod | Coding | public | 28.02.2016 19:07 | 14.10.2024 04:29 |
Reporter | Agent Jones | Assigned To | Daft Mugi | ||
Priority | normal | Severity | minor | Reproducibility | always |
Status | resolved | Resolution | fixed | ||
Product Version | TDM 2.03 | ||||
Target Version | TDM 2.13 | Fixed in Version | TDM 2.13 | ||
Summary | 0004291: List of missions in Downloadable Missions is not cleared if network connection is lost after update of mission list | ||||
Description | In 2.03, trying to download a mission when connection was lost resulted in a crash. In 2.12/64 #10651, it has unwanted cosmetic effects. In both cases the issue seem to stem from that the list of missions in "Downloadable Missions" is not cleared if network connection is lost after update of mission list. Proposed solutions: (both 1 and 2) 1) Clear the list of available missions if network is lost 2) When user clicks "Start Download", check if network is available and inform user if it is not available | ||||
Steps To Reproduce | In 2.03: -Click on Download Missions, download mission list. -Lose your connection. -Select a mission and try to download it. -Click OK when the connection error comes up. -The _selectedMods list won't be wiped unless you change page. If you select something for download again the game will crash. The error occurs at line 596 in DownloadMenu.cpp [gui->SetStateString(va("dl_mission_name_%d", i), modIndex != -1 ? mods[modIndex]->title : "");] In 2.12/64 #10651: * Make sure computer is connected to the network * From main menu, go to Missions - Download missions * The list will update after connecting to the server * Wait for the update to finish * Remove network cable from computer * From the list "Downloadable Missions", select a mission and click "More" * A dialog reads "Failed to download the details XML file" (expected) * From the list "Downloadable Missions", select a mission for download by moving it to the list "Selected for Download" and click "Start Download" * A dialog titled "Mission Download Result" reads "1 mission(s) couldn't be downloaded. Please check your disk space (or maybe some file is write protected) and try again." (text is unexpected since it does not correctly identify the problem) * Click OK * A dialog titled "Unable to Contact Mission Archive" reads "Cannot connect to server" (correctly identifies the problem) * Click OK * The mission is automatically removed from the list "Selected for Download" * The list "Downloadable Missions" keeps the list of the missions (possibly unexpected) * Select a mission and click "Select for Download" * A text appears in "Downloadable Missions" reading "No new mission packages available for download" (expected) but this is very difficult to read because the list of missions were not cleared (unexpected) * Reinsert network cable. * Select a mission and click "Selected for Download". * Nothing happens (unexpected, since the list of missions is visible and missions can be selected) | ||||
Additional Information | as a fix, tell the game to clear the list if we've lost connection. line 73, file DownloadMenu.cpp: _selectedListTop = 0; _selectedMods.Clear(); UpdateGUI(gui); UpdateDownloadProgress(gui); | ||||
Tags | gui | ||||
Looks like the equivalent fix should now be at line 67 and should now be: gui->DeleteStateVar("downloadSelectedList_item_0"); _selectedMods.Clear(); UpdateDownloadProgress(gui); UpdateGUI(gui); |
|
Is this fixed or not? | |
@datiswous: I tested this on 2.12 and I have updated the issue. It does not crash in 2.12, but there are some cosmetic issues which could possibly be resolved by the solution outlined by @nbohr1more. If it is not reassigned, I suggest it be set to state Acknowledged since it was last assigned in 2016 and it seems unlikely anyone is currently working on it. | |
Fixed as part of 0006339. Included in TDM 2.13-dev (dev17121-10869). |
|
I tested this on 2.13/64 #10890: 1) The dialog "check your disk space" still appears (unexpected) 2) The list is now cleared after clicking OK on the dialog "Unable to Contact Mission Archive" (expected) 3) The list view is now completely empty (and it did not add the text "No new mission packages available for download" like before) Overall, the implementation is an improvement and acceptable for the originally reported issue. Perhaps a new issue should be created to tweak for issues 1 and 3 (and some other unexpected behaviors) so that this issue can be set to fixed. |
|
Date Modified | Username | Field | Change |
---|---|---|---|
28.02.2016 19:07 | Agent Jones | New Issue | |
28.02.2016 19:07 | Agent Jones | Status | new => assigned |
28.02.2016 19:07 | Agent Jones | Assigned To | => Agent Jones |
01.01.2023 19:02 | nbohr1more | Relationship added | related to 0005551 |
02.01.2023 03:55 | nbohr1more | Note Added: 0015665 | |
02.01.2023 04:31 | nbohr1more | Note Edited: 0015665 | |
06.08.2024 17:07 | datiswous | Note Added: 0016810 | |
05.10.2024 19:51 | Fiver | Severity | crash => minor |
05.10.2024 19:51 | Fiver | Summary | Downloader crash after connection loss => List of missions in Downloadable Missions is not cleared if network connection is lost after update of mission list |
05.10.2024 19:51 | Fiver | Description Updated | |
05.10.2024 19:51 | Fiver | Steps to Reproduce Updated | |
05.10.2024 19:53 | Fiver | Tag Attached: gui | |
05.10.2024 19:54 | Fiver | Note Added: 0016857 | |
05.10.2024 21:29 | Daft Mugi | Assigned To | Agent Jones => Daft Mugi |
05.10.2024 21:35 | Daft Mugi | Relationship added | related to 0006339 |
05.10.2024 21:36 | Daft Mugi | Note Added: 0016859 | |
05.10.2024 21:37 | Daft Mugi | Status | assigned => feedback |
05.10.2024 21:37 | Daft Mugi | Target Version | => TDM 2.13 |
13.10.2024 20:55 | Fiver | Steps to Reproduce Updated | |
13.10.2024 20:55 | Fiver | Note Added: 0016870 | |
14.10.2024 00:54 | Daft Mugi | Status | feedback => resolved |
14.10.2024 04:29 | nbohr1more | Resolution | open => fixed |
14.10.2024 04:29 | nbohr1more | Fixed in Version | => TDM 2.13 |