View Issue Details

IDProjectCategoryView StatusLast Update
0002056FSSCPSEXPspublic2010-06-23 03:01
ReporterFUBAR-BDHR Assigned ToGoober5000  
PrioritynormalSeverityminorReproducibilityN/A
Status resolvedResolutionfixed 
Product Version3.6.11 
Fixed in Version3.6.13 
Summary0002056: Disable/enable built in messages node processing logic problem
DescriptionWhile looking to update this to take teams and wings as arguments I noticed the logic didn't seem quite right. Portej05 took a quick look and confirmed my suspicions. The sexp current takes #command, <any wingman> or a ship name as arguments (or no argument) The problem is that if a ship is listed before #command or <any wingman> in the list sexp_deal_with_ship_flag will be called and process the entire list thus skipping the processing of #command or <any wingman>

I already have to modify this for accepting teams and wings so I should be able to fix it at the same time.
TagsNo tags attached.

Activities

karajorma

2009-12-02 13:32

administrator   ~0011368

I'm tempted to get the fix for this one in now, you can change that to your complete solution for the problem later.

Goober5000

2010-06-23 03:01

administrator   ~0012124

Fixed by adding an extra parameter to sexp_deal_with_ship_flag to check whether or not to process the entire list.

Fix is in revision 6257.

Issue History

Date Modified Username Field Change
2009-11-27 04:54 FUBAR-BDHR New Issue
2009-12-02 13:32 karajorma Note Added: 0011368
2010-02-06 11:07 karajorma Status new => assigned
2010-02-06 11:07 karajorma Assigned To => karajorma
2010-06-23 03:01 Goober5000 Note Added: 0012124
2010-06-23 03:01 Goober5000 Assigned To karajorma => Goober5000
2010-06-23 03:01 Goober5000 Status assigned => resolved
2010-06-23 03:01 Goober5000 Resolution open => fixed
2010-06-23 03:01 Goober5000 Fixed in Version => 3.6.13