Jump to content

[1.10.2] registerServerCommand Not Working


ViewtifulDom

Recommended Posts

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?

Link to comment
Share on other sites

@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.

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

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.

Link to comment
Share on other sites

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.7.10 is no longer supported by forge, you are on your own.

Link to comment
Share on other sites

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.

Link to comment
Share on other sites

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());
}
}

Link to comment
Share on other sites

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,]";

 

 

 

Link to comment
Share on other sites

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.

Announcements



×
×
  • Create New...

Important Information

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