feedc0de/LogBlock
1
0
Fork 0
forked from LogBlock/LogBlock
Code Pull Requests Activity

Fix lectern logging

Browse Source 
Fixes #761
This commit is contained in:
Brokkonaut
2019-07-09 03:42:33 +02:00
parent e8aaadf37b
commit 6f4ce7e6d0
3 changed files with 20 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
src/main/java/de/diddiz/LogBlock/blockstate/BlockStateCodecLectern.java
View File

@ -34,7 +34,11 @@ public class BlockStateCodecLectern implements BlockStateCodec {
if (conf != null) { if (conf != null) {
book = conf.getItemStack("book"); book = conf.getItemStack("book");
} }
lectern.getSnapshotInventory().setItem(0, book); try {
lectern.getSnapshotInventory().setItem(0, book);
} catch (NullPointerException e) {
//ignored
}
} }
} }

18
src/main/java/de/diddiz/LogBlock/listeners/LecternLogging.java
View File

@ -29,10 +29,18 @@ public class LecternLogging extends LoggingListener {
if (before.getType() == Material.LECTERN && after.getType() == Material.LECTERN) { if (before.getType() == Material.LECTERN && after.getType() == Material.LECTERN) {
Lectern lecternBefore = (Lectern) before.getBlock().getState(); Lectern lecternBefore = (Lectern) before.getBlock().getState();
ItemStack book = lecternBefore.getSnapshotInventory().getItem(0); ItemStack book = lecternBefore.getSnapshotInventory().getItem(0);
lecternBefore.getSnapshotInventory().setItem(0, null); try {
lecternBefore.getSnapshotInventory().setItem(0, null);
} catch (NullPointerException e) {
//ignored
}
lecternBefore.setBlockData(before.getBlockData()); lecternBefore.setBlockData(before.getBlockData());
consumer.queueBlockReplace(Actor.actorFromEntity(event.getPlayer()), lecternBefore, after); consumer.queueBlockReplace(Actor.actorFromEntity(event.getPlayer()), lecternBefore, after);
lecternBefore.getSnapshotInventory().setItem(0, book); try {
lecternBefore.getSnapshotInventory().setItem(0, book);
} catch (NullPointerException e) {
//ignored
}
} }
} }
} }
@ -43,7 +51,11 @@ public class LecternLogging extends LoggingListener {
if (wcfg != null && wcfg.isLogging(Logging.LECTERNBOOKCHANGE)) { if (wcfg != null && wcfg.isLogging(Logging.LECTERNBOOKCHANGE)) {
Lectern oldState = event.getLectern(); Lectern oldState = event.getLectern();
Lectern newState = (Lectern) oldState.getBlock().getState(); Lectern newState = (Lectern) oldState.getBlock().getState();
newState.getSnapshotInventory().setItem(0, null); try {
newState.getSnapshotInventory().setItem(0, null);
} catch (NullPointerException e) {
//ignored
}
org.bukkit.block.data.type.Lectern oldBlockData = (org.bukkit.block.data.type.Lectern) oldState.getBlockData(); org.bukkit.block.data.type.Lectern oldBlockData = (org.bukkit.block.data.type.Lectern) oldState.getBlockData();
org.bukkit.block.data.type.Lectern blockData = (org.bukkit.block.data.type.Lectern) Material.LECTERN.createBlockData(); org.bukkit.block.data.type.Lectern blockData = (org.bukkit.block.data.type.Lectern) Material.LECTERN.createBlockData();
blockData.setFacing(oldBlockData.getFacing()); blockData.setFacing(oldBlockData.getFacing());

1
src/main/java/de/diddiz/util/BukkitUtils.java
View File

@ -327,7 +327,6 @@ public class BukkitUtils {
containerBlocks.add(Material.BARREL); containerBlocks.add(Material.BARREL);
containerBlocks.add(Material.BLAST_FURNACE); containerBlocks.add(Material.BLAST_FURNACE);
containerBlocks.add(Material.SMOKER); containerBlocks.add(Material.SMOKER);
containerBlocks.add(Material.LECTERN);
// Doesn't actually have a block inventory // Doesn't actually have a block inventory
// containerBlocks.add(Material.ENDER_CHEST); // containerBlocks.add(Material.ENDER_CHEST);