It was pretty simple to make, but it doesnt check if a player is in combat when they run the command. Players require the permission sk.pvp to toggle.

Code (Text):

1.  
2. command /combat:
3.   permission: sk.pvp
4.   trigger:
5.     if {combat%player%} is not false:
6.       set {combat%player%} to false
7.       send "&aYou are no longer able to fight" to player
8.     if {combat%player%} is false:
9.       set {combat%player%} to true
10.       send "&aYou are no now able to fight" to player
11.  
12. on damage of player:
13.   if {combat%victim%} is false:
14.     send "&4This player has combat off" to attacker
15.     cancel event
16.   if {combat%attacker%} is false:
17.     cancel event
18.     send "&4You can not attack while combat is off"

 

Code (Text):

1. combat%player% --> combat::%player%

 

whats the difference?

 

I've never seen someone use that kind of variable, it's just hell nah. List variables are way better

 

i learned skript through tutorials i barely paid attention to : p
How would i format variables your way?

 

{test::%player%} for example, the good thing is they can be looped. You cannot do that with normal ones

 

I don't want to necropost but I'm just saying instead of using %player% use %uuid of player% so that if the player changes their username, PVP will still be disabled for that player.