From ac756b67702fc3bedf50e6a05533eb85ba6002c7 Mon Sep 17 00:00:00 2001 From: Vexatos Date: Wed, 24 May 2017 18:26:35 +0200 Subject: [PATCH] Don't display empty stacks. --- src/main/scala/li/cil/oc/common/item/Drone.scala | 2 +- src/main/scala/li/cil/oc/util/ResultWrapper.scala | 3 +++ 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/src/main/scala/li/cil/oc/common/item/Drone.scala b/src/main/scala/li/cil/oc/common/item/Drone.scala index 9f18c7e41..e6df19990 100644 --- a/src/main/scala/li/cil/oc/common/item/Drone.scala +++ b/src/main/scala/li/cil/oc/common/item/Drone.scala @@ -36,7 +36,7 @@ class Drone(val parent: Delegator) extends traits.Delegate with CustomModel { override protected def tooltipExtended(stack: ItemStack, tooltip: util.List[String]): Unit = { if (KeyBindings.showExtendedTooltips) { val info = new DroneData(stack) - for (component <- info.components if component != null) { + for (component <- info.components if !component.isEmpty) { tooltip.add("- " + component.getDisplayName) } } diff --git a/src/main/scala/li/cil/oc/util/ResultWrapper.scala b/src/main/scala/li/cil/oc/util/ResultWrapper.scala index b367400cf..fc844b560 100644 --- a/src/main/scala/li/cil/oc/util/ResultWrapper.scala +++ b/src/main/scala/li/cil/oc/util/ResultWrapper.scala @@ -1,11 +1,14 @@ package li.cil.oc.util +import net.minecraft.item.ItemStack + import scala.math.ScalaNumber object ResultWrapper { def result(args: Any*): Array[AnyRef] = { def unwrap(arg: Any): AnyRef = arg match { case x: ScalaNumber => x.underlying + case x: ItemStack if x.isEmpty => null case x => x.asInstanceOf[AnyRef] } Array(args map unwrap: _*)