elwinar/adeptus

Command-line character sheet tracking tool for the Dark Heresy system.

adeptus

Command-line character sheet tracking tool for the Dark Heresy system.

adeptus's Issues

Panic on suggest method

panic: runtime error: index out of range

goroutine 1 [running]:
main.Character.Suggest(0xc8200e5228, 0x7, 0xc8200f2cf0, 0xc8200f2d20, 0xc8200f2d50, 0xc8200f2d80, 0xc8200f2db0, 0xc8200f2de0, 0xc8200f2e10, 0x1194, ...)
        /home/moxar/Dev/go/src/github.com/elwinar/adeptus/character.go:545 +0x24fc
main.main.func3(0xc8200982d0)
        /home/moxar/Dev/go/src/github.com/elwinar/adeptus/main.go:76 +0xb2
github.com/codegangsta/cli.Command.Run(0x5f1850, 0x7, 0x0, 0x0, 0x0, 0x0, 0x0, 0x6344c0, 0x39, 0x0, ...)
        /home/moxar/Dev/go/src/github.com/codegangsta/cli/command.go:127 +0x1052
github.com/codegangsta/cli.(*App).Run(0xc820096000, 0xc82000a2c0, 0x4, 0x4, 0x0, 0x0)
        /home/moxar/Dev/go/src/github.com/codegangsta/cli/app.go:159 +0xc2f
main.main()
        /home/moxar/Dev/go/src/github.com/elwinar/adeptus/main.go:81 +0x780

Display aptitudes in Print

Add characteristics block after header

Any Sheet is made of a header, a first characteristic block and a group of sessions.

The second block has to be a characteristic block. It must not check the characteristic validity (related to the Universe).

Spell management

History command

We need a command that displays the history of the character:

Each of his upgrades, on a diffrerent line, with it's real cost (computed or not)

Gauge support

Add the gauges support in the character handling

Display on windows

adeptus doesn't display well on windows os's cmd line.

Change of the mark specification

Today, 3 marks exist:

• The default mark * is used when an upgrade is regularly purchased. Its default cost is calculated by the system.
• The Special mark + is used when an upgrade is purchased without affecting future upgrade's costs. For instance, when a character gains or loses permanent characteristic. The cost is mandatory as the system cannot guess it.
• The Free mark - is used when an upgrade is purchased without affecting future upgrade's cost and is free. It's an alias of:

    + My Upgrade [0]

We require to possibility to remove an upgrade from a character. The Free Mark must change. These are the new specs:

• The Default mark * is used when an upgrade is regularly purchased. Its default cost is calculated by the system.
• The Special mark + is used when an upgrade is purchased without affecting future upgrade's costs. Its default value cost is 0.
• The Free Mark - becomes the Loss mark. The upgrade is lost. Its default cost is 0.

suggest command

Rework the errors

We need two types of errors:

• CharacterError, an error that occures when the character sheet or an upgrade is not valid. It is identified by an ErrorCode and holds a line number.
• UniverseError, an error that occures when the universe is not valid. It is absolutely not related to a character sheet. It is identified by an ErrorCode.

Enhance output

The current output is ugly. Improve it.

Refactor print

Check Universe CMD

We need a command that checks the universe.

Universe manager

An interface responsible for making or editting universe would be great.

Test are broken

The culprit is commit 6a56922, dixit git-bisect.
You commit, your problem. :P

Options management

At the character creation, options may be given to the player.
For instance, it can choose Talent A or Talent B. An option system must be provided.

Proposition, assuming the background administrator proposes "Perfect Knowledge" or "INT +5":

name: Sudo
Background: administrator (INT +5)
....

Incorrect error format

message:

unable to create character: line %!d(string=carrier): the background %!s(MISSING) is not defined by the universe

Character sheet:

name:       Paola Piazzini
origin:     Tilea
carrier:    Cleric
role:       Warrior
tarot:      XVIII

Flag universe

The universe file should be a global string flag, as it will be used by all commands.

talents and skill cases

The talents and skills should not be case sensitive.

Propose default cost to gauges

Refactor ApplyUpgrade method

The apply upgrade method is UGLY!

Refactor it.