View Issue Details
|ID||Project||Category||View Status||Date Submitted||Last Update|
|0000385||DarkRadiant||Map Editing||public||21.10.2007 16:23||27.06.2008 23:10|
|Target Version||0.9.7||Fixed in Version||0.9.7|
|Summary||0000385: Don't increment target fields on duplicated entities|
|Description||Create an AI, and give him a targetted patrol point (e.g., path_corner_1).|
Now, duplicate that AI.
The new AI has a "target0" of "path_corner_2".
This target may not necessarily exist. Some obvious problems from this:
-console warnings? Possibly more sinister effects (CPU eating?) from not being able to find the target
-if the target DOES exist, the AI may now be unintentionally targetting the wrong point
-if a new point is created, and then the user targets the AI to that point, that AI now has two targets, target0 (the bad one) and target1 (the correct one).
I haven't explored all possibilities of course, but I think a safe fix might be simply not to increment "target" fields on duplicates. Thoughts?
|Tags||No tags attached.|
|I think not incrementing the target field would be ok, although I don't know if the incrementing is easy to prevent (it's handled very generally by looking for all kinds of "incrementable" spawnargs like name, bind, etc.).|
|Looks good. There could be an argument for not including target fields at all on duplicates... but one could argue the other way (how it is now) just as easily. The user need only delete the target, if they prefer duplicating things instead of creating.|
|21.10.2007 16:23||SneaksieDave||New Issue|
|21.10.2007 18:32||greebo||Note Added: 0000795|
|21.10.2007 18:32||greebo||Status||new => acknowledged|
|11.11.2007 12:46||greebo||Relationship added||has duplicate 0000431|
|27.06.2008 21:47||greebo||Relationship added||child of 0000635|
|27.06.2008 21:47||greebo||Status||acknowledged => assigned|
|27.06.2008 21:47||greebo||Assigned To||=> greebo|
|27.06.2008 21:48||greebo||Status||assigned => resolved|
|27.06.2008 21:48||greebo||Fixed in Version||=> 0.9.7|
|27.06.2008 21:48||greebo||Resolution||open => fixed|
|27.06.2008 21:48||greebo||Projection||none => redesign|
|27.06.2008 21:48||greebo||ETA||none => < 1 month|
|27.06.2008 21:48||greebo||Build||=> 3636|
|27.06.2008 21:48||greebo||Target Version||=> 0.9.7|
|27.06.2008 23:10||SneaksieDave||Status||resolved => closed|
|27.06.2008 23:10||SneaksieDave||Note Added: 0001381|