From efb7b3509aac7ff294d27a3d7a6363d60dcf9ea8 Mon Sep 17 00:00:00 2001 From: zml2008 Date: Mon, 16 Jan 2012 10:18:34 -0800 Subject: [PATCH] Removed support for Permissions2/3 --- CHANGELOG.txt | 4 + .../sk89q/wepif/NijiPermissionsResolver.java | 161 ------------------ .../wepif/PermissionsResolverManager.java | 1 - 3 files changed, 4 insertions(+), 162 deletions(-) delete mode 100644 src/main/java/com/sk89q/wepif/NijiPermissionsResolver.java diff --git a/CHANGELOG.txt b/CHANGELOG.txt index d0788899c..19a1682c6 100644 --- a/CHANGELOG.txt +++ b/CHANGELOG.txt @@ -21,6 +21,10 @@ - Add a cylindrical region selector. - Added initial SpoutAPI compatibility - Allow copying item stacks with damage of > Byte.MAX_VALUE. Closes #972 + - Improved output of YAMLProcessor and added support for root-level comments. (for developers!) + - Removed support for Permissions2/3 + Thanks to everyone for their contributions: + rjwboys, yetanotherx, and GiCodeWarrior 5.0: - Added a //g[en[erate]] command that can generate a shape given a formula diff --git a/src/main/java/com/sk89q/wepif/NijiPermissionsResolver.java b/src/main/java/com/sk89q/wepif/NijiPermissionsResolver.java deleted file mode 100644 index bf0dab73f..000000000 --- a/src/main/java/com/sk89q/wepif/NijiPermissionsResolver.java +++ /dev/null @@ -1,161 +0,0 @@ -// $Id$ -/* - * WorldEdit - * Copyright (C) 2010 sk89q - * - * This program is free software: you can redistribute it and/or modify - * it under the terms of the GNU General Public License as published by - * the Free Software Foundation, either version 3 of the License, or - * (at your option) any later version. - * - * This program is distributed in the hope that it will be useful, - * but WITHOUT ANY WARRANTY; without even the implied warranty of - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the - * GNU General Public License for more details. - * - * You should have received a copy of the GNU General Public License - * along with this program. If not, see . - */ - -package com.sk89q.wepif; - -import com.sk89q.util.yaml.YAMLProcessor; -import org.bukkit.Bukkit; -import org.bukkit.OfflinePlayer; -import org.bukkit.Server; -import org.bukkit.command.PluginCommand; -import org.bukkit.entity.Player; -import org.bukkit.plugin.Plugin; -import org.bukkit.plugin.PluginManager; -import com.nijikokun.bukkit.Permissions.Permissions; - -public class NijiPermissionsResolver implements PermissionsResolver { - private Server server; - private Permissions api; - - public static PermissionsResolver factory(Server server, YAMLProcessor config) { - PluginManager pluginManager = server.getPluginManager(); - try { - Class.forName("com.nijikokun.bukkit.Permissions.Permissions"); - } catch (ClassNotFoundException e) { - return null; - } - - Plugin plugin = pluginManager.getPlugin("Permissions"); - - // Check if plugin is loaded and has Permissions interface - if (plugin == null || !(plugin instanceof Permissions)) { - return null; - } - - // Check for fake permissions - if (config.getBoolean("ignore-nijiperms-bridges", true) && isFakeNijiPerms(plugin)) { - return null; - } - - return new NijiPermissionsResolver(server, (Permissions) plugin); - } - - public void load() { - - } - - public NijiPermissionsResolver(Server server, Permissions plugin) { - this.server = server; - this.api = plugin; - } - - @SuppressWarnings("static-access") - public boolean hasPermission(String name, String permission) { - try { - Player player = server.getPlayerExact(name); - if (player == null) return false; - try { - return api.getHandler().has(player, permission); - } catch (Throwable t) { - return api.Security.permission(player, permission); - } - } catch (Throwable t) { - t.printStackTrace(); - return false; - } - } - - public boolean hasPermission(String worldName, String name, String permission) { - try { - try { - return api.getHandler().has(worldName, name, permission); - } catch (Throwable t) { - return api.getHandler().has(server.getPlayerExact(name), permission); - } - } catch (Throwable t) { - t.printStackTrace(); - return false; - } - } - - @SuppressWarnings("static-access") - public boolean inGroup(String name, String group) { - try { - Player player = server.getPlayerExact(name); - if (player == null) return false; - try { - return api.getHandler().inGroup(player.getWorld().getName(), name, group); - } catch (Throwable t) { - return api.Security.inGroup(name, group); - } - } catch (Throwable t) { - t.printStackTrace(); - return false; - } - } - - @SuppressWarnings("static-access") - public String[] getGroups(String name) { - try { - Player player = server.getPlayerExact(name); - if (player == null) return new String[0]; - String[] groups = null; - try { - groups = api.getHandler().getGroups(player.getWorld().getName(), player.getName()); - } catch (Throwable t) { - String group = api.Security.getGroup(player.getWorld().getName(), player.getName()); - if (group != null) groups = new String[] { group }; - } - if (groups == null) { - return new String[0]; - } else { - return groups; - } - } catch (Throwable t) { - t.printStackTrace(); - return new String[0]; - } - } - - public boolean hasPermission(OfflinePlayer player, String permission) { - return hasPermission(player.getName(), permission); - } - - public boolean hasPermission(String worldName, OfflinePlayer player, String permission) { - return hasPermission(worldName, player.getName(), permission); - } - - public boolean inGroup(OfflinePlayer player, String group) { - return inGroup(player.getName(), group); - } - - public String[] getGroups(OfflinePlayer player) { - return getGroups(player.getName()); - } - - public static boolean isFakeNijiPerms(Plugin plugin) { - PluginCommand permsCommand = Bukkit.getServer().getPluginCommand("permissions"); - - return permsCommand == null || !(permsCommand.getPlugin().equals(plugin)); - } - - public String getDetectionMessage() { - return "Permissions plugin detected! Using Permissions plugin for permissions."; - } -} diff --git a/src/main/java/com/sk89q/wepif/PermissionsResolverManager.java b/src/main/java/com/sk89q/wepif/PermissionsResolverManager.java index 1bbc479c0..96c2f2b61 100644 --- a/src/main/java/com/sk89q/wepif/PermissionsResolverManager.java +++ b/src/main/java/com/sk89q/wepif/PermissionsResolverManager.java @@ -86,7 +86,6 @@ public class PermissionsResolverManager implements PermissionsResolver { protected Class[] availableResolvers = new Class[] { PluginPermissionsResolver.class, PermissionsExResolver.class, - NijiPermissionsResolver.class, DinnerPermsResolver.class, FlatFilePermissionsResolver.class };