Globox1997 / LevelZ

https://modrinth.com/mod/levelz
https://www.curseforge.com/minecraft/mc-mods/levelz
GNU General Public License v3.0
51 stars 50 forks source link

【BUG】MODDED ARMOR ISNT RESTRICTED TO WEAR #183

Open tiigerbones opened 2 years ago

tiigerbones commented 2 years ago

i havent full tested the tools and weapons but when trying to equip armor from -the twilight forest -betterend -betternether -advanced netherite and more...

it doesnt apply level restrictions allowing the player to equip it at any level

xR4YM0ND commented 2 years ago

Try this in the item folder with "item": "minecraft:armor", and replace = true

When the armor is not recognized use the "item": "minecraft:custom_item", and then you have to set every single piece on the list

tiigerbones commented 2 years ago

Try this in the item folder with "item": "minecraft:armor", and replace = true

When the armor is not recognized use the "item": "minecraft:custom_item", and then you have to set every single piece on the list

image

actually thats how i initially did things with replace set to true should i try setting to false?

xR4YM0ND commented 2 years ago

Yeah try this with replace = false and true

Check what the material id this armor has

Normally they are stored in ENums (check code for that)

When its not working than try simmiler material names

And last option -> use custom_item

Bear in mind. When you have a lot of customs the performance lowers

Need more help, hit me up on Globox's Discord (#RAY)

tiigerbones commented 2 years ago

uh lets say for example the advanced netherite mod has advancednetherite:netherite_gold_ingot how would i check the material id or is that the one i should use? image

also sorry i would contact you on discord but i dont see the discord server inv link anywhere

tiigerbones commented 2 years ago

also i tried both false and true with and i get the same result it defines the defense level and everything but players can wear it at any level image

xR4YM0ND commented 2 years ago

https://discord.gg/ZGRDBv3fFg

xR4YM0ND commented 2 years ago

For Record:

Advanced Netherite:

~ If you find more comment down below

tiigerbones commented 2 years ago

The following mod tools:

still give the mined blocked (pickaxe and axe) the farming tools can still perform actions and do get damaged

Globox1997 commented 2 years ago

Added a command which tells the material string with https://github.com/Globox1997/LevelZ/commit/1aaf576c167bd7057aee5b336bb0dc9fd7e3f8fc I highly recommend to not use the custom list

xR4YM0ND commented 2 years ago

In the new version there will be a command

/info material

that displays the material id for the holding item.

Btw for advanced netherite the material id is:

"advancednetherite:itemname"

Mrkuff commented 2 years ago

Very nice mods btw! I love it very much. I'm trying to use it with "MC Dungeons Armors" and there some armor with restriction (like Turtle Armor asking for Defense 3) and other like the "Guard armor set" who don't. My user bypass the "Iron level" with it. This armor only ask for leather pieces surround by iron ingot. How can i define the 'Guard armor set' for asking defense 8 like the iron set? Thank a lot for all!

Mrkuff commented 2 years ago

ok i've activate the dev mode in a creative save and i got this id for the Guard Helmet : "mcda:guards_armor_helmet". But since the receipe for this piece use iron ingot, isnt supposed to be detected as "material": "iron" ? image

xR4YM0ND commented 2 years ago

what happens when you hold the guards armor in your hand and you type /info material ?

Mrkuff commented 2 years ago

it's : Material id: "guards" Can i change the material id on the armor deffinition instead in "MC Dungeons Armors"? what's the list Material id used by Levelz? Seems to have a material id diffrent by kind of armor :\