farrainbow

Use one handler for all map_property packets

3

Also move to Receive.pm and tweak unpack strings a bit.

Move inventory-related Actor::Item methods to a new class based on Actor::Item

4

`Actor::InventoryItem` (to keep the tree simple) or `Actor::Item::InventoryItem`.

Assertion (invIndex must be defined) failed, after running tests

3

Half of the time tests will pass, the other half of the time I get this error: ``` 1..1043 (in cleanup) Assertion (invIndex must be defined) failed! at /home/rachel/Projects/OpenKore/openkore/src/test/../deps/Carp/Assert.pm line...

Gather statistics about which packets are actually used by which server

1

Information to collect: - current date (rounded to the last midnight UTC) - server name - serverType - packet direction: server → client **or** real client (XKore) → server (no...

Remove unused server types 1 - 22

2

Rename type 7 appropriately if still relevant (`servers.txt`). Also remove checks like `$masterServer->{serverType} ==`. Probably use the branches where it's equal to 0.

Implement a hook point for when we're in game and fully received introductory information

2

Some parts for AI (invetory and cart-related, for example) may not have a solid point where all the information is received. For example, if you have no stackable items, the...

Improve talk and talknpc commands

6

- [ ] compatibility with old syntax is a must - [ ] `talk ` - start a conversation (current syntax) - [ ] `talk /regexp/` - start a conversation...

Plan how to refactor more things into Actor objects and ActorLists

2

This is for discussion and planning of possible naming and ways to set up everything. Propose everything there. ## vending shops around ## buying stores around ## chatrooms around ##...

Use `prove` to run tests

1

More in line with how Perl stuff is usually tested. Also harness has better output options and different formats. - [ ] change test files to be self-runnable instead of...

Task::Teleport to replace Misc::useTeleport internals and current AI logic

4

`Task::Teleport` and maybe separate `Task::Respawn` interface. It should wait for map change to occur, possibly retry and fail otherwise. Needed for other AI tasks.