 |
There is no problem in listing current FM for update. No problem in downloading it either: it gets downloaded to {root}/fms/{fm}.pk4, while the actually used archive is {root}/fms/{fm}/{fm}.pk4 The problem happens when we try to accept this new pk4 and copy it to proper location, since that destination location is currently used = locked by TDM process. In the code, this is located in CMissionManager::SearchForNewMods. It looks for new pk4 files and copes them into proper place. It is called on game initialization (alas, that's too late) and on "refreshMissionList" GUI command, which is called when user gos to FM selection screen. Interestingly, the same problem should happen anyway if user manually downloads the FM package and puts it into {root}/fms/ What actually happens is: game says new FM is available, but fails to replace the old pk4 (and does not remove the new one either) --- that can repeat many times. Restarting TDM afresh does not help, unless you restart it with different mission installed =( |
|
|
It seems that idCommonLocal::InitGameArguments is called just before filesystem is initialized. In particular, this code handles current FM selection (check command line arguments, currentfm.txt, presence of .map/.pk4). We can check here if pk4 for the selected FM is present in {foot}/fms/ directory, and replace it if it does. |
|
|
First of all, here is a small generic fix for pushing several GUI messages at once: r9468. Do not try to handle GUI messages while previous mesasge is still active. Regarding the main issue: r9469. Allow downloading update for the currently installed FM, handle it properly. r16326. Added strings for GUI message about updating the currently installed FM. Basically, there are two changes: 1) Accept incoming pk4 files of FMs (from {root}/fms/{fm}.pk4) in idCommonLocal::InitGameArguments. That's very early during engine startup, before filesystem is initialized. The Game is not initialized yet at that moment, so CMissionManager object is not available. To cope with that, I have refactored the necessary methods (SearchForNewMods) into static methods. Had to hack a bit in order to display message about updated mission (see AddToNewModList(newModsList)). 2) When update for the current FM is downloaded, display a new message asking to restart game. The new message is displayed after the old message "N missions downloaded". I have added "Later" button to cancel immediate game restart, although the game won't behave well after that. |
|
|
By the way, a bit of discussion here: https://forums.thedarkmod.com/index.php?/topic/11291-tdm-builtin-download-capability/&do=findComment&comment=462878 |
- Anonymous
