Improve and document event system. Deprecate duplicate methods.

src/main/java/com/massivecraft/factions/event/FPlayerJoinEvent.java

@@ -6,11 +6,8 @@ import org.bukkit.event.Cancellable;
 import org.bukkit.event.Event;
 import org.bukkit.event.HandlerList;
 
-public class FPlayerJoinEvent extends Event implements Cancellable {
+public class FPlayerJoinEvent extends FactionPlayerEvent implements Cancellable {
-    private static final HandlerList handlers = new HandlerList();
 
-    FPlayer fplayer;
-    Faction faction;
     PlayerJoinReason reason;
     boolean cancelled = false;
 
@@ -19,31 +16,14 @@ public class FPlayerJoinEvent extends Event implements Cancellable {
     }
 
     public FPlayerJoinEvent(FPlayer fp, Faction f, PlayerJoinReason r) {
-        fplayer = fp;
+        super(f, fp);
-        faction = f;
         reason = r;
     }
 
-    public FPlayer getFPlayer() {
-        return fplayer;
-    }
-
-    public Faction getFaction() {
-        return faction;
-    }
-
     public PlayerJoinReason getReason() {
         return reason;
     }
 
-    public HandlerList getHandlers() {
-        return handlers;
-    }
-
-    public static HandlerList getHandlerList() {
-        return handlers;
-    }
-
     @Override
     public boolean isCancelled() {
         return cancelled;

src/main/java/com/massivecraft/factions/event/FPlayerLeaveEvent.java

@@ -6,11 +6,9 @@ import org.bukkit.event.Cancellable;
 import org.bukkit.event.Event;
 import org.bukkit.event.HandlerList;
 
-public class FPlayerLeaveEvent extends Event implements Cancellable {
+public class FPlayerLeaveEvent extends FactionPlayerEvent implements Cancellable {
-    private static final HandlerList handlers = new HandlerList();
+
     private PlayerLeaveReason reason;
-    FPlayer FPlayer;
-    Faction Faction;
     boolean cancelled = false;
 
     public enum PlayerLeaveReason {
@@ -18,31 +16,14 @@ public class FPlayerLeaveEvent extends Event implements Cancellable {
     }
 
     public FPlayerLeaveEvent(FPlayer p, Faction f, PlayerLeaveReason r) {
-        FPlayer = p;
+        super(f, p);
-        Faction = f;
         reason = r;
     }
 
-    public HandlerList getHandlers() {
-        return handlers;
-    }
-
-    public static HandlerList getHandlerList() {
-        return handlers;
-    }
-
     public PlayerLeaveReason getReason() {
         return reason;
     }
 
-    public FPlayer getFPlayer() {
-        return FPlayer;
-    }
-
-    public Faction getFaction() {
-        return Faction;
-    }
-
     @Override
     public boolean isCancelled() {
         return cancelled;
@@ -51,9 +32,9 @@ public class FPlayerLeaveEvent extends Event implements Cancellable {
     @Override
     public void setCancelled(boolean c) {
         if (reason == PlayerLeaveReason.DISBAND || reason == PlayerLeaveReason.RESET) {
-            cancelled = false;
+            cancelled = false; // Don't let them cancel factions disbanding.
-            return;
+        } else {
-        }
         cancelled = c;
+        }
     }
 }

src/main/java/com/massivecraft/factions/event/FactionCreateEvent.java

@@ -9,6 +9,7 @@ import org.bukkit.event.Event;
 import org.bukkit.event.HandlerList;
 
 public class FactionCreateEvent extends Event implements Cancellable {
+    
     private static final HandlerList handlers = new HandlerList();
 
     private String factionTag;

src/main/java/com/massivecraft/factions/event/FactionDisbandEvent.java

@@ -9,29 +9,14 @@ import org.bukkit.event.Cancellable;
 import org.bukkit.event.Event;
 import org.bukkit.event.HandlerList;
 
-public class FactionDisbandEvent extends Event implements Cancellable {
+public class FactionDisbandEvent extends FactionEvent implements Cancellable {
-    private static final HandlerList handlers = new HandlerList();
 
-    private boolean cancelled;
+    private boolean cancelled = false;
-    private String id;
     private Player sender;
 
     public FactionDisbandEvent(Player sender, String factionId) {
-        cancelled = false;
+        super(Factions.i.get(factionId));
         this.sender = sender;
-        this.id = factionId;
-    }
-
-    public HandlerList getHandlers() {
-        return handlers;
-    }
-
-    public static HandlerList getHandlerList() {
-        return handlers;
-    }
-
-    public Faction getFaction() {
-        return Factions.i.get(id);
     }
 
     public FPlayer getFPlayer() {

src/main/java/com/massivecraft/factions/event/FactionEvent.java

@@ -0,0 +1,27 @@
+package com.massivecraft.factions.event;
+
+import com.massivecraft.factions.Faction;
+import org.bukkit.event.Event;
+import org.bukkit.event.HandlerList;
+
+public class FactionEvent extends Event {
+
+    private static final HandlerList handlers = new HandlerList();
+    private final Faction faction;
+
+    public FactionEvent(Faction faction) {
+        this.faction = faction;
+    }
+
+    public Faction getFaction() {
+        return this.getFaction();
+    }
+
+    public HandlerList getHandlers() {
+        return handlers;
+    }
+    public static HandlerList getHandlerList() {
+        return handlers;
+    }
+
+}

src/main/java/com/massivecraft/factions/event/FactionPlayerEvent.java

@@ -0,0 +1,18 @@
+package com.massivecraft.factions.event;
+
+import com.massivecraft.factions.FPlayer;
+import com.massivecraft.factions.Faction;
+
+public class FactionPlayerEvent extends FactionEvent {
+
+    private final FPlayer fPlayer;
+
+    public FactionPlayerEvent(Faction faction, FPlayer fPlayer) {
+        super(faction);
+        this.fPlayer = fPlayer;
+    }
+
+    public FPlayer getfPlayer() {
+        return this.fPlayer;
+    }
+}

src/main/java/com/massivecraft/factions/event/FactionRelationEvent.java

@@ -7,6 +7,7 @@ import org.bukkit.event.HandlerList;
 
 
 public class FactionRelationEvent extends Event {
+
     private static final HandlerList handlers = new HandlerList();
 
     private Faction fsender;

src/main/java/com/massivecraft/factions/event/FactionRenameEvent.java

@@ -7,47 +7,28 @@ import org.bukkit.event.Cancellable;
 import org.bukkit.event.Event;
 import org.bukkit.event.HandlerList;
 
-public class FactionRenameEvent extends Event implements Cancellable {
+public class FactionRenameEvent extends FactionPlayerEvent implements Cancellable {
-    private static final HandlerList handlers = new HandlerList();
 
-    private boolean cancelled;
+    private boolean cancelled = false;
-    private FPlayer fplayer;
-    private Faction faction;
     private String tag;
 
     public FactionRenameEvent(FPlayer sender, String newTag) {
-        fplayer = sender;
+        super(sender.getFaction(), sender);
-        faction = sender.getFaction();
         tag = newTag;
-        this.cancelled = false;
-    }
-
-    public Faction getFaction() {
-        return (faction);
-    }
-
-    public FPlayer getFPlayer() {
-        return (fplayer);
     }
 
+    @Deprecated
     public Player getPlayer() {
-        return (fplayer.getPlayer());
+        return getfPlayer().getPlayer();
     }
 
+    @Deprecated
     public String getOldFactionTag() {
-        return (faction.getTag());
+        return getFaction().getTag();
     }
 
     public String getFactionTag() {
-        return (tag);
+        return tag;
-    }
-
-    public HandlerList getHandlers() {
-        return handlers;
-    }
-
-    public static HandlerList getHandlerList() {
-        return handlers;
     }
 
     @Override

src/main/java/com/massivecraft/factions/event/LandClaimEvent.java

@@ -8,51 +8,34 @@ import org.bukkit.event.Cancellable;
 import org.bukkit.event.Event;
 import org.bukkit.event.HandlerList;
 
-public class LandClaimEvent extends Event implements Cancellable {
+public class LandClaimEvent extends FactionPlayerEvent implements Cancellable {
-    private static final HandlerList handlers = new HandlerList();
 
     private boolean cancelled;
     private FLocation location;
-    private Faction faction;
-    private FPlayer fplayer;
 
     public LandClaimEvent(FLocation loc, Faction f, FPlayer p) {
+        super(f, p);
         cancelled = false;
         location = loc;
-        faction = f;
-        fplayer = p;
-    }
-
-    public HandlerList getHandlers() {
-        return handlers;
-    }
-
-    public static HandlerList getHandlerList() {
-        return handlers;
     }
 
     public FLocation getLocation() {
         return this.location;
     }
 
-    public Faction getFaction() {
+    @Deprecated
-        return faction;
-    }
-
     public String getFactionId() {
-        return faction.getId();
+        return getFaction().getId();
     }
 
+    @Deprecated
     public String getFactionTag() {
-        return faction.getTag();
+        return getFaction().getTag();
-    }
-
-    public FPlayer getFPlayer() {
-        return fplayer;
     }
 
+    @Deprecated
     public Player getPlayer() {
-        return fplayer.getPlayer();
+        return getfPlayer().getPlayer();
     }
 
     @Override

src/main/java/com/massivecraft/factions/event/LandUnclaimAllEvent.java

@@ -6,42 +6,24 @@ import org.bukkit.entity.Player;
 import org.bukkit.event.Event;
 import org.bukkit.event.HandlerList;
 
-public class LandUnclaimAllEvent extends Event {
+public class LandUnclaimAllEvent extends FactionPlayerEvent {
-    private static final HandlerList handlers = new HandlerList();
-
-    private Faction faction;
-    private FPlayer fplayer;
 
     public LandUnclaimAllEvent(Faction f, FPlayer p) {
-        faction = f;
+        super(f, p);
-        fplayer = p;
-    }
-
-    public HandlerList getHandlers() {
-        return handlers;
-    }
-
-    public static HandlerList getHandlerList() {
-        return handlers;
-    }
-
-    public Faction getFaction() {
-        return faction;
     }
 
+    @Deprecated
     public String getFactionId() {
-        return faction.getId();
+        return getFaction().getId();
     }
 
+    @Deprecated
     public String getFactionTag() {
-        return faction.getTag();
+        return getFaction().getTag();
-    }
-
-    public FPlayer getFPlayer() {
-        return fplayer;
     }
 
+    @Deprecated
     public Player getPlayer() {
-        return fplayer.getPlayer();
+        return getfPlayer().getPlayer();
     }
 }

src/main/java/com/massivecraft/factions/event/LandUnclaimEvent.java

@@ -8,51 +8,34 @@ import org.bukkit.event.Cancellable;
 import org.bukkit.event.Event;
 import org.bukkit.event.HandlerList;
 
-public class LandUnclaimEvent extends Event implements Cancellable {
+public class LandUnclaimEvent extends FactionPlayerEvent implements Cancellable {
-    private static final HandlerList handlers = new HandlerList();
 
     private boolean cancelled;
     private FLocation location;
-    private Faction faction;
-    private FPlayer fplayer;
 
     public LandUnclaimEvent(FLocation loc, Faction f, FPlayer p) {
+        super(f, p);
         cancelled = false;
         location = loc;
-        faction = f;
-        fplayer = p;
-    }
-
-    public HandlerList getHandlers() {
-        return handlers;
-    }
-
-    public static HandlerList getHandlerList() {
-        return handlers;
     }
 
     public FLocation getLocation() {
         return this.location;
     }
 
-    public Faction getFaction() {
+    @Deprecated
-        return faction;
-    }
-
     public String getFactionId() {
-        return faction.getId();
+        return getFaction().getId();
     }
 
+    @Deprecated
     public String getFactionTag() {
-        return faction.getTag();
+        return getFaction().getTag();
-    }
-
-    public FPlayer getFPlayer() {
-        return fplayer;
     }
 
+    @Deprecated
     public Player getPlayer() {
-        return fplayer.getPlayer();
+        return getfPlayer().getPlayer();
     }
 
     @Override

src/main/java/com/massivecraft/factions/event/PowerLossEvent.java

@@ -7,48 +7,28 @@ import org.bukkit.event.Cancellable;
 import org.bukkit.event.Event;
 import org.bukkit.event.HandlerList;
 
+public class PowerLossEvent extends FactionPlayerEvent implements Cancellable {
 
-public class PowerLossEvent extends Event implements Cancellable {
+    private boolean cancelled = false;
-    private static final HandlerList handlers = new HandlerList();
-
-    private boolean cancelled;
-    private Faction faction;
-    private FPlayer fplayer;
     private String message;
 
     public PowerLossEvent(Faction f, FPlayer p) {
-        cancelled = false;
+        super(f, p);
-        faction = f;
-        fplayer = p;
-    }
-
-    @Override
-    public HandlerList getHandlers() {
-        return handlers;
-    }
-
-    public static HandlerList getHandlerList() {
-        return handlers;
-    }
-
-    public Faction getFaction() {
-        return faction;
     }
 
+    @Deprecated
     public String getFactionId() {
-        return faction.getId();
+        return getFaction().getId();
     }
 
+    @Deprecated
     public String getFactionTag() {
-        return faction.getTag();
+        return getFaction().getTag();
-    }
-
-    public FPlayer getFPlayer() {
-        return fplayer;
     }
 
+    @Deprecated
     public Player getPlayer() {
-        return fplayer.getPlayer();
+        return getfPlayer().getPlayer();
     }
 
     public String getMessage() {