Spigot Permissions

WlanGehtNicht · 31 August 2018

Hey,
ich schreibe derzeit ein Job Plugin. Wenn man sich einen Jobs aussucht bekommt soll man automatisch eine Permissions bekommen. (hat auch geklappt, bis ich die Permissions Abgfrage rein gemacht habe). Jetzt kommt, auch wenn ich keine Permissions habe:

Java:

String jobs = Daten.prefix + "Du hast bereits ein Job, du kannst nicht mehr als 2 Jobs annehmen";

Hier der Code:

Java:

package de.jobs.items;

import org.bukkit.Bukkit;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.inventory.InventoryClickEvent;

import de.jobs.main.Daten;

public class ItemsInventarJobsKlick implements Listener {
   
    String jobs = Daten.prefix + "Du hast bereits ein Job, du kannst nicht mehr als 2 Jobs annehmen";
   
   
   
    @EventHandler
    public void onClick(InventoryClickEvent e) {
        Player p = (Player)e.getWhoClicked();
        if (e.getInventory().getName().equalsIgnoreCase("§9§lJobs")){
            e.setCancelled(true);
             if (e.getCurrentItem().getItemMeta().getDisplayName().equalsIgnoreCase("§6§lMuellmann")){
                 e.setCancelled(true);
                 if(!p.hasPermission("jobs.muell")) {
                     if(!p.hasPermission("jobs.pizzabote")) {
                         if(!p.hasPermission("jobs.taxifahrer")) {
                             if(!p.hasPermission("jobs.fischer")) {
                                 Bukkit.dispatchCommand(p, "pex user " + p.getName() + " add jobs.muell");
                                 p.sendMessage(Daten.prefix + "Du bist nun Müllmann");
                             } else {
                                 p.sendMessage(jobs);
                             }
                         } else {
                             p.sendMessage(jobs);
                         }
                     } else {
                         p.sendMessage(jobs);
                     }
                 } else {
                     p.sendMessage(jobs);
                 }
             } else if(e.getCurrentItem().getItemMeta().getDisplayName().equalsIgnoreCase("§6§lPizzabote")){
                 e.setCancelled(true);
                 if(!p.hasPermission("jobs.muell")) {
                     if(!p.hasPermission("jobs.pizzabote")) {
                         if(!p.hasPermission("jobs.taxifahrer")) {
                             if(!p.hasPermission("jobs.fischer")) {
                                 Bukkit.dispatchCommand(p, "pex user " + p.getName() + " add jobs.pizzabote");
                                 p.sendMessage(Daten.prefix + "Du bist nun Pizzabote");
                             } else {
                                 p.sendMessage(jobs);
                             }
                         } else {
                             p.sendMessage(jobs);
                         }
                     } else {
                         p.sendMessage(jobs);
                     }
                 } else {
                     p.sendMessage(jobs);
                 }
             } else if(e.getCurrentItem().getItemMeta().getDisplayName().equalsIgnoreCase("§6§lTaxifahrer")) {
                 e.setCancelled(true);
                 if(!p.hasPermission("jobs.muell")) {
                     if(!p.hasPermission("jobs.pizzabote")) {
                         if(!p.hasPermission("jobs.taxifahrer")) {
                             if(!p.hasPermission("jobs.fischer")) {
                                 Bukkit.dispatchCommand(p, "pex user " + p.getName() + " add jobs.taxifahrer");
                                 p.sendMessage(Daten.prefix + "Du bist nun Taxifahrer");
                             } else {
                                 p.sendMessage(jobs);
                             }
                         } else {
                             p.sendMessage(jobs);
                         }
                     } else {
                         p.sendMessage(jobs);
                     }
                 } else {
                     p.sendMessage(jobs);
                 }
             } else if(e.getCurrentItem().getItemMeta().getDisplayName().equalsIgnoreCase("§6§lFischer")) {
                 e.setCancelled(true);
                 if(!p.hasPermission("jobs.muell")) {
                     if(!(p.hasPermission("jobs.pizzabote"))) {
                         if(!p.hasPermission("jobs.taxifahrer")) {
                             if(!p.hasPermission("jobs.fischer")) {
                                 Bukkit.dispatchCommand(p, "pex user " + p.getName() + " add jobs.fischer");
                                 p.sendMessage(Daten.prefix + "Du bist nun Fischer");
                             } else {
                                 p.sendMessage(jobs);
                             }
                         } else {
                             p.sendMessage(jobs);
                         }
                     } else {
                         p.sendMessage(jobs);
                     }
                 } else {
                     p.sendMessage(jobs);
                 }
             }
        }
    }

}

Ich habe auch noch ein anderes Problem. Wenn ich das Inventar von der Job aussuche aufmache, dann kommen alle Jobs, außer der Job: Müllmann.

Code:

Java:

package de.jobs.items;

import java.util.ArrayList;

import org.bukkit.Bukkit;
import org.bukkit.Material;
import org.bukkit.entity.Player;
import org.bukkit.inventory.Inventory;
import org.bukkit.inventory.ItemStack;
import org.bukkit.inventory.meta.ItemMeta;

public class ItemsInventarJobs {
   
    public static void run(Player p) {
        Inventory jobs = Bukkit.createInventory(null, 9, "§9§lJobs");
       
        ItemStack muell = new ItemStack(Material.CAULDRON);
        ItemMeta muellmeta = muell.getItemMeta();
        ArrayList<String> lore1 = new ArrayList<>();
        lore1.add("§6Muellmann§8:");
        lore1.add("§7Leere Muelleimer und bringe");
        lore1.add("§7den Muell auf den Müllplatz");
        muellmeta.setLore(lore1);
        muellmeta.setDisplayName("§6§lMuellmann");
        muell.setItemMeta(muellmeta);
       
       
        ItemStack pizza = new ItemStack(Material.BREAD);
        ItemMeta pizzameta = pizza.getItemMeta();
        ArrayList<String> lore2 = new ArrayList<>();
        lore2.add("§6Pizzabote§8:");
        lore2.add("§7Liefere Pizzen an Kunden aus");
        pizzameta.setLore(lore2);
        pizzameta.setDisplayName("§6§lPizzabote");
        pizza.setItemMeta(pizzameta);
       
       
        ItemStack taxi = new ItemStack(Material.MINECART);
        ItemMeta taximeta = taxi.getItemMeta();
        ArrayList<String> lore3 = new ArrayList<>();
        lore3.add("§6Taxifahrer§8:");
        lore3.add("§7Fahre Kunden zu ihrem Wunschort");
        taximeta.setLore(lore3);
        taximeta.setDisplayName("§6§lTaxifahrer");
        taxi.setItemMeta(taximeta);
       
       
        ItemStack fischer = new ItemStack(Material.FISHING_ROD);
        ItemMeta fischermeta = fischer.getItemMeta();
        ArrayList<String> lore4 = new ArrayList<>();
        lore4.add("§6Fischer§8:");
        lore4.add("§7Angel Fische und verkaufe sie");
        fischermeta.setLore(lore4);
        fischermeta.setDisplayName("§6§lFischer");
        fischer.setItemMeta(fischermeta);
       
        jobs.setItem(2, muell);
        jobs.setItem(3, pizza);
        jobs.setItem(5, taxi);
        jobs.setItem(6, fischer);
       
        p.openInventory(jobs);
    }

}

Ich hoffe, dass mir da jemand helfen kann.

DerFrZocker · 31 August 2018

Wenn du OP bist hast du automatisch alle permissions,

WlanGehtNicht schrieb:
Bukkit.dispatchCommand(p, "pex user " + p.getName() + " add jobs.fischer")

Das funktuniert nur, wenn der Spieler auch die permission, hat um permissions zu geben was ein normaler Spieler in der Regel nicht hat.

Und versuche keine solange if else Ketten zu benutzen, die sind sehr fehler anfällig und unübersichtlich.
Es gibt nicht umsonst die 'return' funktion

deleted202832 · 31 August 2018

Dafür benutzt man auch lieber die Pex API

Und rate mal wieso

Code:

CAULDRON

nicht geht wenn es noch

Code:

CAULDRON_ITEM

gibt

WlanGehtNicht · 31 August 2018

@DerFrZocker
Wie lasse ich dann die Console den Befehl ausführen?

DerFrZocker · 31 August 2018

@WlanGehtNicht Ich würde eher wie @RE_Chronex geschrieben hat direkt mit der Pex api arbeiten

WlanGehtNicht · 31 August 2018

ok

Danke

deleted202832 · 31 August 2018

Da ich mir sicher bin das du mit der API nicht arbeiten kannst:

Code:

Bukkit.dispatchCommand(Bukkit.getConsoleSender(), "pex user " + p.getName() + " add jobs.fischer");

WlanGehtNicht · 31 August 2018

ahh ok Danke.
Aber ich hätte mit der API arbeiten können

WlanGehtNicht · 31 August 2018

@RE_Chronex
Aber diese Version geht immer noch nicht

DerFrZocker · 31 August 2018

Hast du dich beim testen auch deop?

WlanGehtNicht · 31 August 2018

oh, das habe ich vergessen.
Jetzt geht es

DerFrZocker · 31 August 2018

Ich würde dir aber noch empfehlen OOP anzuschauen. Gerade bei einem Job Plugin macht es sinn, sich ein Job interface zu machen. Jeder Job ist dan eine eigenes Classe / ein eigenes Object wo das Job interface implementiert. Wenn du dir dann noch eine gescheite Service Classe erstellst, kannst du so ein Plugin mit viel weniger aufwand programmieren und erweitern.

SirYwell · 31 August 2018

Vielleicht solltest du dir mal die Vault API anschauen, bevor du du dich auf PEX beschränkst - das Projekt wird nämlich nicht mehr offiziell weiterentwickelt, und auf Basis von Vault kann es dir egal sein, welches Permission-System letzten Endes auf dem Server läuft.

WlanGehtNicht · 31 August 2018

minecraft-server.eu Europas Größte Serverliste

Spigot Permissions

WlanGehtNicht

Schafhirte

DerFrZocker

Miner

deleted202832

Guest

WlanGehtNicht

Schafhirte

DerFrZocker

Miner

WlanGehtNicht

Schafhirte

deleted202832

Guest

WlanGehtNicht

Schafhirte

WlanGehtNicht

Schafhirte

DerFrZocker

Miner

WlanGehtNicht

Schafhirte

DerFrZocker

Miner

SirYwell

PlotSquared Entwickler

WlanGehtNicht

Schafhirte