[MC-21098] Throwing projectiles at minecarts, boats (with chest), (glow) item frames, and paintings in creative mode will delete them

The Bug:

Throwing projectiles at minecarts, boats, item frames, or paintings in creative mode will delete them.

Steps to Reproduce:

Switch into creative mode and obtain any throwable projectile, for example, some snowballs.
Summon either a minecart, boat, item frame, or painting.
Throw the projectile at the entity you just summoned.
Take note as to whether or not throwing projectiles at minecarts, boats, item frames, or paintings in creative mode will delete them.

Observed Behavior:

Throwing projectiles at minecarts, boats, item frames, or paintings in creative mode will delete them.

Expected Behavior:

Throwing projectiles at minecarts, boats, item frames, or paintings in creative mode would not delete them.

Code Analysis:

Code analysis by @unknown can be found in this comment.

Linked issues

is duplicated by 11

MC-96614 Throwing eggs and snowballs at a boat destroy's the boat Resolved

MC-118205 Snowballs damage certain inanimate entities Resolved

MC-150612 Ender pearls can destroy minecarts in Creative mode Resolved

MC-151271 Ender pearls can destroy boats and boats with chest in Creative mode Resolved

MC-212293 Throwing trident at item frame in creative mode doesn't drop the item in the frame or the frame Resolved

6 more links

relates to 1

MC-78689 Invulnerable entities are hurt by non-melee damage from creative players Reopened

Attachments

MC-21098.mp4

Comments 31

kumasasa 2013-07-07T15:48:53Z

Working as intended.
All entities drop nothing when destroyed by snowballs in creative.

marcono1234 2016-03-07T16:12:29Z

Confirmed for

1.9

The bug is that normally projectiles like snowballs, eggs and enderpearls deal 0 damage, however when you throw them in Creative mode they destroy boats and minecarts instantaneously.

marcono1234 2016-03-07T16:26:51Z

Please link to this comment in the description

The following is based on decompiled version of Minecraft 1.9 using MCP 9.24 beta. All method and class names are the names used in the decompiled version.

The reason why this happens is because the methods net.minecraft.entity.item.EntityMinecart.attackEntityFrom(DamageSource, float) and net.minecraft.entity.item.EntityBoat.attackEntityFrom(DamageSource, float) only test if the player damaging the entity is in creative mode however they should make sure that the damage was dealt directly.

/**
 * Called when the entity is attacked.
 */
public boolean attackEntityFrom(DamageSource source, float amount)
{
    if (!this.worldObj.isRemote && !this.isDead)
    {
        if (this.isEntityInvulnerable(source))
        {
            return false;
        }
        else
        {
            this.setRollingDirection(-this.getRollingDirection());
            this.setRollingAmplitude(10);
            this.setBeenAttacked();
            this.setDamage(this.getDamage() + amount * 10.0F);
            // Replaced this
            //boolean flag = source.getEntity() instanceof EntityPlayer && ((EntityPlayer)source.getEntity()).capabilities.isCreativeMode;
            boolean flag = source.getDamageType().equals("player") && source.isCreativePlayer();

            if (flag || this.getDamage() > 40.0F)
            {
                this.removePassengers();

                if (flag && !this.hasCustomName())
                {
                    this.setDead();
                }
                else
                {
                    this.killMinecart(source);
                }
            }

            return true;
        }
    }
    else
    {
        return true;
    }
}