Jump to content
Search the Forums BEFORE making a new post.
Currently supported: All versions

[1.12.2] Issues about Model and Collision

K10

By K10
in Modder Support

 Share

Recommended Posts

K10 Tree Puncher

K10

Posted
Posted

Hi. I added thin Block looks like wooden slab or wooden pressure plate in my Mod. However, There are some issue :

  • Block model is not loaded (or not adapted).
  • Block has a larger hit box than appearance.

 

BlockRegisterer.java

Spoiler 

package doph.niho.cookmod.init;

import doph.niho.cookmod.block.BaseBlock;
import doph.niho.cookmod.block.BlockManaita;
import doph.niho.cookmod.block.BlockManaita2;
import doph.niho.cookmod.itemblock.BaseItemBlock;
import doph.niho.cookmod.itemblock.ItemBlockManaita;
import doph.niho.cookmod.itemblock.ItemBlockManaita2;
import net.minecraft.block.Block;
import net.minecraft.client.renderer.block.model.ModelBakery;
import net.minecraft.client.renderer.block.model.ModelResourceLocation;
import net.minecraft.item.Item;
import net.minecraft.util.ResourceLocation;
import net.minecraftforge.client.model.ModelLoader;
import net.minecraftforge.event.RegistryEvent;

public class BlockRegisterer {

	public static BaseBlock manaita;
	public static BaseBlock manaita2;

	public static BaseItemBlock manaitaItem;
	public static BaseItemBlock manaitaItem2;

	public static void RegisterModel() {
		RegisterRender(manaita);
		RegisterRender(manaita2);
	}

	public static void RegisterBlock(RegistryEvent.Register<Block> e) {
		e.getRegistry().register(manaita);
		e.getRegistry().register(manaita2);
	}

	public static void RegisterItemBlock(RegistryEvent.Register<Item> e) {
		e.getRegistry().register(manaitaItem);
		e.getRegistry().register(manaitaItem2);
	}

	public static void RegisterRender(Block block, String... subName) {
		int len = subName.length;
		if (len == 0) {
			ModelLoader.setCustomModelResourceLocation(Item.getItemFromBlock(block), 0,
					new ModelResourceLocation(block.getRegistryName(), "inventory"));
		} else {
			ResourceLocation[] rl = new ResourceLocation[subName.length];
			for(int i = 0; i < subName.length; i++) {
				rl[i] = new ResourceLocation(block.getRegistryName() + subName[i]);
			}
			ModelBakery.registerItemVariants(Item.getItemFromBlock(block), rl);
			for (int i = 0; i < len; i++) {
				ModelLoader.setCustomModelResourceLocation(Item.getItemFromBlock(block), i,
						new ModelResourceLocation(block.getRegistryName() + subName[i], "inventory"));
			}
		}
	}

	public static void Init() {
		manaita = new BlockManaita();
		manaita2 = new BlockManaita2();

		manaitaItem = new ItemBlockManaita();
		manaitaItem2 = new ItemBlockManaita2();
	}
}

 

 

BlockManaita.java

Spoiler 

package doph.niho.cookmod.block;

import doph.niho.cookmod.system.Define;
import net.minecraft.block.BlockHorizontal;
import net.minecraft.block.SoundType;
import net.minecraft.block.material.MapColor;
import net.minecraft.block.material.Material;
import net.minecraft.block.properties.IProperty;
import net.minecraft.block.properties.PropertyDirection;
import net.minecraft.block.properties.PropertyEnum;
import net.minecraft.block.state.BlockFaceShape;
import net.minecraft.block.state.BlockStateContainer;
import net.minecraft.block.state.IBlockState;
import net.minecraft.creativetab.CreativeTabs;
import net.minecraft.entity.EntityLivingBase;
import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.item.ItemStack;
import net.minecraft.util.EnumFacing;
import net.minecraft.util.EnumHand;
import net.minecraft.util.IStringSerializable;
import net.minecraft.util.NonNullList;
import net.minecraft.util.ResourceLocation;
import net.minecraft.util.Rotation;
import net.minecraft.util.math.AxisAlignedBB;
import net.minecraft.util.math.BlockPos;
import net.minecraft.util.text.TextComponentTranslation;
import net.minecraft.world.IBlockAccess;
import net.minecraft.world.World;
import net.minecraftforge.fml.relauncher.Side;
import net.minecraftforge.fml.relauncher.SideOnly;

public class BlockManaita extends BaseBlock {

	public static final PropertyEnum<BlockManaita.EnumType> VARIANT = PropertyEnum.<BlockManaita.EnumType>create("meta",
			BlockManaita.EnumType.class);
	public static final PropertyDirection FACING = BlockHorizontal.FACING;
	protected static final AxisAlignedBB X_AXIS_AABB = new AxisAlignedBB(0.0625D, 0.0D, 0.25D, 0.9375D, 0.0625D, 0.75D);
	protected static final AxisAlignedBB Z_AXIS_AABB = new AxisAlignedBB(0.25D, 0.0D, 0.0625D, 0.75D, 0.0625D, 0.9375D);

	public BlockManaita() {
		super(Material.WOOD);
		this.setRegistryName(new ResourceLocation(Define.MODID, "manaita"));
		this.setUnlocalizedName("manaita");
		this.setHardness(0.25F);
		this.setResistance(0.5F);
		this.setSoundType(SoundType.WOOD);
		this.setLightOpacity(1);
		this.setLightLevel(0.0F);
		this.setDefaultState(this.blockState.getBaseState().withProperty(FACING, EnumFacing.NORTH).withProperty(VARIANT,
				BlockManaita.EnumType.OAK));
	}

	public AxisAlignedBB getCollisionBoundingBox(IBlockState state, IBlockAccess worldIn, BlockPos pos) {
		EnumFacing enumfacing = (EnumFacing) state.getValue(FACING);
		return enumfacing.getAxis() == EnumFacing.Axis.X ? X_AXIS_AABB : Z_AXIS_AABB;
	}

	public boolean isFulllCube(IBlockState state) {
		return false;
	}

	public BlockFaceShape getBlockFaceShape(IBlockAccess worldIn, IBlockState state, BlockPos pos, EnumFacing face) {
		return BlockFaceShape.UNDEFINED;
	}

	public boolean isOpequeCube(IBlockState state) {
		return false;
	}

	@SuppressWarnings("deprecation")
	public boolean canPlaceBlockAt(World worldIn, BlockPos pos) {
		IBlockState down = worldIn.getBlockState(pos.down());
		return super.canPlaceBlockAt(worldIn, pos) && (down.isTopSolid() || down.getBlockFaceShape(worldIn, pos.down(), EnumFacing.UP) == BlockFaceShape.SOLID);
	}

	@SuppressWarnings("deprecation")
	public IBlockState getStateForPlacement(World worldIn, BlockPos pos, EnumFacing facing, float hitX, float hitY,
			float hitZ, int meta, EntityLivingBase placer) {
		EnumFacing enumfacing = placer.getHorizontalFacing().rotateY();

		try {
			return super.getStateForPlacement(worldIn, pos, facing, hitX, hitY, hitZ, meta, placer)
					.withProperty(FACING, enumfacing).withProperty(VARIANT, BlockManaita.EnumType.byMetadata(meta));
		} catch (IllegalArgumentException var11) {
			if (!worldIn.isRemote) {
				if (placer instanceof EntityPlayer) {
					placer.sendMessage(new TextComponentTranslation(
							"Invalid damage property. Please pick in [0, 1, 2, 3]", new Object[0]));
				}
			}
			return super.getStateForPlacement(worldIn, pos, facing, hitX, hitY, hitZ, meta, placer)
					.withProperty(FACING, enumfacing).withProperty(VARIANT, BlockManaita.EnumType.OAK);
		}
	}

	public boolean onBlockActivated(World worldIn, BlockPos pos, IBlockState state, EntityPlayer playerIn,
			EnumHand hand, EnumFacing facing, float hitX, float hitY, float hitZ) {
		return true;
	}

	public AxisAlignedBB getBoundingBox(IBlockState state, IBlockAccess source, BlockPos pos) {
		EnumFacing enumfacing = (EnumFacing) state.getValue(FACING);
		return enumfacing.getAxis() == EnumFacing.Axis.X ? X_AXIS_AABB : Z_AXIS_AABB;
	}

	@SideOnly(Side.CLIENT)
	public boolean shouldSideBeRendered(IBlockState blockState, IBlockAccess blockAccess, BlockPos pos,
			EnumFacing side) {
		return true;
	}

	public String getUnlocalizedName(ItemStack item) {
		return BlockManaita.EnumType.byMetadata(item.getMetadata()).getUnlocalizedName();
	}

	public int damageDropped(IBlockState state) {
		return ((BlockManaita.EnumType) state.getValue(VARIANT)).getMetadata();
	}

	public void getSubBlocks(CreativeTabs itemIn, NonNullList<ItemStack> items) {
		for (BlockManaita.EnumType blockmanaita$enumtype : BlockManaita.EnumType.values()) {
			items.add(new ItemStack(this, 1, blockmanaita$enumtype.getMetadata()));
		}
	}

	public MapColor getMapColor(IBlockState state, IBlockAccess worldIn, BlockPos pos) {
		return MapColor.AIR;
	}

	public IBlockState getStateFromMeta(int meta) {
		return this.getDefaultState().withProperty(FACING, EnumFacing.getHorizontal(meta & 4)).withProperty(VARIANT,
				BlockManaita.EnumType.byMetadata(meta));
	}

	public int getMetaFromState(IBlockState state) {
		int i = 0;
		i = i | ((EnumFacing) state.getValue(FACING)).getHorizontalIndex();
		i = i | ((BlockManaita.EnumType) state.getValue(VARIANT)).getMetadata();
		return i;
	}

	public IBlockState withRotation(IBlockState state, Rotation rot) {
		return state.getBlock() != this ? state
				: state.withProperty(FACING, rot.rotate((EnumFacing) state.getValue(FACING)));
	}

	protected BlockStateContainer createBlockState() {
		return new BlockStateContainer(this, new IProperty[] { FACING, VARIANT });
	}

	public static enum EnumType implements IStringSerializable {
		OAK(0, "oak"), SPRUCE(1, "spruce"), BIRCH(2, "birch"), JUNGLE(3, "jungle");

		private static final BlockManaita.EnumType[] META_LOOKUP = new BlockManaita.EnumType[values().length];
		private final int meta;
		private final String name;

		private EnumType(int metaIn, String nameIn) {
			this.meta = metaIn;
			this.name = nameIn;
		}

		public int getMetadata() {
			return this.meta;
		}

		public String toString() {
			return this.name;
		}

		public static BlockManaita.EnumType byMetadata(int meta) {
			if (meta < 0 || meta >= META_LOOKUP.length) {
				meta = 0;
			}
			return META_LOOKUP[meta];
		}

		public String getName() {
			return this.name;
		}

		public String getUnlocalizedName() {
			return this.name;
		}

		static {
			for (BlockManaita.EnumType blockmanaita$enumtype : values()) {
				META_LOOKUP[blockmanaita$enumtype.getMetadata()] = blockmanaita$enumtype;
			}
		}
	}
}

 

 

blockstates\manaita.json

Spoiler 

{
	"variants" : {
		"facing=south,meta=oak" : {"model":"cookmod:block/manaita_oak"},
		"facing=west,meta=oak" : {"model":"cookmod:block/manaita_oak", "y" : 90},
		"facing=north,meta=oak" : {"model":"cookmod:block/manaita_oak", "y" : 180},
		"facing=east,meta=oak" : {"model":"cookmod:block/manaita_oak", "y" : 270},
		"facing=south,meta=spruce" : {"model":"cookmod:block/manaita_spruce"},
		"facing=west,meta=spruce" : {"model":"cookmod:block/manaita_spruce", "y" : 90},
		"facing=north,meta=spruce" : {"model":"cookmod:block/manaita_spruce", "y" : 180},
		"facing=east,meta=spruce" : {"model":"cookmod:block/manaita_spruce", "y" : 270},
		"facing=south,meta=birch" : {"model":"cookmod:block/manaita_birch"},
		"facing=west,meta=birch" : {"model":"cookmod:block/manaita_birch", "y" : 90},
		"facing=north,meta=birch" : {"model":"cookmod:block/manaita_birch", "y" : 180},
		"facing=east,meta=birch" : {"model":"cookmod:block/manaita_birch", "y" : 270},
		"facing=south,meta=jungle" : {"model":"cookmod:block/manaita_jungle"},
		"facing=west,meta=jungle" : {"model":"cookmod:block/manaita_jungle", "y" : 90},
		"facing=north,meta=jungle" : {"model":"cookmod:block/manaita_jungle", "y" : 180},
		"facing=east,meta=jungle" : {"model":"cookmod:block/manaita_jungle", "y" : 270}
	}
}

 

 

models\block\manaita_oak.json

Spoiler 

{
	"parent" : "block/cube_all",
	"textures" : {
		"all" : "block/planks_oak"
	}
}

 

 

models\block\manaita_birch

Spoiler 

{
	"parent" : "cookmod:block/manaita",
	"textures" : {
		"texture" : "blocks/planks_birch"
	}
}

 

 

models\block\manaita

Spoiler 

{
	"parent" : "block/thin_block",
	"textures" : {
		"particle" : "#texture"
	},
	"elements" : [
		{
			"from" : [ 1, 0, 4 ],
			"to" : [ 15, 2, 12],
			"faces" : {
				"down" : { "uv" : [ 1,  4, 15, 12 ], "texture" : "#texture", "cullface" : "down" },
				"up"   : { "uv" : [ 1,  4, 15, 12 ], "texture" : "#texture" },
				"north": { "uv" : [ 1, 14, 15, 16 ], "texture" : "#texture" },
				"south": { "uv" : [ 1, 14, 15, 16 ], "texture" : "#texture" },
				"west" : { "uv" : [ 4, 14, 12, 16 ], "texture" : "#texture" },
				"east" : { "uv" : [ 4, 14, 12, 16 ], "texture" : "#texture" }
			}
		}
	]
}

 

 

Can anyone tell me why model is not adapted and block has strange collision box?

Draco18s Reality Controller

Draco18s

Posted
Posted
1 hour ago, K10 said: 

	public static void RegisterBlock(RegistryEvent.Register<Block> e) {
		e.getRegistry().register(manaita);
		e.getRegistry().register(manaita2);
	}

	public static void RegisterItemBlock(RegistryEvent.Register<Item> e) {
		e.getRegistry().register(manaitaItem);
		e.getRegistry().register(manaitaItem2);
	}

 

You haven't annotated these methods such that the event bus knows about them.

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.

K10 Tree Puncher

K10

Posted
  • Author
Posted

Sorry, I forgot to say that these methods is called from

@SubscribeEvent

public static void OnBlockRegistered(RegistryEvent.Register<Block> e);

@SubscribeEvent

public static void OnItemRegistered(RegistryEvent.Register<Item> e);

K10 Tree Puncher

K10

Posted
  • Author
Posted
Quote

Why do some of your class files not have the .java file extension? How does this code even compile?

I mistook file name when I create file on github because I didn't know way to drag and drop file at first. I already fixed.

 

Quote

Problematic code, issue 2.

There are not enough document about forge tutorial in Japan, so I have to study old article...

 

Quote

You are referencing client-only classes (such as Minecraft and ModelLoader) in common code. This will crash on a server, you must encapsulate this using @SidedProxy.

Oh, I missed it! I have to add it in my code.

 

Quote

ModelBakery.registerItemVariants is not needed if you are using ModelLoader.setCustomModelResourceLocation.

OK, I see.

 

Thank you for your advices. I hava to rewrite the majority of my codes, so I recreate project and restart from first. I will report and ask later.

  • 2 weeks later...
Draco18s Reality Controller

Draco18s

Posted
Posted
12 minutes ago, vpontin said:

Use Github Desktop if you have problems using git via command line.

On 9/14/2017 at 1:34 PM, diesieben07 said:

Please don't ever use this crap.

If you want a good Git client, use SourceTree (I don't know GitKraken, never used it) or (even better) use a good IDE (*cough* IntelliJ *cough*) that has built-in Git support.

 

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.

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.

×
 Share
Go to topic listing

Announcements



  • Recently Browsing

    • No registered users viewing this page.

  • Posts

    • RataSucia
      Please help with a crash.

      By RataSucia · Posted

      I need to know what mod is doing this crash, i mean the mod xenon is doing the crash but i want to know who mod is incompatible with xenon, but please i need to know a solution if i need to replace xenon, i cant use optifine anymore and all the other mods i tried(sodium, lithium, vulkan, etc) doesn't work, it crash the game.
    • Benjaggo
      Strange Crash Due to Error From Missing Class: net.minecraft.client.gui.screens.LoadingOverlay (Minecraft, Forge, 1.20.1)

      By Benjaggo · Posted

      I have been trying to solve a consistent crashing issue on my brother's computer where it will crash during the "Scanning Mod Candidates" phase of the loading process that starts when you click the play button on the Minecraft launcher. The issue seems to stem from a missing library that it mentions in the log file I provide below. I might I'm missing the bigger issue here for a smaller one but hopefully someone can find what I'm missing. Here's all of the stuff that I've been able to figure out so far: 1. It has nothing to do with mods, the crash happened with a real modpack, and even when I made a custom modpack and launched it without putting ANY mods into it (That is where the log file comes from by the way). 2. I have tried to find this class like a file in the Minecraft folders, but I've had no luck finding it (I don't think it works like that, but since I really don't understand how it works, I just figured I'd try). 3. I haven't seen anyone else have this issue before. 4. I know that my modpack (with mods) does work since I've run it on my computer, and it works fantastic. For some reason my brother's computer can't seem to run anything through curseforge. 5. This is for Minecraft version 1.20.1, Minecraft launcher version 3.4.50-2.1.3, forge 47.3.0, and curseforge app version 1.256.0.21056 6. My brother is using a Dell laptop from 6 years ago running Windows 10 (If you think more info on this would help, please ask as I do have it. I'm just choosing not to put it here for now). 7. I have reinstalled the curseforge app and installed Minecraft version 1.20.1. I have not reinstalled Minecraft or forge 47.3.0 but I didn't know if that would help. 8. I had an error code of 1 Please let me know if there is anything else that I am missing that you would like me to add to this post/add in a comment! Lastly, many thanks in advance to whoever can help! ------------- LOG FILE (latest.log) ------------- (from /Users/<NAME OF USER>/cursforge/minecraft/Instances/<THE NAME OF MY EMPTY MODPACK>/logs/latest.log) (This was made after running an empty modpack with same versions for all apps) ("[REDACTED]" is not the actual text from the log, it is me replacing text I figured wouldn't be necessary for fixing and would hurt my privacy) https://pastebin.com/hxXvGGEK ------------- DEBUG.LOG (I realized that I should have put this here first after I had done all of the work on putting latest.log in) -------------------- (again, "[REDACTED]" is not the actual text from the log, it is me replacing text I figured wouldn't be necessary for fixing and would hurt my privacy) https://pastebin.com/Fmh8GHYs
    • Veradigm
      Better Combat causing game crash upon loading into world Minecraft Forge 1.20.1

      By Veradigm · Posted

      That fixed it
    • TheGooch78
      1.19.2 crash curseforge app, 277 mods. Does not close with Exit code

      By TheGooch78 · Posted

      Pastebin... https://pastebin.com/Y3iZ85L5   Brand new profile, does not point to a mod as far as I can tell, my fatal message just has something about mixins. Don't know much about reading logs like this, but am genuinely stuck, please help. Java updated, pc restarted.
    • briankennedy
      COMMENT ON I NEED A HACKER TO RECOVER MONEY FROM BINARY TRADING. HIRE FASTFUND RECOVERY

      By briankennedy · Posted

      Fastfund recovery helps an individual to get back their scammed funds irrespective of nationality, Romance scam funds and Broker's scam, all kinds of scam funds are 100% accurately recovered without disappointment, their goal is to give all those who seek help to recover lost satisfaction of funds recovery within 72 hours After countless hours of research and desperate attempts to find a solution, I stumbled upon FASTFUND RECOVERY. It was like finding an oasis in the middle of a desert. Their website promised to help victims of scams reclaim what was rightfully theirs, and I instantly knew I had to give them a shot. Before diving headfirst into the recovery process, I wanted to make sure that FASTFUND RECOVERY was the real deal. So, I did my due diligence and looked into their expertise and reputation. To my relief, I found that they had an impeccable track record, successfully assisting countless individuals in recovering their lost funds. Their team consisted of experts in cybersecurity and financial fraud, armed with the knowledge and tools needed to tackle even the most intricate scams. With their reputation preceding them, I felt a renewed sense of hope. FASTFUND RECOVERY successfully came to my aid and got back the amount I lost to these scammers and for this, I am sending this article for clarification. The info of FASTFUND RECOVERY is email: Fastfundrecovery8 (@)Gmail (.) com. Web fastfundrecovery(.)com. (W/A 1 807/500/7554)

  • Topics

×
×
  • Create New...

Important Information

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

  I accept