View Issue Details
|ID||Project||Category||View Status||Date Submitted||Last Update|
|0003555||The Dark Mod||Coding||public||14.09.2013 17:13||11.07.2018 11:15|
|Product Version||TDM 2.00|
|Target Version||TDM 2.07||Fixed in Version||TDM 2.07|
|Summary||0003555: TDM crashes if one FM contains a darkmod.txt file with invalid order.|
|Description||By default, the order in the darkmod.txt is Title/Description/Author. However, one would suspect the order to be irrevelant. But if you have a different order, the game crashes as soon as you hit the New Mission button in the main menu.|
|Steps To Reproduce||Unpack any FM, change the order in the darkmod.txt file and pack it again. Then start TDM and hit New Mission.|
|Additional Information||I did not had any FM's installed as I was encountering this. Didn't test whether this plays a role, though.|
|Tags||No tags attached.|
CModInfo::LoadMetaData() is where darkmod.txt is read.
The code expects the fields to be in this order:
Required TDM Version
It will need to change to accept the fields in any order.
Workaround is for authors to put the fields in the expected order.
Since the wiki page for releasing your mission describes the contents of the darkmod.txt file in this order:
Title: The Outpost
Description: Break into a builder outpost to retrieve a gold ingot.
Author: angua & greebo
Required TDM Version: 1.01
I see no reason to change the code to accept _any_ order.
Mission authors should just follow the order given on the wiki page.
|I want to close this issue with a "won't be fixed" resolution.|
You are assuming that everyone is looking at that wiki page. Additionally, even if one does, he may not come to the conclusion, that using a different order would cause an issue (at least I didn't). And if the game is crashing than, the respective FM author will have a happy time searching for the reason ;)
So I think we should not leave it like that. Either the game should accept any order or the game should tell the FM author what he or she did wrong. A plain crash is definetely not good.
Fixed with revision #7563.
In addition to the above mentioned issue, leaving out certain fields could cause display errors.
|14.09.2013 17:13||Obsttorte||New Issue|
|14.09.2013 18:42||grayman||Note Added: 0006179|
|10.04.2016 15:42||Obsttorte||Assigned To||=> Obsttorte|
|10.04.2016 15:42||Obsttorte||Status||new => assigned|
|10.04.2016 18:02||grayman||Note Added: 0008076|
|10.04.2016 18:05||grayman||Note Added: 0008077|
|11.04.2016 06:44||Obsttorte||Note Added: 0008081|
|11.07.2018 11:14||Obsttorte||Note Added: 0010668|
|11.07.2018 11:15||Obsttorte||Status||assigned => resolved|
|11.07.2018 11:15||Obsttorte||Resolution||open => fixed|
|11.07.2018 11:15||Obsttorte||Fixed in Version||=> TDM 2.07|
|11.07.2018 11:15||Obsttorte||Target Version||=> TDM 2.07|