View Issue Details

IDProjectCategoryView StatusLast Update
0002817FSSCPPilot datapublic2013-03-26 10:06
ReporterMjnMixael Assigned Toniffiwan  
PrioritynormalSeverityminorReproducibilityalways
Status resolvedResolutionfixed 
PlatformPCOSWindowsOS VersionWin7
Product Version3.6.19 
Target Version3.7.0 
Summary0002817: Too many warnings about missing campaigns
DescriptionIf you play a campaign and then switch mods to play a different campaign under the same pilot you get at least 3 warning messages that the currently selected campaign can't be found.
Steps To Reproduce1) Create a pilot.
2) Play a campaign, preferably from a mod.
3) Exit FSO and unload that mod.
4) Run FSO and select that same pilot.
Additional InformationYou'll get a warning window on pilot select that the 'Current campaign can't be found'. You'll then get the window that prompts you to go to the campaign room because the 'Current campaign can't be found'. Then, when you select to go to the campaign room, you get another warning window that the 'Current campaign can't be found'.
TagsNo tags attached.

Activities

niffiwan

2013-03-26 10:01

developer  

mantis2817-svn.patch (960 bytes)   
Index: code/menuui/mainhallmenu.cpp
===================================================================
--- code/menuui/mainhallmenu.cpp	(revision 9604)
+++ code/menuui/mainhallmenu.cpp	(working copy)
@@ -605,9 +605,6 @@
 	// handle any sound details
 	main_hall_cull_door_sounds();
 
-	// do any campaign load failure handling
-	mission_campaign_load_failure_popup();
-
 	// process any keypresses/mouse events
 	snazzy_action = -1;
 	code = snazzy_menu_do(Main_hall_mask_data, Main_hall_mask_w, Main_hall_mask_h, Main_hall_num_options, Main_hall_region, &snazzy_action, 1, &key);
Index: code/menuui/readyroom.cpp
===================================================================
--- code/menuui/readyroom.cpp	(revision 9604)
+++ code/menuui/readyroom.cpp	(working copy)
@@ -1693,8 +1693,6 @@
 	} else {
 		Campaign.filename[0] = 0;
 		Campaign.num_missions = 0;
-
-		mission_campaign_load_failure_popup();
 	}
 
 	// we need descriptions too, so "true" it
mantis2817-svn.patch (960 bytes)   

niffiwan

2013-03-26 10:06

developer   ~0014845

Fix committed to trunk@9605.

Related Changesets

fs2open: trunk r9605

2013-03-26 06:53

niffiwan


Ported: N/A

Details Diff
Fix for mantis 2817: remove two redundant popups regarding missing campaigns Affected Issues
0002817
mod - /trunk/fs2_open/code/menuui/mainhallmenu.cpp Diff File
mod - /trunk/fs2_open/code/menuui/readyroom.cpp Diff File

Issue History

Date Modified Username Field Change
2013-03-23 00:21 MjnMixael New Issue
2013-03-26 08:34 niffiwan Assigned To => niffiwan
2013-03-26 08:34 niffiwan Status new => assigned
2013-03-26 10:01 niffiwan File Added: mantis2817-svn.patch
2013-03-26 10:01 niffiwan File Deleted: mantis2817-svn.patch
2013-03-26 10:01 niffiwan File Added: mantis2817-svn.patch
2013-03-26 10:06 niffiwan Changeset attached => fs2open trunk r9605
2013-03-26 10:06 niffiwan Note Added: 0014845
2013-03-26 10:06 niffiwan Status assigned => resolved
2013-03-26 10:06 niffiwan Resolution open => fixed