map the materials if you want such behaviour...
#help-development
vestal dome
quaint mantle
Yea but there might be a list of those items
tardy delta
i guess you have to itertate Material.values()
summer scroll
Make it so the file is on the same directory as the icecube.iml and .gitattributes
quaint mantle
i guess you have to itertate Material.values()
Yea but that gives almost every item
tardy delta
Material#isBlock should
torn shuttle
has anyone ever run into and fixed this issue before?
quaint mantle
but still some blocks will bypass that (like TNT which is in the redstone tab)
mint quiver
hm , thanks u all for help , someone calling me now, gtg
summer scroll
You might want to filter it yourself.
tardy delta
but still some blocks will bypass that (like TNT which is in the redstone tab)
you could make an enumset with the values
neat maple
How much is it allowed to obfuscate a premium plugin on Spigotmc.org?
tardy delta
with the materials i mean
quaint mantle
Aight thanks
neat maple
https://www.spigotmc.org/threads/rule-changes-re-obfuscation.420747/?__cf_chl_f_...
thx
worldly ingot
fervent gate
https://paste.md-5.net/qidikacawo.xml POM.XML
Dependency 'com.github.libraryaddict:LibsDisguises:10.0.27' not found --> Error. Someone told me to use the libraryaddict one. Libsdisguises as it is shown on spigot doesn't work either
wet sigil
Hi. I was making a server with custom textures for guns. Creating the textures/ Custom Model Data wasn't the problem. I don't know how to apply it to the gun. Basically I have iron horse armor that is textured to look like a gun with the custom model data of 1. Does anyone know how to apply this to the crack shot gun version of it? Is there a way to do it in the crack shot config?
tardy delta
what is a crack shot gun?
eternal oxide
a plugin
tardy delta
oh
quaint mantle
Isnβt that plugin popular in 2018 or something?
It is outdated now i guess, iirc the author abandoned it already
vestal dome
event.getPlayer().getItemInHand().setAmount(event.getPlayer().getItemInHand().getAmount() - 1);
why doesn't this remove my item when it reaches only 1 item.... 1-1 = 0 which should remove the item, but it doesn't..
tardy delta
player.updateInventoty()?
vestal dome
why would that work?
tardy delta
idk
vestal dome
it's NOT a visual bug.
tardy delta
and with another amount it works?
glossy venture
wdym inefficient
vestal dome
and I want to ask if this is actually spigot behavior
glossy venture
the code internally would have to do the exact same
tardy delta
thats not inefficient thats just a getter and setter
summer scroll
What version are you on?
summer scroll
I remember on the newer version you can just set the amount to 0
tardy delta
if it becomes 0, remove it makes sense no?
glossy venture
when i did something like that the item showed like a red 0 next to it
summer scroll
I doubt that
glossy venture
idk what i did
summer scroll
What version are you on?
glossy venture
it seems like 1.8
vestal dome
not wrong.
glossy venture
it looks like
fervent gate
https://paste.md-5.net/qidikacawo.xml POM.XML
Dependency 'com.github.libraryaddict:LibsDisguises:10.0.27' not found --> Error. Someone told me to use the libraryaddict one. Libsdisguises as it is shown on spigot doesn't work either
summer scroll
That's probably because of that, the API is not updated yet maybe.
You might want to check the amount and set the item in hand to null or something
vestal dome
.
wet sigil
How would I go about giving a Custom Model Data value to plugin items? (items given by plugin commands)
vestal dome
well that is probably worthless.
my point is simple, if there was a bug, there's a potential chance it should be fixed.
glossy venture
How would I go about giving a Custom Model Data value to plugin items? (items gi...
i think theres ItemMeta#setCustomModelData(int)
tardy delta
yea
wet sigil
i think theres `ItemMeta#setCustomModelData(int)`
in the config of the plugin? or where would I find that. (Im not really good with plugins)
fervent gate
in the config of the plugin? or where would I find that. (Im not really good wit...
You add that in the code
glossy venture
or trying to configure one
wet sigil
configure, Im trying to basically take an item a plugin gives me and add custom model data to it
glossy venture
fervent gate
https://paste.md-5.net/qidikacawo.xml POM.XML
Dependency 'com.github.libraryaddict:LibsDisguises:10.0.27' not found --> Error. Someone told me to use the libraryaddict one. Libsdisguises as it is shown on spigot doesn't work either
summer scroll
my point is simple, if there was a bug, there's a potential chance it should be ...
Well, can't blame spigot too, 1.8 is old
glossy venture
is 1.8 even supported?
vestal dome
Well, can't blame spigot too, 1.8 is old
well I can, because if 1.8 was updated, it SHOULD be fixed.
as I said.
at least that's my mindset.
vestal dome
I mean fortunately
,./version says you're capping
obtuse ferry
I have data for example "a", "b", "c" in python to iterate through this I'd do
for i in ["a", "b", "c"]:
how would i go about doing this in java
summer scroll
,./version says you're capping
You are just not using the latest version on 1.8
vestal dome
sure I'll build 1.8.8 again no problem
fervent gate
I have data for example "a", "b", "c" in python to iterate through this I'd do
`...
I think for(String i : arr)
obtuse ferry
I think ```for(String i : arr)```
ty
fervent gate
Couldn't really find something for this exact issue:
https://paste.md-5.net/qidikacawo.xml POM.XML
Dependency 'com.github.libraryaddict:LibsDisguises:10.0.27' not found --> Error. Someone told me to use the libraryaddict one. Libsdisguises as it is shown on spigot doesn't work either
vocal cloud
1.8.8 sheeesh did i fall back in time
vestal dome
you kinda did... f
tardy delta
some people are living in the past
vestal dome
not that I'm living in the past, I know what I am going to get for maining my plugins 1.8, but I have to prepare for people who refuse to update versions π
vocal cloud
Unless your plugin becomes a staple like luckperms you really don't lol
vestal dome
tell me why I shouldn't.
golden turret
help
vocal cloud
tell me why I shouldn't.
Because unless you're working for someone making a plugin in an older version is less useful as a majority of server owners looking for plugins to use are running newer version s. And because people here are much less inclined to help you.
copper scaffold
I have a question. How can i make something like the plots-Plugin like every user can only build on their own plot. How can i make something like that
quaint mantle
If I wanted to make a jagged / 2d array I would do this right? ik this is more java not spigot but
coordinates = new double[11 + 20 * 2][];
int arrayLocation = 0;
for (double i = 0; i <= Math.PI; i += Math.PI / 20) {
double radius = Math.sin(i);
double y = Math.cos(i);
for (double a = 0; a < Math.PI * 2; a+= Math.PI / 20) {
double x = Math.cos(a) * radius;
double z = Math.sin(a) * radius;
coordinates[arrayLocation++] = new double[] { x, y, z };
}
}
Use a database to store the quadrants? I don't understand the question really setup claim commands and permission levels within each for the amount they can claim
vestal dome
Because unless you're working for someone making a plugin in an older version is...
it's not like I'm asking for forge support on spigot server, spigot 1.8.8 is like spigot 1.18 but "worse",
copper scaffold
Use a database to store the quadrants? I don't understand the question really se...
and how can i set the radius where they can build?
vocal cloud
it's not like I'm asking for forge support on spigot server, spigot 1.8.8 is lik...
Yes so when you want to do anything and we give you the PDC you'll say "can't use it" and we'll roll our eyes because if we actually want to help we'll have to spend even more time figuring it out
vocal cloud
and how can i set the radius where they can build?
You either write your own system or use worldguard
copper scaffold
okay thx
quaint mantle
If I wanted to make a jagged / 2d array I would do this right? *ik this is more ...
Mike yk or anyone?
vocal cloud
I'm not too familiar with it if you know what you're looking for then just test it and see i suppose
quaint mantle
I tested what supposedly causing the error but java wanna be dumb so I have no idea Xd, Iβll just wait for someone to respond I suppose Xd
tardy delta
cute doggo
spiral light
i love that this works:
tardy delta
π΅βπ«
quaint mantle
Brain hurt
spiral light
cant see any problem
obtuse ferry
Is #damage() in terms of hearts or some obscure value?
damage(2) would be 1 heart would it not?
spiral light
1 heart = 2.0d
1/2 heart = 1.0d
1/4 heart = 0.5d
obtuse ferry
π
spiral light
you can have players with 0.1 health and they still alive
tardy delta
yea
quaint mantle
hey, is there some sort of Command execution event in bungeecord?
spiral light
?jd
undone axleBOT
spiral light
maybe chatevent
wary harness
hey how would I get getNearbyEntities with radius
spiral light
World.getnearbyentites iirc
spiral light
nahh i dont wanne live in ancient versions
buoyant viper
actually when did java allow <> instead of the full type
was it 7 or 8
it mightve been 7 bc then 8 were lambdas
write it in java 6 
spiral light
i love that this works:
Actually you could just do: HashMap<ListType, Object> xD
mint mesa
Hello! I was just developing a bit for fun, with my second plugin and I found a bunch of errors that weren't there last time. I was looking for a root for the problem and found this
ok I cant even send images
cool
opal juniper
you have to verify for images
mint mesa
oh hold on
I have @NotNull in front of everything in my CommandExecutor.java file
and this is how I need to set commands apparently
and when I build the project and add it to the plugins folder in my server directory
it says "are you sure your plugin is up to date
chrome beacon
Is your command in the plugin.yml
mint mesa
yes
undone axleBOT
opal juniper
you have not imported the annotations
opal juniper
vale ember
what will java do if i try to us evalueOf on enum and the constant with specified name don't exist? will it throw exception or return null?
mint mesa
lemme check if I accidentally downloaded the wrong spigot version
oh right
I was using 1.18 instead of 1.8
my bad my bad
tardy delta
probably
worldly quest
i was getting this error when doubling hp:
java.lang.IllegalArgumentException: Health must be between 0 and 20.0, but was 40.0. (attribute base value: 20.0)
is it for a player?
worldly quest
a mob
tardy delta
isnt there a .setMaxHealth method?
worldly quest
[10:49:53 ERROR]: Could not pass event CreatureSpawnEvent to mypluginv
oo lemme try that
thanks
tardy delta
why dont people use google tho
signal locust
Anyone willing to help me with Optimizations? I have a massive plugin and am having TPS drop on a specific Class. I have tried to make it as effecient as possible yet still having issues with TPS drop (Server resources are in the single digits during TPS drop)
worldly quest
well it was but its deprecated
latent zephyr
Hello, I want to create custom achievements in my plugin, can I do it without special APIs and such?
tardy delta
heh what did i see
latent zephyr
but yes it is
No.
tardy delta
is 1.8 pvp with the fishing rod spam?
quaint mantle
it is extremely primitive and bound to the mouse capability of clicking
worldly quest
or an setAttribute?
didnt realize u were talking to me but im 99% sure that works
thanks
tardy delta
latent zephyr
it is extremely primitive and bound to the mouse capability of clicking
Okay, It explains everything.
I think this chat is not intended to discuss the mechanics of minecraft.
quaint mantle
Okay, It explains everything.
*I think this chat is not intended to discuss the ...
Oh, yes, the status, it definetly makes sense and is good argument in the discussion
latent zephyr
*Oh, yes, the status, it definetly makes sense and is good argument in the discu...
We did not have a discussion, there was only your not smart statement that did not correspond to the topic of the chat.
I'm not going to continue discussing this, have a good evening.
quaint mantle
how could i create a sphere radius and check if a user walks in it, basically how can i make an array/list of coords in a sphere shape?
tardy delta
should i store like a collection of vanished players in the vanish command class or should i store all my command related collections in one class?
im remaking my whole plugin
quaint mantle
isnt it stored by bukkit itself
tardy delta
uhm maybe but that wasnt my question
each player stores a weakly referenced set with hidden players for them iirc
glossy venture
isnt it stored by bukkit itself
its stored by spigot per player i think
like Player.Spigot#getHiddenPlayers()
spiral light
how could i create a sphere radius and check if a user walks in it, basically ho...
why a list of coords ..... ?
all you need is a location and check the radius you want for players
glossy venture
how could i create a sphere radius and check if a user walks in it, basically ho...
if you want to check if they are in the circle, use something like this:
circleLocation.distance(player.getLocation()) < radius;
like, depends. Do you need that collection outside of the command class?
glossy venture
actually thats a sphere
remove the y component if you want a 2d circle from 0-255
or a cilinder in that case
tardy delta
like, depends. Do you need that collection outside of the command class?
i'm planning on handling all interactions with that collection from within the class
quaint mantle
if you want to check if they are in the circle, use something like this:
```java...
im stupid i didnt know distance was a thing lmfaooo
i was going to create a sphere of cords store it in db and check each individual cords. thats much easier ty
glossy venture
np
quaint mantle
i'm planning on handling all interactions with that collection from within the c...
Welp, you can do some sort of manager for vanished players
delicate lynx
I am trying to drop items. I have a list of materials that is generated. once generated, I then create the items and actually drop them. however, it keep getting "cannot drop air" when anything drops.
for (Material dropMaterial : materials) {
int quantity = new Random().nextInt(maxDrops - minDrops) + minDrops;
ItemStack itemToDrop = new ItemStack(dropMaterial);
if (itemToDrop.getType() != Material.AIR) {
itemToDrop.setAmount(1);
for (int i = 0; i < quantity; i++) {
location.getWorld().dropItemNaturally(location, itemToDrop);
}
}
}
here is the code I am using. I check if the item is air, so not sure what's up. are there other reasons for why and item can be "air?"
tardy delta
dont create the random each time pls
trail bough
hey - making sure that doing an if (sender.teleport(location)) would still teleport a player, correct?
glossy venture
yeah
trail bough
ok good lol
i was 90% sure but i just wanted to make sure it didnt break something somehow
glossy venture
it calls the method so it will do everything it normally does
tardy delta
Welp, you can do some sort of manager for vanished players
i seemed me needless to make a manager for just like 5 small methods or something
quaint mantle
i seemed me needless to make a manager for just like 5 small methods or somethin...
Even 5 methods makes sense if it shares the common logic. It is better than having repeating code in different places
tardy delta
i thought it was good to implement all the logic directly in my command clsass
maiden thicket
is it normal for creaturespawnevent to trigger when the player only is on the server?
glossy venture
what
quaint mantle
i thought it was good to implement all the logic directly in my command clsass
With commands, you just handle user interfaction with your program
tardy delta
i had the idea that were was too little code in my executor
if i handled all the logic somewhere else
glossy venture
CreatureSpawnEvent only triggers if someone is on the server
i honestly dont know
spawn chunks should be loaded
but i dont know if some optimization causes them to not spawn stuff or something
CreatureSpawnEvent is called whenever a living entity (mob or player) spawns
quaint mantle
i had the idea that were was too little code in my executor
no problem! Less code == easier to navigate and debug
glossy venture
wait an ender dragon is actually handled as multiple entities?
i can give permissions to an ender dragon part?
visual tide
can plugin classes be access from inside craftbukkit/spigot?
tardy delta
my code looks weird
glossy venture
can plugin classes be access from inside craftbukkit/spigot?
you mean like from the server implementation?
because if so yes
visual tide
hm odd
hollow arch
Been a while since I've done anything with Spigot plugins, is the bukkit scheduler the recommended tool for scheduling simple actions & delays?
quaint mantle
my code looks weird
Aaaaqwa big font small resolution
visual tide
you mean like from the server implementation?
because im modifying WatchdogThread to call my method when the server crashes and its just getting a classnotfound
tardy delta
Aaaaqwa big font small resolution
better?
fervent gate
How would I implement libsdisguises? This is my build.gradle: https://paste.md-5.net/qadivahebi.rb
tender shard
where is the `PluginLoader` implementation wtf
SimplePluginLoader, isn't it?
glossy venture
SimplePluginLoader, isn't it?
nah thats the plugin manager
its JavaPluginLoader
in org.bukkit.plugin.java
fervent gate
How would I implement libsdisguises? This is my build.gradle: https://paste.md-5...
No error, doesn't work either
tardy delta
:kek:
glossy venture
bruh
glossy venture
How would I implement libsdisguises? This is my build.gradle: https://paste.md-5...
are you 100% sure that the group name is LibsDisguises?
because that doesnt seem right
fervent gate
It is
glossy venture
oh ye
alright
and it doesnt give an error
10.0.21 is the latest version
try that
also it says here that the id=md_5-public but i dont know if that matters
manic crater
I know this is gonna be a noob question and you all will yell at me, but i don't work with guis much but i was wondering how do i open a gui from another class????
quaint mantle
how can i send a user an action bar title? i see sendTitle but i dont see an option for actionbar
manic crater
how can i send a user an action bar title? i see sendTitle but i dont see an opt...
player.sendMessage(ChatMessageType.ACTION_BAR, TextComponent.fromLegacyText(msg in here color codes work as well));
quaint mantle
sendmessage chatmes- yep kk
manic crater
sorry... >_<
quaint mantle
nah ty xd
manic crater
I know this is gonna be a noob question and you all will yell at me, but i don't...
btw really random, but do you know how to do this?
tardy delta
you have to use a runnable for an actionbar to keep it sending iirc
manic crater
^
tardy delta
i tried without and it never showed up
quaint mantle
I know this is gonna be a noob question and you all will yell at me, but i don't...
no idea havent worked with guis yet sorry maybe var dunno
manic crater
no idea havent worked with guis yet sorry maybe var dunno
ok, >_< well thank you for trying anyway,
quaint mantle
you have to use a runnable for an actionbar to keep it sending iirc
its going to be in the player move event so
manic crater
its going to be in the player move event so
Terrible idea,
your server will become laggy and most likely crash.
and i learned that from scoreboards.....
tardy delta
I know this is gonna be a noob question and you all will yell at me, but i don't...
i read "how to work with girls" and i want to know that too π
quaint mantle
i have if statements so it wont run 24/7
quaint mantle
and i learned that from scoreboards.....
f
tardy delta
its going to be in the player move event so
brr that expensive
manic crater
*i have if statements so it wont run 24/7*
Still, it is better to use a runnable.
quaint mantle
kk
quaint mantle
good to know
tardy delta
and if the player stops moving its gone
north trench
How to make a cooldown when it right click
manic crater
okay or that ig,
manic crater
i read "how to work with girls" and i want to know that too π
Nah but in all seriousness do u know how to do that though?
proud basin
why use uuid when you can hash it then use that
tardy delta
Nah but in all seriousness do u know how to do that though?
nah :(
manic crater
rip..
manic crater
why use uuid when you can hash it then use that
#help-development message <- Do u know how to do this?
proud basin
yea
tardy delta
π
manic crater
yeah ik but how do u get it from another class is what im trying to do,
tardy delta
share the inventory instance
proud basin
make an instance of the class
manic crater
can i have a quick example???
tardy delta
i made this
https://github.com/FourteenBrush/MagmaBuildNetwork/blob/main/src/main/java/io/github/FourteenBrush/MagmaBuildNetwork/gui/ShopGui.java
see it all extends GuiCreator
manic crater
Like i said i rarely work with guis, so this is kind of something ive just started doing.
proud basin
ClassA classA = new ClassA();
tardy delta
so i open them by making a new instance of it
fervent gate
also it says here that the id=`md_5-public` but i dont know if that matters
Still refuses to work. Do you have a build.gradle file where it does work?
manic crater
ClassA classA = new ClassA();
oh ok, then i can just use classA.<whatever method here> yeah?
quaint mantle
your server will become laggy and most likely crash.
no.. Move event is not that bad, if you dont do setup stupid things, for sure
proud basin
I believe you can also do new ClassA().mymethod in the openInventory method
glossy venture
Still refuses to work. Do you have a build.gradle file where it does work?
no
sorry then i have no clue
proud basin
oh ok, then i can just use classA.<whatever method here> yeah?
yea
manic crater
I believe you can also do ``new ClassA().mymethod`` in the openInventory method
ohh ok let me give this a try and ill let u know,
tardy delta
is how to work with girls also help-development?
proud basin
yes
tardy delta
oh π³
proud basin
it states above "Ask other questions here"
proud basin
show me your code
manic crater
kk 1 sec let me open up pastebin,
Then this is my main class, https://pastebin.com/wAWkkJyz
tardy delta
static ._.
manic crater
??
tardy delta
nvm
manic crater
private final Inventory inv1; <- Ig i have to change this to private static?? or something?
wintry badger
Hi, how to set the result of PrepareItemCraftEvent ?
i want the result to be null
or just cancel the event if possible
manic crater
Hi, how to set the result of PrepareItemCraftEvent ?
R u trying to remove crafting?
wintry badger
yes without losing the items
manic crater
so u dont want ppl to craft in general??
wintry badger
no no, i just want to disable combining two elytras
so they wont be able to repair
so im checking if PrepareItemCraftEvent inventory contains any elytra
and i will not let it to be crafted
fervent gate
Does anyone have any experience with working with the LibsDisguises API?
wintry badger
and i will not let it to be crafted
okay, setResult is a member of event.getInventory
in case anyone will have the same issue
quaint mantle
Hey, Guys!
Do you know what event should I cancel to prevent players from taken an Item out of an ItemFrame?
Neither PlayerInteractAtEntity, PlayerInteractEntity nor HangingBreakByEntity seem to work
visual tide
does the main thread have to be working (not blocked) for events to work?
ivory sleet
Depends if events are invoked on the main thread
left swift
is it possible for the server to think that, for example, 3 pigs is really one entity?
spiral light
is it possible for the server to think that, for example, 3 pigs is really one e...
you can just not add them to the world .... but then you have to calculate all movements and stuff by yourself (and also all items/names stuff via packets)
\n
quaint mantle
\n
quaint mantle
player.sendMessage(ChatMessageType.ACTION_BAR, TextComponent.fromLegacyText(*msg...
when using this i get the err
sendMessage(net.md_5.bungee.api.ChatMessageType, net.md_5.bungee.api.chat.BaseComponent[])
oops forgot to turn off ping
it seems that uses bungee api?
ye i have to do ```java
player.spigot().sendMessage(ChatMessageType.ACTION_BAR, new TextComponent("text"));
stoic brook
ye i have to do ```java
player.spigot().sendMessage(ChatMessageType.ACTION_BAR, ...
Hehe i have developed action var the whole day. So that code looks like something i have seen to many times today π
river hearth
Database question
worldly quest
how would i check if a certain enchanted book has a certain level of enchant? i want to have players drag and drop books onto weapons in their inventory and add the enchants
but only unsafe enchantments
little elm
Hi. What is the suggested way for persistent data storage? In particular, I would like to store coordinates.
sterile token
How would I loop a Map<Slot, ItemStack> using lambda for returning the ItemStack if the slot given equals to the one in the map?
trail bough
For coordinates you csna just store it in a YML file
sterile token
Hi. What is the suggested way for persistent data storage? In particular, I woul...
YAML and using ConfigurationSection
trail bough
^
sterile token
How would I loop a Map<Slot, ItemStack> using lambda for returning the ItemStack...
I have tried with: ```java
map.entrySet().stream().filter(a -> a.equals(slot)).findFirst().orElse(null)
You have a map yet you're streaming it? Kinda defeats the purpose of having a map
sterile token
You have a map yet you're streaming it? Kinda defeats the purpose of having a ma...
Yeah
I need a map for saving the Buttons of my custom Menu. And then i have to loop throught it for getting the button via slot and checking if slot equals
worldly quest
how would i check if a player is holding a shulker box? would i have to hard code all the colors of each type of box?
vocal cloud
how would i check if a player is holding a shulker box? would i have to hard cod...
instanceof I imagine
worldly quest
ooh ty
vocal cloud
Or you can always do .contains("SHULKER_BOX") on the material
sterile token
You have a map yet you're streaming it? Kinda defeats the purpose of having a ma...
So?
sterile token
I have this Map<Slot, ItemStack>
Im looping it and i want to get the object via slot and check if slot given in method equals to one in the map
worldly quest
is there a way to check if a shulker has a certain item like an enchanted book
undone axleBOT
vocal cloud
Im looping it and i want to get the object via slot and check if slot given in m...
So wait, you have a method that takes in a slot and yet you're looping through a map? You know what a map is used for right?
sterile token
So wait, you have a method that takes in a slot and yet you're looping through a...
Yeah
public MenuButton getButton(int slot) {
return this.buttons.entrySet().stream().filter(i -> i.equals(slot)).findFirst().orElse(null);
}
That my method
It wont work because im comparing the objects not the slot
sterile token
i alr looked at docs
aa ok
vocal cloud
It wont work because im comparing the objects not the slot
Why are you streaming a map? Do you understand how a map works?
sterile token
Why are you streaming a map? Do you understand how a map works?
Because i like using lambda
ivory sleet
Performance 
sterile token
No i have read that lambda perfomance was solved
ivory sleet
Thatβs not it
vocal cloud
Performance <:roocry:744140381473472623>
Trust me I'm already suffering
ivory sleet
The time complexity is still O(n)
vocal cloud
Okay so do you understand what a map is for
sterile token
Okay so do you understand what a map is for
Yeah for mapping 2 values
manic crater
I know this is gonna be a noob question and you all will yell at me, but i don't...
Still need help on this
ivory sleet
just because lambda uses invoke dynamic doesnβt mean everything that uses lambdas are inherently performant
The bytecode instruction is fast sure
vocal cloud
Yeah for mapping 2 values
Maps are a method of mapping 1 value as a key to another value as a value
ivory sleet
But if used incorrectly, like everything else, your performance still drops
sterile token
Maps are a method of mapping 1 value as a key to another value as a value
Yes i know
vocal cloud
So you can query a value instantly
ivory sleet
In fact stream creates a lot of objects under the hood
vocal cloud
Yet for some reason you're streaming it
sterile token
So you can query a value instantly
Yes i know but i need to stream it because i cannot return the object if slot given its not in the map
vocal cloud
Yes i know but i need to stream it because i cannot return the object if slot gi...
if it's not in the map then why stream it what
ivory sleet
itβs faster, itβs more readable in this scenario and itβs more flexible
vocal cloud
Store an int for the slot id then get it with map::get like the good doggo is saying
good doggo
sterile token
just use map::get
I know but what about if slot given its not in map?
ivory sleet
getOrDefault
sterile token
It will explote instead of returning null
ivory sleet
get and null check
computeIfAbsent
computeIfPresent and return identity
there are options
vocal cloud
getOrDefault is what you really want for null prevention
For the love of God don't stream a map 
ivory sleet
If itβs a Map<Integer,V> you could also go with an Int2ObjectMap<V>
paper viper
Store an int for the slot id then get it with map::get like the good doggo is sa...
dont even use a map
bruh
use an array in this case
sterile token
I cannot because my menu works like this:
1- Create a button
2- Then you add it to map
3- Menu add buttons from map
4- On click event i check if click item slot its on the map
ivory sleet
Thonk
worldly quest
is there a way to check if a shulker has a certain item like an enchanted book
does anyone know how to do this? i saw one forum post and when i tried it createInventory was deprecated
ivory sleet
I believe Inventory::contains should do
sterile token
I cannot because my menu works like this:
1- Create a button
2- Then you add it...
That why i cannot use map#get or others methods
worldly quest
I believe Inventory::contains should do
ok ill try that thanks
sterile token
I believe Inventory::contains should do
But im not checking directly the slot
Im checking with custom class
ivory sleet
That wasnβt to you
ivory sleet
Yeah
sterile token
I want to loop trought it, returning object and checking slot given its not null
ivory sleet
Iβm still extremely puzzled why you would use a stream
worldly quest
I believe Inventory::contains should do
how would u get the inventory to check if it contains a item i couldnt figure that out
sterile token
Iβm still extremely puzzled why you would use a stream
Because i can use filter keyword
ivory sleet
1 itβs not a keyword
2 if (boolean) literally exists
sterile token
1 itβs not a keyword
2 if (boolean) literally exists
- Add more lines of code
paper viper
im assuming you are dealing with inventory slots, so its fixed size anyways
ivory sleet
3 more lines of code != less readable code
ivory sleet
Look at sth like triumph gui
sterile token
The one i find uses stream#filter
ivory sleet
They make perfect sense on how to use a Map for inventories
ivory sleet
The one i find uses stream#filter
Link me
sterile token
` public GuiButton getButton(int slot) {
int exists = buttons.keySet().stream().filter(i -> i == slot).findFirst().orElse(-1);
if(exists != -1)
return buttons.get(exists);
return null;
}
`
paper viper
why do people use maps for inventories tho, its fixed size inventory, index based
perfect use of an array
gleaming grove
the slot numbers are fixed
ivory sleet
I mean you basically claim a lot of memory with an array where the majority of its positions arenβt pointing at any objects
Claim is the wrong word
But yeah
Space
Β―_(γ)_/Β―
sterile token
But why using an array?
vocal cloud
why do people use maps for inventories tho, its fixed size inventory, index base...
All that memory taken up for empty slots is a waste however. Maps only use what is assigned
ivory sleet
I mean I personally believe LinkedHashMap is the way to go
sterile token
How i will check that the slot you clicked on menu its the one i setted?
gleaming grove
public GuiButton getButton(int slot) {
//validate slot posiotion
GuiButton button= buttons[slot]
if(button== null)
{
//do sth
}
return button;
}
```because its easier and faster
sterile token
```java
public GuiButton getButton(int slot) {
//validate slot posiotion
...
Allright
buttons[] its an array
Hum
Im not using array
vocal cloud
Don't use arrays too much wasted space. Use a map assign buttons to positions. Much easier and less of a hassle
paper viper
well either way, you really need to rethink your algorithm here (verano)
ivory sleet
myEpicMap.computeIfPresent(event.getSlot(), (key,value) -> {
//for instance verano
return value;
});
sterile token
Don't use arrays too much wasted space. Use a map assign buttons to positions. M...
Its what im doing xdd
ivory sleet
(With the assumption you do not use null values
sterile token
myEpicMap.computeIfPresent(event.getSlot(), (key,value) -> {
//for instance ve...
wait
I will send full code
Im confused
?paste
undone axleBOT
gleaming grove
the slot number for 6 height gui is 54, so Buttons[] = new Buttons[54] takes 54*4 bytes of ram i guess, so it is not much
and every player has only one instance of GUI
paper viper
GuiButton can be a lot of memory tho it depends
gleaming grove
but the array its just list of pointers, so i does not matter how heavy are the object inside
ivory sleet
buoyant viper
:)
sterile token
young knoll
Yeah I donβt think using an array is gonna cripple your server
paper viper
i mean i personally prefer arrays cause speed lol
sterile token
https://paste.md-5.net/oroziyicim.java
That my complete code
young knoll
But hashmaps are hot
vocal cloud
But with a map you don't need to worry about doing if(value == null) or other crap you can just return a default
sterile token
https://paste.md-5.net/oroziyicim.java
So with this, it would work right?
So what do you think?
vocal cloud
public MenuButton getButton(int slot) {
return this.buttons.get(slot);
}
``` isn't this nicer than some crazy foreachit will throw exception if slot is not inside Hashmap
vocal cloud
paper viper
plan your logic first then implement
sterile token
And thanks
sterile token
it will throw exception if slot is not inside Hashmap
That why i was trying to use stram#filter
manic crater
need help, i figured out the inventory cross thing issue, Now a new issue occurred which makes no sense to me. For some reason you can steal items out of the gui, is there a way to fix this??? i tried e.setCancelled(true); but for some reason it wont cancel taking items out.....
paper viper
show code
gleaming grove
but i mean better to validate slot like if(slot > MaxSize || slot < 0) then...
it will be faster the stream
manic crater
@EventHandler
public void onInventoryClick(final InventoryClickEvent e) {
Player p = (Player) e.getWhoClicked();
if (e.getInventory() != inv1) {
return;
}
if (e.getInventory() == inv1) {
e.setCancelled(true);
VoteRewards voterewards = new VoteRewards();
if (e.getRawSlot() == 36) {
p.openInventory(new VoteRewards().SimpleConfig(p));
}
}
return;
}
Example please
young knoll
I normally just null check the value gotten from the map
sterile token
I normally just null check the value gotten from the map
How?
young knoll
computeIfPresent seems fun though
paper viper
sterile token
I cannot use other things because for putting it if it null, because the way how i create a menu
paper viper
you can do that too
sterile token
you can do that too
me?
gleaming grove
{
var position = index >= buttons.length ? buttons.length - 1 : index;
return buttons.get(position);
}``` for example this code returns the last button if slot is invalidBut im not using array
paper viper
me?
Yeah you can supply the second argument the fallback result
if the index isnt valid
sterile token
<https://docs.oracle.com/javase/9/docs/api/java/util/Objects.html#requireNonNull...
I apply that keyword to the method right?
sterile token
Yeah
paper viper
I apply that keyword to the method right?
its not a keyword
you just use the utility method
young knoll
Maps already have getOrDefault
sterile token
Maps already have getOrDefault
Yes but i cannot add if it null, cuz how i create the menu
I will send an example
if not you wont understand me
Omg with intellij my pc its eating 20gb of ram
What a nice IDE
gleaming grove
lol how
sterile token
MenuButton button = new MenuButton(new ItemStack(Material.DIRT), event -> System.out.println("[Menus] Player " + event.getWhoClicked().getName() + " clicked on " + event.getSlot()));
Menu menu = new MenuBuilder().setButton(3, button).build("&6&lTime", 9);
menu.open(player);
Hums
Discord get bug
Nice
manic crater
show code
So any idea on how i could fix this inventory issue from another class: #help-development message
sterile token
```java
MenuButton button = new MenuButton(new ItemStack(Material.DIRT), event -...
That its how i create the menu so if the Button its null i cannot add it
Jack
gleaming grove
so do like this, if the slot of method is out of index, cancel the operation
sterile token
So up to now the code with your change should be working
gleaming grove
{
if(index > MAX_SIZE)
{
return;
}
this.buttons.put(index,button);
}```like this
sterile token
``` public void setButton(int index,ButtonUI button)
{
if(index >...
I already check that i think
gleaming grove
why you have the 8 version of JAVA?
you are doing pluing for older version of MC?
sterile token
why you have the 8 version of JAVA?
Cuz i dont programe up to 1.12.2
gleaming grove
ok
sterile token
Please dont ban me
young knoll
Outdated! Get them!
sterile token
Hahaha in other server i got banend cuz i was not asking for help for newer versions
gleaming grove
you are doing your Core API as a Plugin??? @sterile token
sterile token
you are doing your Core API as a Plugin??? <@!406057762376908800>
This is for a library i will open source on ghb
That i will use for Core
Do you understand like its multi module proejct
Hahaha
gleaming grove
ghb ?
quaint bough
setting nms nbt data
manic crater
ghb i am pretty sure means github
manic crater
Yeah make it simplified and easier for people to use,
BUT speaking of MENUS/GUIS i need help.
sterile token
Yeah make it simplified and easier for people to use,
have you seen the example? Do you think its easy?
sterile token
BUT speaking of MENUS/GUIS i need help.
Me too
I tried and i can take the items
Out from my inventory
sterile token
(The buttons in my case)
sterile token
e.setCancelled(true);
It doesnt work
manic crater
have you seen the example? Do you think its easy?
no i have not yet to see the example :)
manic crater
It doesnt work
oh?
wait before i help u just wondering,
do u know how to fix my problem?
gleaming grove
``` public void setButton(int index,ButtonUI button)
{
if(index >...
it that code the answer for your problem? @sterile token
manic crater
My problem i got going on is i can still drag items out of the inventory BUT the inventory is in a diff class aside from my main class,
and i dont know how to cancel it out...
Cuz i tried e.setCancelled(true); and it wont work for some reason...
sterile token
it that code the answer for your problem? <@!406057762376908800>
The methods its the same as mine
The problems is that i will an exception when the clicked slot its not on the map
gleaming grove
if(buttons.contains(index) == false) {return} can you just do that in the OnInventoryClick?
sterile token
no i have not yet to see the example :)
MenuButton button = new MenuButton(new ItemStack(Material.DIRT), event -> System.out.println("[Menus] Player " + event.getWhoClicked().getName() + " clicked on " + event.getSlot()));
Menu menu = new MenuBuilder().setButton(3, button).build("&6&lTime", 9);
menu.open(player);
could anyone help me out π₯Ί #932424700389367828
young knoll
if(buttons.contains(index) == false) {return} can you just do that in the OnInve...
== false
Wat
sterile token
if(buttons.contains(index) == false) {return} can you just do that in the OnInve...
I dont have a getter for the map
manic crater
```java
MenuButton button = new MenuButton(new ItemStack(Material.DIRT), event -...
That kind of seems easy... just gonna take a little to get used to,
young knoll
Check if the value returned from the map is null
sterile token
I already checking that on InvclickEvent i think
manic crater
@EventHandler
public void onInventoryClick(final InventoryClickEvent e) {
Player p = (Player) e.getWhoClicked();
if (e.getInventory() != inv1) {
return;
}
if (e.getInventory() == inv1) {
e.setCancelled(true);
VoteRewards voterewards = new VoteRewards();
if (e.getRawSlot() == 36) {
p.openInventory(new VoteRewards().SimpleConfig(p));
}
}
}
It wont cancel it...... pls help someone i beg ;-;
gleaming grove
but the getButton has not this if if(buttons.contains(index)) { return null; }
manic crater
Do u know how to fix my issue..
gleaming grove
this is why you get erros
young knoll
No it isnβt
sterile token
this is why you get erros
Aaa
Now i udnerstand
Im getting error because the slot its not on the map
Yo urefer to that?
sterile token
It just returns null
But so i wont get an exception or yes?
young knoll
No
sterile token
Im confused between you and Jack
gleaming grove
```
@EventHandler
public void onInventoryClick(final InventoryClickEvent e...
if this is not working that means e.getInventory() is not inv1
sterile token
jack
manic crater
if this is not working that means e.getInventory() is not inv1
Well that makes no sense, how can i make it "inv1" do i have to close the other gui??
gleaming grove
if the key is not inside the Map and you use map.get(key) the exception will be thrown
sterile token
Another question i need to cancel the button movement from my MenuListener or from the button Consumer?
young knoll
No it wonβt
sterile token
No it wonβt
I tihin its return nulls
young knoll
Correct
sterile token
Another question i need to cancel the button movement from my MenuListener or fr...
Do you agree?
manic crater
if this is not working that means e.getInventory() is not inv1
here im gonna send a pastebin of my code, and maybe you can help me find what i did wrong...
gleaming grove
No it wonβt
sorry, the exception will be thrown in C#
sterile token
Another question i need to cancel the button movement from my MenuListener or fr...
for this
candid plover
how to load world?
sterile token
how to load world?
have you read the javadocs?
Or search at least on google: "Spigotmc how to load a world?"
sterile token
Or search at least on google: "Spigotmc how to load a world?"
This sometimes always have a solution
young knoll
Using WorldCreator
sterile token
here im gonna send a pastebin of my code, and maybe you can help me find what i ...
I recommend you using pastebin from md5 (?paste)
undone axleBOT
candid plover
Or search at least on google: "Spigotmc how to load a world?"
Yes, i using Bukkit.getServer().createWorld(new WorldCreator("serverBuilds"));
sterile token
Because it format the code
manic crater
candid plover
But if I put two, it only initializes one.
sterile token
https://paste.md-5.net/bajapovogo.java
i will check it
manic crater
i will check it
Alright thank you,
Im having a e.setCancelled(true); issue, where it wont cancel out item clicking in the gui, so just trying to get help on wtf i did wrong
gleaming grove
https://paste.md-5.net/bajapovogo.java
i think the problem is you use first constructor instead of secound
so the inv1 is never initialized
manic crater
well how do i go about initializing inv1?
gleaming grove
this.inv1 = null;
plugin = instance;
this();
}```ok ok,
sterile token
``` public EXPGui(VoteRewards instance) {
this.inv1 = null;
plug...
And second error he is not checking about the top inventory and no the player one
manic crater
ill update u
gleaming grove
do this code so the two constructors will be triggered
lost matrix
Good thing loops arent available in java
manic crater
error 101,
gleaming grove
public EXPGui(VoteRewards instance) {
this.inv1 = null;
plugin = instance;
}
Dude we would be here for ages if that was the case.
manic crater
public EXPGui(VoteRewards instance) {
this.inv1 = null;
plugin =...
Thats how it was before hand..
sterile token
XD
Jack so do i have to cancel item movement on my MenuListener or in the Consumer (InventoryClickEvent )
gleaming grove
sry the this(); should be the first thing
manic crater
ok ok, no worries, just give me a second to see if this is gonna work,
if it does i will gladly let u know :)
it doesn't wanna work.... skldjhf
young knoll
Real programming languages donβt need loops
lost matrix
+ i get this error now apparently.
Why do you need 2 constructors? Makes absolutely no sense here.
manic crater
Why do you need 2 constructors? Makes absolutely no sense here.
well i dont,
i tried without one of them, but it doesnt wanna work.
quaint bough
can you add nms nbt data to an extended class of itemstack?
sterile token
Why do you need 2 constructors? Makes absolutely no sense here.
In command handler i have multiple constructors. Hahaha. So it wrong??
manic crater
and use the loop plz
Will later, just wanna fix this set cancelled issue first,
lost matrix
can you add nms nbt data to an extended class of itemstack?
You cant extend ItemStack. ItemStacks are handled quite differently.
If you want to store data in an ItemStack just use its PDC
sterile token
for(int i=0;i<44;i++){ inv.1.setItem(i,......):}
HATE LOOP
sterile token
@gleaming grove listen
He forgot something
And you ignore me
PEOPLE he is not checking if the inventory he is clicking its the TOP one
lost matrix
e.getInventory() == inv1
Dont check Objects with == unless you want to check for identity
manic crater
Okay so what should i replace it with?
gleaming grove
e.getInvenotry().equals(inv1)
lost matrix
.equals
sterile token
``e.getInventory() == inv1``
Dont check Objects with ``==`` unless you want to c...
If you use Equals do internally that
SO dont fuck
Equals internally do an ==
ivory sleet
Not really
manic crater
Should i do the same with this?
gleaming grove
And you ignore me
sry what is your issue?
ivory sleet
Equals internally do an ==
Thatβs just the Object::equals implementation
derivatives may actually end up implementing it themselves
sterile token
sry what is your issue?
I was telling YOU that he forgot to checking if the invetory its the TOP one if not the event wont be cancelled
young knoll
Iβm pretty sure inventories are referentially comparable
sterile token
Thatβs just the Object::equals implementation
Ah allright
manic crater
I was telling YOU that he forgot to checking if the invetory its the TOP one if ...
oh ok, so what should i do then?
lost matrix
I was telling YOU that he forgot to checking if the invetory its the TOP one if ...
No he doesnt. The inventory is the primary inventory involved in the event. This part is fine.
sterile token
No he doesnt. The inventory is the primary inventory involved in the event. This...
Really?
ivory sleet
Yeah coll for most part? Altho iirc Bukkit sometimes create delegates
manic crater
i wanna give veranO123's idea a go, and see wtf happens,
lost matrix
Yes getInventory is always the top inventory unless there is only the players inventory present
manic crater
ah ok
quaint bough
You cant extend ItemStack. ItemStacks are handled quite differently.
If you want...
I am using 1.12.2 tho, but youre saying you cant extend itemstack, but you can? I am doing that right now I know that you dont get the object after its been added to the players inventory. But extending still works in order to create multiple items and some sort of abstraction if you know what I mean
manic crater
so then ig not then.
sterile token
No he doesnt. The inventory is the primary inventory involved in the event. This...
So i can remove this from my code??
analog prairie
Hiw can I add or remove thinks to Inventory
manic crater
here ill send an updated version of the gui thing and u tell me what i need to fix.
sterile token
Hiw can I add or remove thinks to Inventory
Inventory#setItem(slot, ItemStack) and Inventory#Remove()
lost matrix
I am using 1.12.2 tho, but youre saying you cant extend itemstack, but you can? ...
Extending the Interface ItemStack does exactly nothing as you will always get a new instance of ItemStack every so often.
gleaming grove
private final Inventory inv1;
public EXPGui(VoteRewards instance) {
plugin = instance;
inv1 = Bukkit.createInventory(null, 45, "Experimental Options");
inv1.setItem(36, createGuiItem(Material.ARROW, ChatColor.GREEN + "Back A Page", ChatColor.YELLOW + "Click to flip"));
inv1.setItem(0, createGuiItem(Material.BLACK_STAINED_GLASS_PANE, ChatColor.GREEN + ""));
inv1.setItem(1, createGuiItem(Material.BLACK_STAINED_GLASS_PANE, ChatColor.GREEN + ""));
inv1.setItem(2, createGuiItem(Material.BLACK_STAINED_GLASS_PANE, ChatColor.GREEN + ""));
inv1.setItem(3, createGuiItem(Material.BLACK_STAINED_GLASS_PANE, ChatColor.GREEN + ""));
}```use just one constructor
sterile token
use just one constructor
Now can u help me?
lost matrix
So i can remove this from my code??
should be event.getInventory().equals(menu.getInventory()) instead
sterile token
I can remove this check here?
and second thing: For calling item moving from gui i have to cancell it on MenuListener or in the Consumer (InventoyClickEvent)??
manic crater
```
private final Inventory inv1;
public EXPGui(VoteRewards i...
oki uh ill see 1 sec,
sterile token
should be ``event.getInventory().equals(menu.getInventory())`` instead
Ah allright
quaint bough
Extending the Interface ItemStack does exactly nothing as you will always get a ...
So how do you create custom items then within their own class?
lost matrix
I can remove this check here?
and second thing: For calling item moving from gu...
But you are tracking the inv opened by a player so you can also just remove that check all together actually...
manic crater
```
private final Inventory inv1;
public EXPGui(VoteRewards i...
Still letting me pick up items.
lost matrix
So how do you create custom items then within their own class?
By using a supported spigot version and accessing its PDC
sterile token
But you are tracking the inv opened by a player so you can also just remove that...
And the second one?
young knoll
So how do you create custom items then within their own class?
I use pdc to save a unique name to the item and then map said unique name to a custom item class
gleaming grove
but do you removed the first constructor? @manic crater
manic crater
but do you removed the first constructor? <@!773769547932237834>
ye
quaint bough
im 1.12.2 tho
manic crater
The thing is,
lost matrix
And the second one?
You just need to check if the menu is null and if the button is null. Everything else should be obsolete.
lost matrix
im 1.12.2 tho
π€·
ivory sleet
Yikes
sterile token
You just need to check if the menu is null and if the button is null. Everything...
I have done all that. But i still can move the items
manic crater
I dont know how to fix this issue,
manic crater
Learn JAVA
analog prairie
What's InOffHand
lost matrix
I have done all that. But i still can move the items
Then cancel the event
manic crater
its the item in ur off hand.
ivory sleet
Thatβs like the shield hand whes
manic crater
What's InOffHand
https://www.spigotmc.org/wiki/how-to-learn-java-programming/
Honestly learn this before u jump into such "big" projects for no reason.
sterile token
Then cancel the event
I actually NOW that i have to cancell it BUT WHERE?? On the MenuListener or Consumer?
ivory sleet
Do you play minecraft? Itβs quite hard to understand some of the methods if you donβt even know the content.
quaint bough
π€·
but I am not really extending itemstack just making a structure around the constructor. After its initialized i dont touch it does that make more sense?
manic crater
yeah same with that,
lost matrix
I actually NOW that i have to cancell it BUT WHERE?? On the MenuListener or Cons...
Wherever you think it makes sense. Depends on your menu implementation. Just cancel it somewhere.
manic crater
anyway i can still drag items out of the inv... and i fr have no clue how to fix this.. ive tried google, doing it on my own etc. and nothing seems to wanna work..
ivory sleet
Jelles usually when you have a custom item you use pdc to identify it, then a factory method to create it
analog prairie
It is difficult for people to understand it's actual use in these English
sterile token
Wherever you think it makes sense. Depends on your menu implementation. Just can...
Cancelling it on MenuListener doesnt affect i have tried
ivory sleet
the Bukkit ItemStack class is simply a middleman and a context provider
manic crater
It is difficult for people to understand it's actual use in these English
https://www.spigotmc.org/wiki/how-to-learn-java-programming/ BTW! you make 0 sense.
lost matrix
but I am not really extending itemstack just making a structure around the const...
CraftBukkit will always produce instances of CraftItemStack as their implementation of ItemStack.
You can extend that all you want but it will never get called.
ivory sleet
It is difficult for people to understand it's actual use in these English
Yeah, learning is just that hard, donβt give up tho, but maybe you wanna take a look at the javadocs for instance and if you lack experience in navigating those, then maybe thatβs your first stepping stone.
lost matrix
Cancelling it on MenuListener doesnt affect i have tried
Always cancel if menu is not null.
To also prevent bottom inv clicks
gleaming grove
It is difficult for people to understand it's actual use in these English
offhand means the left hand
analog prairie
Yeah, learning is just that hard, donβt give up tho, but maybe you wanna take a ...
Is there a wiki page for Inventory?
lost matrix
?jd
undone axleBOT
sterile token
Always cancel if menu is not null.
setCancel(true) right and should be before Button checking?
analog prairie
π€©
sterile token
Allrgiht thanks
sterile token
What happens?
manic crater
I can still drag items out of the gui..
sterile token
or i will cry about it
I have been crying with my menu since a week HAHAHA
manic crater
even after i cancelled it..
manic crater
I have been crying with my menu since a week HAHAHA
AHHHHH
sterile token
even after i cancelled it..
I havin the same yeat
lost matrix
I can still drag items out of the gui..
cancel the event
sterile token
cancel the event
Not works
manic crater
cancel the event
I did...
light parcel
Can EntityDamageEvent and EntityDamageByEntity event get thrown at the same time? e.g. player hits a player
manic crater
@EventHandler
public void onInventoryClick(final InventoryClickEvent e) {
Player p = (Player) e.getWhoClicked();
if (e.getInventory() != inv1) {
return;
}
if (e.getInventory().equals(inv1)) {
e.setCancelled(true);
VoteRewards voterewards = new VoteRewards();
if (e.getRawSlot() == 36) {
p.openInventory(new VoteRewards().SimpleConfig(p));
}
}
}
See
it was cancelled to begin with.
gleaming grove
Bukkit.getConsoleSender.sendMessage("IT WORKS");
e.setCancelled(true);
VoteRewards voterewards = new VoteRewards();
if (e.getRawSlot() == 36) {
p.openInventory(new VoteRewards().SimpleConfig(p));
}
}``` @manic crater replace your code with that and see if message will be sendIt does. If not then either:
The event is not called
or
Menu is null
Check for both
sterile token
It does. If not then either:
The event is not called
or
Menu is null
Check for b...
Menu its not null because i debug that
manic crater
As this doesn't work....
lost matrix
Menu its not null because i debug that
Show some code
manic crater
Ill just make it send a msg.
sterile token
Show some code
sterile token
Ill just make it send a msg.
Yeah that how you should debug
manic crater
it doesnt send a msg to console.
lost matrix
Debug it again to see if the menu is really not null.
Then check if your button reverses the cancellation of your event
sterile token
Debug it again to see if the menu is really not null.
Then check if your button ...
I already do it before
gleaming grove
```
@EventHandler
public void onInventoryClick(final InventoryClickEvent e...
@EventHandler
public void onInventoryClick(final InventoryClickEvent e) {
Bukkit.getConsoleSender().sendMessage("1");
Player p = (Player) e.getWhoClicked();
if (e.getInventory() != inv1) {
return;
}
Bukkit.getConsoleSender().sendMessage("2");
e.setCancelled(true);
VoteRewards voterewards = new VoteRewards();
if (e.getRawSlot() == 36) {
p.openInventory(new VoteRewards().SimpleConfig(p));
}
}``` try this and check the messagesDo it again. If menu is not null then you cant move items
sterile token
Do it again. If menu is not null then you cant move items
Remember that in MenuButton i have a consumer of InventoryClickEven t
I think thats why i can move the items
I think you havent seen the code i sent on paste
lost matrix
I think thats why i can move the items
only if the consumer reverses the cancellation. Otherwise it doesnt matter.
if (e.getInventory() != inv1) {
return;
}
I thought we established not to use identity equality with Objects...
manic crater
```java
if (e.getInventory() != inv1) {
return;
...
well i dont know what to replace "!=" with.
cuz i dont think the word "Doesn't" will work
brave sparrow
.equals
manic crater
yeah but im checking though if the inventory doesn't equal that,.
The whole point of that is so other's inventories doesn't act like the gui
lost matrix
yeah but im checking though if the inventory doesn't equal that,.
then negate the statement
ivory sleet
```java
if (e.getInventory() != inv1) {
return;
...
Yeah, unfortunately the spigot wiki uses it so probably why other devs also decide to stick to it.
brave sparrow
For objects == just checks the memory location, not the actual object data
Which isnβt what you want
manic crater
i changed it, and i can still drag the items out
manic crater
I don't know how to add item to player
First off using "e" instead of "event" is faster,
2nd, Please learn the literal BASIC MOVEMENT OF JAVA!
young knoll
Use addsItem rather than setItem
young knoll
First off using "e" instead of "event" is faster,
2nd, Please learn the literal...
Donβt use one letter variables
manic crater
Donβt use one letter variables
y
gleaming grove
First off using "e" instead of "event" is faster,
2nd, Please learn the literal...
why e is faster then event?
manic crater
why e is faster then event?
because then you don't have to type out "vent" every time.
young knoll
They arenβt descriptive
manic crater
well in events they are.
young knoll
Any ide will let you autocomplete it
manic crater
and it is kind of really obvious on what an event is.
ivory sleet
Yeah I mean theyβre both fine. The name of a variable should be proportional to the size of its scope; so e and event is fine, whilst event does in fact make it more explicit, a short event method would still make it clear that e is for event. 
sterile token
I alread recommend him learning how to use Intellij Idea Community
lost matrix
i changed it, and i can still drag the items out
This is your general setup:
public class CustomInvListener implements Listener {
private final Inventory customInv;
public CustomInvListener() {
this.customInv = Bukkit.createInventory(null, 3 * 9, "Cool Inv");
setupInv();
}
private void setupInv() {
// Fill your inv with garbage
}
@EventHandler
public void onClick(InventoryClickEvent event) {
Inventory primary = event.getInventory();
if (!customInv.equals(primary)) {
return;
}
event.setCancelled(true);
// Custom inventory was clicked here
}
}
Okay but why wont what i have work???
young knoll
Eclipse lets you autocomplete variables just fine lmao
sterile token
Please i need a logical explanation of how player inventory works and custom one. Because when you open a custom inventory his invetory its still open
That will helpme to understand more
ivory sleet
The player inventory is always open (iirc)
manic crater
Please i need a logical explanation of how player inventory works and custom one...
Just close the inventory when they close that inventory.....
lost matrix
Please i need a logical explanation of how player inventory works and custom one...
You probably have a Map<UUID, Menu> right?
brave sparrow
The playerβs inventory cannot be hidden if another inventory is open
Thatβs just Minecraft
ivory sleet
π₯²
lost matrix
Oh thats what he meant...
sterile token
You probably have a Map<UUID, Menu> right?
Yes
undone axleBOT
manic crater
yep
its just the dragging that causes issues,
like u can drag items out and i dont want that to happen...
lost matrix
https://paste.md-5.net/yelovebipe.java
First of all: DuckDuckGo the shortcut for auto formatting for your IDE, press it and then paste again.
manic crater
???
analog prairie
manic crater
wdym duckduck go?
lost matrix
google it.. -.-
manic crater
ik duckduckgo is a web browser.
just like
still okay whatever who cares about duckduck go for rn
late sonnet
You can check the docs.. need pass a Material in the construction of itemstack
lost matrix
Queue a search request with the search engine that you fancy.
manic crater
literally when i first started out in java not ONCE did i need to ask for help regarding giving an item...
lost matrix
literally when i first started out in java not ONCE did i need to ask for help r...
You use eclipse, right?
manic crater
yeah ctrl s is really helpful
lost matrix
https://paste.md-5.net/litixiyope.java
- make VoteRewards plugin private
- Group your fields:
private final VoteRewards plugin;
private final Inventory inv1;
public EXPGui(VoteRewards instance) {
this();
plugin = instance;
}
- rename your method
EXPConfigto something likeopenGUIand make it return void - remove the second constructor and merge them
analog prairie
manic crater
analog prairie
Is this error?
manic crater
Is this error?
Test it out then find out, honestly no one is gonna spoon feed you basic things.
analog prairie
I got api error
lost matrix
public void openGUI(HumanEntity entity) {
entity.openInventory(inv1);
}
id assume combind this(); and plugin = instance in EXPGui?
lost matrix
id assume combind this(); and plugin = instance in EXPGui?
remove the call to your other constructor and merge them
kind hatch
instance = this
analog prairie
[08:56:00] [Server thread/ERROR]: Fatal error trying to convert ExpTech_Economy v22w04-pre1:eventlistener/eventlistener.class
org.bukkit.plugin.AuthorNagException: No legacy enum constant for GRASS_BLOCK. Did you forget to define a modern (1.13+) api-version in your plugin.yml? ~[spigot-1.18.1-R0.1-SNAPSHOT.jar:3398-Spigot-f4ff00f-f3828bb]
at org.bukkit.craftbukkit.v1_18_R1.CraftServer.reload(CraftServer.java:941) ~[spigot-1.18.1-R0.1-SNAPSHOT.jar:3398-Spigot-f4ff00f-f3828bb]
at org.bukkit.Bukkit.reload(Bukkit.java:749) ~[spigot-api-1.18.1-R0.1-SNAPSHOT.jar:?]
at org.bukkit.command.defaults.ReloadCommand.execute(ReloadCommand.java:27) ~[spigot-api-1.18.1-R0.1-SNAPSHOT.jar:?]
at org.bukkit.command.SimpleCommandMap.dispatch(SimpleCommandMap.java:149) ~[spigot-api-1.18.1-R0.1-SNAPSHOT.jar:?]
at org.bukkit.craftbukkit.v1_18_R1.CraftServer.dispatchCommand(CraftServer.java:821) ~[spigot-1.18.1-R0.1-SNAPSHOT.jar:3398-Spigot-f4ff00f-f3828bb]
at org.bukkit.craftbukkit.v1_18_R1.CraftServer.dispatchServerCommand(CraftServer.java:806) ~[spigot-1.18.1-R0.1-SNAPSHOT.jar:3398-Spigot-f4ff00f-f3828bb]
at net.minecraft.server.dedicated.DedicatedServer.bf(DedicatedServer.java:453) ~[spigot-1.18.1-R0.1-SNAPSHOT.jar:3398-Spigot-f4ff00f-f3828bb]
at net.minecraft.server.dedicated.DedicatedServer.b(DedicatedServer.java:429) ~[spigot-1.18.1-R0.1-SNAPSHOT.jar:3398-Spigot-f4ff00f-f3828bb]
at net.minecraft.server.MinecraftServer.a(MinecraftServer.java:1206) ~[spigot-1.18.1-R0.1-SNAPSHOT.jar:3398-Spigot-f4ff00f-f3828bb]
at net.minecraft.server.MinecraftServer.w(MinecraftServer.java:1034) ~[spigot-1.18.1-R0.1-SNAPSHOT.jar:3398-Spigot-f4ff00f-f3828bb]
at net.minecraft.server.MinecraftServer.lambda$0(MinecraftServer.java:304) ~[spigot-1.18.1-R0.1-SNAPSHOT.jar:3398-Spigot-f4ff00f-f3828bb]
at java.lang.Thread.run(Thread.java:833) [?:?]
Add api-version: 1.13 to your plugin.yml
lost matrix
```
[08:56:00] [Server thread/ERROR]: Fatal error trying to convert ExpTech_Econ...
The stack trace literally asks you:
Did you forget to define a modern (1.13+) api-version in your plugin.yml?
analog prairie
is this necessary
manic crater
```
[08:56:00] [Server thread/ERROR]: Fatal error trying to convert ExpTech_Econ...
add api-version: 1.13 in your plugin.yml
analog prairie
Or is this only necessary to use the new version of the block
manic crater
.....
kind hatch
is this necessary
Yes it's necessary if you are creating a plugin post 1.13
manic crater
You dont even know basic java and you're getting people who are x1000 better to help u
when yet they normally redirect you to learn java,.
manic crater
then do api-version: 1.17
lost matrix
But I use 1.18
then insert 1.18
kind hatch
Then your api version would be 1.18
manic crater
oh wait 1.18 api is out now?
hm thats weird, bc for me it wont let me use 1.18 as a spigot jar file... and only 1.17.1 and under...
manic crater
then insert 1.18
now im getting an intilization error,
kind hatch
The point of that flag is to prevent the legacy code from running. Which it will do if no flag is provided.
The flag should really be set to the lowest version your plugin supports. (At least down to 1.13)
sterile token
A recommendation about APIS, would you call a project an API when the project works over an API?
ivory sleet
A recommendation about APIS, would you call a project an API when the project wo...
Maybe
sterile token
A recommendation about APIS, would you call a project an API when the project wo...
I think its a really opened question
ivory sleet
I mean you can have several apis inside your project
manic crater
I think its a really opened question
yeah that was my bad, but fr though in this scenario,
sterile token
I mean you can have several apis inside your project
Yeah but i dont know if correct calling something an API when its over/using another API
ivory sleet
Even if itβs not an interface
lost matrix
No dont. The openGUI method is fine as it is. No need to change it to anything else than void.
I can now see the problem emerging. You are creating several new instances of your GUI inventory. None of which are known by the registered listener.
ivory sleet
Yeah but i dont know if correct calling something an API when its over/using ano...
Well what you got?
manic crater
No dont. The openGUI method is fine as it is. No need to change it to anything e...
ok so then wat do i do about the error then?
sterile token
Well what you got?
Like for example an easy way of commands for spigot/bukkit will be called a CommandHandler or a CommandAPI?
lost matrix
ok so then wat do i do about the error then?
Treat your listener as a singleton
manic crater
Treat your listener as a singleton
Okay but ive been doing that for a while now...
sterile token
Yeah so its really strange the usage of word API on miencraft plugins
lost matrix
Okay but ive been doing that for a while now...
Your code tells me otherwise
sterile token
Because for my buiness owner if told him an API he dont understand why i called API
ivory sleet
I mean the whole api thing is kinda softy defined
kind hatch
When you really think about it, every single project is an API. It's just that we don't refer to it as one until another project depends on it.
manic crater
Your code tells me otherwise
well.... can u help me make it then a singleton??
ivory sleet
Object orientation is about making apis for yourself that hide the complexity and details of things (which has proven to be both good and bad)
lost matrix
well.... can u help me make it then a singleton??
Sure. Only construct a single instance of your listener and then never again.
manic crater
Sure. Only construct a single instance of your listener and then never again.
erm ok.......
sterile token
Object orientation is about making apis for yourself that hide the complexity an...
I ask because i talk with my work owner that works on .NET If i tell him about an api. He didnt understand because the menaning on spigot development its different
ivory sleet
In a sense itβs probably very different from the general api term
Which usually is an inferred reference to a web api
sterile token
In a sense itβs probably very different from the general api term
That answer i was waiting