Javadoc wants <p>, not <p/>

src/main/java/com/onarandombox/MultiverseCore/api/Core.java

@@ -16,7 +16,7 @@ import org.bukkit.entity.Player;
 
 /**
  * Multiverse 2 Core API
- * <p/>
+ * <p>
  * This API contains a bunch of useful things you can get out of Multiverse in general!
  */
 public interface Core {

src/main/java/com/onarandombox/MultiverseCore/api/MVDestination.java

@@ -21,7 +21,7 @@ import org.bukkit.util.Vector;
 public interface MVDestination {
     /**
      * Returns the identifier or prefix that is required for this destination.
-     * <p/>
+     * <p>
      * Portals have a prefix of "p" for example and OpenWarp (third party plugin) uses "ow". This is derived from a
      * hash and cannot have duplicate values. Read that as your plugin cannot use 'p' because it's already used.
      * Please check the wiki when adding a custom destination!
@@ -32,7 +32,7 @@ public interface MVDestination {
 
     /**
      * Allows you to determine if a Destination is valid for the type it thinks it is.
-     * <p/>
+     * <p>
      * An example of this would be the exact destination. A valid string would be: e:0,0,0 where an invalid one would
      * be e:1:2:3. The first string would return true the second would return false. This is simply a convenience
      * method
@@ -48,11 +48,11 @@ public interface MVDestination {
 
     /**
      * Returns the location a specific entity will spawn at.
-     * <p/>
+     * <p>
      * To just retrieve the location as it is stored you can just pass null, but be warned some destinations may return
      * null back to you if you do this. It is always safer to pass an actual entity. This is used so things like
      * minecarts can be teleported.
-     * <p/>
+     * <p>
      * Do not forget to use {@link #getVelocity()} as destinations can use this too!
      *
      * @param entity The entity to be teleported.
@@ -63,7 +63,7 @@ public interface MVDestination {
 
     /**
      * Returns the velocity vector for this destination.
-     * <p/>
+     * <p>
      * Plugins wishing to fully support MVDestinations MUST implement this.
      *
      * @return A vector representing the speed/direction the player should travel when arriving
@@ -72,7 +72,7 @@ public interface MVDestination {
 
     /**
      * Sets the destination string.
-     * <p/>
+     * <p>
      * This should be used when you want to tell this destination object about a change in where it should take people.
      * The destination param should be match the result from {@link #getIdentifier()}. A valid example would be that if
      * {@link #getIdentifier()} returned "ow" our destination string could be "ow:TownCenter" but could not be
@@ -85,7 +85,7 @@ public interface MVDestination {
 
     /**
      * Returns true if the destination is valid and players will be taken to it.
-     * <p/>
+     * <p>
      * Even if destinations are in the correct format (p:MyPortal) MyPortal may not exist, and therefore this would
      * return false.
      *
@@ -95,7 +95,7 @@ public interface MVDestination {
 
     /**
      * Gives you a general friendly description of the type of destination.
-     * <p/>
+     * <p>
      * For example, the PlayerDestination sets this to "Player". You can use this to show where a player will be taken.
      *
      * @return A friendly string description of the type of destination.
@@ -104,7 +104,7 @@ public interface MVDestination {
 
     /**
      * Gives you a specific name of the destination.
-     * <p/>
+     * <p>
      * For example, the PlayerDestination sets this to The Player's Name.
      *
      * @return A friendly string stating the name of the destination.
@@ -114,7 +114,7 @@ public interface MVDestination {
     /**
      * Returns a string that can easily be saved in the config that contains all the details needed to rebuild this
      * destination.
-     * <p/>
+     * <p>
      * ex: e:0,0,0:50:50
      *
      * @return The savable config string.
@@ -123,9 +123,9 @@ public interface MVDestination {
 
     /**
      * Returns the permissions string required to go here.
-     * <p/>
+     * <p>
      * ex: multiverse.access.world
-     * <p/>
+     * <p>
      * NOTE: This is NOT the permission to use the teleport command.
      *
      * @return the permissions string required to go here.
@@ -134,7 +134,7 @@ public interface MVDestination {
 
     /**
      * Should the Multiverse SafeTeleporter be used?
-     * <p/>
+     * <p>
      * If not, MV will blindly take people to the location specified.
      *
      * @return True if the SafeTeleporter will be used, false if not.

src/main/java/com/onarandombox/MultiverseCore/api/MVWorldManager.java

@@ -19,7 +19,7 @@ import java.util.List;
 
 /**
  * Multiverse 2 World Manager API
- * <p/>
+ * <p>
  * This API contains all of the world managing
  * functions that your heart desires!
  */
@@ -142,7 +142,7 @@ public interface MVWorldManager {
 
     /**
      * Loads the Worlds & Settings for any worlds that bukkit loaded before us.
-     * <p/>
+     * <p>
      * This way people will _always_ have some worlds in the list.
      */
     public void loadDefaultWorlds();

src/main/java/com/onarandombox/MultiverseCore/api/MultiverseWorld.java

@@ -17,7 +17,7 @@ import java.util.List;
 
 /**
  * The API for a Multiverse Handled World.
- * <p/>
+ * <p>
  * Currently INCOMPLETE
  */
 public interface MultiverseWorld {
@@ -97,7 +97,7 @@ public interface MultiverseWorld {
 
     /**
      * Sets the environment of a world.
-     * <p/>
+     * <p>
      * Note: This will ONLY take effect once the world is unloaded/reloaded.
      *
      * @param environment A {@link org.bukkit.World.Environment}.
@@ -127,7 +127,7 @@ public interface MultiverseWorld {
 
     /**
      * Gets the alias of this world.
-     * <p/>
+     * <p>
      * This alias allows users to have a world named "world" but show up in the list as "FernIsland"
      *
      * @return The alias of the world as a String.
@@ -472,7 +472,7 @@ public interface MultiverseWorld {
 
     /**
      * Sets whether or not Multiverse should auto-load this world.
-     * <p/>
+     * <p>
      * True is default.
      *
      * @param autoLoad True if multiverse should autoload this world the spawn, false if not.
@@ -489,7 +489,7 @@ public interface MultiverseWorld {
     /**
      * Sets whether or not a player who dies in this world will respawn in their
      * bed or follow the normal respawn pattern.
-     * <p/>
+     * <p>
      * True is default.
      *
      * @param autoLoad True if players dying in this world respawn at their bed.

src/main/java/com/onarandombox/MultiverseCore/event/MVWorldPropertyChangeEvent.java

@@ -14,9 +14,9 @@ import org.bukkit.event.Event;
 
 /**
  * This event is fired *before* the property is actually changed.
- * <p/>
+ * <p>
  * If it is cancled, no change will happen.
- * <p/>
+ * <p>
  * If you want to get the values of the world before the change, query the world.
  * If you want to get the value being changed, use getProperty()
  */

src/main/java/com/onarandombox/MultiverseCore/utils/PermissionTools.java

@@ -106,7 +106,7 @@ public class PermissionTools {
 
     /**
      * Checks to see if player can go to a world given their current status.
-     * <p/>
+     * <p>
      * The return is a little backwards, and will return a value safe for event.setCancelled.
      *
      * @param fromWorld  The MultiverseWorld they are in.

src/test/java/com/onarandombox/MultiverseCore/test/utils/MockBlock.java

@@ -263,7 +263,7 @@ public class MockBlock implements Block{
     /**
      * Captures the current state of this block. You may then cast that state
      * into any accepted type, such as Furnace or Sign.
-     * <p/>
+     * <p>
      * The returned object will never be updated, and you are not guaranteed that
      * (for example) a sign is still a sign after you capture its state.
      *
@@ -352,7 +352,7 @@ public class MockBlock implements Block{
 
     /**
      * Checks if this block is empty.
-     * <p/>
+     * <p>
      * A block is considered empty when {@link #getType()} returns {@link org.bukkit.Material#AIR}.
      *
      * @return true if this block is empty
@@ -364,7 +364,7 @@ public class MockBlock implements Block{
 
     /**
      * Checks if this block is liquid.
-     * <p/>
+     * <p>
      * A block is considered liquid when {@link #getType()} returns {@link org.bukkit.Material#WATER}, {@link
      * org.bukkit.Material#STATIONARY_WATER}, {@link org.bukkit.Material#LAVA} or {@link
      * org.bukkit.Material#STATIONARY_LAVA}.

src/test/java/com/onarandombox/MultiverseCore/test/utils/TestCommandSender.java

@@ -87,7 +87,7 @@ public class TestCommandSender implements CommandSender {
 
     /**
      * Gets the value of the specified permission, if set.
-     * <p/>
+     * <p>
      * If a permission override is not set on this object, the default value of the permission will be returned.
      *
      * @param name Name of the permission
@@ -100,7 +100,7 @@ public class TestCommandSender implements CommandSender {
 
     /**
      * Gets the value of the specified permission, if set.
-     * <p/>
+     * <p>
      * If a permission override is not set on this object, the default value of the permission will be returned
      *
      * @param perm Permission to get
@@ -174,7 +174,7 @@ public class TestCommandSender implements CommandSender {
 
     /**
      * Recalculates the permissions for this object, if the attachments have changed values.
-     * <p/>
+     * <p>
      * This should very rarely need to be called from a plugin.
      */
     @Override