Implemented fixed destination block using * in .gate file. This is the recommended method of doing an exit point for custom gates, as the automatic method doesn't work in a lot of cases.

Split networks up in memory, can now use same name in different networks. As a result, fixed gates must now specify a network.
Added the ability to have a private gate, which only you can activate. Use the 'P' option to create.
Fixed but not AlwaysOn gates now open the destination gate.
Fixed gates now show their network. Existing fixed gates are added to the default network (Sorry! It had to be done)

README

@@ -79,6 +79,12 @@ gate-folder - The folder containing your .gate files
 =============
   Changes
 =============
+[Version 0.12]
+ - Implemented fixed destination block using * in .gate file. This is the recommended method of doing an exit point for custom gates, as the automatic method doesn't work in a lot of cases.
+ - Split networks up in memory, can now use same name in different networks. As a result, fixed gates must now specify a network.
+ - Added the ability to have a private gate, which only you can activate. Use the 'P' option to create.
+ - Fixed but not AlwaysOn gates now open the destination gate.
+ - Fixed gates now show their network. Existing fixed gates are added to the default network (Sorry! It had to be done)
 [Version 0.11]
  - Fuuuu- Some code got undid and broke everything. Fixed.
 [Version 0.10]

src/net/TheDgtl/Stargate/Gate.java

@@ -22,6 +22,7 @@ public class Gate {
     public static final int ANYTHING = -1;
     public static final int ENTRANCE = -2;
     public static final int CONTROL = -3;
+    public static final int EXIT = -4;
     private static HashMap<String, Gate> gates = new HashMap<String, Gate>();
     private static HashMap<Integer, ArrayList<Gate>> controlBlocks = new HashMap<Integer, ArrayList<Gate>>();
 
@@ -31,6 +32,7 @@ public class Gate {
     private RelativeBlockVector[] entrances = new RelativeBlockVector[0];
     private RelativeBlockVector[] border = new RelativeBlockVector[0];
     private RelativeBlockVector[] controls = new RelativeBlockVector[0];
+    private RelativeBlockVector exitBlock = null;
     private HashMap<RelativeBlockVector, Integer> exits = new HashMap<RelativeBlockVector, Integer>();
     private int portalBlockOpen = Material.PORTAL.getId();
     private int portalBlockClosed = Material.AIR.getId();
@@ -56,9 +58,11 @@ public class Gate {
             for (int x = 0; x < layout[y].length; x++) {
                 Integer id = layout[y][x];
 
-                if (id == ENTRANCE) {
+                if (id == ENTRANCE || id == EXIT) {
                     entranceList.add(new RelativeBlockVector(x, y, 0));
                     exitDepths[x] = y;
+                    if (id == EXIT)
+                    	this.exitBlock = new RelativeBlockVector(x, y, 0);
                     //bottom = y;
                 } else if (id == CONTROL) {
                     controlList.add(new RelativeBlockVector(x, y, 0));
@@ -122,6 +126,8 @@ public class Gate {
                         symbol = ' ';
                     } else if (id == CONTROL) {
                         symbol = '-';
+                    } else if (id == EXIT) {
+                    	symbol = '*';
                     } else if (reverse.containsKey(id)) {
                         symbol = reverse.get(id);
                     } else {
@@ -163,6 +169,9 @@ public class Gate {
     public HashMap<RelativeBlockVector, Integer> getExits() {
         return exits;
     }
+    public RelativeBlockVector getExit() {
+    	return exitBlock;
+    }
 
     public int getControlBlock() {
         return types.get('-');
@@ -189,7 +198,7 @@ public class Gate {
             for (int x = 0; x < layout[y].length; x++) {
                 int id = layout[y][x];
 
-                if (id == ENTRANCE) {
+                if (id == ENTRANCE || id == EXIT) {
                     if (topleft.modRelative(x, y, 0, modX, 1, modZ).getType() != 0) {
                         return false;
                     }
@@ -246,6 +255,8 @@ public class Gate {
 
                         if (symbol.equals('.')) {
                             id = ENTRANCE;
+                        } else if (symbol.equals('*')) {
+                        	id = EXIT;
                         } else if (symbol.equals(' ')) {
                             id = ANYTHING;
                         } else if (symbol.equals('-')) {
@@ -356,7 +367,7 @@ public class Gate {
             {ANYTHING, Portal.OBSIDIAN, Portal.OBSIDIAN, ANYTHING},
             {Portal.OBSIDIAN, ENTRANCE, ENTRANCE, Portal.OBSIDIAN},
             {CONTROL, ENTRANCE, ENTRANCE, CONTROL},
-            {Portal.OBSIDIAN, ENTRANCE, ENTRANCE, Portal.OBSIDIAN},
+            {Portal.OBSIDIAN, EXIT, ENTRANCE, Portal.OBSIDIAN},
             {ANYTHING, Portal.OBSIDIAN, Portal.OBSIDIAN, ANYTHING},
         };
         HashMap<Character, Integer> types = new HashMap<Character, Integer>();

src/net/TheDgtl/Stargate/Portal.java

@@ -33,9 +33,10 @@ public class Portal {
     public static final int SIGN = 68;
     public static final int BUTTON = 77;
     private static final HashMap<Blox, Portal> lookupBlocks = new HashMap<Blox, Portal>();
-    private static final HashMap<String, Portal> lookupNames = new HashMap<String, Portal>();
     private static final HashMap<Blox, Portal> lookupEntrances = new HashMap<Blox, Portal>();
-    private static final ArrayList<String> allPortals = new ArrayList<String>();
+    private static final ArrayList<Portal> allPortals = new ArrayList<Portal>();
+    private static final HashMap<String, ArrayList<String>> allPortalsNet = new HashMap<String, ArrayList<String>>();
+    private static final HashMap<String, HashMap<String, Portal>> lookupNamesNet = new HashMap<String, HashMap<String, Portal>>();
     private Blox topLeft;
     private int modX;
     private int modZ;
@@ -60,6 +61,7 @@ public class Portal {
     private boolean hidden = false;
     private Player activePlayer;
     private boolean alwaysOn = false;
+    private boolean priv = false;
     private World world;
     private long openTime;
 
@@ -67,7 +69,7 @@ public class Portal {
             float rotX, SignPost id, Blox button,
             String dest, String name,
             boolean verified, String network, Gate gate,
-            String owner, boolean hidden, boolean alwaysOn) {
+            String owner, boolean hidden, boolean alwaysOn, boolean priv) {
         this.topLeft = topLeft;
         this.modX = modX;
         this.modZ = modZ;
@@ -84,6 +86,7 @@ public class Portal {
         this.owner = owner;
         this.hidden = hidden;
         this.alwaysOn = alwaysOn;
+        this.priv = priv;
         this.world = topLeft.getWorld();
         
         if (this.alwaysOn && !this.fixed) {
@@ -105,17 +108,6 @@ public class Portal {
         return false;
     }
 
-    private boolean pastGrace() {
-    	if (isAlwaysOn()) return false; // Ignore always open fixed gates.
-        if (manipGrace(false, false)) {
-            return manipGrace(true, false);
-        }
-        if (isActive() || isOpen()) {
-            return true;
-        }
-        return false; // else.
-    }
-
     public boolean isOpen() {
         return isOpen || isAlwaysOn();
     }
@@ -127,6 +119,10 @@ public class Portal {
     public boolean isHidden() {
     	return hidden;
     }
+    
+    public boolean isPrivate() {
+    	return priv;
+    }
 
     public boolean open(boolean force) {
         return open(null, force);
@@ -145,7 +141,7 @@ public class Portal {
         openTime = System.currentTimeMillis() / 1000;
         Stargate.openList.add(this);
         // Open remote gate
-        if (!isFixed()) {
+        if (!isAlwaysOn()) {
             player = openFor;
             manipGrace(true, true);
 
@@ -162,7 +158,7 @@ public class Portal {
 
     public void close(boolean force) {
     	if (isAlwaysOn() && !force) return; // Never close an always open gate
-        if (!isFixed()) {
+        if (!isAlwaysOn()) {
         	Portal end = getDestination();
 
 	        if (end != null && end.isOpen()) {
@@ -258,43 +254,50 @@ public class Portal {
     }
 
     public Location getExit(Location traveller, Portal origin) {
-    	// Move the "entrance" to the first portal block up at the current x/z
+    	Location loc = null;
-    	// "Exits" seem to only consist of the lowest Y coord
+    	// Check if the gate has an exit block
-    	int bX = traveller.getBlockX();
+    	 if (gate.getExit() != null) {
-    	int bY = traveller.getBlockY();
+    		 Blox exit = getBlockAt(gate.getExit());
-    	int bZ = traveller.getBlockZ();
+    		 loc = exit.modRelativeLoc(0D, 0D, 1D, traveller.getYaw(), traveller.getPitch(), modX, 1, modZ);
-    	while (traveller.getWorld().getBlockTypeIdAt(bX, bY, bZ) == gate.getPortalBlockOpen())
+    	 } else {
-    		bY --;
+	    	// Move the "entrance" to the first portal block up at the current x/z
-    	bY++;
+	    	// "Exits" seem to only consist of the lowest Y coord
-    	// End
+	    	int bX = traveller.getBlockX();
-        Blox entrance = new Blox(world, bX, bY, bZ);
+	    	int bY = traveller.getBlockY();
-        HashMap<Blox, Integer> originExits = origin.getExits();
+	    	int bZ = traveller.getBlockZ();
-        HashMap<Blox, Integer> destExits = this.getExits();
+	    	while (traveller.getWorld().getBlockTypeIdAt(bX, bY, bZ) == gate.getPortalBlockOpen())
+	    		bY --;
+	    	bY++;
+	    	// End
+	    	
+	        Blox entrance = new Blox(world, bX, bY, bZ);
+	        HashMap<Blox, Integer> originExits = origin.getExits();
+	        HashMap<Blox, Integer> destExits = this.getExits();
+	
+	        if (originExits.containsKey(entrance)) {
+	            int position = (int)(((float)originExits.get(entrance) / originExits.size()) * destExits.size());
+	            Blox exit = getReverseExits().get(position);
+	            // Workaround for different size gates. Just drop them at the first exit block.
+	            if (exit == null) {
+	            	exit = (Blox)getReverseExits().values().toArray()[0];
+	            }
+	            if (exit != null) {
+	                loc = exit.modRelativeLoc(0D, 0D, 1D, traveller.getYaw(), traveller.getPitch(), modX, 1, modZ);
+	            }
+	        }
+    	}
+    	if (loc != null) {
+            Block block = world.getBlockAt(loc.getBlockX(), loc.getBlockY(), loc.getBlockZ());
 
-        if (originExits.containsKey(entrance)) {
+            if (block.getType() == Material.STEP) {
-        	
+                loc.setY(loc.getY() + 0.5);
-            int position = (int)(((float)originExits.get(entrance) / originExits.size()) * destExits.size());
-            Blox exit = getReverseExits().get(position);
-            // Workaround for different size gates. Just drop them at the first exit block.
-            if (exit == null) {
-            	exit = (Blox)getReverseExits().values().toArray()[0];
             }
 
-            if (exit != null) {
+            loc.setPitch(traveller.getPitch());
-                Location loc = exit.modRelativeLoc(0D, 0D, 1D, traveller.getYaw(), traveller.getPitch(), modX, 1, modZ);
+            return loc;
-                Block block = world.getBlockAt(loc.getBlockX(), loc.getBlockY(), loc.getBlockZ());
+    	}
 
-                if (block.getType() == Material.STEP) {
+        Stargate.log.log(Level.WARNING, "No position found calculating route from " + this + " to " + origin);
-                    loc.setY(loc.getY() + 0.5);
-                }
-
-                loc.setPitch(traveller.getPitch());
-                return loc;
-            }
-        }
-
-        Stargate.log.log(Level.WARNING, "No position found calculting route from " + this + " to " + origin);
-        Stargate.log.info(originExits.get(entrance) + " / " + originExits.size() + " * " + destExits.size());
         return traveller;
     }
 
@@ -321,7 +324,7 @@ public class Portal {
     }
 
     public Portal getDestination() {
-        return Portal.getByName(destination);
+        return Portal.getByName(destination, getNetwork());
     }
 
     public String getDestinationName() {
@@ -356,13 +359,12 @@ public class Portal {
         drawSign(true);
         Stargate.activeList.add(this);
         activePlayer = player;
-        for (String dest : allPortals) {
+        for (String dest : allPortalsNet.get(getNetwork().toLowerCase())) {
-            Portal portal = getByName(dest);
+            Portal portal = getByName(dest, getNetwork());
-            if (	(portal.getNetwork().equalsIgnoreCase(network)) && 				// In the network
+            if (	(!dest.equalsIgnoreCase(getName())) && 							// Not this portal
-            		(!dest.equalsIgnoreCase(getName())) && 							// Not this portal
             		(!portal.isHidden() || Stargate.hasPerm(player, "stargate.hidden", player.isOp()) || portal.getOwner().equals(player.getName()))	// Is not hidden, player can view hidden, or player created
             	) {
-                destinations.add(dest);
+                destinations.add(portal.getName());
             }
         }
     }
@@ -422,6 +424,7 @@ public class Portal {
         } else {
             if (isFixed()) {
                 id.setText(++done, "To: " + destination);
+                id.setText(++done, " (" + network + ") ");
             } else {
                 manipGrace(true, true);
                 int index = destinations.indexOf(destination);
@@ -535,7 +538,7 @@ public class Portal {
     public void unregister() {
     	Stargate.log.info("[Stargate] Unregistering gate " + getName());
     	close(true);
-        lookupNames.remove(getName().toLowerCase());
+        lookupNamesNet.get(getNetwork().toLowerCase()).remove(getName().toLowerCase());
 
         for (Blox block : getFrame()) {
             lookupBlocks.remove(block);
@@ -550,7 +553,8 @@ public class Portal {
             lookupEntrances.remove(entrance);
         }
 
-        allPortals.remove(getName());
+        allPortals.remove(this);
+        allPortalsNet.get(getNetwork().toLowerCase()).remove(getName().toLowerCase());
 
         if (id.getBlock().getType() == Material.WALL_SIGN) {
             id.setText(0, getName());
@@ -560,8 +564,8 @@ public class Portal {
             id.update();
         }
 
-        for (String originName : allPortals) {
+        for (String originName : allPortalsNet.get(getNetwork().toLowerCase())) {
-            Portal origin = Portal.getByName(originName);
+            Portal origin = Portal.getByName(originName, getNetwork());
             if ((origin != null) && (origin.isAlwaysOn()) && (origin.getDestinationName().equalsIgnoreCase(getName())) && (origin.isVerified())) {
                 origin.close(true);
             }
@@ -583,8 +587,9 @@ public class Portal {
     }
 
     private void register() {
-    	Stargate.log.info("[Stargate] Registering gate " + getName());
+        if (!lookupNamesNet.containsKey(getNetwork().toLowerCase()))
-        lookupNames.put(getName().toLowerCase(), this);
+        	lookupNamesNet.put(getNetwork().toLowerCase(), new HashMap<String, Portal>());
+        lookupNamesNet.get(getNetwork().toLowerCase()).put(getName().toLowerCase(), this);
 
         for (Blox block : getFrame()) {
             lookupBlocks.put(block, this);
@@ -599,7 +604,11 @@ public class Portal {
             lookupEntrances.put(entrance, this);
         }
 
-        allPortals.add(getName());
+        allPortals.add(this);
+        // Check if this network exists
+        if (!allPortalsNet.containsKey(getNetwork().toLowerCase()))
+        	allPortalsNet.put(getNetwork().toLowerCase(), new ArrayList<String>());
+        allPortalsNet.get(getNetwork().toLowerCase()).add(getName().toLowerCase());
     }
 
     @Override
@@ -619,20 +628,18 @@ public class Portal {
         String options = filterName(id.getText(3));
         boolean hidden = (options.indexOf('h') != -1 || options.indexOf('H') != -1);
         boolean alwaysOn = (options.indexOf('a') != -1 || options.indexOf('A') != -1);
+        boolean priv = (options.indexOf('p') != -1 || options.indexOf('P') != -1);
         
         // Can not create a non-fixed always-on gate.
         if (alwaysOn && destName.length() == 0) {
         	alwaysOn = false;
         }
 
-        if ((name.length() < 1) || (name.length() > 11) || (getByName(name) != null)) {
-            return null;
-        }
         if ((network.length() < 1) || (network.length() > 11)) {
             network = Stargate.getDefaultNetwork();
         }
-        if (destName.length() > 0) {
+        if ((name.length() < 1) || (name.length() > 11) || (getByName(name, network) != null)) {
-            network = "";
+            return null;
         }
 
         int modX = 0;
@@ -695,18 +702,18 @@ public class Portal {
         	button = topleft.modRelative(buttonVector.getRight(), buttonVector.getDepth(), buttonVector.getDistance() + 1, modX, 1, modZ);
         button.setType(BUTTON);
         }
-        portal = new Portal(topleft, modX, modZ, rotX, id, button, destName, name, true, network, gate, player.getName(), hidden, alwaysOn);
+        portal = new Portal(topleft, modX, modZ, rotX, id, button, destName, name, true, network, gate, player.getName(), hidden, alwaysOn, priv);
 
         // Open always on gate
         if (portal.isAlwaysOn()) {
-        	Portal dest = Portal.getByName(destName);
+        	Portal dest = Portal.getByName(destName, portal.getNetwork());
         	if (dest != null)
         		portal.open(true);
         }
         
         // Open any always on gate pointing at this gate
-        for (String originName : allPortals) {
+        for (String originName : allPortalsNet.get(portal.getNetwork().toLowerCase())) {
-        	Portal origin = Portal.getByName(originName);
+        	Portal origin = Portal.getByName(originName, portal.getNetwork());
         	if (origin != null && origin.isAlwaysOn() && origin.getDestinationName().equalsIgnoreCase(portal.getName()) && origin.isVerified()) 
         		origin.open(true);
         }
@@ -716,8 +723,10 @@ public class Portal {
         return portal;
     }
 
-    public static Portal getByName(String name) {
+    public static Portal getByName(String name, String network) {
-        return lookupNames.get(name.toLowerCase());
+        if (!lookupNamesNet.containsKey(network.toLowerCase())) return null;
+        return lookupNamesNet.get(network.toLowerCase()).get(name.toLowerCase());
+        
     }
 
     public static Portal getByEntrance(Location location) {
@@ -738,8 +747,7 @@ public class Portal {
         try {
             BufferedWriter bw = new BufferedWriter(new FileWriter(loc, false));
 
-            for (String name : allPortals) {
+            for (Portal portal : allPortals) {
-                Portal portal = Portal.getByName(name);
                 StringBuilder builder = new StringBuilder();
                 Blox sign = new Blox(portal.id.getBlock());
                 Blox button = portal.button;
@@ -769,6 +777,8 @@ public class Portal {
                 builder.append(portal.isHidden());
                 builder.append(':');
                 builder.append(portal.isAlwaysOn());
+                builder.append(':');
+                builder.append(portal.isPrivate());
 
                 bw.append(builder.toString());
                 bw.newLine();
@@ -784,9 +794,10 @@ public class Portal {
         String location = Stargate.getSaveLocation();
 
         lookupBlocks.clear();
-        lookupNames.clear();
+        lookupNamesNet.clear();
         lookupEntrances.clear();
         allPortals.clear();
+        allPortalsNet.clear();
 
         if (new File(location).exists()) {
         	int l = 0;
@@ -816,17 +827,15 @@ public class Portal {
                     Blox topLeft = new Blox(world, split[6]);
                     Gate gate = (split[7].contains(";")) ? Gate.getGateByName("nethergate.gate") : Gate.getGateByName(split[7]);
 
-                    String fixed = (split.length > 8) ? split[8] : "";
+                    String dest = (split.length > 8) ? split[8] : "";
                     String network = (split.length > 9) ? split[9] : Stargate.getDefaultNetwork();
+                    if (network.isEmpty()) network = Stargate.getDefaultNetwork();
                     String owner = (split.length > 10) ? split[10] : "";
                     boolean hidden = (split.length > 11) ? split[11].equalsIgnoreCase("true") : false;
                     boolean alwaysOn = (split.length > 12) ? split[12].equalsIgnoreCase("true") : false;
+                    boolean priv = (split.length > 13) ? split[13].equalsIgnoreCase("true") : false;
 
-                    if (fixed.length() > 0) {
+                    Portal portal = new Portal(topLeft, modX, modZ, rotX, sign, button, dest, name, false, network, gate, owner, hidden, alwaysOn, priv);
-                        network = "";
-                    }
-
-                    Portal portal = new Portal(topLeft, modX, modZ, rotX, sign, button, fixed, name, false, network, gate, owner, hidden, alwaysOn);
                     portal.close(true);
                     // Verify portal integrity/register portal
                     if (!portal.isVerified() || !portal.checkIntegrity()) {
@@ -842,8 +851,7 @@ public class Portal {
                 
                 // Open any always-on gates. Do this here as it should be more efficient than in the loop.
                 int OpenCount = 0;
-                for (String srcName : allPortals) {
+                for (Portal portal : allPortals) {
-                	Portal portal = Portal.getByName(srcName);
                 	if (portal == null) continue;
                 	if (!portal.isAlwaysOn()) continue;
 					if (!portal.wasVerified()) continue;
@@ -864,8 +872,7 @@ public class Portal {
     
     public static void closeAllGates() {
     	Stargate.log.info("Closing all stargates.");
-    	for (String name : allPortals) {
+    	for (Portal p : allPortals) {
-    		Portal p = Portal.getByName(name);
     		p.close(true);
     	}
     }
@@ -873,14 +880,4 @@ public class Portal {
     public static String filterName(String input) {
         return input.replaceAll("[\\|:#]", "").trim();
     }
-
-    public static Portal getNextOpen() {
-        for (String name : allPortals) {
-            Portal tmp = getByName(name);
-            if (tmp.pastGrace()) {
-                return tmp;
-            }
-        }
-        return null;
-    }
 }

src/net/TheDgtl/Stargate/Stargate.java

@@ -35,7 +35,6 @@ import org.bukkit.util.config.Configuration;
 
 // Permissions
 import com.nijikokun.bukkit.Permissions.Permissions;
-import com.nijiko.permissions.PermissionHandler;
 
 /**
  * Stargate.java - Plug-in for hey0's minecraft mod.
@@ -44,7 +43,7 @@ import com.nijiko.permissions.PermissionHandler;
  */
 public class Stargate extends JavaPlugin {
 	// Permissions
-	public static PermissionHandler Permissions = null;
+	public static Permissions Permissions = null;
 	
     private final bListener blockListener = new bListener();
     private final pListener playerListener = new pListener();
@@ -82,9 +81,19 @@ public class Stargate extends JavaPlugin {
 
     public void onEnable() {
         PluginDescriptionFile pdfFile = this.getDescription();
+        pm = getServer().getPluginManager();
+        
+        /* Lamesauce, they broke this. No way to check build number anymore as far as I know D:
+        String cbVerStr = CraftServer.class.getPackage().getImplementationVersion();
+        int cbVersion = Integer.parseInt(cbVerStr.substring(cbVerStr.length() - 3));
+        if (cbVersion < 319) {
+        	log.info("[" + pdfFile.getName() + " v." + pdfFile.getVersion() + "] CraftBukkit build " + cbVersion + " too old to run Stargate.");
+        	pm.disablePlugin(this);
+        	return;
+        }*/
+        
         log.info(pdfFile.getName() + " v." + pdfFile.getVersion() + " is enabled.");
         
-    	pm = getServer().getPluginManager();
     	config = this.getConfiguration();
 		
     	pm.registerEvent(Event.Type.BLOCK_FLOW, blockListener, Priority.Normal, this);
@@ -139,12 +148,22 @@ public class Stargate extends JavaPlugin {
 	}
 	
 	private void migrate() {
-    	// Migrate old stargates if applicable.
+		// Only migrate if new file doesn't exist.
-        File oldFile = new File("stargates/locations.dat");
+		File newFile = new File(portalFile);
-        if (oldFile.exists()) {
+		if (!newFile.exists()) {
-        	Stargate.log.info("[Stargate] Migrated existing locations.dat");
+			// Migrate REALLY old stargates if applicable
-            oldFile.renameTo(new File(portalFile));
+			File olderFile = new File("stargates.txt");
-        }
+			if (olderFile.exists()) {
+				Stargate.log.info("[Stargate] Migrated old stargates.txt");
+				olderFile.renameTo(newFile);
+			}
+	    	// Migrate old stargates if applicable.
+	        File oldFile = new File("stargates/locations.dat");
+	        if (oldFile.exists()) {
+	        	Stargate.log.info("[Stargate] Migrated existing locations.dat");
+	            oldFile.renameTo(newFile);
+	        }
+		}
         
         // Migrate old gates if applicaple
         File oldDir = new File("stargates");
@@ -170,16 +189,20 @@ public class Stargate extends JavaPlugin {
         Portal destination = gate.getDestination();
 
         if (!gate.isOpen()) {
-        	if ((!gate.isFixed()) && (gate.getActivePlayer() != player)) {
+        	if ((!gate.isFixed()) && gate.isActive() &&  (gate.getActivePlayer() != player)) {
         		gate.deactivate();
                 if (!denyMsg.isEmpty()) {
                     player.sendMessage(ChatColor.RED + denyMsg);
                 }
+        	} else if (gate.isPrivate() && !gate.getOwner().equals(player.getName()) && !hasPerm(player, "stargate.private", player.isOp())) {
+            	if (!denyMsg.isEmpty()) {
+            		player.sendMessage(ChatColor.RED + denyMsg);
+            	}
         	} else if ((destination == null) || (destination == gate)) {
                 if (!invMsg.isEmpty()) {
                     player.sendMessage(ChatColor.RED + invMsg);
                 }
-            } else if ((destination.isOpen()) && (!destination.isFixed())) {
+            } else if ((destination.isOpen()) && (!destination.isAlwaysOn())) {
                 if (!blockMsg.isEmpty()) {
                     player.sendMessage(ChatColor.RED + blockMsg);
                 }
@@ -195,7 +218,7 @@ public class Stargate extends JavaPlugin {
     	Plugin perm = pm.getPlugin("Permissions");
 
 	    if(perm != null) {
-	    	Stargate.Permissions = ((Permissions)perm).getHandler();
+	    	Stargate.Permissions = ((Permissions)perm);
 	    } else {
 	    	log.info("[" + this.getDescription().getName() + "] Permission system not enabled.");
 	    }
@@ -307,6 +330,10 @@ public class Stargate extends JavaPlugin {
 	                    if ((!portal.isOpen()) && (!portal.isFixed())) {
 	                        portal.cycleDestination(player);
 	                    }
+                	} else {
+                		if (!denyMsg.isEmpty()) {
+                			player.sendMessage(denyMsg);
+                		}
                 	}
                 }
             }
@@ -375,9 +402,10 @@ public class Stargate extends JavaPlugin {
         }
     }
     
-    public static Boolean hasPerm(Player player, String perm, Boolean def) {
+    @SuppressWarnings("static-access")
+	public static Boolean hasPerm(Player player, String perm, Boolean def) {
     	if (Permissions != null)
-    		return Permissions.has(player, perm);
+    		return Permissions.Security.permission(player, perm);
     	return def;
     }
     

src/plugin.yml

@@ -1,6 +1,6 @@
 name: Stargate
 main: net.TheDgtl.Stargate.Stargate
-version: 0.11
+version: 0.12
 description: Stargate mod for Bukkit
 author: Drakia
 website: http://www.thedgtl.net