Tinkers' Mechworks

Tinkers' Mechworks

Mods
18M Downloads

Illegal extra prefix TMechworks for name TMechworks:Dynamo

RoyCurtis opened this issue ยท 1 comments

commented

Forge is reporting that TMechworks registers both a block and tile entity using "TMechworks:Dynamo", which it sees as an extra prefix that is not necessary.

Details

  • Minecraft : 1.7.10
  • Forge: 10.13.0.1207 w/ LiteLoader 1.7.10_03
  • Mechworks: TMechworks-1.7.10-0.2.9
  • Construct: TConstruct-1.7.10-1.7.0d3.1

Log

[18:28:57] [Client thread/WARN]: * Illegal extra prefix TMechworks for name TMechworks:Dynamo, invalid registry invocation/invalid name?
[18:28:57] [Client thread/WARN]: *  at cpw.mods.fml.common.registry.GameData.registerItem(GameData.java:755)
[18:28:57] [Client thread/WARN]: *  at cpw.mods.fml.common.registry.GameRegistry.registerBlock(GameRegistry.java:215)
[18:28:57] [Client thread/WARN]: *  at cpw.mods.fml.common.registry.GameRegistry.registerBlock(GameRegistry.java:171)
[18:28:57] [Client thread/WARN]: *  at cpw.mods.fml.common.registry.GameRegistry.registerBlock(GameRegistry.java:160)
[18:28:57] [Client thread/WARN]: *  at tmechworks.common.MechContent.registerBlocks(MechContent.java:81)
[18:28:57] [Client thread/WARN]: *  at tmechworks.common.MechContent.<init>(MechContent.java:44)...
[18:28:57] [Client thread/WARN]: ****************************************

Proposed fixes

Lines 81 and 82 of /src/main/java/tmechworks/common/MechContent.java should have "TMechworks:Dynamo" replaced with just "Dynamo".

I did want to try this change myself for testing and pull request purposes, but testing from source fails; Minecraft complains that ForgeMultipart requires 1.7.2, not 1.7.10. I see in the Gradle dependency list generated by IntelliJ that it is trying to use ForgeMultipart 1.7.2; I do not know how or where to fix this.

commented

the dynamo is a block that should be removed it is actually an empty class