issues: 218616162
This data as json
id | node_id | number | title | user | state | locked | assignee | milestone | comments | created_at | updated_at | closed_at | author_association | active_lock_reason | draft | pull_request | body | reactions | performed_via_github_app | state_reason | repo | type |
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
218616162 | MDExOlB1bGxSZXF1ZXN0MTEzNzMyMDEw | 189 | Inventory Phase 1 and Drop Item | 25460553 | closed | 0 | 2 | 2017-03-31T21:01:27Z | 2017-05-01T16:41:56Z | 2017-04-03T03:00:33Z | CONTRIBUTOR | 0 | ACEmulator/ACE/pulls/189 | This is at a good place for code review and possible merge. Once approved, this will allow others to start on the numerous tasks around inventory management. You can spawn a Training wand in your backpack or on the ground. That is just throw away code to test with. It is in the debug handler. usage: @ctw me OR @ctw ground Once the item is spawned, you can then drop it on the ground. Included in this PR. Starts to address Issue #144 World object now has inventory dictionary. Added this here as anything that can be a container needs to have this ability. Not all world objects are containers - so also added property IsContainer Added methods to: - Add to inventory - Remove from inventory - Lookup item in inventory - Handle Drop item: 1. Calls lookup 2. Update Container burden 3. Sets some properties in preparation for going 3D 4. GameMessageUpdatePosition. 5. Alerts Landblock to new arrival 6. Removes the item the player's inventory Added GameActionDropItem This handles the client's request to drop an item. Added GameMessagePutObjectIn3d As usual - any coding faux pas are not by design. Feedback is wanted and welcome. If there is a better way to do what I did - please let me know. | {"url": "https://api.github.com/repos/ACEmulator/ACE/issues/189/reactions", "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0} | 79078680 | pull |