Solved skull getOwner warning

Discussion in 'Spigot Plugin Development' started by SickSkillz, Jun 29, 2016.

Thread Status:
Not open for further replies.
  1. I have the following code to check if a skull has the owner Luck but getOwner() becomes getOwner()

    this is my blockbreak code:
    Code (Text):
    @EventHandler
      public void breakListener(BlockBreakEvent event)
      {
        Player p = event.getPlayer();
        if (event.getBlock().getType() == Material.SKULL)
        {
          Skull luckyblock = (Skull)event.getBlock().getState();
          if (luckyblock.getOwner().equalsIgnoreCase("Luck"))
          {
            event.setCancelled(true);
         
            event.getPlayer().getWorld().getBlockAt(event.getBlock().getLocation()).setType(Material.AIR);
            if (this.config.getBoolean("settings.BroadcastBlockBreak")) {
              Bukkit.broadcastMessage(ChatColor.GOLD + "[SLB] " + event.getPlayer().getDisplayName() + "Has broken a Lucky Block!");
            }
            int howManyPossibleHappenstances = 53;
         
            int randomChoice = 1 + (int)(Math.random() * howManyPossibleHappenstances);
         
            Location l = event.getBlock().getState().getLocation();
            switch (randomChoice)
            {
            case 1:
              if (this.config.getBoolean("surprises.enabled.LuckyTools")) {
                luckyTools(1, l, p);
              }
              break;
            case 2:
              if (this.config.getBoolean("surprises.enabled.LuckyTools")) {
                luckyTools(2, l, p);
              }
              break;
            case 3:
              if (this.config.getBoolean("surprises.enabled.LuckyTools")) {
                luckyTools(3, l, p);
              }
              break;
            case 4:
              if (this.config.getBoolean("surprises.enabled.LuckyTools")) {
                luckyTools(4, l, p);
              }
              break;
            case 5:
              if (this.config.getBoolean("surprises.enabled.LuckyTools")) {
                luckyTools(5, l, p);
              }
              break;
            case 6:
              if (this.config.getBoolean("surprises.enabled.LuckyTools")) {
                luckyTools(6, l, p);
              }
              break;
            case 7:
              if (this.config.getBoolean("surprises.enabled.enderPearls")) {
                enderPearls(l, p);
              }
              break;
            case 8:
              if (this.config.getBoolean("surprises.enabled.explosion")) {
                explosion2(l, p);
              }
              break;
            case 9:
              if (this.config.getBoolean("surprises.enabled.explosion")) {
                explosion3(l, p);
              }
              break;
            case 10:
              if (this.config.getBoolean("surprises.enabled.explosion")) {
                explosion(l, p);
              }
              break;
            case 11:
              if (this.config.getBoolean("surprises.enabled.wither")) {
                wither(l, p);
              }
              break;
            case 12:
              if (this.config.getBoolean("surprises.enabled.woodTools")) {
                woodTools(l, p);
              }
              break;
            case 13:
              if (this.config.getBoolean("surprises.enabled.witch")) {
                witch(l, p);
              }
              break;
            case 14:
              if (this.config.getBoolean("surprises.enabled.stoneTools")) {
                stoneTools(l, p);
              }
              break;
            case 15:
              if (this.config.getBoolean("surprises.enabled.Zombies")) {
                Zombie(l, p);
              }
              break;
            case 16:
              if (this.config.getBoolean("surprises.enabled.goldTools")) {
                goldTools(l, p);
              }
              break;
            case 17:
              if (this.config.getBoolean("surprises.enabled.lightningCreeper")) {
                lightningcreeper(l, p);
              }
              break;
            case 18:
              if (this.config.getBoolean("surprises.enabled.leatherArmor")) {
                leatherArmor(l, p);
              }
              break;
            case 19:
              if (this.config.getBoolean("surprises.enabled.ghast")) {
                ghast(l, p);
              }
              break;
            case 20:
              if (this.config.getBoolean("surprises.enabled.goldArmor")) {
                goldArmor(l, p);
              }
              break;
            case 21:
              if (this.config.getBoolean("surprises.enabled.creepers")) {
                creepers(l, p);
              }
              break;
            case 22:
              if (this.config.getBoolean("surprises.enabled.gold")) {
                gold(l, p);
              }
              break;
            case 23:
              if (this.config.getBoolean("surprises.enabled.ironTools")) {
                ironTools(l, p);
              }
              break;
            case 24:
              if (this.config.getBoolean("surprises.enabled.diamondTools")) {
                diamondTools(l, p);
              }
              break;
            case 25:
              if (this.config.getBoolean("surprises.enabled.beacon")) {
                beacon(l, p);
              }
              break;
            case 26:
              if (this.config.getBoolean("surprises.enabled.ironArmor")) {
                ironArmor(l, p);
              }
              break;
            case 27:
              if (this.config.getBoolean("surprises.enabled.enderChests")) {
                enderChests(l, p);
              }
              break;
            case 28:
              if (this.config.getBoolean("surprises.enabled.diamondArmor")) {
                diamondArmor(l, p);
              }
              break;
            case 29:
              if (this.config.getBoolean("surprises.enabled.dragonEgg")) {
                dragonEgg(l, p);
              }
              break;
            case 30:
              if (this.config.getBoolean("surprises.enabled.enchantmentTable")) {
                enchantmentTable(l, p);
              }
              break;
            case 31:
              if (this.config.getBoolean("surprises.enabled.horseArmor")) {
                horseArmor(l, p);
              }
              break;
            case 32:
              if (this.config.getBoolean("surprises.enabled.obsidian")) {
                obsidian(l, p);
              }
              break;
            case 33:
              if (this.config.getBoolean("surprises.enabled.TNT")) {
                TNT(l, p);
              }
              break;
            case 34:
              if (this.config.getBoolean("surprises.enabled.bowAndArrows")) {
                bowAndArrows(l, p);
              }
              break;
            case 35:
              if (this.config.getBoolean("surprises.enabled.brewingItems")) {
                brewingItems(l, p);
              }
              break;
            case 36:
              if (this.config.getBoolean("surprises.enabled.Redstone")) {
                Redstone(l, p);
              }
              break;
            case 37:
              if (this.config.getBoolean("surprises.enabled.booksAndBookshelves")) {
                booksAndBookshelves(l, p);
              }
              break;
            case 38:
              if (this.config.getBoolean("surprises.enabled.meats")) {
                meats(l, p);
              }
              break;
            case 39:
              if (this.config.getBoolean("surprises.enabled.vegetables")) {
                vegetables(l, p);
              }
              break;
            case 40:
              if (this.config.getBoolean("surprises.enabled.PumpKins")) {
                PumpKins(l, p);
              }
              break;
            case 41:
              if (this.config.getBoolean("surprises.enabled.JackOLanterns")) {
                JackOLanterns(l, p);
              }
              break;
            case 42:
              if (this.config.getBoolean("surprises.enabled.Halloween1")) {
                halloween1(l, p);
              }
              break;
            case 43:
              if (this.config.getBoolean("surprises.enabled.Halloween2")) {
                halloween2(l, p);
              }
              break;
            case 44:
              if (this.config.getBoolean("surprises.enabled.Halloween3")) {
                halloween3(l, p);
              }
              break;
            case 45:
              if (this.config.getBoolean("surprises.enabled.GiantZombie")) {
                GiantZombie(l, p);
              }
              break;
            case 46:
              if (this.config.getBoolean("surprises.enabled.FlyingSkeleton")) {
                FlyingSkeleton(l, p);
              }
              break;
            case 47:
              if (this.config.getBoolean("surprises.enabled.FlyingexpBottles")) {
                FlyingexpBottles(l, p);
              }
              break;
            case 48:
              if (this.config.getBoolean("surprises.enabled.FlyingTnT")) {
                FlyingTnT(l, p);
              }
              break;
            case 49:
              if (this.config.getBoolean("surprises.enabled.RandomMobs")) {
                RandomMobs(l, p);
              }
              break;
            case 50:
              if (this.config.getBoolean("surprises.enabled.RandomAnimals")) {
                RandomAnimals(l, p);
              }
              break;
            case 51:
              if (this.config.getBoolean("surprises.enabled.ObsidianCage")) {
                build1(l, p);
              }
              break;
            case 52:
              if (this.config.getBoolean("surprises.enabled.Minerals")) {
                Minerals(l, p);
              }
              break;
            case 53:
              if (this.config.getBoolean("surprises.enabled.Elytra")) {
                Elytra(l, p);
              }
              break;
            case 54:
              if (this.config.getBoolean("surprises.enabled.Shield")) {
                Shield(l, p);
              }
              break;
            }
          }
        }
      }
     
  2. I wouldn't recommend this unless absolutely needed. The reason it is deprecated is because the method is outdated and there is a new, better method to use.
     
  3. yeah I know there is a method but this method is also efficient and edible also I didn't even suggest it, I just said use it to suppress not suggesting you have to use it.
     
  4. Okay thanks for your help. But I want to make the check work on 1.10 and 1.9. So do I need to use getOwner() and the suppress warning?
     
  5. Well if getOwningPlayer() is not in the 1.9 version of Spigot, then you can use getOwner() for your 1.9 version. Just keep in mind if you try to use your plugin on later Spigot servers,there is a chance your plugin will break if that method is removed.
     
    • Agree Agree x 1
Thread Status:
Not open for further replies.