View Issue Details [ Jump to Notes ] | [ Issue History ] [ Print ] | ||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||
---|---|---|---|---|---|---|---|---|---|
0003539 | Spring engine | (No Category) | public | 2013-03-08 15:48 | 2013-03-08 19:23 | ||||
Reporter | Jools | ||||||||
Assigned To | Kloot | ||||||||
Priority | normal | Severity | minor | Reproducibility | always | ||||
Status | closed | Resolution | no change required | ||||||
Product Version | 93.0 | ||||||||
Target Version | Fixed in Version | ||||||||
Summary | 0003539: Line attack not working on water | ||||||||
Description | When using line attack, vlaunch units cannot target water. Instead of firing, they will move towards the target. | ||||||||
Steps To Reproduce | Make two merls, use line attack, choose target on water. Line attack command is given by: select units, press A, hold alt, draw attack line with right mouse button pressed. | ||||||||
Additional Information | Version: 93.0 (OMP) | ||||||||
Tags | No tags attached. | ||||||||
Checked infolog.txt for Errors | |||||||||
Attached Files |
|
Notes | |
Jools (reporter) 2013-03-08 15:52 |
Correction: Line attack command is given by: select units, press A, hold alt, draw attack line with *left* mouse button pressed. |
cleanrock (reporter) 2013-03-08 16:39 |
Its not line attack specific. Bug is present if u mark multiple merls and attack a point on sea. Attacking sea point with only one merl works fine. Attacking land point with multiple merl works fine. |
Kloot (developer) 2013-03-08 16:45 Last edited: 2013-03-08 16:45 |
attacking sea point with multiple merl also works fine (XTA, not BA) |
Jools (reporter) 2013-03-08 17:00 |
True, attacking multiple sea points with normal attack works in xta 9,725 p1.7 and svn. But line attack doesn't work. In BA 7.72, both line attack and normal (direct) attack doesn't work if you have multiple ranged units. |
cleanrock (reporter) 2013-03-08 17:12 |
kloot, isnt this a bug anyway ? If one BA merl will attack sea shouldnt multiple merls be able to do that ? |
Kloot (developer) 2013-03-08 17:16 |
of course, I was just ruling out possibilities (in the case of BA there is a gadget complicating matters) |
Kloot (developer) 2013-03-08 17:39 Last edited: 2013-03-08 18:22 |
cause of this bug in BA with LuaUI disabled (no line-drawing): gadget:AllowCommand in unit_prevent_range_hax.lua issues attack orders on the ground, which breaks units that cannot shoot into water when a user placed the original order on the water *surface* cause of this bug with LuaUI enabled (line-drawing): unit_customformations.lua uses Spring.GetGroundHeight, which returns the *underwater* height if lines are dragged across water and this leaks through to TryTarget --> see above |
Kloot (developer) 2013-03-08 18:24 |
conclusion: engine code is fine, widgets need updating to deal with non-waterweapon units |
Jools (reporter) 2013-03-08 18:57 |
Only way I can post is to re-open, feel free to just close if this is also included in the resolution: In XTA, with luaui disabled, the bug is still present when using area attack button instead of normal attack order. The one where you draw a white circle. Is the error a similar one also in that button? I always thought the area attack button was part of the engine. |
Jools (reporter) 2013-03-08 19:15 |
Ok, nevermind. There is a gadget called areaattack.lua. It must be the one that breaks the attack like in the ba case. |
Jools (reporter) 2013-03-08 19:18 |
Eh, tried to close it. I can't. |
Kloot (developer) 2013-03-08 19:23 |
Reopen if area-attack also doesn't work with luarules disabled. |
Issue History | |||
Date Modified | Username | Field | Change |
---|---|---|---|
2013-03-08 15:48 | Jools | New Issue | |
2013-03-08 15:48 | Jools | File Added: screen00168.png | |
2013-03-08 15:52 | Jools | Note Added: 0010004 | |
2013-03-08 16:39 | cleanrock | Note Added: 0010005 | |
2013-03-08 16:45 | Kloot | Note Added: 0010006 | |
2013-03-08 16:45 | Kloot | Note Edited: 0010006 | View Revisions |
2013-03-08 17:00 | Jools | Note Added: 0010007 | |
2013-03-08 17:12 | cleanrock | Note Added: 0010008 | |
2013-03-08 17:16 | Kloot | Note Added: 0010009 | |
2013-03-08 17:39 | Kloot | Note Added: 0010010 | |
2013-03-08 18:22 | Kloot | Note Edited: 0010010 | View Revisions |
2013-03-08 18:24 | Kloot | Note Added: 0010011 | |
2013-03-08 18:24 | Kloot | Status | new => closed |
2013-03-08 18:24 | Kloot | Assigned To | => Kloot |
2013-03-08 18:24 | Kloot | Resolution | open => no change required |
2013-03-08 18:57 | Jools | Note Added: 0010012 | |
2013-03-08 18:57 | Jools | Status | closed => feedback |
2013-03-08 18:57 | Jools | Resolution | no change required => reopened |
2013-03-08 19:15 | Jools | Note Added: 0010013 | |
2013-03-08 19:15 | Jools | Status | feedback => assigned |
2013-03-08 19:18 | Jools | Note Added: 0010014 | |
2013-03-08 19:23 | Kloot | Note Added: 0010015 | |
2013-03-08 19:23 | Kloot | Status | assigned => closed |
2013-03-08 19:23 | Kloot | Resolution | reopened => no change required |
2014-01-16 16:06 | abma | Category | Spring MT => (No Category) |