From 78ba57b8415598231d343bb5a4a774a5ac9cb902 Mon Sep 17 00:00:00 2001 From: sekwah41 Date: Tue, 8 Oct 2013 22:46:12 +0100 Subject: [PATCH] Started storing the config values inside variables for easier access, also it stops the config from being loaded every time it needs to check a variable. --- .../src/com/sekwah/advancedportals/Listeners.java | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/Advanced Portals/src/com/sekwah/advancedportals/Listeners.java b/Advanced Portals/src/com/sekwah/advancedportals/Listeners.java index 0357e42..b9039ae 100644 --- a/Advanced Portals/src/com/sekwah/advancedportals/Listeners.java +++ b/Advanced Portals/src/com/sekwah/advancedportals/Listeners.java @@ -16,8 +16,14 @@ public class Listeners implements Listener { private final AdvancedPortalsPlugin plugin; + // The needed config values will be stored so they are easier to access later + // an example is in the interact event in this if statement if((!UseOnlyServerAxe || event.getItem().getItemMeta().getDisplayName().equals("§eP... + private boolean UseOnlyServerAxe = false; + public Listeners(AdvancedPortalsPlugin plugin) { this.plugin = plugin; + ConfigAccessor config = new ConfigAccessor(plugin, "Config.yml"); + this.UseOnlyServerAxe = config.getConfig().getBoolean("UseOnlyServerMadeAxe"); plugin.getServer().getPluginManager().registerEvents(this, plugin); } @@ -36,12 +42,11 @@ public class Listeners implements Listener { Player player = event.getPlayer(); Location blockloc = event.getClickedBlock().getLocation(); if(player.hasPermission("AdvancedPortals.create")){ - ConfigAccessor config = new ConfigAccessor(plugin, "Config.yml"); // UseOnlyServerMadeAxe being set to true makes is so only the axe generated by the server can be used so other iron axes can be used normally, // by default its false but it is a nice feature in case the user wants to use the axe normally too, such as a admin playing survival or it being used // as a weapon. - if((!config.getConfig().getBoolean("UseOnlyServerMadeAxe") || event.getItem().getItemMeta().getDisplayName().equals("§ePortal Region Selector")) && event.getPlayer().getItemInHand().getTypeId() == Material.IRON_AXE.getId()) { + if((!UseOnlyServerAxe || event.getItem().getItemMeta().getDisplayName().equals("§ePortal Region Selector")) && event.getPlayer().getItemInHand().getTypeId() == Material.IRON_AXE.getId()) { // This checks if the action was a left or right click and if it was directly effecting a block. if(event.getAction() == Action.LEFT_CLICK_BLOCK) {