mirrors/0ad
1
0
Fork 2
mirror of https://gitea.wildfiregames.com/0ad/0ad.git synced 2026-09-21 20:06:40 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix crossed unit-to-target pairing

Browse Source 
sortEntitiesForEngagement projects both attackers and targets from
the same origin (avgAttackers) onto the same vector,
so reversing the target array was incorrect.
This commit is contained in:
Atrik
2026-09-11 10:07:23 +02:00
committed by Vantha
parent c3ace13b54
commit 191549e51c
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
binaries/data/mods/public/gui/session/input.js
+3 -3
View File
@@ -2045,7 +2045,8 @@ function distributeAttackOrders(attackers, targets)
* @param {number[]} attackers - Array of attacking entity IDs.
* @param {number[]} targets - Array of target entity IDs.
* @returns {Object} { attackers: number[], targets: number[] } - Both arrays sorted
* for cross-line engagement (targets automatically reversed).
* along the same shared axis so corresponding indices pair up
* for realistic line engagement.
*/
function sortEntitiesForEngagement(attackers, targets)
{
@@ -2066,10 +2067,9 @@ function sortEntitiesForEngagement(attackers, targets)
const sortedAttackers = sortEntitiesAlongLine(attackers, avgAttackers, avgTargets);
const sortedTargets = sortEntitiesAlongLine(targets, avgAttackers, avgTargets);
// Reverse targets so the leftmost attacker pairs with leftmost target
return {
"attackers": sortedAttackers,
"targets": sortedTargets.reverse()
"targets": sortedTargets
};
}