Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Consolidate inventory selectInventoryItem and dropInventoryItem into one swap command #148

Closed
DaveyBiggers opened this issue Jul 7, 2016 · 1 comment
Closed

Consolidate inventory selectInventoryItem and dropInventoryItem into one swap command #148

DaveyBiggers opened this issue Jul 7, 2016 · 1 comment
Assignees
@DaveyBiggers
Labels
P3
Milestone
Camel

Comments

@DaveyBiggers
Copy link
Member

At the moment, to swap items in slots 12 and 13, say, the agent needs to do this:

agent_host.sendCommand("selectInventoryItem 12")
agent_host.sendCommand("dropInventoryItem 13")

This dates from a dark time when all commands had to be of the form<verb> <parameter>.
It's time to replace this with a single "swapInventoryItems 12 13" call.
@DaveyBiggers DaveyBiggers self-assigned this Jul 7, 2016
@timhutton timhutton added this to the Camel milestone Jul 8, 2016
@timhutton timhutton added the P3 label Jul 8, 2016
@DaveyBiggers DaveyBiggers mentioned this issue Jul 14, 2016
Merged
@DaveyBiggers
Copy link
Member Author

Fixed in #190.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@DaveyBiggers DaveyBiggers

Labels
P3
Projects
None yet
Milestone
Camel
Development

No branches or pull requests
2 participants
@timhutton @DaveyBiggers