search

bravoserver / bravo

UNMAINTAINED: Reverse-engineered Minecraft stuff. Worked with 1.4.x protocol
http://bravoserver.org/
Other
206 stars 45 forks source link

Improve support for chests #256

Closed squiddy closed 13 years ago

squiddy commented 13 years ago

Right now, chest are loaded from / saved to disk and can be placed. For 1.6 I want (I'll write code for it) players to be able to destroy chests (with its contents spawned as pickups), interact with their inventory and build large chests.

MostAwesomeDude commented 13 years ago

Adding to 1.6. I'm totally in favor of this.

gtaylor commented 13 years ago

Squiddy, have you started on this yet?

squiddy commented 13 years ago

@gtaylor: Yeah, I started. Unfortunately I've been quite busy and didn't get around to continue the work. I had the feeling you wanted to look at it too, am I right?

Sorry for my absence.

gtaylor commented 13 years ago

@squiddy: I poked at it, but couldn't figure out the slots argument to the make-window packet. I'm not exactly sure what it is used for, the the protocol docs on MC Coalition's wiki only serve to confuse further. I imagine I've git a knowledge gap about how the protocol fits together as a whole, rather than this being overly complicated.

mjevans commented 13 years ago

If chests used to work before, now they don't. :(

gtaylor commented 13 years ago

@mjevans I've never seen them working before, don't think they were ever implemented.

MostAwesomeDude commented 13 years ago

Recent work on inventories may have closed this bug. Adding to the latest milestone.

mcsakoff commented 13 years ago

Can be closed as implemented.

MostAwesomeDude commented 13 years ago

Closing in the hope that the recent inventory patches are sufficient for this.