#153 closed enhancement (released)
Default name for armies
Reported by: | ibboard | Owned by: | Henri |
---|---|---|---|
Priority: | minor | Milestone: | WarFoundry 0.1 |
Component: | WarFoundry-GUIs | Version: | Unknown or N/A |
Keywords: | army name default papercut | Cc: | |
Blocked By: | Blocking: |
Description
Apparently users like Snowblizz may feel lazy/unimaginative and want the app to name their army for them. This would be a GUI change rather than an API change.
Change History (16)
comment:1 Changed 10 years ago by
comment:2 Changed 10 years ago by
Status: | new → confirmed |
---|
Move tickets to new "confirmed" status to show that they've been checked
comment:3 Changed 10 years ago by
Milestone: | → WarFoundry 0.2 |
---|
comment:4 Changed 10 years ago by
Keywords: | papercut added |
---|---|
released: | → no |
Version: | → Unknown or N/A |
comment:5 Changed 9 years ago by
Owner: | set to Henri |
---|---|
Status: | confirmed → accepted |
comment:6 Changed 9 years ago by
I've been looking at this since you note it is a GUI thing and I've managed to get a simple version working. Ie just taking the race name + army, and getting the point value.
I am however curious, if I wanted to use a "name" pulled from the datafile would there be any more involved changes, I can figure out so much that the schema needs to include the new attribute, but it does also require changes to the Warfoundry.API, yes? I've managed to track down the Warfoundry.Objects.Races but I can quite fathom how it all fits together.
comment:7 Changed 9 years ago by
To pull a basic name you'll need to add a few bits to the API:
1) Add a property to the Race object ("DefaultArmyName" or something sensible) with a setter, a getter and a variable behind it
2) Add a definition to the schema (an attribute on the <race> tag is probably most appropriate)
3) Add the loading of the value to the WarFoundryXmlRaceFactory - we can have multiple factories, but XML is the main one for now and it just parses the XML file and creates the necessary objects.
4) Add some tests in the IBBoard.WarFoundry.API.Tests project to make sure that it validates and loads correctly, and fails if you give it an invalid value.
As a future extension, we could make this fall back to the game system if nothing is defined at the race level and let the race name be used (e.g. "{0} horde" would become "Dwarf Horde" "Human Horde" etc).
If you want any more suggestions and pointers, just start a topic on the forums :)
comment:8 Changed 9 years ago by
comment:9 Changed 9 years ago by
comment:10 Changed 9 years ago by
comment:11 Changed 9 years ago by
comment:12 Changed 9 years ago by
comment:13 Changed 9 years ago by
comment:14 Changed 9 years ago by
Resolution: | → fixed |
---|---|
Status: | accepted → closed |
comment:16 Changed 9 years ago by
Milestone: | WarFoundry 0.2 → WarFoundry 0.1 |
---|
Snowblizz's suggestion is to use "[Race name] Army ([points value])" or to extend races to define what a default army size is (e.g. "Host" for Elves, "Throng" for Dwarves, etc).