Jump to content
View in the app

A better way to browse. Learn more.

Forge Forums

A full-screen app on your home screen with push notifications, badges and more.

To install this app on iOS and iPadOS
  1. Tap the Share icon in Safari
  2. Scroll the menu and tap Add to Home Screen.
  3. Tap Add in the top-right corner.
To install this app on Android
  1. Tap the 3-dot menu (⋮) in the top-right corner of the browser.
  2. Tap Add to Home screen or Install app.
  3. Confirm by tapping Install.

Featured Replies

Posted

I'm trying to create a mana system to keep track of a player's pool of mana.

Unfortunately, I'm having huge trouble registering it. I've tried every combination of .class and new() but I just can't seem to get it to do anything other than give the error "The method register(Class<T>, Capability.IStorage<T>, Class<? extends T>) in the type CapabilityManager is not applicable for the arguments (Class<IBaseManaCapability>, PlayerManaCapabilityHandler, Class<PlayerManaCapability>)".

 

Here's some code:

IBaseManaCapability

 

package com.nosrick.masterofmagic.capabilities;

public interface IBaseManaCapability extends IBaseCapability
{
public int GetMana();
public void SetMana(int mana);
}

 

 

PlayerManaCapability

 

package com.nosrick.masterofmagic.capabilities;

public class PlayerManaCapability implements IBaseManaCapability
{
protected int m_Mana = 0;

@Override
public int GetMana() 
{
	return m_Mana;
}

@Override
public void SetMana(int mana) 
{
	m_Mana = mana;
}
}

 

 

PlayerManaCapabilityHandler

 

package com.nosrick.masterofmagic.capabilities;

import net.minecraft.nbt.NBTBase;
import net.minecraft.nbt.NBTTagCompound;
import net.minecraft.util.EnumFacing;
import net.minecraftforge.common.capabilities.Capability;
import net.minecraftforge.common.capabilities.Capability.IStorage;
import net.minecraftforge.common.capabilities.CapabilityManager;

public class PlayerManaCapabilityHandler implements IStorage<PlayerManaCapability> 
{
@Override
public NBTBase writeNBT(Capability<PlayerManaCapability> capability, PlayerManaCapability instance,
		EnumFacing side) 
{
	NBTTagCompound manaTags = new NBTTagCompound();
	manaTags.setInteger("mana", instance.GetMana());
	return manaTags;
}

@Override
public void readNBT(Capability<PlayerManaCapability> capability, PlayerManaCapability instance, EnumFacing side,
		NBTBase nbt) 
{
	NBTTagCompound manaTags = (NBTTagCompound)nbt;
	instance.SetMana(manaTags.getInteger("mana"));
}
}

 

 

And the factory, PlayerManaFactory

 

package com.nosrick.masterofmagic.capabilities;

import java.util.concurrent.Callable;

public class PlayerManaFactory implements Callable<PlayerManaCapability> 
{
@Override
public PlayerManaCapability call() throws Exception 
{
	return new PlayerManaCapability();
}

}

 

 

Now, here is how I'm trying to register it:

CapabilityManager.INSTANCE.register(IBaseManaCapability.class, new PlayerManaCapabilityHandler(), PlayerManaCapability.class);

I've tried a few different methods - including trying to use the factory - but they all give me the same error.

 

What am I doing wrong?

Just went through an almost identical process I'll update this response with some specifics but for now you can take a look at my github. You'll want to look at DefaultDrafter and DrafterProvider.

 

 

Update:

IBaseManaCapability should be a regular interface(doesn't extend anything)

 

PlayerManaCapability should implement IBaseManaCapability(this is the default implementation)

For saving the players mana and loading it you would do something like this:

public class PlayerManaCapability implements IBaseManaCapability{

//implement methods
   public static class ManaStorage implements IStorage<IBaseManaCapability>{
      public static final ManaStorage manaStorage = new ManaStorage ();
      //implement methods of IStorage (read and write nbt)
   }
}

 

You can break ManaStorage into a different file like you have it but if you leave it as an inner class it lets you access the instance.privateVariables for read and write without extra method calls.

 

register it like this:

CapabilityManager.INSTANCE.register(IBaseManaCapability.class, ManaStorage .manaStorage , PlayerManaCapability .class);

 

update 2:

your factory should be a provider meaning something like this:

public class ManaProvider implements ICapabilityProvider, INBTSerializable{
   @CapabilityInject(IBaseManaCapability .class)
   public static Capability<IBaseManaCapability > MANAPLAYER = null;

// implement methods
}

 

 

You will also want to hook into these two events

@SubscribeEvent
public void AttachCapability(AttachCapabilitiesEvent.Entity e)

@SubscribeEvent
public void onClonePlayer(PlayerEvent.Clone e)

Current Project: Armerger 

Planned mods: Light Drafter  | Ore Swords

Looking for help getting a mod off the ground? Coding  | Textures

  • Author

Register takes a class first, an instance of the storage second and a class third

That's exactly what I've done here:

CapabilityManager.INSTANCE.register(IBaseManaCapability.class, new PlayerManaCapabilityHandler(), PlayerManaCapability.class);

 

PlayerManaCapabilityHandler is the storage.

  • Author

I am not 100% sure with it, but I think that ur PlayerManaCapabilityHandler needs to implement IBaseManaCapability

Just tried modifying the code; no such luck, I'm afraid.

  • Author

your handler should be implementing IStorage<IBaseManaCapability>

Thanks! It's registering, now. Just got to get the rest of it working now.

 

Thanks again!

  • Author

Your welcome, and it is almost exactly what I have been going through. Let me know if you need help getting the client to update properly, that bit took a lot of trouble shooting for me.

I'd love some help with that.

 

But the main problem I'm having right now is that the player doesn't have the custom capability, so any hasCapability() call always returns false. Is the addition process automated, or do I have to do it myself?

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

Important Information

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

Configure browser push notifications

Chrome (Android)
  1. Tap the lock icon next to the address bar.
  2. Tap Permissions → Notifications.
  3. Adjust your preference.
Chrome (Desktop)
  1. Click the padlock icon in the address bar.
  2. Select Site settings.
  3. Find Notifications and adjust your preference.