| 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 |
| ||||||||
 Relationships |
|
 Relationships |
| Notes |
|
|
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. |
| Notes |
| Issue History |
|||
| 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 |
| Issue History |