development

[dense galleon]

yeah ill try it later, just cant really program now and was curious to see what you guys thought about it

[dusty frost]

Shift + Z Z

[minor summit]

vim users be typing in wingdings to quit

[dusty frost]

Yeah I mean once you understand the basics of modal editing it's really easy

Just getting there I suppose lol

[ancient chasm]

blasphemy

[river solstice]

nano 🫡

[ancient chasm]

I personally wouldn't do it, as that's a lot of unnecessary packets that are bound to clog the pipeline up. I would assume lower speed connections would be choppier/glitchier with that method

I mean it's a small packet, but it would probably be easier to just move the time back every few ticks

[dense drift]

Does anyone know why jetty would throw this exception out of nowhere? https://pastebin.com/EP4SWhMX
Code: https://pastebin.com/7eKSHxuj

[minor summit]

i mean even if you send it once every second... twice a second, the game already sends many more packets with much more frequency already

[ancient chasm]

yeah, I just feel like adding an extra time update packet to every player every tick isn't the best idea

[minor summit]

well yeah maybe not on every tick

i'd do it once or twice a second

[ancient chasm]

yeah, that was their original idea for it (every tick)

[minor summit]

ah

[dense drift]

<not my code>

        return playerPermissions.computeIfAbsent(id, k -> createPlayerPermissionSet());
        if (playerPermissions.containsKey(id)) {
            return playerPermissions.get(id);
        } else {
            PlayerPermissionSet set = createPlayerPermissionSet();

            playerPermissions.put(id, set);

            return set;
        }```
The first return will replace anything that's below, right?

I love some C-style code 🤤

[minor summit]

that won't even compile lol

[dense drift]

yeah I know smh

[dense drift]

one char variables and obscure code

[minor summit]

it's not like we're the ones maintaining it ¯_(ツ)_/¯

[dense drift]

yeah but still annoying to read

[dense galleon]

So, when a player disconnects from any backend server in my bungeecord network, their data gets saved on the database right

Now, I want to do so that if a player connects to another server, the data does this:

1. Gets saved on the database
2. Is removed from the player's current server
3. Is loaded onto the player's new server

Though, the issue is that I'm afraid a player will manage to connect to the server before the data is saved on the database?

[dusty frost]

I would look at plugins like HuskSync or MPDB. HuskSync deals with this problem by using Redis, as that makes the latency very low. https://github.com/WiIIiam278/HuskSync

[dense drift]

❤️

[dense galleon]

what is a redis server..?

[icy shadow]

A server running the redis db

[dense galleon]

Ah

[icy shadow]

I am very smart

[dusty frost]

lmao

i was gonna say it's basically a cache server that also can do message passing and stuff since it's all in memory

so you can really quickly save data and then read it immediately after

[icy shadow]

I think my description is better

[dusty frost]

ofc

[dense galleon]

I was just thinking that since there is only one way for players to go from server to server, to simply save the player data using a CompletableFuture

And then teleport the player only when the data is saved

[dusty frost]

how are you gonna get that CompletableFuture between servers?

[dense galleon]

i mean i wont

That is just used to trigger an event, and once the event is triggered the player is teleported to the server

[dusty frost]

oh I see, yeah that is not the best idea

the way HuskSync does it is it just puts people in limbo and cancels their interactions and movements until it loads the data

which is generally faster than the time it takes for them to load the new world and everything

so you don't even notice

[dense galleon]

Yeah but I gotta figure out Redis and all that

[dusty frost]

I mean you don't have to

the same principle can apply to anything, it would just take a lot longer with like an actual db

[dense galleon]

Wdym

[dusty frost]

have them wait at the end rather than at the beginning

there's no reason to make them wait for the data to save

[dense galleon]

I mean thing is

The way I teleport players between servers is through a system which, once the player begins the teleportation process, takes 2-3 seconds to teleport them due to the animation playing

Kinda works like a loading screen in a game

[dusty frost]

great

so even more time to read from the db

[dense galleon]

Yeah but what I mean is

[dusty frost]

that shouldn't factor into it though, since admins and people who have /server can bypass that

[dense galleon]

That I can save the data when the player is about to teleport, and then teleport them away once its finished

[dusty frost]

if your db takes 2-3 seconds to save, you have bigger problems

and regardless, you don't need them on the server to save their data

[dense galleon]

nah it doesn't, that waiting time is mostly just to make the teleportation satisfying

[dense galleon]

But I need the data on the database to retrieve their updated data

[dense galleon]

I've already disabled that command and implemented my own

(for players)

[dusty frost]

yeah so wait for it on the other end

i guess this works better with a messaging system like Redis where the data is only there when it is done

[dense galleon]

But that's harder to implement with what I already got compared to having the player wait in their current server

It shouldn't make much of a difference anyway, which side they wait on?

[dusty frost]

I guess it makes less of a difference if you have a weird animation

but on normal servers, it just instantly sends you and then gets your data when you get there

makes it simpler to reason about

[dusty frost]

but anyways, if you just put the player in a limbo state and cancel all their events when they are still loading, you can just wait for the data to be loaded and bam

i suppose you could do the opposite and hold them on the first server until you're 100% sure their data has been saved

but you still have to wait for the data to be loaded anyways, so it makes sense just to do all of the waiting on the destination server

[dense galleon]

I suppose this error means I am not opening/closing connections through HikariCP properly? https://paste.helpch.at/emenenofag.php

The error is pointing at this method:

    public Connection getConnection() {
        try {
            return dataSource.getConnection();
        } catch (SQLException e) {
            e.printStackTrace();
            return null;
        }
    }```

This is my saveAllData() method as well since it's probably what's wrong: https://paste.helpch.at/zukahomabe.java

[river solstice]

Catch exception throw exception

[dense galleon]

Yeah lol idek why I wrote that my bad

But that's not what's causing the issue

[tender thicket]

hey so

I made a provider class that takes a result set and maps all rows and inserts them in order into a list
I need to know
ResultSetProvider.getAllRows(ResultSet set) returns a CompletableFuture and I am supplying the resultset in a CompletableFuture#supplyAsync and returning a ResultSetProvider.
When the ResultSetProvider#getAllRows(ResultSet) is called, the ResultSet is provided through a PreparedStatement that's being initialized in a try-with-resources
Since I am using a completeable future and returning the value with .join(), will I need to be concerned about the result set autoclosing before the provider can map it?
the provider is mapping to a List<Map<String, Object> (might change to set but I actually cannot confirm that we are not storing identical values)

I asked this somewhere else but I'm asking here for a differential

[minor summit]

depends when you're joining, if inside or outside the try-with

usually proper solutions are, either having a callback to close the connection/stmt/rs when things are done, or handing the resources over to the other thread so they are closed by that other thread

with the callback option you could probably get away with

Connection c = ...
PreparedStatement ps = ...
ResultSet rs = ...
return CompletableFuture.supplyAsync(() -> ...)
  .handle((r, t) -> {
      try (c; ps; rs) {
      } catch (SQLException ex) {
        throw new CompletionException(ex); // handle downstream
      }
  });

[stuck hearth]

What I would suggest is

  try (PreparedStatement statement = connection.prepareStatement("SELECT * FROM bans"))
        {
            try (ResultSet r = statement.executeQuery()) {
                return doSomethingTo(r);
            }
        }```
or alternatively 
```java
IntermediaryData data = null;
try (PreparedStatement statement = connection.prepareStatement("SELECT * FROM bans")) {
  try (ResultSet r = statement.executeQuery()) {
    data = loadIntermediaryData(r);
  }
}
doSomethingTo(data);```

[icy shadow]

you sly dog

[stuck hearth]

😮

[minor summit]

at that point you're no longer using futures

[stuck hearth]

Yes

[minor summit]

which, tbf is another way to solve the problem

[icy shadow]

yes which weve established is probably good

[stuck hearth]

Don't look at Bev Ben

[minor summit]

i mean it's not necessarily good

you just have to know how to handle stuff

[minor summit]

either of these approaches will work

[icy shadow]

no advertising please

[stuck hearth]

Arguably I did the opposite

So I should be promoted

[dense galleon]

Bump

[icy shadow]

uh that sounds more like a problem on the database side

or you've misconfigured something

if any connections throw that exception

[dense galleon]

i tried it both with my university database (lol) and my private one and the issue was present in both cases

        // Configure and initialize the HikariDataSource
        HikariConfig config = new HikariConfig();
        config.setJdbcUrl("jdbc:mysql://" + host + ":" + port + "/" + database);
        config.setUsername(username);
        config.setPassword(password);

        // Optional: Set additional HikariCP settings
        config.setMaximumPoolSize(30); // Adjust the pool size as needed

        dataSource = new HikariDataSource(config);``` this is how I configure the HikariConfig

[tender thicket]

@minor summit @icy shadow thanks for the help again; also ideally i didn't really want to be using futures because these would end up being costly operations that could probably block the main thread as data is being collected when players join and leave

[icy shadow]

um

what?

those 2 things seem orthogonal (good word)

[minor summit]

if the operation is expensive, it's gonna be expensive regardless of thread

[icy shadow]

yeah lol

and if it's expensive that's all the more reason to be doing it off the main thread

just to make sure, you are doing the database queries asynchronously, right?

[minor summit]

if it's sync, it's gonna block the current thread
if it's async and you join, it's gonna block the current thread

[tender thicket]

hmmm i want to do this in a way that is non-blocking and asynchronous

and i'm not sure @icy shadow this is a class file that someone else had created and i'm trying to fix.

[minor summit]

joining means blocking until it's done

[tender thicket]

Yeah i get that and I wanted to make sure that the result was returned before we did any operations on the ResultSetProvider

[icy shadow]

if it's not then you have a big problem

[tender thicket]

What does an async sql call look like

so far everything is being done in twr blocks

[icy shadow]

thats not really anything to do with it

[tender thicket]

but i think that's about ti

it

[minor summit]

jdbc is by nature, blocking io, best you can do is throw the whole thing into another thread

[icy shadow]

it would just be called in some future (supplyAsync) or with the bukkit scheduler or whatever

^

[tender thicket]

i dont necessarily see anything anywhere that indicates it's asynchronous

[minor summit]

if you want non-blocking sql, look into something like r2dbc

[tender thicket]

what if i wrapped everything with Mono / Flux

[icy shadow]

lol

[tender thicket]

I've been trying to learn r2dbc but that's not an option for this project as of this moment

Due to restrictions from the parent organization

[icy shadow]

just put it in a future man and forget about it

[minor summit]

throwing it in a mono/flux isn't any better than throwing it in a completablefuture

[icy shadow]

it doesnt need to be fancy

[dusky harness]

btw kinda unrelated to your question but can't there technically be a method to make nearly everything non-blocking?
Since for things that can't return like a stream of bytes (like downloading a file), it's going to be blocking a thread, anyways, right?
So just make a method that starts a new thread and runs a callback when complete

[icy shadow]

that is still technically blocking, it's just blocking a different thread lol

[minor summit]

basically this

[tender thicket]

A future, or a completablefuture

[minor summit]

a cf is a future

[tender thicket]

is there not a Future<T> class?

[minor summit]

sure but it's ass

[tender thicket]

ohhh and I assume CF is a child?

[minor summit]

it's an interface and it's very lacking

[tender thicket]

got it

[minor summit]

cf is a subclass of future if tht's what you mean

details don't matter

return cf

[tender thicket]

Okay so I should put all the sql calls into completable futures?

[minor summit]

throw the thing in cf.supplyasync

[tender thicket]

What about calls that are only updating the DB and not actually receiving data?

[icy shadow]

still should be async

[tender thicket]

okay cool

[icy shadow]

it's the IO that's the problem, not the actual database call

[minor summit]

runasync ig, returning a future allows for stuff like chaining when things are done etc

but yes you want to run it async

[icy shadow]

you could have a connection that for some reason takes a few seconds to open, you absolutely do not want the main thread to be blocking for that

[minor summit]

i've seen people in luckperms with db ping of 500ms 💀

[tender thicket]

Okay so I should return a completable future for the calls and then provide a callback to run afterwards, and then wherever i'm using the sql data i should use the methods provided by the CF to manipulate and store/return ?

or can i do the completable future work within the method body and not have to change the code signature?

[icy shadow]

yeah nothing about that is obviously bad, check your database logs would be a good start

[icy shadow]

is it happening on every connection?

[icy shadow]

uhhh im not really sure what u mean here but CF's are a replacement for callbacks

[tender thicket]

ohh okay so i meant like

can i do the completable future work within the method body and then just return the result as normal so the code signature doesn't change, or should i return the completablefuture and make the changes to the methods uses?

to be honest if you saw this codebase i'm working on you'd be questioning why i'm even bothering to work on it

quite a disaster

[icy shadow]

still not sure what method you're actually talking about but tldr is no

without blocking you cant turn a CF into a normal value

[minor summit]

if you need to "return the result as normal" you'd need to wait until the result is there, which means blocking

[icy shadow]

^^

[tender thicket]

ok well I need to make sure that this never blocks the main thread

that's the only condition

[minor summit]

then return the cf, which means changing the signature and adapting at call sites

[tender thicket]

okay

thanks

What methods from the CF wont block the thread? // edited because i asked the wrong thing

and if i put a try with resources in a supplyAsync that will still open and autoclose when the CF is called correct?

or do i need to close manually in a finally block

[minor summit]

supplyAsync will run the supplier code in another thread
you want to perform all the work involving using the connection/stmt/resultset inside the supplier function, inside a try-with, not storing them in fields or collections you then return and pass around

[tender thicket]

ok so I want to return a completablefuture of any particular object that i want to acquire through the database, otherwise a CompletableFuture<Void> if the call doesn't require a return value?

[minor summit]

basically

[tender thicket]

and then use something like #whenCompleteAsync() or #thenAcceptAsync() for any other work I might need to do in the areas where the CF is being called?

or can i just run the regular non-async call off the CF since it's supplied async?

[minor summit]

these continuation methods you would use if you want to run something after you are sure the async action has completed

e.g.

println("started");
var someFuture = ...;
someFuture.thenRunAsync(() -> println("completed"));
println("uh idk");

this will (most likely) print:
started
uh idk
(and after a bit) completed

[tender thicket]

So, for CompletableFuture<Void>, i am returning null at the end of the block because a void has no value?

just making sure I'm understanding this correctly

[gray dagger]

hey guys...anyone can help me with a dev problem?

i try to create a plugin...at the moment, it looks pretty good...

this plugin isnt very difficult bcs its my first but i here is my problem:

the plugin works with /reward (then you get a dayli reward)
iam worked with cool down (but idk how i can make that its isnt in milli seconds bcs i need 24hours)
but that not the problem for now

the problem is...if the player has a full inventory then the plugin say you "here is your dayli reward" and dont give you anywhat...

what i need todo that the plugin check if you have a empty slot in your inventory that the plugin can give you the dayli reward?

and what i need todo that the plugin say "pls make an empty slot before you can get your reward"?

[tender thicket]

How can I just tell a completablefuture to execute without doing anything else? Just call it?

[icy shadow]

what

[tender thicket]

ohh

nevermind

should I be calling .complete() or .completeAsync()

[dense drift]

?

[icy shadow]

that... depends?

[tender thicket]

if it's being supplied async

would calling complete() block the thread?

i need to avoid blocking the thread entirely;

[gray dagger]

nobody can help me??

[dense drift]

then dont force it to complete?

[icy shadow]

why would you complete it if it's being supplied asynchronously

it's 1 or the other

supply is the same as "do this, then complete() the result"

[tender thicket]

Okay so then if plugin.sql.removeAdmin(admin) returns a CompletableFuture, when I call it, the completable future will execute and complete by itself?

[icy shadow]

well yes, eventually

i think it would be wise to read up on CFs actually work

[tender thicket]

ok

[icy shadow]

or at least how you actually use them lol

[tender thicket]

so if i don't need to wait for the future to complete, and i am not interested in the result, i can just call it. Otherwise, I should use .join() if i need the object directly, or one of the other methods if I don't wish to block the thread and I'm not storing the object for use later on. This is correct?

I've been reading the docs by the way I just don't quite understand some of the things fully that it explains

[icy shadow]

you should not be using join or get most of the time

because those will block

theyre just like callbacks mixed with streams, usually you'd use things like thenApply and whenComplete

[tender thicket]

Right but in some cases here we need to cache the value returned by certain futures. Is there a better option than .join() or .get() that will allow us to return that value so we can store it?

[icy shadow]

whenComplete(addToCache)

you cannot get the value without blocking, period

no way around it

all you can do is use the callback functions, or block (which is a good idea in a few situations)

[tender thicket]

okay

thank you

[feral raptor]

What are the JUnit 5 equivalents of Result & Failure objects from Junit 4 (for test suites)?

I can't seem to find any valid documentation on it. I.e I have this test suite (for j4)

        Result result = JUnitCore.runClasses(AppTestSuite.class);

        for (Failure failure : result.getFailures()) {
            System.out.println("[JUNIT 4.11 TEST SUITE #1] " + failure.toString());
        }

        System.out.println("\nTime elapsed: " + result.getRunTime() + "ms");
        System.out.println("Total Tests: " + result.getRunCount() + " Ignores: " + result.getIgnoreCount() + " Failures: " + result.getFailureCount());
    }

[minor summit]

huh I never saw a single person manually dispatching unit tests lmao

[feral raptor]

its just for practice

I literally cannot get anything to function with Junit 5 though, it seems like everything is just fragmented and thrown all over the place

[sterile hinge]

You’re probably looking for https://junit.org/junit5/docs/current/user-guide/#launcher-api-execution

[tired olive]

you can use a summary generating listener:

        SummaryGeneratingListener listener = new SummaryGeneratingListener();

        LauncherFactory.create()
                .execute(LauncherDiscoveryRequestBuilder.request()
                        .selectors(DiscoverySelectors.selectClass(...))
                        .listeners()
                        .build(), listener);

        List<Failure> failures = listener.getSummary().getFailures()

[gray dagger]

?

[dense drift]

player.getInventory().getFirstEmptySlot() !== -1
this means they have at least one slot empty

[gray dagger]

oh okk then i only forget the !== -1 thank you 🙂

[gray dagger]

and can you help to solve the timer problem?

that the cooldown dont say 8400000 milliseconds to whait...that the cooldown say 24hours:minutes:seconds to whait?

i have try with SimpleDateFormat but i think i dont do it correct 😦 bcs i try out 4 hours...

I either got an error when executing the command ingame or the time was still displayed in milliseconds 😦

[dense drift]

SDF is for date, not time

you can try this https://github.com/PlaceholderAPI/Statistics-Expansion/blob/ae4a6eb7bfeace7cc57deef41a7318163f85f757/src/main/java/com/extendedclip/papi/expansion/mcstatistics/StatisticsUtils.java#L135-L168 but you need to convert the time in seconds first

[gray dagger]

mhh ok thank you, i will try out...

1 last question to this...
need i say the cooldown that it need format or will it do byself?

i mean like

this.cooldown.get(formatTime)

[dense drift]

?

[gray dagger]

cooldown is the placeholder from the time in milliseconds...

need i say the placeholder that it need work with formatTime? or will it know with this?

i can send you the jar file if you want look how i have code it...

(iam not very good...i start to lern java at the moment)

[dense drift]

I can't go through basics rn. You need to pass to the formatTime method the time in seconds, and it will return a formatted string which then you can use wherever you need

[gray dagger]

ok that was my ask 😄

but need i do it with the variable cooldown wich i use to use save the seconds or need i do it with the seconds directly? bcs then its the string equalsTime

[forest jay]

Barry How can ItemStack#getItemMeta be nullable? Do some items just not have meta?

[icy shadow]

air does not have meta

im not barry sorry

[dense galleon]

No, only after 30

Which is the amount of pools I've got

So it definitely looks like I am not closing the connection pools appropriately

Is there some other code I should show other than that? idk what else might be at fault

It looks like it happens only in autosave logic

[dusty frost]

Yes you need to show where you use each connection you get. Iirc you either use a try-with-resources or you manually release the connection after using it

[dense galleon]

I am pretty sure I am doing the first?

[dusty frost]

Show please!

[dense galleon]

Yeah sorry I was pasting the code to the paste

https://paste.helpch.at/yujavojoje.java

The error specifically points at dbManager.getConnection().setAutoCommit(true);

[icy shadow]

ah i got it

dbManager.getConnection().setAutoCommit(true); this opens a new connection, and never closes it

you'll probably need a nested twr if you need the auto-committing

[dense galleon]

Ahhhh shit that makes sense

[icy shadow]

same goes for the rollback obviously

[dense galleon]

I've just got to replace it with this right?

            try (Connection connection = dbManager.getConnection()){
                connection.setAutoCommit(true);```

[icy shadow]

pretty much

[dense galleon]

And the rollback yeah

[dusty frost]

Yeah I would do a search in your entire codebase for every dB Manager.getConnection() and make sure

[dense galleon]

So basically i should NEVER use dbManager.getConnection() since if I do that shit will go wrong

[dense galleon]

yeah there

We basically said the same thing haha

[dusty frost]

Well you just gotta make sure you free it after every use

It's like doing memory management in C yk

[dense galleon]

Yeah, just that the try catch does it for me

[dusty frost]

Yes

[icy shadow]

i mean... you have to use it to get a connection lol, but apart from that no

the smart thing to do would be encapsulate the whole getConnection away with some usingConnection(Consumer<Connection>) method

[dense galleon]

Yeah what I meant is that I should only use it after creating a var holding that in the try brackets

[dense galleon]

I have got no idea what that means tbh

[dusty frost]

Now you're thinking with portals!

[icy shadow]

public <T> useConnection(Function<Connection, T> f) {
  try (Connection c = getConnection()) {
    return f.apply(c);
  }
}
``` for example

then you never actually touch getConnection except in that method

[dense galleon]

Ahh yeah okay I get that

I just haven't really used functions before tbh

Or I mean, made use of them myself

[icy shadow]

functional programming gateway drug

we'll have you monadding in no time

[dense galleon]

i honestly just havent found a use for them in my own code

i mean I use the built in ones or any that any of the PaperAPI may use

but I never used them in code I wrote myself

[dusty frost]

Start doing web dev, you'll find a lot of uses for closures lmao

I stg nearly everything in react is a closure

[dense galleon]

next year in uni I am doing web development and 3d web development

So maybe then

[dusty frost]

3d web development 🤯

[dense galleon]

What

is that bad 😂 it's an optional module I had, it was either that or some other weird one

[dusty frost]

What is that lol

Like Three.js?

[dense galleon]

The weird one? It's "Technology-Enhanced Learning Environments"

[dusty frost]

no what is 3d web development

what does that mean

screens are 2d

like i feel like the second class for web dev shouldn't be immediately 3d shit yk

[split harbor]

Code:

    @EventHandler
    public void onInventoryClick(InventoryClickEvent e) {
        if(e.getCurrentItem().getType() == Material.BEDROCK) e.setCancelled(true);
    }

    @EventHandler
    public void onInventoryCreative(InventoryCreativeEvent e) {
        if(e.getCurrentItem().getType() == Material.BEDROCK) e.setCancelled(true);
    }```

Problem:
https://i-have.c0ronavir.us/cbw45MvT

no embeds?

[river solstice]

https://tenor.com/view/embed-no-no-embed-megamind-megamind-meme-gif-25261934

[dusty frost]

not at your rank lol

also why is that your image hosting domain

[river solstice]

set current item to null (I THINK)

[minor summit]

useConnection(identity())

gottem

[split harbor]

cause it's funny

[river solstice]

https://tenor.com/view/so-funny-sarcastic-so-funny-i-forgot-to-laugh-should-i-laugh-i-laughed-gif-25558527

[icy shadow]

!!!!!

[split harbor]

now it deletes the original item, but i still pick up a copy and can place it

the issue is, I'm trying to make an item just act like a button, you click it and something happens but the item doesn't get affected. Currently in creative when the item is clicked it gets copied to the cursor and i can do whatever i want with it. In survival it acts like it should but creative is a problem.

[hoary scarab]

Yes creative is the problem. There is also a separate event in the spigot api for clicking in creative.
For some reason alot of the inventory interaction for creative is handled client side.

[split harbor]

I used this event but it doesn't change anything.

@EventHandler
    public void onInventoryCreative(InventoryCreativeEvent e) {
        if(e.getCurrentItem().getType() == Material.BEDROCK) {
            e.setCurrentItem(null);
            e.setCancelled(true);
        }
    }```

it still copies the item.

[hoary scarab]

Yeah, like I said alot of the creative inventory happens client sided.

[split harbor]

So there is nothing I can do about it? I just have to circumvent it?

[hoary scarab]

Not in its current state. You would need a mod or something to send more packets to geter a better indication of what the player is doing.

[dense galleon]

What should be launched first? The Bungeecord instance or the backend servers?

[mental cypress]

I don't think it really matters? I typically do proxy first.

[river solstice]

you won't be able to connect to the backend without proxy

and with proxy you won't be able to connect to the backend if it's off

really doesn't matter, but proxy would typically boot faster allowing players in, and in the meantime while they connect the backend would spin up

[dense galleon]

Yeah okay cool cool

[dusty frost]

they're completely detached 😌

makes doing maintenance and stuff super nice as you can just go through the backend servers one by one to update

[dense galleon]

epic

[manic robin]

If i add player.getInventory().clear(); into my code, my server will crash with this message,

[20:17:54] [Server thread/ERROR]: Encountered an unexpected exception
java.lang.AssertionError: TRAP
    at net.minecraft.server.v1_16_R3.ItemStack.checkEmpty(ItemStack.java:99) ~[spigot-1.16.5.jar:2991-Spigot-018b9a0-f3f3094]
    at net.minecraft.server.v1_16_R3.ItemStack.setCount(ItemStack.java:890) ~[spigot-1.16.5.jar:2991-Spigot-018b9a0-f3f3094]
    at net.minecraft.server.v1_16_R3.PlayerInteractManager.a(PlayerInteractManager.java:407) ~[spigot-1.16.5.jar:2991-Spigot-018b9a0-f3f3094]
    at net.minecraft.server.v1_16_R3.PlayerConnection.a(PlayerConnection.java:1425) ~[spigot-1.16.5.jar:2991-Spigot-018b9a0-f3f3094]
    at net.minecraft.server.v1_16_R3.PacketPlayInBlockPlace.a(PacketPlayInBlockPlace.java:28) ~[spigot-1.16.5.jar:2991-Spigot-018b9a0-f3f3094]
    at net.minecraft.server.v1_16_R3.PacketPlayInBlockPlace.a(PacketPlayInBlockPlace.java:1) ~[spigot-1.16.5.jar:2991-Spigot-018b9a0-f3f3094]
    at net.minecraft.server.v1_16_R3.PlayerConnectionUtils.lambda$0(PlayerConnectionUtils.java:19) ~[spigot-1.16.5.jar:2991-Spigot-018b9a0-f3f3094]

    at net.minecraft.server.v1_16_R3.TickTask.run(SourceFile:18) ~[spigot-1.16.5.jar:2991-Spigot-018b9a0-f3f3094]
    at net.minecraft.server.v1_16_R3.IAsyncTaskHandler.executeTask(SourceFile:144) ~[spigot-1.16.5.jar:2991-Spigot-018b9a0-f3f3094]
    at net.minecraft.server.v1_16_R3.IAsyncTaskHandlerReentrant.executeTask(SourceFile:23) ~[spigot-1.16.5.jar:2991-Spigot-018b9a0-f3f3094]
    at net.minecraft.server.v1_16_R3.IAsyncTaskHandler.executeNext(SourceFile:118) ~[spigot-1.16.5.jar:2991-Spigot-018b9a0-f3f3094]
    at net.minecraft.server.v1_16_R3.MinecraftServer.bb(MinecraftServer.java:942) ~[spigot-1.16.5.jar:2991-Spigot-018b9a0-f3f3094]
    at net.minecraft.server.v1_16_R3.MinecraftServer.executeNext(MinecraftServer.java:935) ~[spigot-1.16.5.jar:2991-Spigot-018b9a0-f3f3094]
    at net.minecraft.server.v1_16_R3.IAsyncTaskHandler.awaitTasks(SourceFile:127) ~[spigot-1.16.5.jar:2991-Spigot-018b9a0-f3f3094]
    at net.minecraft.server.v1_16_R3.MinecraftServer.sleepForTick(MinecraftServer.java:919) ~[spigot-1.16.5.jar:2991-Spigot-018b9a0-f3f3094]
    at net.minecraft.server.v1_16_R3.MinecraftServer.w(MinecraftServer.java:851) ~[spigot-1.16.5.jar:2991-Spigot-018b9a0-f3f3094]
    at net.minecraft.server.v1_16_R3.MinecraftServer.lambda$0(MinecraftServer.java:164) ~[spigot-1.16.5.jar:2991-Spigot-018b9a0-f3f3094]
    at java.lang.Thread.run(Thread.java:748) [?:1.8.0_202]

does someone know a fix?

[split harbor]

@EventHandler
public void onInteract(PlayerInteractEvent e) {
    Bukkit.broadcastMessage(e.getClickedBlock() != null ? e.getClickedBlock().getType().toString() : " air");
    if(e.getClickedBlock() != null && e.getClickedBlock() instanceof Container) {
        Bukkit.broadcastMessage("container");
        if(StaffManager.isInStaffMode(e.getPlayer())) e.getPlayer().setGameMode(GameMode.SPECTATOR);
        e.getPlayer().setVelocity(new Vector(0, 0 , 0));
    }
}

it's giving me "chest" and "barrel" but their not instanceof Container?

or am i using it wrong?

[proud pebble]

getClickedBlock() returns Block, Container is a subinterface of BlockState

so you want to get the blockstate

[split harbor]

thank you

[proud pebble]

so (Container) e.getClickedBlock().getBlockState()

also remember that getBlockState returns a copy of the blockstate, so you have to setblockstate or whatever for changes made to take effect

or i believe thats correct

[split harbor]

all g I just wanted to detect clicking a container e.getClickedBlock().getState() instanceof Container worked fine

[proud pebble]

good

[lyric gyro]

skript wise would armor stand interact trigger if an armor stand was shot by an arrow

is this also wrong 😭

[dense drift]

possible

idk how many skript users we have and how many of them are watching this channel

[split harbor]

idk anything about skript anymore but i doubt it

[dense galleon]

What's the best course of action to add custom recipes to my Paper server, using my items made with custom model data?
The main issue is that all my custom items are just "minecraft:coal" with custom model data, meaning that any piece of coal can be used to replace the ingredients.

[dense drift]

https://hub.spigotmc.org/javadocs/spigot/org/bukkit/inventory/ShapedRecipe.html#setIngredient(char,org.bukkit.inventory.RecipeChoice) I assume this would do what you want

If that's not enough (e.g. the item can have a different name / lore) I assume you can also use the PrepareCrafting event and keep your recipes somewhere and then check if the item is completelly matching and not only its material.

[dense galleon]

yeah that's what I think i'll have to do

But then, how can I have it pop up in the recipe book

[dense drift]

You register the recipes first

And then on the event you check the item

[feral raptor]

shouldnt #5 not match given this expression?

[minor summit]

well, no, it's just matching it multiple times

[dense galleon]

Right so I create the recipes, then register them, and then handle everything in the event

[dusty frost]

yeah there's a lot of 1-3 e's in that line

[dense galleon]

But then, how about having the recipe show up when the player picks up an item or does an advancement or something? I guess I will have to do all that with code instead of being able to do it through jsons right?

[dusky harness]

theres something that can make the regex start at the first

i dont remember on the top of my dead

[dense drift]

Ah yes, you need to give them access to the recipes somehow. Unless you need it to work like vanilla recipes, where you unlock them by collecting certain items, I would just give access when the players enters the server

[dense galleon]

Yeah I need it to work like Vanilla

[wintry grove]

so I'm currently looking for a way of taking a list and then sorting it based on input so I can get the stuff that has been added to it in order (based on input, as I said), I was thinking of using a set but I dont exactly know if thats the best thing to do

so what should I use?

a set?

take a list and then sort it based on collections.sort?

[dusty frost]

sets inherently have no order, so would probably not be a wise choice for an ordering scenario as you describe

you should probably just take a list and sort it however, it's not super clear what your intention for sorting it is "based on input?"

[wintry grove]

so sorting a list with Collections.sort()?

[dusty frost]

there are lots of ways of sorting, that is one of them

[wintry grove]

so when I loop the list to get the values in it I get them in order of input

so lets say I push some stuff into a list

then I sort it to later get the values inside it in order

thats what I need to do

[dusty frost]

what is "in order" in this context

insertion order?

[wintry grove]

yes thats what I meant

[dusty frost]

it most likely is already ordered that way

unless you construct the list in a specific way, that's how lists are generally built lol

[wintry grove]

didnt know that

its been a while since I have coded

but ty

[split harbor]

I sit here for 2 hours at 2am to find where in my code I have an infinite loop and then I see this: for(int index = loc.getBlockY(); index > -63; index++) {

index++

-63

just noticed wrong channel mb

[minor summit]

that will eventually break

[dusty frost]

thank god java doesn't have unsigned integers or you'd be waiting twice as long!

[minor summit]

no if it had unsigned then yes it would loop forever

[forest jay]

I am using FAWE, and I am getting a weird error about the schematics I am using. I created them on 1.19.3, but they wont paste (I am using the API): https://paste.denizenscript.com/View/108289

Is there something I am doing wrong?

[dense galleon]

How does the Java logger work exactly

I set its level to FINEST, shouldn't anything printed out with .fine() be shown?

Since it's a lever higher than FINEST?

        System.out.println(Bukkit.getLogger().getLevel());
        Bukkit.getLogger().fine(itemKey.getKey() + " has flags: " + this.flags);``` 
The logger level shown from the first line is `FINEST`, but for some reason the message below it doesn't display in the console?

[dense drift]

just log with info

[dense galleon]

I wanna split up the importance of logging messages though

[dense drift]

I know that doesn't answer your question but it works

[dense galleon]

Ughh

[dense drift]

ah

[dense galleon]

I am logging with info as well, which works, but instead of constantly commenting out/in stuff over and over again i'd rather just set different logging levels

[dense drift]

are they for debug?

[dense galleon]

Yeah

[dense drift]

and also, don't use the bukkit logger directly

[dense galleon]

I know I can write my own system

Huh wdym

[dense drift]

use your plugin logger

[dense galleon]

Oh sh

I forgot about that

Yeah my bad

I think that's the issue

the bukkit logger's logging level is set elsewhere

[dense drift]

and you can probably enable debug level somehow and use that

[dense galleon]

Hmm i suppose there is no way to make custom recipes using items with custom model data work as seamlessly as normal Minecraft recipes?

The recipe book tells the player that he cannot craft the custom item even if they got the right ingredients in theit inventory

Does anyone know if there is any way to fix this?

The recipe book not correctly displaying whether the player has the correct items to craft something?

[forest jay]

were there any changes to custom model data and unicode image replacements between 1.17 and 1.19? I am using the same resource pack, and no custom model datas or unicode image replacements are working.

[stuck hearth]

;d PrepareInventoryResultEvent

smh

https://hub.spigotmc.org/javadocs/spigot/org/bukkit/event/inventory/PrepareInventoryResultEvent.html

Oh I did it backward that's why

Nvm DocDex said no anyway

[dense galleon]

What

[stuck hearth]

Unrelated to you

Just read the linky link

[dense galleon]

Oke

[dense drift]

You can create advancements I guess and give them the recipe

[stuck hearth]

Does creating a recipe actually make it craftable then?

[dense drift]

Yes ofc

[stuck hearth]

Doesn't seem right, but I'll take your word for it

[dense drift]

What doesnt seem right?

I mean, yes, simply creating the object wont make it work, you need to register it through Bukkit#registerRecipe or smth

[dense galleon]

This is the issue, not that

[dense drift]

Hm

[dense galleon]

The recipe can be crafted, but the recipe book believes it cannot

I suppose that can be fixed through packets...?

[stuck hearth]

Oh neat

[dense drift]

The book might be client side

[dense galleon]

yeah that's the thing it probably is

[dense drift]

What happens if you use the recipe give cmd?

[broken elbow]

I'm pretty sure you can edit the book but idk if it is possible using spigot's api

[dense galleon]

I already have the recipe

[dense drift]

And how does the recipe code look like?

[dense galleon]

https://paste.helpch.at/datinupoqi.java

[dense drift]

Also, try to give yourself a stick with model data for example and see if the torch recipe is red too

[dense galleon]

it's not optimal rn as it's a whole method for a single recipe, but first I wanna make sure this method works before i refactori t

[dense galleon]

i'm not sure how to do that tbh

[dense drift]

Try /minecraft:give stick{CustomModelData:1}

And you also need the recipe which is unlocked by getting coal btw

[dense galleon]

it's not red

[dense drift]

Do these shards also have a name?

Or smth else than CMD

[dense galleon]

yeah they got a display name if that's what you mean

And other flags I created myself

[dense drift]

Yeah

Idk what the recipechoice checks for to see if the item is correct

[dense galleon]

yeah they've got 1/2 persistent data

[dense drift]

But still, if the item can be craftable it should work I guess

[dense galleon]

        @Override
        public boolean test(@NotNull ItemStack t) {
            for (ItemStack match : choices) {
                if (t.isSimilar(match)) {
                    return true;
                }
            }

            return false;
        }```

[dense drift]

Yeah but isSimilar goes deeper

And for that you need to check spigot source

[dense galleon]

I tried creating my own RecipeChoice but Bukkit didn't like that

[dense drift]

It says that in the description lol

d;recipechoice

[uneven lantern]

org.bukkit.inventory.RecipeChoice

public interface RecipeChoice
extends Cloneable, Predicate<ItemStack>```
RecipeChoice has 2 extensions, 3 super interfaces, 2 implementing classes, and  3 methods.

Description:

Represents a potential item match within a recipe. All choices within a recipe must be satisfied for it to be craftable. This class is not legal for implementation by plugins!

Requested by: GabyTM | RO • spigot

[dense galleon]

:c

I mean

Nvm yeah idk what I can do

[stuck hearth]

sealed interfaces in 20 years

[dense galleon]

Are packets through ProtocolLib not an option

[dense drift]

Idk

[dense galleon]

I suppose the PaperAPI doesn't fully support this

So I'll just have to deal with the recipe book displaying the custom recipes in red

[dense drift]

I guess if you register it as a normal recipe (with materials instead of choices) it works fine?

[dense galleon]

The issue though with that is the recipe will result available at the wrong times

Since all my items are Coal with custom data

[dense drift]

yeah I just said to test

[dense galleon]

Yeah that does work

I genuinely feel like the solution might be using packets through protocollib

Nah what

In a server without any plugins, when I try clicking on the recipe for the torch it doesn't work

[dense galleon]

This, inside of RecipeChoice, is deprecated... but why?

The explanation is extremely vague

[dense drift]

d;recipechoice#getitemstack

[uneven lantern]

org.bukkit.inventory.RecipeChoice#getItemStack

@Deprecated @NotNull
ItemStack getItemStack()```

Description:

Gets a single item stack representative of this stack choice.

Deprecation Message:

for compatibility only

Returns:

a single representative item

Requested by: GabyTM | RO • spigot

[dense drift]

great

[dense galleon]

Yeah rip

[dense drift]

sometimes spigot annotates stuff with deprecate to warn users about something, e.g. Bukkit#getOfflinePlayer(String)

but this makes no sense

[stuck hearth]

I wonder if it doesn't actually work in the newer versions and was kept in purely for compatibility?

Though if that were the case I'd expect it to return null

[feral raptor]

public List<String> noX(List<String> strings) {
  return strings.stream()
  .map(e -> e.replaceAll("[Xx]",""))
  .collect(Collectors.toList());
}

is using String#replaceAll overkill here (regex)

(remove all occurances of "X" from a List of strings)

[dense drift]

could do a simple replace probably

replace("X", "").replace("x", "")

[forest jay]

how can I get the formatted name of a material? for example: Material.DIAMOND_SWORD into "Diamond Sword"

[dense drift]

Component.translatable(Material.DIAMOND_SWORD.key()) or smth around those lines

[forest jay]

that gives me a TranslatableComponent, but how does that become a string?

toString() just print json

[dense drift]

It does not become a string

What do you want to do?

The component will be rendered depending on client's language

[forest jay]

I want it turned into a string that I can manipulate

[dense drift]

Welp, for that you would need to handle the names on your own

[stuck hearth]

PlainTextSerializer might work

[dense galleon]

In all honesty, I've got to say GPT-4 is pretty great when it comes to the PaperAPI
I've been using it to write documentation for me, and other than making a couple minor corrections it's been really great

[stuck hearth]

Your questions are making sense now

[dense galleon]

I mean yeah, i'm talking to ChatGPT to figure out how to do some things within the PaperAPI it's really good

[dusky harness]

u bought gpt 4? 👀

[dense galleon]

Though if not used correctly it's just a spoonfeeding machine which breaks every other method

[dense galleon]

Yeah, I use it for uni 😂

[stuck hearth]

it's still a spoonfeeding machine either way 😉

[dusky harness]

💀

[dense galleon]

For sure, but it speeds up things a LOT

After half a decade of writing plugins, if I don't have a lot of free time with uni/work to work on this stuff

[stuck hearth]

Yes, as a spoonfeeding machine, it does make spoonfeeding a lot faster

[dense galleon]

Spoonfeeding is mostly shit for beginners, after years of programming if I don't have a lot of free time to work with the PaperAPI it makes it a less annoying yknow

[dusky harness]

github copilot is literally just spoonfeeding
it's not good for beginners but useful for advanced developers

[dense galleon]

Github co-pilot is kinda wack tbh

I mean

It's good i guess

[dusky harness]

its free for students so

¯_(ツ)_/¯

[dense galleon]

But very often it writes code which makes 0 sense

That is true

And it does speed up repetitive stuff quite a lot

[dusky harness]

for me its just mainly useful for repetitive code

not like completely repetitive but similar patterns

[dense galleon]

^ 😂

[dusky harness]

ah

🥲

yeah

didn't see that

[dense galleon]

yeah, ChatGPT is good as it can explain parts of the API or help refactor code

[dusky harness]

mhm but you can't input the whole project

although

maybe gpt 4 u can?

not a big project

but i think gpt 4 can store more memory

[dense galleon]

Nah, it shits itself if you write more than like 2000 characters or something

[dusky harness]

oh lol

[dense galleon]

It's got a memory of 64k characters or something?

Though after a while it begins hallucinating methods that do not exist within the API

[dusky harness]

💀

[dense drift]

Translatable components are player dependent

[stuck hearth]

Oh that makes sense, I haven't touched those specifically yet.

[dusty frost]

yeah the point is that you send it to them in the chat or whatever and then their client translates it in their language or somethin

[dusky harness]

Paper has a method

I think

ItemStack#get18NDisplayName or smth

[forest jay]

I am just doing a quick ```java
String formattedName = Arrays.stream(material.name().toLowerCase().replace("_", " ").split("\s+"))
.map(word -> word.substring(0, 1).toUpperCase() + word.substring(1))
.collect(Collectors.joining(" "));

[dusky harness]

a lot of it is different but if that's ok then it works

[forest jay]

I am not using Paper, so this is the next best choice (that I can think of)

[stuck hearth]

didn't think of using paper?

[forest jay]

my server instance isnt paper

[stuck hearth]

Well that didn't clarify anything

pinged me to tell me something we already knew lol

[forest jay]

oh wait, I misunderstood you, mb

idk what I thought

[stuck hearth]

Oh paper api instead of paper server maybe,

[forest jay]

yeah, something like that

[stuck hearth]

Makes sense, my bad 😳

[dusty frost]

🤨

wtf is is then

[forest jay]

spigot ig

[dusty frost]

wtf why

[forest jay]

idk, never bothered to replace it with paper/purpur/pufferfish

I will once into production tho

[dusty frost]

you're missing out big time

especially considering you could use a bunch of APIs that would make your life a lot better

[stuck hearth]

Paper is a drop-in replacement too, not terribly confusing

[forest jay]

I know, I just havent done it

[dusty frost]

well this is as good a time as any

considering you could literally use part of the paper api

[stuck hearth]

yeah submit to peer pressure

[dusty frost]

i mean it's in their best interest

[stuck hearth]

I agree, they should submit to peer pressure

That's why Im pressuring them as a peer

[forest jay]

isnt using the paper api more than just adding a dependancy? I remember some sort of dev kit or something I had to set up

[dusty frost]

nope

you just swap out in your gradle file

it's also easier to setup a dev server for than spigot, since you don't need to run buildtools or anything

[stuck hearth]

https://bstats.org/global/bukkit 58% of servers use paper now apparently.
Wondering what that 6% unknown is

[dusty frost]

66% if you include Purpur

also worth noting only 24% is Spigot

[echo briar]

For A authentication plugin should i have it check for the players name for authentication or their uuid?

[river solstice]

lowercase/uppercase name I would guess

[stuck hearth]

Depends on what you're authenticating for, if you're an online mode server UUID works just fine.
If you're an offline mode server Idfk

[river solstice]

because won't the UUID change if you joined with mycoolnickname and later with MyCoolNickname?

[stuck hearth]

In offline mode yes I believe so

[dense drift]

you usually block the other variants of one name

if someone joined with Username, then someone else can not join with USERNAME

[river solstice]

well I suppose you'd save the name of the player with capitalization on the first join and just check if it matches it on other attempts

[dense drift]

no

because that will allow all variants of one name

[river solstice]

hm

[dense drift]

UsErNaMe.upperCase() is the same as username.upperCase()

[river solstice]

Map<NameLowerCase, NameActual> names?
onJoin
actual = names.get(joinedName.lowerCase)
actual == joinedName?

[dense drift]

You can look at what AuthMe does, but it depends on your server type I guess, online or offline.

or just store the name directly?

[river solstice]

¯_(ツ)_/¯

[stuck hearth]

Over complicating things is the true sign if an engineer

[echo briar]

Thank you everyone else for pitching in too, I’ve gone ahead and stored the name along with any type of variant and it seems to work perfectly! I didn’t think to do that so thank you

[river solstice]

you really shoudn't need to store all the possible variants

[echo briar]

It doesn’t store all the varieties but instead saves just the actual name then upon checking it will try all those varieties on the name, hence making the player login if it matches

[proud pebble]

you store either a uppercase or lower case version, but with the first vwrsion if the username

<String,String> map
map.put(username.toUppercase(),username)

then check if the value in the map that corrisponds to username.toUppercase() is the same, if not deny access

[dense drift]

You dont want to allow users to join with various versions of an username, if this is for Minecraft.

[minor summit]

TreeSet moment

[hollow sorrel]

How can I fix the NULL-Exception Error?
https://paste.ofcode.org/5WiUavs8btB22RESpyieHT
My code:
https://paste.ofcode.org/R3KTNsPAM97xWiZprdnEy6

[broken elbow]

that is not a null exception error. that's a number format exception

[hollow sorrel]

oh i see

[broken elbow]

And it comes from line 32.
long duration = Long.valueOf(args[1].substring(0, args[1].length() - 1));

args[1].substring(0, args[1].length() - 1) this is not a number. it is an empty string

[hollow sorrel]

oh, that's my bad prob thx for helping

[shy canopy]

@EventHandler
    public void on(ProjectileHitEvent event) {
        event.getEntity().remove();
        if (event.getEntity().getShooter() instanceof Player attacker) {
            if (event.getHitEntity() instanceof Player victim) {
                if (GameManager.getPlayerTeam(attacker) == GameManager.getPlayerTeam(victim)) {
                    event.setCancelled(true);
                }
                else if (victim.getHealth() - victim.getda <= 0) {
                    killReward(attacker);
                    Bukkit.broadcast(MessageManager.messageManager(attacker.getName()+ " <red>shoot <white> "+ victim.getName()));
                }
            }
        }
    }

It's currently spaghet but how i can check did projectile kill entity?

[river solstice]

uh oh arrow code

@EventHandler
public void onProjectileHit(ProjectileHitEvent event) {
    if (event.getEntity() instanceof Arrow arrow) {
        if (arrow.getShooter() instanceof Player shooter) {
            Entity hitEntity = event.getHitEntity();
            if (hitEntity instanceof LivingEntity target) {
                if (target.getHealth() <= 0) {
                    // The projectile killed the entity
                }
            }
        }
    }
}

here's more arrow code for ya bud

[shy canopy]

So you basically say this would work?

@EventHandler
    public void on(ProjectileHitEvent event) {
        event.getEntity().remove();
        if (event.getEntity().getShooter() instanceof Player attacker) {
            if (event.getHitEntity() instanceof Player victim) {
                if (GameManager.getPlayerTeam(attacker) == GameManager.getPlayerTeam(victim)) {
                    event.setCancelled(true);
                }
                else if (victim.getHealth()  <= 0) {
                    killReward(attacker);
                    Bukkit.broadcast(MessageManager.messageManager(attacker.getName()+ " <red>shoot <white> "+ victim.getName()));
                }
            }
        }
    }

[forest jay]

yeah, most likely. also, try and use return statements instead of nested ifs. for example:

    @EventHandler
    public void on(ProjectileHitEvent event) {
        event.getEntity().remove();
        if (!(event.getEntity().getShooter() instanceof Player attacker))
            return;
        
        if (!(event.getHitEntity() instanceof Player victim)) 
            return;
        
        if (GameManager.getPlayerTeam(attacker) == GameManager.getPlayerTeam(victim)) {
            event.setCancelled(true);
            return;
        }
        if (victim.getHealth()  <= 0) {
            killReward(attacker);
            Bukkit.broadcast(MessageManager.messageManager(attacker.getName()+ " <red>shoot <white> "+ victim.getName()));
        }
    }

It makes it far more readable and understandable.

[shy canopy]

Thx!

[shy canopy]

This return health that I had before i got shot

[river solstice]

famous last words, left for technical debt, forgotten jira task by now

[hoary scarab]

Anyone know the mappings for EntityPose? I found Pose as the unmapped but its different from entitypose.

[lyric gyro]

getting an error on this int length = Integer.parseInt((String) this.getConfig().get("test.test.test"));
error: class java.lang.Integer cannot be cast to class java.lang.String
how would i rework this to fix it?

[hoary scarab]

getConfig().getInt("test.test.test")

[lyric gyro]

thank you so much! <3

[feral raptor]

any idea why neither of these EventHandler methods are being invoked?

    @EventHandler
    public void cancelDragEvent(InventoryDragEvent e) {
        e.setCancelled(true);
        Bukkit.broadcastMessage("inventorydragevent called");
    }

    @EventHandler
    public void onClickEvent(InventoryClickEvent e) {
        Bukkit.broadcastMessage("inventoryclickevent called");
        if (e.getClick().equals(ClickType.RIGHT)) {
            this.player.sendMessage(ChatColor.translateAlternateColorCodes('&', "&a&l[BitCore]&a Right clicking will sell the item!"));
            // player.playSound(player.getLocation(), Sound., null, 0, 0);            
        } else if (e.getClick().equals(ClickType.LEFT)) {
            this.player.sendMessage(ChatColor.translateAlternateColorCodes('&',"&a&l[BitCore]&a Left clicking will buy the item!"));
        }
    }

[hoary scarab]

Did you register the listener?

[feral raptor]

Thanks

[hoary scarab]

Np

[proud pebble]

thats called the enum is no lomger an enum while unmapped

[hoary scarab]

Yeah I just matched it myself. I moved on lol

[proud pebble]

https://nms.screamingsandals.org/1.19.4/net/minecraft/world/entity/Pose.html

using screamingsandals might be helpful

[hoary scarab]

👍

[proud pebble]

certainly makes figuring out what the unmapped version of a mapped class, method or variable is called

cus navigating the minecraft code is hard sometimes

[hoary scarab]

Especially when you have to find all remapps and moved code from 1.8.4 to 1.19.4 lol

[lyric gyro]

created a public variable at the beginning of the main class public List<String> playerlist;
edited it in a function within the main class this.playerlist.add(p.getUniqueId().toString());
error cannot invoke "java.util.List.add(Object)" because "this.playerlist" is null
i assume i am getting the error due to changing the variable within the function although i do not know how i would keep the variable public while being able to edit it within the function.

[hoary scarab]

You never initialize the variable playerList... playerList = new ArrayList<>();

[lyric gyro]

god im stupid, thank you

[hoary scarab]

Np.

[feral raptor]

    @EventHandler
    public void cancelDragEvent(InventoryDragEvent e) {
        e.setCancelled(true);
        Bukkit.broadcastMessage("inventorydragevent called");
    }

Still doesn't get called, unsure why. Listener is registered, not sure if its because it shares the same class as another Listener (I dont think that should be a problem)

[dusky harness]

show the whole class along with the class where you register it

that code itself looks fine, so maybe theres something else that could be causing the issue

[feral raptor]

https://gist.github.com/BitCashMC/488423259252840f777f7a61c4119c22

First class is where the listener is registered, 2nd is where the eventhandler method is

Also, (screenshot from the provided code ^), for some reason this code doesn't work either. the method is invoked, but neither of the conditionals are passing

[flint kernel]

Enums should be compared using == rather than #equals

[feral raptor]

Ah alright, thanks for letting me know ^

[feral raptor]

found what my problems were:

• NPE (player was null, using getWhoClicked() instead to access the interacting player)
• clickevent cancellation state wasn't false (thought only drag was necessary)

[glacial comet]

I have an expansion hook in my plugin which I can see in my server console is successfully registered on server startup, but none of my placeholders are working. Here is my onRequest method of my hook:

@Override
    public String onRequest(OfflinePlayer player, String params) {
        if (params.equalsIgnoreCase("timelock_server_name")) {
            return plugin.getConfig().getString("placeholders.timelock_server_name", "My Server");
        }

        if (params.equalsIgnoreCase("timelock_unlock_at")) {
            return Timelock.unlockAt;
        }

        if (params.equalsIgnoreCase("timelock_lock_at")) {
            return Timelock.lockAt;
        }

        return null; // Placeholder is unknown by the Expansion
    }```

Running /papi list shows my hook.

The hook is registered in my onEnable method of my plugin class:

// Register placeholders with PlaceholderAPI.
new TimelockPlaceholderExpansion(this).register();```

When I do /papi parse <player name> %timelock_server_name% it just returns my placeholder, %timelock_server_name%

Does anybody know what could be wrong here?

[dusty frost]

You don't include your expansion name in the match. So for your current setup to work, you'd need to do %timelock_timelock_server_name%

So just get rid of the first timelock_ in all the checks!

@glacial comet

[glacial comet]

This works! Thank you very much.

In the identifier, I had actually put my full package name identifer instead of just timelock so I had to fix that, too.

[slow wharf]

looking for a dev that knows to work with deluxemenus and make me a custom shop, i could pay up to 5$ cuz its a very basic thing.

[icy shadow]

#1091573125046612079

[stuck hearth]

Is it $5 because it's basic, or $5 because you only have $5 🤔

[lyric gyro]

is there an expression to get the coordinate of the other edge of a schematic (the point not entered into the editsession)?

[feral raptor]

https://paste.helpch.at/ahetolaved.java

Any idea as to why the two EventHandlers aren't executing their code?

It seems as though if (!(e.getInventory() == gui)) return; is returning true.

[pulsar ferry]

Try using equals instead

Also why !(e.getInventory() == gui) and not e.getInventory() != gui?

[minor summit]

are you registering the listener?

[feral raptor]

what fixed it was having the registered Listener instance be the same as the one being used by the CommandExecutor used to open it

Actually, what would be even smarter is to just.. not initialize gui during instantiation..

[feral raptor]

less nesting ig

wait, no. that doesnt make sense what I've said.

You're right

i'm starting to wonder if its just more practical for the majority of this class (aside from EventHandlers) to just be static. I mean, if its made to construct only one gui, then why instantiate it?

Instead of just having one instance of the class used for everything from listener registration to accessing the guis functionalities, instantiate it once to register the events, and the rest of the functionality remains static

[proud pebble]

id still instantiate it

[feral raptor]

yeah fair, hate the idea of making it all static anyway

[dense drift]

Objects are compared with equals()

[north hollow]

Sorry, sent in the wrong channel. If that message was for me

[dense drift]

Has anyone built this plugin? https://github.com/Xiao-MoMi/Custom-Crops it doesn't have all the required libraries added in build.gradle / the libs folder

The free plugins are easy to get, but then there's this battle pass plugin that used to be OS but the new developer made is private 🙂

[pulsar ferry]

Just comment out the code that uses those libs, ez 😃

[dense drift]

yeah I wanted to avoid that but it seems like the only option

[dense drift]

@pulsar ferry I wonder if gradle can be configured to ignore certain classes

[tight junco]

Anyone know what

compileJava {
  options.compilerArgs += ['-parameters']
}
``` Would be when compiling with kotlin 1.8.20 cause compileKotlin is deprecated

i tink anyway

[sterile hinge]

what

that should be only relevant for Java code anyways, unless Kotlin has a similar feature

[tight junco]

i think i found what i need anyway

[pulsar ferry]

tasks.withType<KotlinCompile> {
    kotlinOptions.javaParameters = true
}

[tight junco]

yeah i found that

[stuck hearth]

https://kotlinlang.org/docs/gradle-compiler-options.html

[proud pebble]

maybe they have an api plugin that you can use as a replacement?

like some devs have a basically empty jar with the class names and methods that wouldnt do anything, but are the same locations as the premium plugin location to basically work

interfaces

like the bukkit classes

[dense drift]

Nah, the dev is greedy and didn't made and api, unless the wiki is outdated

[proud pebble]

fun

[fading stag]

customers:
    - name: "X"
    age: "Y"
    - name: "A"
    age: "B"

What is the best way to get customers as some type of list (maybe a list of maps with String keys and String values etc) in plugin config?

[dense drift]

I think that's a List of Map<String, Object> or something like that

[proud pebble]

config.get("customers") would return an object, in this case a memory section

[minor summit]

no

it would return a list

[proud pebble]

i was sure it was a memory section

[minor summit]

i mean in this case customers is a list of sections

[proud pebble]

ohhh i see

[fading stag]

Are we talking about these? https://hub.spigotmc.org/javadocs/spigot/org/bukkit/configuration/MemorySection.html

[proud pebble]

actually how do you cast an object as a list without it complaining that it might not be

[minor summit]

getMapList("customers")

https://jd.papermc.io/paper/1.19/org/bukkit/configuration/ConfigurationSection.html#getMapList(java.lang.String)

[fading stag]

Is it paper spesific?

[minor summit]

no

[fading stag]

Thanks

[wind tapir]

"'setPlaceholders(org.bukkit.entity.Player, java.lang.String)' is scheduled for removal in version 2.10.8"

what is the other syntax please ?

[proud pebble]

uhhh placeholderapi is at 2.11.3

it was undeprecated

i remember seeing that awhile ago

as far as i can tell from looking at papi's wiki page on github theres no other method for setting placeholders, or atleast i cant find them, maybe im blind idk

[wind tapir]

yes thanks I updated it

0 placeholder hook(s) registered! ????

        total = PlaceholderAPI.setPlaceholders(player, total);```

[proud pebble]

did you download the bungee expansion?

through the ecloud command usually

[wind tapir]

uhh no

"Bungee
by clip" ?

[proud pebble]

yes

[wind tapir]

ok thanks

[wind tapir]

the problem is:

        at fr.idaamo.essentials.Listners.PlayerListener.onJoin(PlayerListener.java:83) ~[?:?]
        at sun.reflect.GeneratedMethodAccessor70.invoke(Unknown Source) ~[?:?]
        at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) ~[?:1.8.0_312]
        at java.lang.reflect.Method.invoke(Method.java:498) ~[?:1.8.0_312]
        at org.bukkit.plugin.java.JavaPluginLoader$1.execute(JavaPluginLoader.java:306) ~[server.jar:git-Spigot-21fe707-741a1bd]
        ... 14 more```

and the scoreboard says "%bungee_total%" not the total

[proud pebble]

whats on line 83

id assume it nped before it could replace the text or smth

[wind tapir]

nah I tested a mysql thing

but the bungee doesn't have to do anything with that

        total = PlaceholderAPI.setPlaceholders(player, total);```

        {
            Score second = objective.getScore("§7Connecté: " + total);
            second.setScore(8);
        }else{
            Score second = objective.getScore("§7Connectés: " + total);
            second.setScore(8);
        }```

I did this

[dense drift]

is the bungee expansion registered? 🙂

[proud pebble]

well you still didnt answer whats on line 83 in the playerlistener class

so telling us that doesnt give info on what was null

[wind tapir]

I removed it

it's not important

[wind tapir]

no, how do I do that please ?

um

I just didn't install it correctly

[proud pebble]

so you do /papi ecloud download bungee
then /papi reload

[wind tapir]

ye I installed it already, there is no bungee_total anymore but it says 0 but I'm in the server...

[dense drift]

it takes around 30s for placeholders to update

[wind tapir]

so what do I have to do ?

[dense drift]

wait

[proud pebble]

im assuming it polls all of the placeholders every 30 seconds?

[wind tapir]

lessgoo thank you all

[dense drift]

Would be possible to publish a jar to github packages? I don't have the src, only the jar 🥶

[lyric gyro]

anyone here used the LitebansAPI before? Im trying to listen to ban events but the wiki isnt great

[proud pebble]

https://gitlab.com/ruany/LiteBansAPI/-/wikis/Examples

first result shows how to listen to ban unban events

[lyric gyro]

i dont see Events.get()

[proud pebble]

are you importing the dependency correctly.

?

[lyric gyro]

ye

[proud pebble]

and you clicked the little maven or gradle icon after making changes

[lyric gyro]

Yes

<dependency> <groupId>com.gitlab.ruany</groupId> <artifactId>LiteBansAPI</artifactId> <version>0.3.4</version> <scope>provided</scope> </dependency>

[proud pebble]

and the repo link is correct?

[lyric gyro]

Yep, <repository> <id>jitpack.io</id> <url>https://jitpack.io</url> </repository>

[proud pebble]

try typing litebans.api.Events

dont copy and paste

it should have auto completion things for it

[lyric gyro]

Yes it does

[proud pebble]

middle click Events

[lyric gyro]

i have it imported already

[proud pebble]

if your using intellij atleast it should show some form of source

[lyric gyro]

hmm yes im using intelij but nothing happens on middle click

[proud pebble]

do you see any methods at all?

[lyric gyro]

it just shows Events on the autocompletion

[proud pebble]

so auto complete it and check if you can find any methods

[lyric gyro]

only thing is Listener - which is a class inside of Events

[proud pebble]

your just doing Events and not new Events right

[lyric gyro]

yea i didnt create an instance of it, im just using Events

[proud pebble]

weird

go find your external libraries, find litebandapi and look through and find the Events class

double click it, it should open

should be below your project structure

[lyric gyro]

can we switch to dms or somewhere where i can post images?

[proud pebble]

post the images to imgur and post here

[lyric gyro]

did you mean external libraries by here or my pom.xml https://i.imgur.com/gafgAgh.png

[proud pebble]

neither

the libraries below where your pom xml is found

[lyric gyro]

oh 🤦‍♂️

[proud pebble]

should be a directory right at the bottom of the list

[lyric gyro]

Yep im there

im at the Events class

[proud pebble]

open it

it should show a preview of the class's contents

[lyric gyro]

yes it does

[proud pebble]

does it show the get method?

[lyric gyro]

yes

        if (instance == null) {
            throw new MissingImplementationException();
        } else {
            return instance;
        }
    }```

[proud pebble]

then you should beable to use the get method

[lyric gyro]

maybe its something with maven then

[proud pebble]

if its there you should beable to use it

[lyric gyro]

https://i.imgur.com/fQsaPLY.png https://i.imgur.com/ehO3a6Z.png

[proud pebble]

Events is abstract, so it should be green afaik

unless its not hmm

not green i mean

maybe its interfaces that are green

mind posting the full class to paste.helpch.at

just ctrl + a ctrl + c

ofcourse post the link here

[lyric gyro]

https://paste.helpch.at/otelijipup.typescript

[proud pebble]

i ment your class

[lyric gyro]

oh lol

https://paste.helpch.at/kikenemube.kotlin

[proud pebble]

wheres the access modifier

and the variable type

you cannot have a statement in the main scope of the class

it has to either be a variable or a method

no wonder it doesnt work

[lyric gyro]

bruhhh im so stupid

my bad sorry for that wasted time aha

[proud pebble]

you can have a statement in a method

[lyric gyro]

ye i just made one

thank you

[proud pebble]

you can in theory have a statement as a variable as long as it returns a type

statements that return void, cannot

[spare marsh]

Folia ETA?

[stuck hearth]

Tomorrow

[spare marsh]

Wow

I will have to test this out then

GitHub Release ETA?

[stuck hearth]

Next near maybe yesterday

[spare marsh]

Very exciting to hear

[pulsar ferry]

https://github.com/PaperMC/Folia

[stuck hearth]

I released it for you

[delicate jasper]

How would I go about loading and pasting a schematic at a certain location using the WorldEdit API

[sterile hinge]

Take a look at their documentation, it has an example for that

[tight junco]

https://worldedit.enginehub.org/en/latest/api/examples/clipboard/#schematic-examples

[dense drift]

I need to replace some item frames with display entities, and by some I mean quite a lot spread across the world (custom furniture made by oraxen). Is there any way I could loop through all the chunks that are generated, to load them and do the replacement? The plan B would be to use ChunkLoadEvent and do the replacement progressively as players load chunks but I would really like to do the replacement once for good.

Edit: Since the map is pre-generated I guess I can just get the coordinates of the chunks that are in the corners and use a for loop.

[marble nimbus]

Hey, what's the best way to get the plain String of item#displayName and each of item#lore? with their text colors as & or §?

[dense drift]

LegacyComponentSerializer

[marble nimbus]

I tried LegacyComponentSerializer.legacyAmpersand().serialize(itemStack.displayName())
But I am getting "&fchat.square_brackets"

[broken elbow]

ItemMeta#displayName

get it from the meta not from the item

[minor summit]

woo let's go poor naming

[spare marsh]

Sooo it’s coming out today still?

[broken elbow]

Folia is already public

[broken elbow]

Matt sent a link here

[spare marsh]

So this plugin supports folia?

[dense drift]

no

[spare marsh]

That’s what I was getting at..

[broken elbow]

What plugin exactly?

I didn't see you mention any plugins

[spare marsh]

Deluxe Menus

[solar zephyr]

no, but I would like to, the developers hope that you will make support for the core

[spare marsh]

kernel?

[solar zephyr]

flolia

[spare marsh]

me?

[spare marsh]

?

@solar zephyr

[solar zephyr]

I just said that I hope that the developers will make support

[spare marsh]

Oh when I read " the developers hope that you will make support " I thought you were referring to me lol

[river solstice]

a lot of extra work to add support for another fork that's has entirely different internals lol

[spare marsh]

That is true, but you guys are very talented ( :

[river solstice]

unless it becomes more popular than paper itself and has a clean way of making sure to not allow racing conditions, then I don't see developers adding support for it by default

[spare marsh]

hmm if this isn't planned I wouldn't mind assembling a small team of people to help me add Folia support. The biggest road block is that this Deluxe Menu doesn't have a GitHub yet. At least from what I know.

If you would be interested in this proposal let me know. I would still need the GitHub to Deluxe Menu so if you could send me it ( it can be through dms or here ) if you approve of this I would appreciate it.

[solar zephyr]

the core has already taken 20% of the used cores in a week + multithreading is a very weighty argument and a breakthrough in servers

[dense drift]

I think most things that needs to be changed are regarding tasks

I hope you guys know that Folia is far from being ready for production. As for DeluxeMenus, I don't have an answer regarding when it will support Folia.

And I don't think many of us have enough resources to test the changes during development.

[hazy summit]

Is there any resource on how to preverse comments when using Configurations in Bungee plugins (on save)?

Whenever I set a new value and save the config, all my comments disappear

https://paste.md-5.net/quboxexava.cs

[solar zephyr]

I checked with Deluxemenus they say they don't have it yet

[spiral mural]

So I am kinda silly bad at trying at softdepends.. what's the proper way of handling soft depends?

did worldguard softdepend and it just went ballistic trying to find class even when disabled.

[stuck hearth]

Check the plugin manager if the plugin is enabled, and don't try and instantiate classes that don't exist

[proud pebble]

if you require those classes for your plugin to function correctly, then use depend. if not then do what doctor zod said

[spiral mural]

appreciate it ^^

[stuck canopy]

Is there a way I can get a Crafting Table Recipe by its id (String)?

are recipes stored with their ids?

[dense drift]

NamespaceKeys on 1.13+

[stuck canopy]

does it work for default crafting recipes too? or for just custom recipes from a plugin?

[dense drift]

all recipes have a key

[proud pebble]

since they are all registered to the exact same place afaik

[stuck canopy]

oh

[tight junco]

this doesn't seem like developer questions

[sage parcel]

oh sorry

[lament grove]

Hi dont work papi dump

https://paste.helpch.at/cijipiguqu.css

[forest jay]

I have a plugin that expands barrels, and due to some issues, I need to create a custom loop that transfers items when hoppers interact. How can I do this at server startup without looping through every single block? I just want to check if there is a hopper above/below the barrel

[hoary scarab]

Continued from #off-topic
Try looping tileentities

[forest jay]

yeah, I am trying that now, just seeing if there were any alternatives.

Looping through TileEntities on ChunkLoad seems to be the best method.

[hoary scarab]

Depending what you are doing you could loop through the world's nbtdata

[forest jay]

I have 3 base components, and I want send them in one line. How can I add the base components together to make one line?

Should I use BaseComponent#addExtra?

[pulsar ferry]

d;adventure Component#join

[uneven lantern]

net.kyori.adventure.text.Component#join

@@Contract(pure=true) @@NotNull
static Component join(@NotNull @NotNull JoinConfiguration config, @NotNull @NotNull ComponentLike @NotNull ... components)```

Description:

Joins components using the configuration in config.

Since:

4.9.0

Returns:

the resulting component

Parameters:

config - the join configuration
components - the components

Page 0 • Requested by: Matt • adventure

[dense drift]

^

[stuck canopy]

how are these two ItemStacks not similar to each other

for some reason ItemStack#isSimilar returns false

[broken elbow]

well one is AIR x 1 and the other is AIR x 0

[broken elbow]

ah

[sterile hinge]

amount 0 sounds sus, does it work for other stack sizes?

[proud pebble]

perhaps the 2 stacks have other data that is different

because from looking at this youve set the itemstack to have a material of air, which doesnt remove extra data that the itemstack might have had before hand

[stuck canopy]

I am logging out ItemStack#toString

so I am pretty sure thats all the data

[sterile hinge]

why would you trust a method that can give you arbitrary values instead of using a debugger

[past ibex]

amount=0 means it is air

oh, empty items aren't similar to empty items

   public static boolean isSame(ItemStack p_41747_, ItemStack p_41748_) {
      if (p_41747_ == p_41748_) {
         return true;
      } else {
         return !p_41747_.isEmpty() && !p_41748_.isEmpty() ? p_41747_.sameItem(p_41748_) : false;
      }
   }

that's intentional

[river solstice]

yes, p numbers

[fading stag]

I'm doing a plugin with dynamic inventories, what is the best way to check is it my gui at InventoryClickEvent? I mean is there anything like a uniqueid of menu or sth else?

[dense drift]

you can check the InventoryHandler instance

inventory.getHandler() instanceof MyHandler

[fading stag]

what should "MyHandler" be exactly? I create inventories using Bukkit.createInventory

[dense drift]

d;Bukkit#createInventory

[uneven lantern]

org.bukkit.Bukkit#createInventory

@NotNull
public static Inventory createInventory(@Nullable InventoryHolder owner, @NotNull InventoryType type)
throws IllegalArgumentException```

Description:

Creates an empty inventory with the specified type. If the type is InventoryType.CHEST, the new inventory has a size of 27; otherwise the new inventory has the normal size for its type.
InventoryType.WORKBENCH will not process crafting recipes if created with this method. Use HumanEntity.openWorkbench(Location, boolean) instead.
InventoryType.ENCHANTING will not process ItemStacks for possible enchanting results. Use HumanEntity.openEnchanting(Location, boolean) instead.

Returns:

a new inventory

Parameters:

owner - the holder of the inventory, or null to indicate no holder
type - the type of inventory to create

Throws:

IllegalArgumentException - if the InventoryType cannot be viewed.

Page 0 • Requested by: GabyTM | RO • spigot

[dense drift]

holder*

you create a custom class that implements InventoryHolder and on click you check if the holder of the inventory is an instance of your custom holder

[fading stag]

understood it, thanks

[atomic trail]

This warning is incorrect right?

Or am I just too tired

It could be left click something

[tight junco]

they cancel each other out i believe

[past ibex]

!=

[atomic trail]

How, if it's not right click on block or air it could still be left click on block or air?

It's not the only two possible actions

[tight junco]

what are you trying to make it do

require only left clicks or only right clicks

[atomic trail]

Only right clicks

Well return if it's not right click

[tight junco]

if statements can be very much fucked in my brain so give me a second

but there is if (!event.isRightClick())

[atomic trail]

Oh really?

[tight junco]

because it could also be PHYSICAL action believe it or not

[atomic trail]

Ah yeah, with action.isright... etc tho

[atomic trail]

Dont really understand the whole physical thing

[tight junco]

copilot for the win

yeah you want &&

physical is pressure plates

"ass pressure" because md_5 thinks hes hilarious

[atomic trail]

Would that be considered a player interaction? Well ig it makes sense actually

[tight junco]

do i remember the command

d;spigot Action

[uneven lantern]

org.bukkit.event.block.Action

public enum Action
extends Enum<Action>```
Action has 1 extensions, 3 all implementations, 2 methods, and  5 fields.

Requested by: Ori • spigot

[tight junco]

wack

you're so ass

i hate this bot

[atomic trail]

Lmao

[stuck hearth]

Get good

[tight junco]

[atomic trail]

Thank you

[dusty frost]

you already have an answer for what to do, but the reason it's giving you that warning is like this: if the action IS Action.RIGHT_CLICK_AIR, then the first boolean is false and the second is true, so therefore the whole thing is true. If the action IS Action.RIGHT_CLICK_BLOCK, then the first one is true, so it's true. And if it's any other action, they're both true. So there's no way it can be false since it is always one thing, it can't be both of them, which is the only way you return false.

[atomic trail]

Ohhh yeah that makes sense, I should really get some sleep lol

Ty

Btw quick question, what would a material string be? For example item.getType().toString()

If it was a netherite sword for example

[dusty frost]

try it and see!

[atomic trail]

Yeah I can do that, but havn't set up any testing yet so would take some time

[dusty frost]

likely just the name, but I think there's a function like Material#name() that gives you the serializable name

[atomic trail]

Just wondering if anyone knows

[atomic trail]

I need it pretty exactly though, for this ugly code lol

        String type;
        String itemMaterial = item.getType().toString();
        switch(itemMaterial) {
            case "1" -> type = "sword";
            case "2" -> type = "shovel";
            case "3" -> type = "axe";
            case "4" -> type = "pickaxe";
            default -> {
                return;
            }
        }

Ignore those 1, 2, 3 and 4

Those would be the material to string

[dusty frost]

okay I don't think I understand what you want here

because you can just switch case over an enum

and it's like, actually type safe and stuff

[atomic trail]

How would I do that?

Ohhh

Got it...

[dusty frost]

lol

[atomic trail]

Thanks

I'm going to bed now lol, 5 am

[dusty frost]

yeah sidenote that's like the whole point of enums lol

[atomic trail]

Yeah makes sense, and I would've probably just thought of that if I was awake lol

[forest jay]

That doesnt work with BaseComponents, and I am not sure how to convert as the docs for Adventure also don't explain how to do hover and click events.