Attempted to set registry name with existing registry name

[Dongle12]

I am having an issue with porting a mod from 1.11 to 1.12.2. I think I have all the block and item registration setup correctly, but when I go to launch the game, it crashes right away and gives an error about existing registry names. I am not sure what I have to change to move past this. 

 

Crash log is here:  https://pastebin.com/pujTMbfa

 

I am not even sure what code I would need to provide to be more helpful on my end, so please let me know.

 

Thank you so much

[Dongle12]

  On 12/17/2017 at 10:35 AM, diesieben07 said:

You are calling setRegistryName twice on the same object, this is not allowed. We need to see your code to know where.

Expand  

I solve the problem, but I just want to make sure I have my answer straight. When I register a Tile Entity, it is the same thing as registering a block correct? I.E. I do not need to register both a block and a tile entity for the same object.

[Draco18s]

  On 12/17/2017 at 6:19 PM, Dongle12 said:

I solve the problem, but I just want to make sure I have my answer straight. When I register a Tile Entity, it is the same thing as registering a block correct? I.E. I do not need to register both a block and a tile entity for the same object.

Expand  

No, it is not the same thing. TileEntities should be rare, but in either case, if you have one, you do need to register it.

However, that is not your problem here. Registering two items is not the same as registering the same thing twice. 

[Dongle12]

  On 12/17/2017 at 6:51 PM, Draco18s said:

No, it is not the same thing. TileEntities should be rare, but in either case, if you have one, you do need to register it.

However, that is not your problem here. Registering two items is not the same as registering the same thing twice. 

Expand  

 

Alright thanks, turns out my problem wasn't solved. I have tried limiting changing the code around to ensure that it doesn't register the same thing twice, but I think I do not fully grasp registering or am missing something and registering the block twice by accident.

 

The crash log starts with this: java.lang.IllegalStateException: Attempted to set registry name with existing registry name! New: BlockCompressedTorcherino Old: torcherino:blocktorcherino, and then points to line #9 of the BlockCompressedTorcherino class which has the following. 

 

public final class BlockCompressedTorcherino extends BlockTorcherino {
    public BlockCompressedTorcherino() {
        setRegistryName("BlockCompressedTorcherino");
        setUnlocalizedName("torcherino.compressed_torcherino");
    }

    @Override
    public TileEntity createNewTileEntity(final World world, final int i) {
        return new TileCompressedTorcherino();
    }
}

 

Line # 9 is the setRegistryName() function call.

 

From there I do the block registering in the CommonProxy(). The block registration code looks like the following: 

 

    @SubscribeEvent
    public static void registerBlocks(RegistryEvent.Register<Block> event) {
    	
    	event.getRegistry().register(new BlockTorcherino());
    	GameRegistry.registerTileEntity(TileTorcherino.class, "torcherino_tile");

    	event.getRegistry().register(new BlockCompressedTorcherino());
    	GameRegistry.registerTileEntity(TileCompressedTorcherino.class, "compressed_torcherino_tile");

    	event.getRegistry().register(new BlockDoubleCompresedTorcherino());
    	GameRegistry.registerTileEntity(TileDoubleCompressedTorcherino.class, "double_compressed_torcherino_tile");
    }

 

It runs into trouble at the line event.getRegistry().register(new BlockCompressedTorcherino()); 

 

I think this issue may stem from the BlockCompressedTorcherino extending the BlockTorcherino class which has code for setting registry name already in its constructor: setRegistryName("blocktorcherino"); 

 

 

[Dongle12]

  On 12/17/2017 at 9:44 PM, diesieben07 said:

• Problematic code, issue 4, 10 and 13.

• Indeed.

Expand  

 

Alright thank you for the link to the code changes. I seem to have it working, although my solution seems a bit hacked together to me. I fixed it by removing the setRegistryName() call from the super and subclasses, and then in my CommonProxy(), I create new objects, and then set their appropriate names there. Is this a correct approach to fixing this issue?

 

[Draco18s]

Code style issue 1