View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0005074 | Spring engine | Lua | public | 2016-02-09 06:39 | 2016-02-09 12:04 |
| Reporter | Google_Frog | Assigned To | |||
| Priority | normal | Severity | major | Reproducibility | always |
| Status | resolved | Resolution | fixed | ||
| Product Version | 100.0+git | ||||
| Summary | 0005074: Spring.IsPosIn{Los, Radar, AirLos} and Spring.GetPositionLosState now require allyTeamID | ||||
| Description | These four callouts now crash if called from LuaUI without an allyTeamID argument. In 100.0 the callout used the allyTeamID of the player running the widget. I assume the same held for unsynced luarules. In the dev engine these callouts now cause an error such as this one: [f=0020640] [cawidgets.lua] Error: Error in Update(): [string "LuaUI/Widgets/gui_defenserange.lua"]:755: bad argument 0000004 to 'spGetPositionLosState' (Invalid allyTeam) I think that the 4th argument should go back to being optional. Otherwise a lot of code will need to be updated. | ||||
| Tags | No tags attached. | ||||
| Checked infolog.txt for Errors | |||||
|
|
I made a temporary change so I can continue testing other things. https://github.com/ZeroK-RTS/Zero-K/commit/fd09055443c5b2a15bfbd72d3d8f155dc6689aca |
|
|
Fix 3f33ff37db15c656173dfd6c9c88a3a8d7a27781 committed to develop branch: fix 0005074, repo: spring changeset id: 6584 |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2016-02-09 06:39 | Google_Frog | New Issue | |
| 2016-02-09 07:02 | Google_Frog | Note Added: 0015705 | |
| 2016-02-09 12:04 | Changeset attached | => spring develop 3f33ff37 | |
| 2016-02-09 12:04 | Anonymous | Note Added: 0015706 | |
| 2016-02-09 12:04 | Anonymous | Status | new => resolved |
| 2016-02-09 12:04 | Anonymous | Resolution | open => fixed |
