[1.11] Capability Not Syncing Properly

[Lambda]

Hey there,

 

*Packets are being to used to sync everything*

 

SO I have a capability that adds: level, corruption, and experience. Simple. However, when I'm using an Item to add to the capability, it displays for 1 frame, then resets, here are some pictures of whats happening:

 

 

So, on connect, it displays normally:

However, when I right click with the item, It seems to display a weird number for 1 frame, then reset.

Note: that local debugging shows the values are always 0 however.

phew, that took awhile, anyways, then it resets to this:

 

 

 

 

Okay, so I am syncing the client every time the value is changed and when the player connects..

So here is the capability data:

public class CapabilityLevelData extends LevelHandler
{
    private EntityLivingBase entity;

    public CapabilityLevelData(){ super(500); }

    public EntityLivingBase getEntity() {
        return entity;
    }

    public void setEntity(EntityLivingBase entity)
    {
        this.entity = entity;
    }

    public float addExperienceInternal(float experienceToAdd) {
        float toReturn = this.addExperience(experienceToAdd);
        sendPacket();
        return toReturn;
    }

    public int addLevelsInternal(int levelsToAdd) {
        int toReturn = this.addLevels(levelsToAdd);
        sendPacket();
        return toReturn;
    }

    public int removeLevelsInternal(int levelsToRemove) {
        int toReturn = this.removeLevels(levelsToRemove);
        sendPacket();
        return toReturn;
    }

    public int addCorruptionInternal(int corruptionToAdd) {
        int toReturn = this.addCorruption(corruptionToAdd);
        sendPacket();
        return toReturn;
    }

    public int removeCorruptionInternal(int corruptionToRemove) {
        int toReturn = this.removeCorruption(corruptionToRemove);
        sendPacket();
        return toReturn;
    }
//todo
public void updateExpForLevelUp() {
    float currentXP = this.getExperience();
    int currentLevel = this.getLevels();
    int currentCorruption = this.getCorruption();

    float currentXPToLevelUp = currentLevel * (levelUpIncrement + currentCorruption);

    if(currentXP >= currentXPToLevelUp) {
        this.addLevelsInternal(1);
        this.setExpereince(0);
    }

    if(currentLevel <= 0) {
        this.setLevel(1);
    }

    if(currentLevel >= 100) {
        this.setLevel(100);
    }
}


    @Override
    public float addExperience(float experienceToAdd) {
        float currentExp = this.getExperience();

        float returnedExp = currentExp+=experienceToAdd;

        this.setExpereince(returnedExp);

        return returnedExp;
    }

    @Override
    public int addLevels(int levelsToAdd) {
        int currentLevel = this.getLevels();

        int returnedLevel = currentLevel+=levelsToAdd;

        this.setLevel(returnedLevel);
        sendPacket();
        return returnedLevel;
    }

    @Override
    public int removeLevels(int levelsToRemove) {
        int currentLevel = this.getLevels();

        int returnedLevel = currentLevel-+levelsToRemove;

        this.setLevel(returnedLevel);
        sendPacket();
        return returnedLevel;
    }

    @Override
    public int addCorruption(int corruptionToAdd) {
        int currentCurruption = this.getCorruption();

        int returnedCorruption = currentCurruption+=corruptionToAdd;

        this.setCorruption(returnedCorruption);
        sendPacket();
        return returnedCorruption;
    }

    @Override
    public int removeCorruption(int corruptionToRemove) {
        int currentCurruption = this.getCorruption();

        int returnedCorruption = currentCurruption-=corruptionToRemove;

        this.setCorruption(returnedCorruption);
        sendPacket();
        return returnedCorruption;
    }

    public void readFromNBT(NBTTagCompound compound){
        this.setLevel(compound.getInteger("Level"));
        this.setCorruption(compound.getInteger("Corruption"));
        this.setExpereince(compound.getFloat("Experience"));
    }

    public void writeToNBT(NBTTagCompound compound){
        compound.setInteger("Level", this.getLevels());
        compound.setInteger("Corruption", this.getCorruption());
        compound.setFloat("Experience", this.getExperience());
    }

    public NBTBase writeData()
    {
        NBTTagCompound tag = new NBTTagCompound();
        tag.setInteger("Level", level);
        tag.setInteger("Corruption", levelCorruption);
        tag.setFloat("Experience", experience);
        return tag;
    }

    public void readData(NBTBase nbt)
    {
        NBTTagCompound tag = (NBTTagCompound) nbt;
        this.setLevel(tag.getInteger("Level"));
        this.setCorruption(tag.getInteger("Corruption"));
        this.setExpereince(tag.getFloat("Experience"));
    }

    public int getLevels() {
        return level;
    }

    public int getCorruption() {
        return levelCorruption;
    }

    public float getExperience() { return experience; }

    public int setLevel(int levelToSet) {
        return this.level = levelToSet;
    }

    public int setCorruption(int corruptionToSet) {
        return this.levelCorruption = corruptionToSet;
    }

    public float setExpereince(float experienceToSet) { return this.experience = experienceToSet; }

    public void sendPacket() {
        if(entity instanceof EntityPlayerMP && entity != null) {
            PlentifulMisc.network.sendTo(new LevelNetworkSyncClient(this.getLevels()),  (EntityPlayerMP) entity);
            PlentifulMisc.network.sendTo(new CorruptionNetworkSyncClient(this.getCorruption()), (EntityPlayerMP) entity);
            PlentifulMisc.network.sendTo(new ExperienceNetworkSyncClient(this.getExperience()), (EntityPlayerMP) entity);
        }
    }
}

 

I'm thinking that its because I'm using 3 different networks for 3 different values, maybe I can store it into one? Last time I did that, I get an error... here is how I'm registering them:

        network.registerMessage(LevelClientNetworkHandler.class, LevelNetworkSyncClient.class, 1, Side.CLIENT);
        network.registerMessage(CorruptionClientNetworkHandler.class, CorruptionNetworkSyncClient.class, 2, Side.CLIENT);
        network.registerMessage(ExperienceClientNetworkHandler.class, ExperienceNetworkSyncClient.class, 3, Side.CLIENT);

,

 

then how I'm modifying the cap:

    public ActionResult<ItemStack> onItemRightClick(World worldIn, EntityPlayer playerIn, EnumHand itemStackIn)
    {
        if(!worldIn.isRemote) {
            if (playerIn.hasCapability(CapabilityLevel.LEVEL, null)) {
                CapabilityLevelData cap = playerIn.getCapability(CapabilityLevel.LEVEL, null);
                cap.addExperienceInternal(200);
                if(playerIn.isSneaking()) {
                    cap.removeLevelsInternal(10);
                }
               return new ActionResult<>(EnumActionResult.SUCCESS, playerIn.getHeldItem(itemStackIn));
            }
        }
        return new ActionResult(EnumActionResult.PASS, playerIn.getHeldItem(itemStackIn));
    }
}

 

Thanks for your time,

 

[Lambda]

Edit:

Using the UpdateEntity event, and it seems to add fine, maybe its something to do with my item?

[hugo_the_dwarf]

On your item use, you are checking that the word is sever, which is good. However you "could" get away with running that on both server and client (remove the !word.isremote if statement) but if you want it server only make sure you send the update packet right after.