remove the iml file
remove the iml file
add base gitignore
convert to maven project with craftbukkit + bukkit dependencies
change package name to com.gmail.theawesomegem.uhc2
Cleanup SpawnPoint:
public members => private
added setters and getters
variable should start with a lowercase char
Cleanup Primary:
variable game was never set so onDisable would probably throw an NPE
variables should start with a lowercase char
no need to log onEnable and onDisable, Bukkit already does that
Cleanup PlayerData:
public members => private
added setter and getter
variables should start with a lowercase char
Cleanup PlayerData:
public members => private
added setter and getter
variables should start with a lowercase char
don't store a public singleton instance of the game in the Game class, pass the instance of the game to the commandhandler on construction
rename variables to lowercase
no need to initalize values in the constructor
Cleanup Game:
name all method to lowercase starter char
removed the disable seeing as it was going to do nothing anyway
no need to make an array for vararg method
no need for 2 if statements
no need to check instanceof again
adding to the location does nothing, you need to teleport them to the new location
empty else statement
add the workaround for the bug BUKKIT-5216 - PR - https://github.com/Bukkit/CraftBukkit/pull/1374
in the 'freeze' type behaviour don't lock the direction they are facing, just their x and z coordinates
redundant else statements
why return true on true and false on false?