diff --git a/src/com/massivecraft/massivecore/command/type/sender/TypeSenderIdAbstract.java b/src/com/massivecraft/massivecore/command/type/sender/TypeSenderIdAbstract.java index a43ebcb0..e2596733 100644 --- a/src/com/massivecraft/massivecore/command/type/sender/TypeSenderIdAbstract.java +++ b/src/com/massivecraft/massivecore/command/type/sender/TypeSenderIdAbstract.java @@ -110,15 +110,17 @@ public abstract class TypeSenderIdAbstract extends TypeAbstract // All of our subclasses return null if senderId is null. // Thus we don't need to check for that being null, but only check ret. - // If presence is online or local ... - if (this.presence == SenderPresence.LOCAL || this.presence == SenderPresence.ONLINE) + // If presence is online or local and the target is not visible for the sender then throw an error. + if + ( + (this.presence == SenderPresence.LOCAL || this.presence == SenderPresence.ONLINE) + && + senderId != null + && + !MixinVisibility.get().isVisible(senderId, sender) + ) { - // ... and the target is not visible for the sender ... - if (!MixinVisibility.get().isVisible(senderId, sender)) - { - // .. throw an error. - throwError(arg); - } + throwError(arg); } // Create & populate Ret