clay with inventory | SpigotMC | Page 1

steady fractal Oct 23, 2021, 10:42 PM

#

@wintry heath and @sick cairn

wintry heath Oct 23, 2021, 10:43 PM

#

You cant assign a TileState to a random Block. It has to be a TileState as well.

steady fractal Oct 23, 2021, 10:43 PM

#

hm

sick cairn Oct 23, 2021, 10:43 PM

#

why are you even using reflection?

#

can you pls explain what's your goal?

steady fractal Oct 23, 2021, 10:44 PM

#

to override the block

sick cairn Oct 23, 2021, 10:44 PM

#

that's a very bad idea

steady fractal Oct 23, 2021, 10:44 PM

#

my goal is to add inventory to any block

#

allowing them to receive items from hopper

sick cairn Oct 23, 2021, 10:44 PM

#

oh yeah well

steady fractal Oct 23, 2021, 10:44 PM

#

1.12

wintry heath Oct 23, 2021, 10:44 PM

#

steady fractal my goal is to add inventory to any block

This has to be handled externally. There is no way to implement that natively.

sick cairn Oct 23, 2021, 10:44 PM

#

oh shit 1.12 I'm out lol, sorry

steady fractal Oct 23, 2021, 10:44 PM

#

:c

sick cairn Oct 23, 2021, 10:45 PM

#

my knowledge about pre 1.13 is very limited

wintry heath Oct 23, 2021, 10:45 PM

#

Oh then the idea i just cooked up in my head wont work too...

steady fractal Oct 23, 2021, 10:45 PM

#

say it

wintry heath Oct 23, 2021, 10:45 PM

#

PDC of Chunks

steady fractal Oct 23, 2021, 10:45 PM

#

sad

#

:C

sick cairn Oct 23, 2021, 10:46 PM

#

well you cannot simply register new block / tileentity classes and expect them to work

#

that's not what the API was designed for

wintry heath Oct 23, 2021, 10:46 PM

#

You need to completely emulate all the properties of a BlockInventoryHolder TileState

sick cairn Oct 23, 2021, 10:46 PM

#

wintry heath PDC of Chunks

only exists on 1.16.3+

wintry heath Oct 23, 2021, 10:46 PM

#

jep

sick cairn Oct 23, 2021, 10:47 PM

#

yeah but they are using 1.12.2^^

#

sooo

#

let me rephrase

#

you want hoppers be able to put stuff into your custom block's inventories

#

right?

steady fractal Oct 23, 2021, 10:48 PM

#

yes

sick cairn Oct 23, 2021, 10:48 PM

#

if so: there's NO OTHER WAY than checking inventory/hopper events and moving those items manually

wintry heath Oct 23, 2021, 10:48 PM

#

Events arent fired

steady fractal Oct 23, 2021, 10:48 PM

#

^

sick cairn Oct 23, 2021, 10:48 PM

#

well of course events are fired

steady fractal Oct 23, 2021, 10:48 PM

#

no

sick cairn Oct 23, 2021, 10:48 PM

#

events for items ENTERING the hopper are

steady fractal Oct 23, 2021, 10:48 PM

#

yes

wintry heath Oct 23, 2021, 10:49 PM

#

He has to get all TileStates when a chunk loads, filter the hopper states out and manually tick them depending on their orientation.

sick cairn Oct 23, 2021, 10:49 PM

#

and then you have to the rest manually

#

I mean what do you expect? of course spigot won't call an event when a hopper doesnt actually attempt to move stuff from a hopper to your custom dirt block^^

#

you will have to check when items ENTER your hopper and then do the rest on your own

#

there DEFINITELY is no other way, sorry

steady fractal Oct 23, 2021, 10:50 PM

#

:C

#

i was using some breakpoints

#

and i saw that the event was fired in my custom block

sick cairn Oct 23, 2021, 10:52 PM

#

I made a hopper filtering plugin a few weeks ago ( https://www.spigotmc.org/resources/filtered-hoppers.96037/ ) so I know how hoppers are working in 1.17, aaaaaand: in 1.12.2, the API will definitely not have more methods, soooo I can garantuee: no way! There are no events for hoppers checking blockstates that don't extend/implement container :/

steady fractal Oct 23, 2021, 10:52 PM

#

my block is a container :D

sick cairn Oct 23, 2021, 10:52 PM

#

steady fractal and i saw that the event was fired in my custom block

you just CANNOT create new blocks

steady fractal Oct 23, 2021, 10:53 PM

#

override

sick cairn Oct 23, 2021, 10:53 PM

#

yes but hear me out

#

of course you can create a new class that extends/implements container

#

but never ever will there exist any REAL block in your world that actually has your custom block as its blockstate

#

the API isn't designed to do that

#

that's why MATERIAL is an enum

steady fractal Oct 23, 2021, 10:54 PM

#

im not using the api :D

#

nms

#

i can place the block

#

i can override the methods

sick cairn Oct 23, 2021, 10:55 PM

#

believe me or not, you will not get it to work in this way

steady fractal Oct 23, 2021, 10:55 PM

#

the only problem here is that the hopper is not transfering items

sick cairn Oct 23, 2021, 10:55 PM

#

of course it doesn't

#

because

#

vanilla MC works like this:

steady fractal Oct 23, 2021, 10:55 PM

#

wintry heath Oct 23, 2021, 10:56 PM

#

steady fractal i can override the methods

You are currently changing the bahavior of all clay type Blocks. Not just certain ones.

sick cairn Oct 23, 2021, 10:56 PM

#

Is the block next to me a hopper? If yes, transfer it. If no, don't. And your block is NOT a hopper, it's just something else

steady fractal Oct 23, 2021, 10:56 PM

#

ik

#

i want something that works now

sick cairn Oct 23, 2021, 10:57 PM

#

I'm doing spigot development for more than 3 years now and have sold quite a lot of premium plugins until today. In my opinion, what you are looking for is simply not possible. Of course you can recreate the behaviour you're looking for, but you cannot just do it the way you're currently trying to use. Sure, I might be wrong, but tbh I am very very sure that I aint.

#

Sooo to rephrase:

#

you have to rewrite all the hopper mechanics

#

you cannot simply trick the game into thinking your custom block is a container

#

it just will never work

wintry heath Oct 23, 2021, 10:58 PM

#

Its possible but he will have to use a LOT of workarounds.
The hardest problem will be serialization.

sick cairn Oct 23, 2021, 10:59 PM

#

noo nooo

#

well

#

I never looked at the source code that actually saves the world data

#

anyway, let's just agree on the fact that listening to the event that tells you when items ENTER the hopper and then doing the remaining stuff manually is the only viable solution^^

#

ooooor

#

you could fork spigot, create a PR that adds the new events you need, and ask md_5 to merge it. But he probably will never do because hoppers already are one of the largest lag machines that exist in spigot

steady fractal Oct 24, 2021, 12:19 AM

#

cool

#

the messages i said that were sent to console

#clay with inventory