Remove obsolete TODOs.
Some of them have been mangled somehow, and others just straight up aren't relevant anymore. Tidy up!
This commit is contained in:
@@ -514,7 +514,6 @@ public class ArenaImpl implements Arena
|
|||||||
|
|
||||||
// Teleport players, give full health, initialize map
|
// Teleport players, give full health, initialize map
|
||||||
for (Player p : arenaPlayers) {
|
for (Player p : arenaPlayers) {
|
||||||
// TODO figure out how people die in lobby and get sent to spectator area early
|
|
||||||
// Remove player from spec list to avoid invincibility issues
|
// Remove player from spec list to avoid invincibility issues
|
||||||
if (inSpec(p)) {
|
if (inSpec(p)) {
|
||||||
specPlayers.remove(p);
|
specPlayers.remove(p);
|
||||||
|
|||||||
@@ -134,12 +134,6 @@ public class ArenaListener
|
|||||||
this.region = arena.getRegion();
|
this.region = arena.getRegion();
|
||||||
this.monsters = arena.getMonsterManager();
|
this.monsters = arena.getMonsterManager();
|
||||||
|
|
||||||
/*
|
|
||||||
* TODO: Figure out if this is really a good idea + It saves needing all
|
|
||||||
* those methods in Arena.java + It is relatively simple + It would be
|
|
||||||
* fairly easy to implement an observer pattern - More private fields -
|
|
||||||
* Uglier code
|
|
||||||
*/
|
|
||||||
ConfigurationSection s = arena.getSettings();
|
ConfigurationSection s = arena.getSettings();
|
||||||
this.softRestore = s.getBoolean("soft-restore", false);
|
this.softRestore = s.getBoolean("soft-restore", false);
|
||||||
this.softRestoreDrops = s.getBoolean("soft-restore-drops", false);
|
this.softRestoreDrops = s.getBoolean("soft-restore-drops", false);
|
||||||
|
|||||||
@@ -77,7 +77,6 @@ public class MAGlobalListener implements Listener
|
|||||||
// //
|
// //
|
||||||
///////////////////////////////////////////////////////////////////////////
|
///////////////////////////////////////////////////////////////////////////
|
||||||
|
|
||||||
//TODO watch block physics, piston extend, and piston retract events
|
|
||||||
@EventHandler(priority = EventPriority.HIGHEST)
|
@EventHandler(priority = EventPriority.HIGHEST)
|
||||||
public void blockBreak(BlockBreakEvent event) {
|
public void blockBreak(BlockBreakEvent event) {
|
||||||
for (Arena arena : am.getArenas())
|
for (Arena arena : am.getArenas())
|
||||||
|
|||||||
@@ -68,7 +68,6 @@ public class ClassChests {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
// TODO: Make number of searches configurable
|
|
||||||
// First check the pillar below the sign
|
// First check the pillar below the sign
|
||||||
Block block = findChestBelow(blockBelow, 6);
|
Block block = findChestBelow(blockBelow, 6);
|
||||||
|
|
||||||
|
|||||||
@@ -423,10 +423,6 @@ public class WaveParser
|
|||||||
}
|
}
|
||||||
|
|
||||||
private static List<PotionEffect> getPotionEffects(Arena arena, String name, ConfigurationSection config) {
|
private static List<PotionEffect> getPotionEffects(Arena arena, String name, ConfigurationSection config) {
|
||||||
/*
|
|
||||||
* TODO: Make this consistent with the rest somehow
|
|
||||||
* - Things only work for Players (currently)
|
|
||||||
*/
|
|
||||||
List<String> effects = config.getStringList("effects");
|
List<String> effects = config.getStringList("effects");
|
||||||
if (effects == null || effects.isEmpty()) {
|
if (effects == null || effects.isEmpty()) {
|
||||||
String value = config.getString("effects", null);
|
String value = config.getString("effects", null);
|
||||||
|
|||||||
@@ -54,8 +54,6 @@ public class DefaultWave extends AbstractWave
|
|||||||
break;
|
break;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
//TODO: Remember amount-multiplier.
|
|
||||||
// Return the map.
|
|
||||||
return monsters;
|
return monsters;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user