OpenSource/PaperMC
OpenSource/PaperMC
1
0
Fork 0
mirror of https://github.com/PaperMC/Paper.git synced 2024-11-24 16:18:02 +01:00
Code Issues Projects Releases Packages Wiki Activity

Catch plugins setting null Material or BlockData to blocks

Browse source
This commit is contained in:
md_5 2018-10-13 09:43:05 +11:00
parent 8ab46ff7c6
commit e7ced970d2
2 changed files with 5 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/main/java/org/bukkit/craftbukkit/block/CraftBlock.java
View file

@ -134,6 +134,7 @@ public class CraftBlock implements Block {
@Override
public void setType(Material type, boolean applyPhysics) {
Preconditions.checkArgument(type != null, "Material cannot be null");
setBlockData(type.createBlockData(), applyPhysics);
}
@ -144,6 +145,7 @@ public class CraftBlock implements Block {
@Override
public void setBlockData(BlockData data, boolean applyPhysics) {
Preconditions.checkArgument(data != null, "BlockData cannot be null");
setTypeAndData(((CraftBlockData) data).getState(), applyPhysics);
}

3
src/main/java/org/bukkit/craftbukkit/block/CraftBlockState.java
View file

@ -1,5 +1,6 @@
package org.bukkit.craftbukkit.block;
import com.google.common.base.Preconditions;
import net.minecraft.server.BlockPosition;
import org.bukkit.Location;
import org.bukkit.block.Block;
@ -98,6 +99,7 @@ public class CraftBlockState implements BlockState {
@Override
public void setBlockData(BlockData data) {
Preconditions.checkArgument(data != null, "BlockData cannot be null");
this.data = ((CraftBlockData) data).getState();
}
@ -121,6 +123,7 @@ public class CraftBlockState implements BlockState {
}
public void setType(final Material type) {
Preconditions.checkArgument(type != null, "Material cannot be null");
if (this.getType() != type) {
this.data = CraftMagicNumbers.getBlock(type).getBlockData();
}