Jump to content

crops 1.14


Guest

Recommended Posts

Post your project as a git repository, because that's the easiest way for me to look at everything.

Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable.  If you think this is the case, JUST REPORT ME.  Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice.

 

Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked.

 

DO NOT PM ME WITH PROBLEMS. No help will be given.

Link to comment
Share on other sites

  • Replies 96
  • Created
  • Last Reply

Top Posters In This Topic

Change this to final or the annotation doesn't work.

https://github.com/drmdgg/mcraft/blob/master/java/drmdgg/marijuanacraft/init/blocks/BlockInit.java#L15

 

Also, you don't need to implement IGrowable, CropsBlock already does.

https://github.com/drmdgg/mcraft/blob/master/java/drmdgg/marijuanacraft/init/blocks/plants/MPlant.java#L29

 

What is this? Why is this here? You aren't even using it anyway.

https://github.com/drmdgg/mcraft/blob/master/java/drmdgg/marijuanacraft/init/blocks/plants/MPlant.java#L40-L49

 

You are still assigning to fields directly:

https://github.com/drmdgg/mcraft/blob/master/java/drmdgg/marijuanacraft/MarijuanaCraft.java#L165

 

Why do you have both a BlockInit and a BlockList? Your BlockList is also not using annotations, also not using final, and also being assigned to directly.

 

As far as I can tell, you haven't changed anything I've asked you to change. Or if you have, you haven't updated your git repo, which makes my searching for problems completely pointless because its not your code (it is outdated).

Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable.  If you think this is the case, JUST REPORT ME.  Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice.

 

Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked.

 

DO NOT PM ME WITH PROBLEMS. No help will be given.

Link to comment
Share on other sites

1 hour ago, Draco18s said:

Change this to final or the annotation doesn't work.

https://github.com/drmdgg/mcraft/blob/master/java/drmdgg/marijuanacraft/init/blocks/BlockInit.java#L15

 

Also, you don't need to implement IGrowable, CropsBlock already does.

https://github.com/drmdgg/mcraft/blob/master/java/drmdgg/marijuanacraft/init/blocks/plants/MPlant.java#L29

 

What is this? Why is this here? You aren't even using it anyway.

https://github.com/drmdgg/mcraft/blob/master/java/drmdgg/marijuanacraft/init/blocks/plants/MPlant.java#L40-L49

 

You are still assigning to fields directly:

https://github.com/drmdgg/mcraft/blob/master/java/drmdgg/marijuanacraft/MarijuanaCraft.java#L165

 

Why do you have both a BlockInit and a BlockList? Your BlockList is also not using annotations, also not using final, and also being assigned to directly.

 

As far as I can tell, you haven't changed anything I've asked you to change. Or if you have, you haven't updated your git repo, which makes my searching for problems completely pointless because its not your code (it is outdated).

I sincerely hope this is better. but its not running now:

https://github.com/drmdgg/mcraft/tree/master/java/drmdgg/marijuanacraft/java/drmdgg/marijuanacraft

and I don't fully understand this log:

https://pastebin.com/nXAFMzkc

Link to comment
Share on other sites

Your item group is incorrect.

https://github.com/drmdgg/mcraft/blob/master/java/drmdgg/marijuanacraft/java/drmdgg/marijuanacraft/MJItemGroup.java#L7

 

You must use ModItemGroup and pass a Supplier<ItemStack> as at the point that the ItemGroup is constructed your icon item is null and vanilla code is Dumb about it.

 

Here's a working example:

https://github.com/Draco18s/ReasonableRealism/blob/1.14.4/src/main/java/com/draco18s/industry/ExpandedIndustry.java#L59

 

 

Also, your repo is not correctly set up:

https://github.com/drmdgg/mcraft

That should contain the build.gradle file, and other files at that root directory. The way you've done it has completely fucked up where your resources are.

For example, it should look like this:

https://github.com/Draco18s/ReasonableRealism

Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable.  If you think this is the case, JUST REPORT ME.  Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice.

 

Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked.

 

DO NOT PM ME WITH PROBLEMS. No help will be given.

Link to comment
Share on other sites

2 hours ago, Draco18s said:

Your item group is incorrect.

https://github.com/drmdgg/mcraft/blob/master/java/drmdgg/marijuanacraft/java/drmdgg/marijuanacraft/MJItemGroup.java#L7

 

You must use ModItemGroup and pass a Supplier<ItemStack> as at the point that the ItemGroup is constructed your icon item is null and vanilla code is Dumb about it.

 

Here's a working example:

https://github.com/Draco18s/ReasonableRealism/blob/1.14.4/src/main/java/com/draco18s/industry/ExpandedIndustry.java#L59

 

 

Also, your repo is not correctly set up:

https://github.com/drmdgg/mcraft

That should contain the build.gradle file, and other files at that root directory. The way you've done it has completely fucked up where your resources are.

For example, it should look like this:

https://github.com/Draco18s/ReasonableRealism

okay, I changed everything you told me to, but I'm still getting this:

https://pastebin.com/tRCKKfee

 

and as for the repo. that's just the repo, not my workstation

 

https://github.com/drmdgg/mcraft/tree/master/java/drmdgg/marijuanacraft

 

Edited by Guest
Link to comment
Share on other sites

1 hour ago, diesieben07 said:

Thats not how this works. Your repository must be in your project root (where the build.gradle is). The MDK comes with a .gitignore file for a reason. But you're not actually using Git, so all this is pointless. Use Git. Yes, Github != Git.

 

This makes it so nobody can actually just clone your repo and test this locally.

sorry about that. I have to run out for a while...but if it's testing you want, go to this link: 

it's the best I can offer right now.

Edited by Guest
Link to comment
Share on other sites

1 hour ago, diesieben07 said:

Thats not how this works. Your repository must be in your project root (where the build.gradle is). The MDK comes with a .gitignore file for a reason. But you're not actually using Git, so all this is pointless. Use Git. Yes, Github != Git.

 

This makes it so nobody can actually just clone your repo and test this locally.

 

33 minutes ago, diesieben07 said:

I am not downloading a random zip file of the internet.

I wouldn't want to give a virus to anyone, especially those willing to help. here is the git repo the way draco has his set up

https://github.com/drmdgg/marijuanacraft

Link to comment
Share on other sites

Great. You changed ONE thing in your ItemList to be final.

https://github.com/drmdgg/marijuanacraft/blob/master/src/main/java/drmdgg/marijuanacraft/lists/ItemList.java#L14

How about the rest of them, hmmm?

Ditto your block list:

https://github.com/drmdgg/marijuanacraft/blob/master/src/main/java/drmdgg/marijuanacraft/lists/BlockList.java#L22

 

The reason you're still getting null is because your marijuana_bud is null because @ObjectHolder only works on static final fields.

https://mcforge.readthedocs.io/en/1.13.x/concepts/registries/#injecting-registry-values-into-fields

Edited by Draco18s

Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable.  If you think this is the case, JUST REPORT ME.  Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice.

 

Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked.

 

DO NOT PM ME WITH PROBLEMS. No help will be given.

Link to comment
Share on other sites

40 minutes ago, Draco18s said:

Great. You changed ONE thing in your ItemList to be final.

https://github.com/drmdgg/marijuanacraft/blob/master/src/main/java/drmdgg/marijuanacraft/lists/ItemList.java#L14

How about the rest of them, hmmm?

Ditto your block list:

https://github.com/drmdgg/marijuanacraft/blob/master/src/main/java/drmdgg/marijuanacraft/lists/BlockList.java#L22

 

The reason you're still getting null is because your marijuana_bud is null because @ObjectHolder only works on static final fields.

https://mcforge.readthedocs.io/en/1.13.x/concepts/registries/#injecting-registry-values-into-fields

alright. I changed all the stuff to final for the ObjectHolder. https://github.com/drmdgg/marijuanacraft

 

still getting the same error

https://pastebin.com/Fu3bCQKz

Link to comment
Share on other sites

Set a breakpoint here:

https://github.com/drmdgg/marijuanacraft/blob/master/src/main/java/drmdgg/marijuanacraft/MarijuanaCraft.java#L88

Run in debug mode and inspect the value of BlockList.mplant

Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable.  If you think this is the case, JUST REPORT ME.  Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice.

 

Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked.

 

DO NOT PM ME WITH PROBLEMS. No help will be given.

Link to comment
Share on other sites

24 minutes ago, Draco18s said:

Set a breakpoint here:

https://github.com/drmdgg/marijuanacraft/blob/master/src/main/java/drmdgg/marijuanacraft/MarijuanaCraft.java#L88

Run in debug mode and inspect the value of BlockList.mplant

gradlew runclient --debug?

 

and i'm only getting the same log and crash report (besides. Minecraft won't even get to the loading screen.)

Link to comment
Share on other sites

19 minutes ago, DrMDGG said:

gradlew runclient --debug?

No debug mode in your IDE. Which one are you using?

VANILLA MINECRAFT CLASSES ARE THE BEST RESOURCES WHEN MODDING

I will be posting 1.15.2 modding tutorials on this channel. If you want to be notified of it do the normal YouTube stuff like subscribing, ect.

Forge and vanilla BlockState generator.

Link to comment
Share on other sites

1 minute ago, DrMDGG said:

eclipse

Excellent cause I don't know anything about IntelliJ. First set a breakpoint in your code by right clicking off to the left of the line you want to pause at and click "toggle breakpoint". Then up at the top next to the play button there is one that looks like a bug this is the debug run configuration thingy. Use this to run the runClient configuration.

VANILLA MINECRAFT CLASSES ARE THE BEST RESOURCES WHEN MODDING

I will be posting 1.15.2 modding tutorials on this channel. If you want to be notified of it do the normal YouTube stuff like subscribing, ect.

Forge and vanilla BlockState generator.

Link to comment
Share on other sites

37 minutes ago, Animefan8888 said:

Excellent cause I don't know anything about IntelliJ. First set a breakpoint in your code by right clicking off to the left of the line you want to pause at and click "toggle breakpoint". Then up at the top next to the play button there is one that looks like a bug this is the debug run configuration thingy. Use this to run the runClient configuration.

glad to hear it!

it said that there were 2 blocks that it was unable to load. one was a mistake on my part, and the other being the block mplant. it hadn't been registered. so I added it to the registry under registerBlocks. now the game boots again. but the texture still isn't there. the block breaks like a normal crop would, but its a square, black and pink. 

now I checked through all the json files and textures and they're all there.

Link to comment
Share on other sites

Just now, DrMDGG said:

now I checked through all the json files and textures and they're all there.

Post the console output.

VANILLA MINECRAFT CLASSES ARE THE BEST RESOURCES WHEN MODDING

I will be posting 1.15.2 modding tutorials on this channel. If you want to be notified of it do the normal YouTube stuff like subscribing, ect.

Forge and vanilla BlockState generator.

Link to comment
Share on other sites

8 minutes ago, DrMDGG said:

as I said, I double checked the json files against the ones for wheat, and they all check out it seems.

Yes judging by those errors it appears your block doesn't have the AGE property in it. Which doesn't make sense because it extends CropsBlock which adds the property. Try overriding fillStateContainer in your plants Block class and add your age property to the builder.

Also you can remove this. You never use it.

VANILLA MINECRAFT CLASSES ARE THE BEST RESOURCES WHEN MODDING

I will be posting 1.15.2 modding tutorials on this channel. If you want to be notified of it do the normal YouTube stuff like subscribing, ect.

Forge and vanilla BlockState generator.

Link to comment
Share on other sites

2 hours ago, DrMDGG said:

eclipse

image.png.faeae6bab63f48e3380614a5f5061e3e.png

Or

image.png.f351b327cb87d7239d8ef2d310bd7b15.png

Or

F11

Or

https://lmgtfy.com/?q=eclipse+run+debug

Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable.  If you think this is the case, JUST REPORT ME.  Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice.

 

Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked.

 

DO NOT PM ME WITH PROBLEMS. No help will be given.

Link to comment
Share on other sites

9 hours ago, Animefan8888 said:

Yes judging by those errors it appears your block doesn't have the AGE property in it. Which doesn't make sense because it extends CropsBlock which adds the property. Try overriding fillStateContainer in your plants Block class and add your age property to the builder.

Also you can remove this. You never use it.

okay. I removed that section. but I am having trouble finding the fillstatecontainer in my plants block class. 

 

theres one in "Clone" for my tree, but nothing in my MPlant class unless I'm blind

what line should I place the override?

Link to comment
Share on other sites

13 minutes ago, DrMDGG said:

okay. I removed that section. but I am having trouble finding the fillstatecontainer in my plants block class. 

fillStateContainer is a method in the Block class. If you are having a hard time overriding the method you can press ctrl+space and type the method name to get suggestions.

VANILLA MINECRAFT CLASSES ARE THE BEST RESOURCES WHEN MODDING

I will be posting 1.15.2 modding tutorials on this channel. If you want to be notified of it do the normal YouTube stuff like subscribing, ect.

Forge and vanilla BlockState generator.

Link to comment
Share on other sites

5 minutes ago, Animefan8888 said:

fillStateContainer is a method in the Block class. If you are having a hard time overriding the method you can press ctrl+space and type the method name to get suggestions.

my plants block class? or the block class. 

and also, I've tried pressing ctrl + space and typing fillStateContainer but nothing comes up along with "setAge" or "getAge" or "Age"

Link to comment
Share on other sites

1 minute ago, DrMDGG said:

or the block class. 

It exists in the Block class. You need to override it in your Block class I believe is called MPlant.

VANILLA MINECRAFT CLASSES ARE THE BEST RESOURCES WHEN MODDING

I will be posting 1.15.2 modding tutorials on this channel. If you want to be notified of it do the normal YouTube stuff like subscribing, ect.

Forge and vanilla BlockState generator.

Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Unfortunately, your content contains terms that we do not allow. Please edit your content to remove the highlighted words below.
Reply to this topic...

×   Pasted as rich text.   Restore formatting

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

Announcements




  • Recently Browsing

    • No registered users viewing this page.
  • Posts

    • Update your AMD/ATI drivers - get the drivers from their website - do not update via system  
    • As the title says i keep on crashing on forge 1.20.1 even without any mods downloaded, i have the latest drivers (nvidia) and vanilla minecraft works perfectly fine for me logs: https://pastebin.com/5UR01yG9
    • Hello everyone, I'm making this post to seek help for my modded block, It's a special block called FrozenBlock supposed to take the place of an old block, then after a set amount of ticks, it's supposed to revert its Block State, Entity, data... to the old block like this :  The problem I have is that the system breaks when handling multi blocks (I tried some fix but none of them worked) :  The bug I have identified is that the function "setOldBlockFields" in the item's "setFrozenBlock" function gets called once for the 1st block of multiblock getting frozen (as it should), but gets called a second time BEFORE creating the first FrozenBlock with the data of the 1st block, hence giving the same data to the two FrozenBlock :   Old Block Fields set BlockState : Block{minecraft:black_bed}[facing=east,occupied=false,part=head] BlockEntity : net.minecraft.world.level.block.entity.BedBlockEntity@73681674 BlockEntityData : id:"minecraft:bed",x:3,y:-60,z:-6} Old Block Fields set BlockState : Block{minecraft:black_bed}[facing=east,occupied=false,part=foot] BlockEntity : net.minecraft.world.level.block.entity.BedBlockEntity@6d1aa3da BlockEntityData : {id:"minecraft:bed",x:2,y:-60,z:-6} Frozen Block Entity set BlockState : Block{minecraft:black_bed}[facing=east,occupied=false,part=foot] BlockPos{x=3, y=-60, z=-6} BlockEntity : net.minecraft.world.level.block.entity.BedBlockEntity@6d1aa3da BlockEntityData : {id:"minecraft:bed",x:2,y:-60,z:-6} Frozen Block Entity set BlockState : Block{minecraft:black_bed}[facing=east,occupied=false,part=foot] BlockPos{x=2, y=-60, z=-6} BlockEntity : net.minecraft.world.level.block.entity.BedBlockEntity@6d1aa3da BlockEntityData : {id:"minecraft:bed",x:2,y:-60,z:-6} here is the code inside my custom "freeze" item :    @Override     public @NotNull InteractionResult useOn(@NotNull UseOnContext pContext) {         if (!pContext.getLevel().isClientSide() && pContext.getHand() == InteractionHand.MAIN_HAND) {             BlockPos blockPos = pContext.getClickedPos();             BlockPos secondBlockPos = getMultiblockPos(blockPos, pContext.getLevel().getBlockState(blockPos));             if (secondBlockPos != null) {                 createFrozenBlock(pContext, secondBlockPos);             }             createFrozenBlock(pContext, blockPos);             return InteractionResult.SUCCESS;         }         return super.useOn(pContext);     }     public static void createFrozenBlock(UseOnContext pContext, BlockPos blockPos) {         BlockState oldState = pContext.getLevel().getBlockState(blockPos);         BlockEntity oldBlockEntity = oldState.hasBlockEntity() ? pContext.getLevel().getBlockEntity(blockPos) : null;         CompoundTag oldBlockEntityData = oldState.hasBlockEntity() ? oldBlockEntity.serializeNBT() : null;         if (oldBlockEntity != null) {             pContext.getLevel().removeBlockEntity(blockPos);         }         BlockState FrozenBlock = setFrozenBlock(oldState, oldBlockEntity, oldBlockEntityData);         pContext.getLevel().setBlockAndUpdate(blockPos, FrozenBlock);     }     public static BlockState setFrozenBlock(BlockState blockState, @Nullable BlockEntity blockEntity, @Nullable CompoundTag blockEntityData) {         BlockState FrozenBlock = BlockRegister.FROZEN_BLOCK.get().defaultBlockState();         ((FrozenBlock) FrozenBlock.getBlock()).setOldBlockFields(blockState, blockEntity, blockEntityData);         return FrozenBlock;     }  
    • It is an issue with quark - update it to this build: https://www.curseforge.com/minecraft/mc-mods/quark/files/3642325
  • Topics

×
×
  • Create New...

Important Information

By using this site, you agree to our Terms of Use.