Opened 11 years ago

Closed 9 years ago

#33 closed enhancement (fixed)

Add centralised requirement checker

Reported by: ibboard Owned by: ibboard
Priority: major Milestone: WarFoundry API 0.2
Component: WarFoundry-API Version: Unknown or N/A
Keywords: Cc:
Blocked By: Blocking: #35, #36, #104

Description (last modified by ibboard)

For improved plugability, requirements should be checked through a centralised "requirement handler". This would allow better handling of Warhammer 40K's roster-style or Warhammer's numbers based on army size restrictions through plugins by giving a central point to add the requirements to. It would also separate out "one blob, one job" functionality by removing functions like CanAddToArmy from classes like UnitType.

Change History (11)

comment:1 Changed 11 years ago by ibboard

Blocking: 35 added

comment:2 Changed 11 years ago by ibboard

Blocking: 36 added

comment:3 Changed 11 years ago by ibboard

Owner: ibboard deleted

Remove owner so that the team can more easily see bugs. Using default owners seems good at first but is bad practice when a team can be working on any part of the project.

comment:4 Changed 10 years ago by ibboard

Blocking: 104 added

comment:5 Changed 10 years ago by ibboard

Milestone: WarFoundry 0.1WarFoundry 0.2

Move some tickets to v0.2 milestone

comment:6 Changed 10 years ago by ibboard

Status: newconfirmed

Move tickets to new "confirmed" status to show that they've been checked

comment:7 Changed 9 years ago by ibboard

Milestone: WarFoundry 0.2WarFoundry API 0.2

Separate API milestone from UI milestone

comment:8 Changed 9 years ago by ibboard

Description: modified (diff)
Summary: Add centralised requirement handlerAdd centralised requirement checker
Version: Unknown or N/A

comment:9 Changed 9 years ago by ibboard

In [898]:

Re #33: Centralised requirement handling

  • Use central requirement handler to check each selection

comment:10 Changed 9 years ago by ibboard

In [926]:

Re #33: Add centralised requirement checker

  • Add test for getting "allows adding" failure messages from handler

comment:11 Changed 9 years ago by ibboard

Owner: set to ibboard
Resolution: fixed
Status: confirmedclosed

In [927]:

Fixes #33: Add centralised requirement checker

  • Implement validation with messages for adding

(Other features added under #27 and not separately marked)

Note: See TracTickets for help on using tickets.