I've just started practicing with Forge modding, and I'm trying to create a simple command that prints "It works!" to the server chat, just so I can prove that it does, in fact, work.

 

It doesn't, of course.  I can't seem to use registerServerCommand() in the code below:

 

Main.java

import net.minecraftforge.fml.common.Mod;
import net.minecraftforge.fml.common.Mod.EventHandler;
import net.minecraftforge.fml.common.Mod.Instance;
import net.minecraftforge.fml.common.event.FMLServerStartingEvent;

@Mod(modid = Main.MODID, name = Main.MODNAME, version = Main.VERSION)
public class Main {

    public static final String MODID = "testmod";
    public static final String MODNAME = "Domo's Test Mod";
    public static final String VERSION = "1.0";
        
    @Instance
    public static Main instance = new Main();
        
    @EventHandler
    public void load(FMLServerStartingEvent event) {
event.registerServerCommand(new TestCommand());
    }
}

 

TestCommand.java

import java.util.ArrayList;
import java.util.List;

import net.minecraft.command.CommandException;
import net.minecraft.command.ICommand;
import net.minecraft.command.ICommandSender;
import net.minecraft.server.MinecraftServer;
import net.minecraft.util.math.BlockPos;
import net.minecraft.util.text.TextComponentString;

public class TestCommand implements ICommand
{
private final List aliases;

public TestCommand() 
    { 
        aliases = new ArrayList(); 
        aliases.add("testcommand"); 
        aliases.add("tc"); 
    }

@Override
public int compareTo(ICommand arg0) {
	return 0;
}

@Override
public String getCommandName()
{
	return "testcommand"; 
}

@Override
public String getCommandUsage(ICommandSender sender)
{
	return "testcommand"; 
}

@Override
public List<String> getCommandAliases()
{
	return this.aliases;
}

@Override
public void execute(MinecraftServer server, ICommandSender sender, String[] args) throws CommandException
{
	sender.addChatMessage(new TextComponentString("It works!"));
}

@Override
public boolean checkPermission(MinecraftServer server, ICommandSender sender) {
	return false;
}

@Override
public List<String> getTabCompletionOptions(MinecraftServer server, ICommandSender sender, String[] args,
		BlockPos pos) {
	return null;
}

@Override
public boolean isUsernameIndex(String[] args, int index) {
	return false;
} 
}

 

I'll be honest - I'm not good at this, there are significant holes in my understanding of Java and Forge's whole structure.  I've been using Minecraft modding as practice, and it's helped me immensely in becoming a better programmer, but I still run into obstacles I can't handle.  Point being - I can't solve this, although I have a feeling it's a REALLY easy fix that I'm not seeing.  Can anyone shed some light on this?

@Instance
public static Main instance = new Main();

 

Do not instantiate your own main class, the @Instance annotation does it for you as Forge loads your mod.

Do not instantiate your own main class, the @Instance annotation does it for you as Forge loads your mod.

 

I took out that whole line, all that's left is '@Instance', and it's giving me an error:

 

The annotation @Mod.Instance is disallowed for this location

I had that line in the first place after following this advice from a tutorial:

 

The next thing we need in your main class is an instance of it. Forge uses this instance to have a reference to your mod in order to communicate with it. If we don't create the instance on our own, Forge creates one for us. The problem of this method is that we have no access on the instance anymore, so I would recommend to create the instance manually.

1. You shouldn't implement ICommand (unless needed). Use (extend) CommandBase.

 

2.

Do not instantiate your own main class, the @Instance annotation does it for you as Forge loads your mod.

I took out that whole line, all that's left is '@Instance', and it's giving me an error:

 

Learn what "instantiate" means. Annotation has to be assigned to something and that is field declaration.

1. You shouldn't implement ICommand (unless needed). Use (extend) CommandBase.

 

Extending CommandBase hasn't fixed the error, it's still telling me the same thing.

 

I'd like to ignore any issues with my mod instance, as it is not causing me any immediate problems.  Thank you for your suggestions, though.

Maybe it's because you're returning

false

no matter what from

checkPermission

. If you had extended

CommandBase

and not overridden all these methods, you wouldn't have this issue.

Maybe it's because you're returning

false

no matter what from

checkPermission

. If you had extended

CommandBase

and not overridden all these methods, you wouldn't have this issue.

 

Alright, this is what it looks like now, as per your suggestion:

 

TestCommand.java

import net.minecraft.command.CommandBase;
import net.minecraft.command.CommandException;
import net.minecraft.command.ICommandSender;
import net.minecraft.server.MinecraftServer;
import net.minecraft.util.text.TextComponentString;

public class TestCommand extends CommandBase
{

@Override
public String getCommandName() {
	return "testcommand";
}

@Override
public String getCommandUsage(ICommandSender sender) {

	return "testcommand";
}

@Override
public void execute(MinecraftServer server, ICommandSender sender, String[] args) throws CommandException
{
	sender.addChatMessage(new TextComponentString("It works!"));
}

}

 

I realized I never posted the error I was getting from registerServerCommand:

The method registerServerCommand(l) in the type FMLServerStartingEvent is not applicable for the arguments (TestCommand)

Main.java

import net.minecraftforge.fml.common.Mod;
import net.minecraftforge.fml.common.Mod.EventHandler;
import net.minecraftforge.fml.common.Mod.Instance;
import net.minecraftforge.fml.common.event.FMLServerStartingEvent;

@Mod(modid = Main.MODID, name = Main.MODNAME, version = Main.VERSION)
public class Main {

    public static final String MODID = "testmod";
    public static final String MODNAME = "Domo's Test Mod";
    public static final String VERSION = "1.0";
        
    @Instance
    public static Main instance;
        
    @EventHandler
public void load(FMLServerStartingEvent event) {
	event.registerServerCommand(new TestCommand());
}
}

In case it wasn't clear, even after making these changes, it still does not work.  Does anyone know why that method can't be found, or why it suggests casting TestCommand as an 'l' type?

Hi,

Quick notes.

 

1) First you have no packages in the java source. (e.g package com.mymods.testcommandmod; )

 

2) I do the register in server start.

 

@EventHandler
public void serverStart(FMLServerStartingEvent event) {

	MinecraftServer server = event.getServer();
	ICommandManager command = server.getCommandManager();
	ServerCommandManager manager = (ServerCommandManager) command;
	ICommand cmd;
	if (strFiles == null) {
		cmd = new ExecuteFile(server);

		manager.registerCommand(cmd);
	} else {
// This sends the directory list of files to the command for tabcompletion of file names. strFiles is loaded in the server once. 
		cmd = new ExecuteFile(server, mydirProperty.getString(), strFiles);
		manager.registerCommand(cmd);
	}

}

 

3) By the Way : Do the recommended housekeeping

 

I set up the apache logger and set accepted versions, et al.

public static final Logger logger = LogManager.getLogger(MODID);

 

And in the @MOD annotation, I added the maven formatted acceptedVersions; and required-after

e.g.

public static final String REQUIRED_AFTER = "required-after:Forge@[12.16.0.1887,]";