Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#127 closed defect (released)

Unit creation always assigns default equipment

Reported by: ibboard Owned by: ibboard
Priority: major Milestone: WarFoundry 0.1
Component: WarFoundry-API Version:
Keywords: army load unit defaultequipment Cc:
Blocked By: Blocking:

Description

While working on #54, unit equipment selection doesn't change anything. This is because when a Unit is created it auto-populates itself with default equipment. This is useful for normal "create new unit" situations, but means that "load unit with existing data" situations can end up with equipment that they don't want/need.

Loading default equipment should only be done when we know that it is a new unit because created that needs default equipment.

Change History (7)

comment:1 Changed 10 years ago by ibboard

Owner: set to ibboard
Status: newaccepted

comment:2 Changed 10 years ago by ibboard

(In r216) Re #127: Unit creation always assigns default equipment

  • Add and use new constructor for Unit that doesn't assign default values

Re #54: Add Army support to WarFoundryFactory

  • Load unit name and unit size

comment:3 Changed 10 years ago by ibboard

r216 appears to fix the problem.

comment:4 Changed 10 years ago by ibboard

Resolution: fixed
Status: acceptedclosed

comment:5 Changed 10 years ago by ibboard

released: 1

Mark released changes as released

comment:6 Changed 10 years ago by ibboard

released: 1yes

Mark released fixes as released using radio values

comment:7 Changed 10 years ago by ibboard

Resolution: fixedreleased

Mark fix as released under a previous version

Note: See TracTickets for help on using tickets.