Explicitly prevent breaking when not sneaking.

The default behaviour. This TODO seems to be outdated, at least in this
instance, as it falls back.

Author: noobanidus

common/src/main/java/noobanidus/mods/lootr/common/entity/LootrChestMinecartEntity.java

@@ -100,8 +100,9 @@ public boolean isInvulnerableTo(DamageSource source) {
       } else if (!source.getEntity().isShiftKeyDown()) {
         ((Player) source.getEntity()).displayClientMessage(Component.translatable("lootr.message.cart_should_sneak").setStyle(LootrAPI.getChatStyle()), false);
         ((Player) source.getEntity()).displayClientMessage(Component.translatable("lootr.message.should_sneak2", Component.translatable("lootr.message.cart_should_sneak3").setStyle(Style.EMPTY.withBold(true))).setStyle(LootrAPI.getChatStyle()), false);
-        // TODO: I think this is broken
-      } else if (source.getEntity().isShiftKeyDown()) {
+        return true;
+      } else //noinspection RedundantIfStatement
+        if (source.getEntity().isShiftKeyDown()) {
         return false;
       }
     } else {