player-death is very redundant. Changing to entity-death.

src/main/java/com/sk89q/rulelists/KnownAttachment.java

@@ -37,7 +37,6 @@ public enum KnownAttachment {
     PLAYER_SPAWN("player-spawn"),
     PLAYER_RESPAWN("player-respawn"),
     PLAYER_QUIT("player-quit"),
-    PLAYER_DEATH("player-death"),
     ENTITY_EXPLODE("entity-explode"),
     ENTITY_DAMAGE("entity-damage"),
     ENTITY_DEATH("entity-death"),

src/main/java/com/sk89q/worldguard/bukkit/WorldGuardPlayerListener.java

@@ -848,7 +848,7 @@ public void onPlayerDeath(PlayerDeathEvent event) {
         WorldConfiguration wcfg = cfg.get(player.getWorld());
 
         // RuleLists
-        RuleSet rules = wcfg.getRuleList().get(KnownAttachment.PLAYER_DEATH);
+        RuleSet rules = wcfg.getRuleList().get(KnownAttachment.ENTITY_DEATH);
         BukkitContext context = new BukkitContext(event);
         context.setTargetEntity(player);
         rules.process(context);

src/main/resources/builtin_rules.yml

@@ -23,7 +23,7 @@
           set-data: 0
 - setting: protection.disable-xp-orb-drops
   rules:
-    - when: player-death
+    - when: entity-death
       then:
         - ?: set-drop
           set-xp: 0