View Issue Details [ Jump to Notes ] | [ Issue History ] [ Print ] | ||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||
---|---|---|---|---|---|---|---|---|---|
0002056 | FSSCP | SEXPs | public | 2009-11-26 23:54 | 2010-06-22 23:01 | ||||
Reporter | FUBAR-BDHR | ||||||||
Assigned To | Goober5000 | ||||||||
Priority | normal | Severity | minor | Reproducibility | N/A | ||||
Status | resolved | Resolution | fixed | ||||||
Product Version | 3.6.11 | ||||||||
Target Version | Fixed in Version | 3.6.13 | |||||||
Summary | 0002056: Disable/enable built in messages node processing logic problem | ||||||||
Description | While 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. | ||||||||
Tags | No tags attached. | ||||||||
Attached Files |
|
![]() |
|
karajorma (administrator) 2009-12-02 08:32 |
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 (administrator) 2010-06-22 23:01 |
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. |
![]() |
|||
Date Modified | Username | Field | Change |
---|---|---|---|
2009-11-26 23:54 | FUBAR-BDHR | New Issue | |
2009-12-02 08:32 | karajorma | Note Added: 0011368 | |
2010-02-06 06:07 | karajorma | Status | new => assigned |
2010-02-06 06:07 | karajorma | Assigned To | => karajorma |
2010-06-22 23:01 | Goober5000 | Note Added: 0012124 | |
2010-06-22 23:01 | Goober5000 | Assigned To | karajorma => Goober5000 |
2010-06-22 23:01 | Goober5000 | Status | assigned => resolved |
2010-06-22 23:01 | Goober5000 | Resolution | open => fixed |
2010-06-22 23:01 | Goober5000 | Fixed in Version | => 3.6.13 |