Jump to content
Search In
  • More options...
Find results that contain...
Find results in...

[Solved] Planting crops on other blocks besides farmland (1.8.9)


Ronaldi2001
 Share

Recommended Posts

I am adding on to my mod More Items and i'm adding material crops. I am making my diamond crop right now and I want the seed to plant the crop on a block of diamond. I added the diamond block where farmland should be but it still wont let me plant it on it.

 

Diamond Plant Registry:

GameRegistry.registerItem(diamond_seeds = new MIItemSeeds(MoreItemsCrops.diamond_crop, Blocks.diamond_block, "diamond_seeds"), "diamond_seeds");	

~ Ronaldi2001

Link to comment
Share on other sites

I'm pretty sure the farmland-block needs to cooperate as well.  Which the diamond block doesn't know anything about crops, so the calls made to allow planting fail.

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

Reeds (sugar cane) can be planted on any of several types of blocks. Can they be imitated? Do they need co-operation from sand, dirt etc?

 

Walk through the reed code to see if you can extend that class (or its parent) to achieve your purpose.

 

EDIT: Oh, and netherwart is a "bush" that grows on soul sand. Maybe that's what you should imitate.

The debugger is a powerful and necessary tool in any IDE, so learn how to use it. You'll be able to tell us more and get better help here if you investigate your runtime problems in the debugger before posting.

Link to comment
Share on other sites

There is a chunk of code somewhere that handles valid placement that wass't in the BlockCrops class, I just don't remember where it was.

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

I'm looking in net.minecraft.item.ItemSeeds#onItemUse right now and there's this if statement:

 

/**
     * Called when a Block is right-clicked with this Item
     */
    public boolean onItemUse(ItemStack stack, EntityPlayer playerIn, World worldIn, BlockPos pos, EnumFacing side, float hitX, float hitY, float hitZ)
    {
        if (side != EnumFacing.UP)
        {
            return false;
        }
        else if (!playerIn.canPlayerEdit(pos.offset(side), side, stack))
        {
            return false;
        }
        else if (worldIn.getBlockState(pos).getBlock().canSustainPlant(worldIn, pos, EnumFacing.UP, this) && worldIn.isAirBlock(pos.up()))
        {
            worldIn.setBlockState(pos.up(), this.crops.getDefaultState());
            --stack.stackSize;
            return true;
        }
        else
        {
            return false;
        }
    }

 

So short of creating a coremod and altering the diamond block's canSustainPlant function, you should make an item that alters the diamond block, eg turns it into a custom diamond block which overrides this method and returns true. My advice would be to take a look at net.minecraft.block.Block#canSustainPlant and net.minecraft.block.Block#canHarvestBlock methods. I may just be talking out of my ass though, someone with more knowledge can and should correct me if I am.

Link to comment
Share on other sites

getBlock().canSustainPlant()

 

That's the one.  Go look at Block#canSustainPlant

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

However, we're not trying to plant wheat here, so we don't even need to work around ItemSeeds#onItemUse. We're trying to plant some custom thingy, so let's override the custom thing's onItemUse method to do a different test. Have it copy the early branches of the seed's version, and then look for diamond block instead of calling canSustainPlant.

 

PS: It would help if you could post the source code to MIItemSeeds. I think that's where you need to override onItemUse.

The debugger is a powerful and necessary tool in any IDE, so learn how to use it. You'll be able to tell us more and get better help here if you investigate your runtime problems in the debugger before posting.

Link to comment
Share on other sites

Diamond Seeds Class:

package ronaldi2001.MoreItems.crops;

import net.minecraft.block.Block;
import net.minecraft.block.BlockBush;
import net.minecraft.block.material.Material;
import net.minecraft.init.Blocks;
import net.minecraft.init.Items;
import net.minecraft.item.Item;
import net.minecraft.util.BlockPos;
import net.minecraft.util.EnumFacing;
import net.minecraft.world.World;
import ronaldi2001.MoreItems.init.MoreItemsItems;

public class MIDiamondCrop extends BlockBush {

public MIDiamondCrop(String name) {
	this.setUnlocalizedName(name);
}

public Item getSeed() {
	return MoreItemsItems.diamond_seeds;
}

public Item getCrop() {
	return Items.diamond;	
}


}

 

MIItemSeeds Class:

package ronaldi2001.MoreItems.items;

import net.minecraft.block.Block;
import net.minecraft.item.ItemSeeds;
import ronaldi2001.MoreItems.main.MoreItems;

public class MIItemSeeds extends ItemSeeds {

public MIItemSeeds(Block crops, Block soil, String name) {
	super(crops, soil);
	this.setUnlocalizedName(name);
	this.setCreativeTab(MoreItems.tabMoreItems);
}

}

~ Ronaldi2001

Link to comment
Share on other sites

You need to override #onItemUse in your custom seed class; from there, you can do any type of checking you want and plant your crop on any type of block you want. As your code is now, your seed uses the same logic as its parent class, the vanilla ItemSeed, which makes all sorts of checks for plant type / sustainability and you don't want any of that.

 

Here's an example of a seed that can be planted only on stone / rock-based blocks that are next to lava. As you can see, there is nothing that requires you to make the same checks as the vanilla seeds do.

Link to comment
Share on other sites

Guest
This topic is now closed to further replies.
 Share



×
×
  • Create New...

Important Information

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