Fix syntax errors: double semicolon, invalid 00 literals; add NOTES.md for upstream bug
This commit is contained in:
24
NOTES.md
Normal file
24
NOTES.md
Normal file
@@ -0,0 +1,24 @@
|
||||
# Notes
|
||||
|
||||
## Upstream Bug - `fetch_params.sqf` Line 158
|
||||
|
||||
**File:** `scripts/shared/fetch_params.sqf`
|
||||
**Line:** 158
|
||||
**Type:** Logic bug (copy-paste error)
|
||||
|
||||
The `default` case in the `switch (GRLIB_civilian_activity)` block writes to the wrong variable:
|
||||
|
||||
```sqf
|
||||
// Line 153-159
|
||||
switch (GRLIB_civilian_activity) do {
|
||||
case 0: {GRLIB_civilian_activity = 0;};
|
||||
case 1: {GRLIB_civilian_activity = 0.5;};
|
||||
case 2: {GRLIB_civilian_activity = 1;};
|
||||
case 3: {GRLIB_civilian_activity = 2;};
|
||||
default {GRLIB_csat_aggressivity = 1;}; // BUG: should be GRLIB_civilian_activity
|
||||
};
|
||||
```
|
||||
|
||||
This is a copy-paste error from the `GRLIB_csat_aggressivity` block above (lines 144-151). If the mission param `Civilians` receives an out-of-range value, enemy aggressivity gets silently overwritten instead of defaulting civilian activity.
|
||||
|
||||
Present in the upstream KP Liberation repository at the same location. Not introduced by this mission's setup.
|
||||
Reference in New Issue
Block a user