On Damage GUI

[VampTheSkripter]

Skript Version: Skript 2.2-dev27
Minecraft Version: 1.8-1.12

Okay, so, I want this skript and it says attacker cant be outside of a damage event! I don't know whats wrong, so please fix this. @ShaneBee

Full Code:

on damage:
    if attacker is a player:
        if victim is a player:
            cancel event
            create a gui with virtual chest named "&4&lAttacks" with 1 row:
                make gui slot 0 to diamond named "&4Punch" with lore "&cDamage&8: &65":
                    damage victim by 5 hearts
                    send "&4You punched %victim% by 5 hearts!" to attacker
                    send "&7+ &a10 Experience"
                    add 10 to {Exp.%attacker%}
                if {Lvl.%attacker%} is greater than 4:
                    make gui slot 1 to diamond named "&4Mega Punch" with lore "&cDamage&8: &68":
                        damage victim by 8 hearts
                        send "&4You MEGA Punched %victim% by 8 hearts!" to attacker
                        send "&7+ &a25 Experience"
                        add 25 to {Exp.%attacker%}
                if {Lvl.%attacker%} is greater than 7:
                    make gui slot 2 to diamond named "&4Slam" with lore "&cDamage&8: &610":
                        damage victim by 10 hearts
                        send "&4You slammed %victim% to the ground by 10 hearts!" to attacker

Errors on Reload:

Console Errors:

[None]

Addons using:
SkQuery, skUtilities, SkRayFall, Skellet, & Skript-Mirror

Troubleshooting:

Have you tried searching the docs? Yes
Have you tried searching the forums? Yes
What other methods have you tried to fix it? I have tried to put it in on click, that gave more errors.

 

[TPGamesNL]

why do you tag shane lol

 

[Runakai]

lol why not just use this

on damage:
    if attacker is a player:
        if victim is a player:
            set {_victim} to victim
            set {_attacker} to attacker
            cancel event
            create a gui with virtual chest named "&4&lAttacks" with 1 row:
                make gui slot 0 to diamond named "&4Punch" with lore "&cDamage&8: &65":
                    damage {_victim} by 5 hearts
                    send "&4You punched %{_victim}% by 5 hearts!" to {_attacker}
                    send "&7+ &a10 Experience"
                    add 10 to {Exp.%{_attacker}%}
                if {Lvl.%{_attacker}%} is greater than 4:
                    make gui slot 1 to diamond named "&4Mega Punch" with lore "&cDamage&8: &68":
                        damage {_victim} by 8 hearts
                        send "&4You MEGA Punched %{_victim}% by 8 hearts!" to {_attacker}
                        send "&7+ &a25 Experience"
                        add 25 to {Exp.%{_attacker}%}
                if {Lvl.%{_attacker}%} is greater than 7:
                    make gui slot 2 to diamond named "&4Slam" with lore "&cDamage&8: &610":
                        damage {_victim} by 10 hearts
                        send "&4You slammed %{_victim}% to the ground by 10 hearts!" to {_attacker}