Workaround for a bug that broke the packet handling

This commit is contained in:
Ellpeck 2016-01-11 22:33:54 +01:00
parent dbf8093ea0
commit 85428e885d
2 changed files with 43 additions and 30 deletions

View file

@ -39,6 +39,8 @@ public class TileEntityAtomicReconstructor extends TileEntityInventoryBase imple
private boolean activateOnceWithSignal; private boolean activateOnceWithSignal;
private int oldEnergy; private int oldEnergy;
public boolean syncSlotsNextTime;
public TileEntityAtomicReconstructor(){ public TileEntityAtomicReconstructor(){
super(1, "reconstructor"); super(1, "reconstructor");
} }
@ -109,11 +111,12 @@ public class TileEntityAtomicReconstructor extends TileEntityInventoryBase imple
super.writeSyncableNBT(compound, sync); super.writeSyncableNBT(compound, sync);
compound.setInteger("CurrentTime", this.currentTime); compound.setInteger("CurrentTime", this.currentTime);
this.storage.writeToNBT(compound); this.storage.writeToNBT(compound);
}
@Override if(this.syncSlotsNextTime){
public boolean shouldSyncSlots(){ this.writeSlotsToCompound(compound);
return true; compound.setBoolean("ShouldSync", true);
this.syncSlotsNextTime = false;
}
} }
@Override @Override
@ -121,16 +124,22 @@ public class TileEntityAtomicReconstructor extends TileEntityInventoryBase imple
super.readSyncableNBT(compound, sync); super.readSyncableNBT(compound, sync);
this.currentTime = compound.getInteger("CurrentTime"); this.currentTime = compound.getInteger("CurrentTime");
this.storage.readFromNBT(compound); this.storage.readFromNBT(compound);
if(compound.getBoolean("ShouldSync")){
this.readSlotsFromCompound(compound);
}
} }
@Override @Override
public void setInventorySlotContents(int i, ItemStack stack){ public void setInventorySlotContents(int i, ItemStack stack){
super.setInventorySlotContents(i, stack); super.setInventorySlotContents(i, stack);
this.syncSlotsNextTime = true;
this.sendUpdate(); this.sendUpdate();
} }
@Override @Override
public ItemStack decrStackSize(int i, int j){ public ItemStack decrStackSize(int i, int j){
this.syncSlotsNextTime = true;
this.sendUpdate(); this.sendUpdate();
return super.decrStackSize(i, j); return super.decrStackSize(i, j);
} }

View file

@ -41,7 +41,20 @@ public abstract class TileEntityInventoryBase extends TileEntityBase implements
@Override @Override
public void writeSyncableNBT(NBTTagCompound compound, boolean isForSync){ public void writeSyncableNBT(NBTTagCompound compound, boolean isForSync){
super.writeSyncableNBT(compound, isForSync); super.writeSyncableNBT(compound, isForSync);
if(!isForSync || this.shouldSyncSlots()){ if(!isForSync){
this.writeSlotsToCompound(compound);
}
}
@Override
public void readSyncableNBT(NBTTagCompound compound, boolean isForSync){
super.readSyncableNBT(compound, isForSync);
if(!isForSync){
this.readSlotsFromCompound(compound);
}
}
public void writeSlotsToCompound(NBTTagCompound compound){
if(this.slots.length > 0){ if(this.slots.length > 0){
NBTTagList tagList = new NBTTagList(); NBTTagList tagList = new NBTTagList();
for(int currentIndex = 0; currentIndex < slots.length; currentIndex++){ for(int currentIndex = 0; currentIndex < slots.length; currentIndex++){
@ -55,16 +68,8 @@ public abstract class TileEntityInventoryBase extends TileEntityBase implements
compound.setTag("Items", tagList); compound.setTag("Items", tagList);
} }
} }
}
public boolean shouldSyncSlots(){ public void readSlotsFromCompound(NBTTagCompound compound){
return false;
}
@Override
public void readSyncableNBT(NBTTagCompound compound, boolean isForSync){
super.readSyncableNBT(compound, isForSync);
if(!isForSync || this.shouldSyncSlots()){
if(this.slots.length > 0){ if(this.slots.length > 0){
NBTTagList tagList = compound.getTagList("Items", 10); NBTTagList tagList = compound.getTagList("Items", 10);
for(int i = 0; i < tagList.tagCount(); i++){ for(int i = 0; i < tagList.tagCount(); i++){
@ -76,7 +81,6 @@ public abstract class TileEntityInventoryBase extends TileEntityBase implements
} }
} }
} }
}
@Override @Override
public int getInventoryStackLimit(){ public int getInventoryStackLimit(){