Mercurial > repos > IBDev-IBBoard.WarFoundry.API
comparison api/Objects/UnitEquipmentItem.cs @ 138:c11c0da01bbc
Fixes #147: "Replace Weapon" button doesn't always enable (regression)
r167 introduced the bug by triggering an "add item to unit type" call before the mutex group was set
* Added optional mutex group parameter to constructor
* Remove setter on mutex group
author | IBBoard <dev@ibboard.co.uk> |
---|---|
date | Sun, 13 Sep 2009 18:00:52 +0000 |
parents | 2f3cafb69799 |
children | 0c0e14f03785 |
comparison
equal
deleted
inserted
replaced
137:f58051572ec7 | 138:c11c0da01bbc |
---|---|
23 private double costMultiplier; | 23 private double costMultiplier; |
24 private RoundType roundType; | 24 private RoundType roundType; |
25 private string mutexGroup; | 25 private string mutexGroup; |
26 private UnitType unitType; | 26 private UnitType unitType; |
27 | 27 |
28 public UnitEquipmentItem(EquipmentItem equipmentItem, UnitType equipmentFor) | 28 public UnitEquipmentItem(EquipmentItem equipmentItem, UnitType equipmentFor) : this(equipmentItem, equipmentFor, "") |
29 { | |
30 //Do nothing extra | |
31 } | |
32 | |
33 public UnitEquipmentItem(EquipmentItem equipmentItem, UnitType equipmentFor, string mutexGroup) | |
29 { | 34 { |
30 item = equipmentItem; | 35 item = equipmentItem; |
31 unitType = equipmentFor; | 36 unitType = equipmentFor; |
37 this.mutexGroup = mutexGroup; | |
32 unitType.AddEquipmentItem(this); | 38 unitType.AddEquipmentItem(this); |
33 } | 39 } |
34 | 40 |
35 public override string Name | 41 public override string Name |
36 { | 42 { |
100 } | 106 } |
101 | 107 |
102 public string MutexGroup | 108 public string MutexGroup |
103 { | 109 { |
104 get { return mutexGroup; } | 110 get { return mutexGroup; } |
105 set { mutexGroup = (value == null ? "" : value.Trim()); } | |
106 } | 111 } |
107 | 112 |
108 public UnitType EquipmentForUnit | 113 public UnitType EquipmentForUnit |
109 { | 114 { |
110 get { return unitType; } | 115 get { return unitType; } |