Refactor the RegionEvent into two subclasses for future operations

src/main/java/us/camin/regions/RegionCreateEvent.java

@@ -0,0 +1,38 @@
+package us.camin.regions;
+
+/**
+ * This file is part of Regions
+ *
+ * Regions is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation, either version 3 of the License, or
+ * (at your option) any later version.
+ *
+ * Regions is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with Regions.  If not, see <http://www.gnu.org/licenses/>.
+ *
+ */
+
+import org.bukkit.event.Event;
+import org.bukkit.event.HandlerList;
+
+public class RegionCreateEvent extends RegionEvent {
+    public static final HandlerList s_handlers = new HandlerList();
+
+    public RegionCreateEvent(Region region) {
+        super(region);
+    }
+
+    public HandlerList getHandlers() {
+        return s_handlers;
+    }
+
+    public static HandlerList getHandlerList() {
+        return s_handlers;
+    }
+}

src/main/java/us/camin/regions/RegionEvent.java

@@ -21,19 +21,12 @@ package us.camin.regions;
 import org.bukkit.event.Event;
 import org.bukkit.event.HandlerList;
 
-public class RegionEvent extends Event {
+public abstract class RegionEvent extends Event {
     private static final HandlerList s_handlers = new HandlerList();
-    public final EventType type;
     public final Region region;
 
-    public enum EventType {
+    public RegionEvent(Region region) {
-        Added,
-        Removed
-    }
-
-    public RegionEvent(Region region, EventType type) {
         this.region = region;
-        this.type = type;
     }
 
     @Override

src/main/java/us/camin/regions/RegionEventHandler.java

@@ -36,14 +36,15 @@ public class RegionEventHandler implements Listener {
     }
 
     @EventHandler
-    public void onRegionEvent(RegionEvent event) {
+    public void onRegionEvent(RegionRemoveEvent event) {
-        if (event.type == RegionEvent.EventType.Added) {
+        Marker marker = m_set.findMarkerByLabel(event.region.name());
-            Location loc = event.region.location();
+        marker.deleteMarker();
-            MarkerIcon icon = m_api.getMarkerIcon("default");
+    }
-            m_set.createMarker(null, event.region.name(), loc.getWorld().getName(), loc.getX(), loc.getY(), loc.getZ(), icon, false);
+
-        } else if (event.type == RegionEvent.EventType.Removed) {
+    @EventHandler
-            Marker marker = m_set.findMarkerByLabel(event.region.name());
+    public void onRegionEvent(RegionCreateEvent event) {
-            marker.deleteMarker();
+        Location loc = event.region.location();
-        }
+        MarkerIcon icon = m_api.getMarkerIcon("default");
+        m_set.createMarker(null, event.region.name(), loc.getWorld().getName(), loc.getX(), loc.getY(), loc.getZ(), icon, false);
     }
 }

src/main/java/us/camin/regions/RegionManager.java

@@ -73,7 +73,7 @@ public class RegionManager {
         if (!m_regions.containsKey(worldName))
             m_regions.put(worldName, new ArrayList<Region>());
         if (m_regions.get(worldName).add(r)) {
-            m_pm.callEvent(new RegionEvent(r, RegionEvent.EventType.Added));
+            m_pm.callEvent(new RegionCreateEvent(r));
         }
         return false;
     }
@@ -83,7 +83,7 @@ public class RegionManager {
         log.fine("Removing region "+r.name()+" from "+r.location());
         if (m_regions.containsKey(worldName)) {
             if (m_regions.get(worldName).remove(r)) {
-                m_pm.callEvent(new RegionEvent(r, RegionEvent.EventType.Removed));
+                m_pm.callEvent(new RegionRemoveEvent(r));
             }
             return true;
         }

src/main/java/us/camin/regions/RegionRemoveEvent.java

@@ -0,0 +1,39 @@
+package us.camin.regions;
+
+/**
+ * This file is part of Regions
+ *
+ * Regions is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation, either version 3 of the License, or
+ * (at your option) any later version.
+ *
+ * Regions is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with Regions.  If not, see <http://www.gnu.org/licenses/>.
+ *
+ */
+
+import org.bukkit.event.Event;
+import org.bukkit.event.HandlerList;
+
+public class RegionRemoveEvent extends RegionEvent {
+    public static final HandlerList s_handlers = new HandlerList();
+
+    public RegionRemoveEvent(Region region) {
+        super(region);
+    }
+
+    public HandlerList getHandlers() {
+        return s_handlers;
+    }
+
+    public static HandlerList getHandlerList() {
+        return s_handlers;
+    }
+}
+