2025-07-27 16:17 CEST
  

View Issue Details [ Jump to Notes ] [ Issue History ] [ Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0000612Spring engineGeneralpublic2007-08-28 11:312013-01-20 14:35
ReporterKDR_11k 
Assigned ToKloot 
PrioritynormalSeverityminorReproducibilityalways
StatusresolvedResolutionfixed 
Product Version 
Target VersionFixed in Version 
Summary0000612: Weapons target transported units
DescriptionA weapon that cannot target the transport because of onlytargetcategory or whatever can still target the loaded units and thus kill the transport.

Reproduction:
Using Gundam, spawn a dodai_2 and a xamel on opposing teams. Notice that the xamel makes no attempt to attack the dodai_2. Now make the dodai_2 load a ground unit, e.g. a zaku and fly it to the xamel. The xamel will suddently shoot at the combo even though it cannot shoot at the dodai_2.
TagsNo tags attached.
Checked infolog.txt for Errors
Attached Files
  • patch file icon dont_target_transported.patch (394 bytes) 2007-10-10 16:13 - 
    Index: Weapon.cpp
===================================================================
--- Weapon.cpp	(revision 4535)
+++ Weapon.cpp	(working copy)
@@ -639,7 +639,9 @@
 	if (weaponDef->stockpile && !numStockpiled) {
 		return false;
 	}
-
+	if(unit && unit->toBeTransported) {
+		return false;
+	}
 	float3 dif=pos-weaponMuzzlePos;
 	float heightDiff; // negative when target below owner
    patch file icon dont_target_transported.patch (394 bytes) 2007-10-10 16:13 +

-Relationships
+Relationships

-Notes

~0001335

tombom (reporter)

2007-10-10 15:35

 Should transported units ever be targeted?

~0001383

FLOZi (reporter)

2007-10-30 21:46

 I was about to say 'perhaps if using isfirebase', but on second thoughts, I think a seperate tag would be preferable.

~0001384

trepan (reporter)

2007-10-31 03:00

 Why 'toBeTransported' instead of 'transporter' in the patch?

~0001390

tombom (reporter)

2007-10-31 12:26

 ignore the patch it probably doesn't work at all

~0009621

Kloot (developer)

2013-01-20 14:35

 modrule "targetableTransportedUnits"
+Notes

-Issue History
Date Modified Username Field Change
2007-08-28 11:31 KDR_11k New Issue
2007-10-10 15:35 tombom Note Added: 0001335
2007-10-10 16:13 tombom File Added: dont_target_transported.patch
2007-10-30 21:46 FLOZi Note Added: 0001383
2007-10-31 03:00 trepan Note Added: 0001384
2007-10-31 12:26 tombom Note Added: 0001390
2013-01-20 14:35 Kloot Note Added: 0009621
2013-01-20 14:35 Kloot Status new => resolved
2013-01-20 14:35 Kloot Resolution open => fixed
2013-01-20 14:35 Kloot Assigned To => Kloot
+Issue History