From c02306b55e67d33e0b1e582977727b61db9b35c5 Mon Sep 17 00:00:00 2001 From: Wesley Wolfe Date: Mon, 17 Dec 2012 01:16:28 -0600 Subject: Add ItemMeta factory and interfaces. This adds BUKKIT-15 Included with ItemMeta is a new serializable class Color. PotionEffects are now serializable. --- src/main/java/org/bukkit/Bukkit.java | 5 + src/main/java/org/bukkit/Color.java | 339 +++++++++++++++++++ src/main/java/org/bukkit/DyeColor.java | 57 +++- src/main/java/org/bukkit/Material.java | 2 +- src/main/java/org/bukkit/Server.java | 10 + src/main/java/org/bukkit/Utility.java | 15 + .../bukkit/configuration/ConfigurationSection.java | 39 +++ .../org/bukkit/configuration/MemorySection.java | 20 +- .../serialization/ConfigurationSerialization.java | 4 + .../java/org/bukkit/inventory/ItemFactory.java | 82 +++++ src/main/java/org/bukkit/inventory/ItemStack.java | 266 ++++++++++++--- .../java/org/bukkit/inventory/meta/BookMeta.java | 114 +++++++ .../java/org/bukkit/inventory/meta/ItemMeta.java | 108 ++++++ .../bukkit/inventory/meta/LeatherArmorMeta.java | 26 ++ .../java/org/bukkit/inventory/meta/MapMeta.java | 23 ++ .../java/org/bukkit/inventory/meta/PotionMeta.java | 69 ++++ .../java/org/bukkit/inventory/meta/Repairable.java | 31 ++ .../java/org/bukkit/inventory/meta/SkullMeta.java | 33 ++ src/main/java/org/bukkit/potion/PotionEffect.java | 117 ++++++- .../java/org/bukkit/potion/PotionEffectType.java | 8 + src/test/java/org/bukkit/ColorTest.java | 365 +++++++++++++++++++++ .../configuration/ConfigurationSectionTest.java | 2 +- .../bukkit/configuration/file/TestEnchantment.java | 55 ---- .../configuration/file/YamlConfigurationTest.java | 53 --- 24 files changed, 1655 insertions(+), 188 deletions(-) create mode 100644 src/main/java/org/bukkit/Color.java create mode 100644 src/main/java/org/bukkit/Utility.java create mode 100644 src/main/java/org/bukkit/inventory/ItemFactory.java create mode 100644 src/main/java/org/bukkit/inventory/meta/BookMeta.java create mode 100644 src/main/java/org/bukkit/inventory/meta/ItemMeta.java create mode 100644 src/main/java/org/bukkit/inventory/meta/LeatherArmorMeta.java create mode 100644 src/main/java/org/bukkit/inventory/meta/MapMeta.java create mode 100644 src/main/java/org/bukkit/inventory/meta/PotionMeta.java create mode 100644 src/main/java/org/bukkit/inventory/meta/Repairable.java create mode 100644 src/main/java/org/bukkit/inventory/meta/SkullMeta.java create mode 100644 src/test/java/org/bukkit/ColorTest.java delete mode 100644 src/test/java/org/bukkit/configuration/file/TestEnchantment.java (limited to 'src') diff --git a/src/main/java/org/bukkit/Bukkit.java b/src/main/java/org/bukkit/Bukkit.java index e2506bc1..ba5f6d74 100644 --- a/src/main/java/org/bukkit/Bukkit.java +++ b/src/main/java/org/bukkit/Bukkit.java @@ -26,6 +26,7 @@ import org.bukkit.plugin.messaging.Messenger; import org.bukkit.scheduler.BukkitScheduler; import com.avaje.ebean.config.ServerConfig; +import org.bukkit.inventory.ItemFactory; /** * Represents the Bukkit core, for version and Server singleton handling @@ -390,4 +391,8 @@ public final class Bukkit { public static WarningState getWarningState() { return server.getWarningState(); } + + public static ItemFactory getItemFactory() { + return server.getItemFactory(); + } } diff --git a/src/main/java/org/bukkit/Color.java b/src/main/java/org/bukkit/Color.java new file mode 100644 index 00000000..e5ddc422 --- /dev/null +++ b/src/main/java/org/bukkit/Color.java @@ -0,0 +1,339 @@ +package org.bukkit; + +import java.util.Map; + +import org.apache.commons.lang.Validate; +import org.bukkit.configuration.serialization.ConfigurationSerializable; +import org.bukkit.configuration.serialization.SerializableAs; + +import com.google.common.collect.ImmutableMap; + +/** + * A container for a color palette. This class is immutable; the set methods return a new color. + * The color names listed as fields are HTML4 standards, but subject to change. + */ +@SerializableAs("Color") +public final class Color implements ConfigurationSerializable { + private static final int BIT_MASK = 0xff; + + /** + * White, or (0xFF,0xFF,0xFF) in (R,G,B) + */ + public static final Color WHITE = fromRGB(0xFFFFFF); + + /** + * Silver, or (0xC0,0xC0,0xC0) in (R,G,B) + */ + public static final Color SILVER = fromRGB(0xC0C0C0); + + /** + * Gray, or (0x80,0x80,0x80) in (R,G,B) + */ + public static final Color GRAY = fromRGB(0x808080); + + /** + * Black, or (0x00,0x00,0x00) in (R,G,B) + */ + public static final Color BLACK = fromRGB(0x000000); + + /** + * Red, or (0xFF,0x00,0x00) in (R,G,B) + */ + public static final Color RED = fromRGB(0xFF0000); + + /** + * Maroon, or (0x80,0x00,0x00) in (R,G,B) + */ + public static final Color MAROON = fromRGB(0x800000); + + /** + * Yellow, or (0xFF,0xFF,0x00) in (R,G,B) + */ + public static final Color YELLOW = fromRGB(0xFFFF00); + + /** + * Olive, or (0x80,0x80,0x00) in (R,G,B) + */ + public static final Color OLIVE = fromRGB(0x808000); + + /** + * Lime, or (0x00,0xFF,0x00) in (R,G,B) + */ + public static final Color LIME = fromRGB(0x00FF00); + + /** + * Green, or (0x00,0x80,0x00) in (R,G,B) + */ + public static final Color GREEN = fromRGB(0x008000); + + /** + * Aqua, or (0x00,0xFF,0xFF) in (R,G,B) + */ + public static final Color AQUA = fromRGB(0x00FFFF); + + /** + * Teal, or (0x00,0x80,0x80) in (R,G,B) + */ + public static final Color TEAL = fromRGB(0x008080); + + /** + * Blue, or (0x00,0x00,0xFF) in (R,G,B) + */ + public static final Color BLUE = fromRGB(0x0000FF); + + /** + * Navy, or (0x00,0x00,0x80) in (R,G,B) + */ + public static final Color NAVY = fromRGB(0x000080); + + /** + * Fuchsia, or (0xFF,0x00,0xFF) in (R,G,B) + */ + public static final Color FUCHSIA = fromRGB(0xFF00FF); + + /** + * Purple, or (0x80,0x00,0x80) in (R,G,B) + */ + public static final Color PURPLE = fromRGB(0x800080); + + /** + * Orange, or (0xFF,0xA5,0x00) in (R,G,B) + */ + public static final Color ORANGE = fromRGB(0xFFA500); + + private final byte red; + private final byte green; + private final byte blue; + + /** + * Creates a new Color object from a red, green, and blue + * + * @param red integer from 0-255 + * @param green integer from 0-255 + * @param blue integer from 0-255 + * @return a new Color object for the red, green, blue + * @throws IllegalArgumentException if any value is strictly >255 or <0 + */ + public static Color fromRGB(int red, int green, int blue) throws IllegalArgumentException { + return new Color(red, green, blue); + } + + /** + * Creates a new Color object from a blue, green, and red + * + * @param blue integer from 0-255 + * @param green integer from 0-255 + * @param red integer from 0-255 + * @return a new Color object for the red, green, blue + * @throws IllegalArgumentException if any value is strictly >255 or <0 + */ + public static Color fromBGR(int blue, int green, int red) throws IllegalArgumentException { + return new Color(red, green, blue); + } + + /** + * Creates a new color object from an integer that contains the red, green, and blue bytes in the lowest order 24 bits. + * + * @param rgb the integer storing the red, green, and blue values + * @return a new color object for specified values + * @throws IllegalArgumentException if any data is in the highest order 8 bits + */ + public static Color fromRGB(int rgb) throws IllegalArgumentException { + Validate.isTrue((rgb >> 24) == 0, "Extrenuous data in: ", rgb); + return fromRGB(rgb >> 16 & BIT_MASK, rgb >> 8 & BIT_MASK, rgb >> 0 & BIT_MASK); + } + + /** + * Creates a new color object from an integer that contains the blue, green, and red bytes in the lowest order 24 bits. + * + * @param bgr the integer storing the blue, green, and red values + * @return a new color object for specified values + * @throws IllegalArgumentException if any data is in the highest order 8 bits + */ + public static Color fromBGR(int bgr) throws IllegalArgumentException { + Validate.isTrue((bgr >> 24) == 0, "Extrenuous data in: ", bgr); + return fromBGR(bgr >> 16 & BIT_MASK, bgr >> 8 & BIT_MASK, bgr >> 0 & BIT_MASK); + } + + private Color(int red, int green, int blue) { + Validate.isTrue(red >= 0 && red <= BIT_MASK, "Red is not between 0-255: ", red); + Validate.isTrue(green >= 0 && green <= BIT_MASK, "Red is not between 0-255: ", green); + Validate.isTrue(blue >= 0 && blue <= BIT_MASK, "Red is not between 0-255: ", blue); + + this.red = (byte) red; + this.green = (byte) green; + this.blue = (byte) blue; + } + + /** + * Gets the red component + * + * @return red component, from 0 to 255 + */ + public int getRed() { + return BIT_MASK & red; + } + + /** + * Creates a new Color object with specified component + * + * @param red the red component, from 0 to 255 + * @return a new color object with the red component + */ + public Color setRed(int red) { + return fromRGB(red, getGreen(), getBlue()); + } + + /** + * Gets the green component + * + * @return green component, from 0 to 255 + */ + public int getGreen() { + return BIT_MASK & green; + } + + /** + * Creates a new Color object with specified component + * + * @param green the red component, from 0 to 255 + * @return a new color object with the red component + */ + public Color setGreen(int green) { + return fromRGB(getRed(), green, getBlue()); + } + + /** + * Gets the blue component + * + * @return blue component, from 0 to 255 + */ + public int getBlue() { + return BIT_MASK & blue; + } + + /** + * Creates a new Color object with specified component + * + * @param blue the red component, from 0 to 255 + * @return a new color object with the red component + */ + public Color setBlue(int blue) { + return fromRGB(getRed(), getGreen(), blue); + } + + /** + * + * @return An integer representation of this color, as 0xRRGGBB + */ + public int asRGB() { + return getRed() << 16 | getGreen() << 8 | getBlue() << 0; + } + + /** + * + * @return An integer representation of this color, as 0xBBGGRR + */ + public int asBGR() { + return getBlue() << 16 | getGreen() << 8 | getRed() << 0; + } + + /** + * Creates a new color with its RGB components changed as if it was dyed with the colors passed in, replicating + * vanilla workbench dyeing + * + * @param colors The DyeColors to dye with + * @return A new color with the changed rgb components + */ + // TODO: Javadoc what this method does, not what it mimics. API != Implementation + public Color mixDyes(DyeColor... colors) { + Validate.noNullElements(colors, "Colors cannot be null"); + + Color[] toPass = new Color[colors.length]; + for (int i = 0; i < colors.length; i++) { + toPass[i] = colors[i].getColor(); + } + + return mixColors(toPass); + } + + /** + * Creates a new color with its RGB components changed as if it was dyed with the colors passed in, replicating + * vanilla workbench dyeing + * + * @param colors The colors to dye with + * @return A new color with the changed rgb components + */ + // TODO: Javadoc what this method does, not what it mimics. API != Implementation + public Color mixColors(Color... colors) { + Validate.noNullElements(colors, "Colors cannot be null"); + + int totalRed = this.getRed(); + int totalGreen = this.getGreen(); + int totalBlue = this.getBlue(); + int totalMax = Math.max(Math.max(totalRed, totalGreen), totalBlue); + for (Color color : colors) { + totalRed += color.getRed(); + totalGreen += color.getGreen(); + totalBlue += color.getBlue(); + totalMax += Math.max(Math.max(color.getRed(), color.getGreen()), color.getBlue()); + } + + float averageRed = totalRed / (colors.length + 1); + float averageGreen = totalGreen / (colors.length + 1); + float averageBlue = totalBlue / (colors.length + 1); + float averageMax = totalMax / (colors.length + 1); + + float maximumOfAverages = Math.max(Math.max(averageRed, averageGreen), averageBlue); + float gainFactor = averageMax / maximumOfAverages; + + return Color.fromRGB((int) (averageRed * gainFactor), (int) (averageGreen * gainFactor), (int) (averageBlue * gainFactor)); + } + + @Override + public boolean equals(Object o) { + if (!(o instanceof Color)) { + return false; + } + final Color that = (Color) o; + return this.blue == that.blue && this.green == that.green && this.red == that.red; + } + + @Override + public int hashCode() { + return asRGB() ^ Color.class.hashCode(); + } + + public Map serialize() { + return ImmutableMap.of( + "RED", getRed(), + "BLUE", getBlue(), + "GREEN", getGreen() + ); + } + + @SuppressWarnings("javadoc") + public static Color deserialize(Map map) { + return fromRGB( + asInt("RED", map), + asInt("GREEN", map), + asInt("BLUE", map) + ); + } + + private static int asInt(String string, Map map) { + Object value = map.get(string); + if (value == null) { + throw new IllegalArgumentException(string + " not in map " + map); + } + if (!(value instanceof Number)) { + throw new IllegalArgumentException(string + '(' + value + ") is not a number"); + } + return ((Number) value).intValue(); + } + + @Override + public String toString() { + return "Color:[rgb0x" + Integer.toHexString(getRed()).toUpperCase() + Integer.toHexString(getGreen()).toUpperCase() + Integer.toHexString(getBlue()).toUpperCase() + "]"; + } +} diff --git a/src/main/java/org/bukkit/DyeColor.java b/src/main/java/org/bukkit/DyeColor.java index 531323fe..251b3321 100644 --- a/src/main/java/org/bukkit/DyeColor.java +++ b/src/main/java/org/bukkit/DyeColor.java @@ -12,73 +12,76 @@ public enum DyeColor { /** * Represents white dye */ - WHITE(0x0), + WHITE(0x0, Color.WHITE), /** * Represents orange dye */ - ORANGE(0x1), + ORANGE(0x1, Color.fromRGB(0xD87f33)), /** * Represents magenta dye */ - MAGENTA(0x2), + MAGENTA(0x2, Color.fromRGB(0xB24CD8)), /** * Represents light blue dye */ - LIGHT_BLUE(0x3), + LIGHT_BLUE(0x3, Color.fromRGB(0x6699D8)), /** * Represents yellow dye */ - YELLOW(0x4), + YELLOW(0x4, Color.fromRGB(0xE5E533)), /** * Represents lime dye */ - LIME(0x5), + LIME(0x5, Color.fromRGB(0x7FCC19)), /** * Represents pink dye */ - PINK(0x6), + PINK(0x6, Color.fromRGB(0xF27FA5)), /** * Represents gray dye */ - GRAY(0x7), + GRAY(0x7, Color.fromRGB(0x4C4C4C)), /** * Represents silver dye */ - SILVER(0x8), + SILVER(0x8, Color.fromRGB(0x999999)), /** * Represents cyan dye */ - CYAN(0x9), + CYAN(0x9, Color.fromRGB(0x4C7F99)), /** * Represents purple dye */ - PURPLE(0xA), + PURPLE(0xA, Color.fromRGB(0x7F3FB2)), /** * Represents blue dye */ - BLUE(0xB), + BLUE(0xB, Color.fromRGB(0x334CB2)), /** * Represents brown dye */ - BROWN(0xC), + BROWN(0xC, Color.fromRGB(0x664C33)), /** * Represents green dye */ - GREEN(0xD), + GREEN(0xD, Color.fromRGB(0x667F33)), /** * Represents red dye */ - RED(0xE), + RED(0xE, Color.fromRGB(0x993333)), /** * Represents black dye */ - BLACK(0xF); + BLACK(0xF, Color.fromRGB(0x191919)); private final byte data; + private final Color color; private final static Map BY_DATA = Maps.newHashMap(); + private final static Map BY_COLOR = Maps.newHashMap(); - private DyeColor(final int data) { + private DyeColor(final int data, Color color) { this.data = (byte) data; + this.color = color; } /** @@ -90,6 +93,15 @@ public enum DyeColor { return data; } + /** + * Gets the color that this dye represents + * + * @return The {@link Color} that this dye represents + */ + public Color getColor() { + return color; + } + /** * Gets the DyeColor with the given data value * @@ -100,9 +112,20 @@ public enum DyeColor { return BY_DATA.get(data); } + /** + * Gets the DyeColor with the given color value + * + * @param color Color value to get the dye by + * @return The {@link DyeColor} representing the given value, or null if it doesn't exist + */ + public static DyeColor getByColor(final Color color) { + return BY_COLOR.get(color); + } + static { for (DyeColor color : values()) { BY_DATA.put(color.getData(), color); + BY_COLOR.put(color.getColor(), color); } } } diff --git a/src/main/java/org/bukkit/Material.java b/src/main/java/org/bukkit/Material.java index 0488f471..020c1396 100644 --- a/src/main/java/org/bukkit/Material.java +++ b/src/main/java/org/bukkit/Material.java @@ -15,7 +15,7 @@ import com.google.common.collect.Maps; * An enum of all material ids accepted by the official server + client */ public enum Material { - AIR(0), + AIR(0, 0), STONE(1), GRASS(2), DIRT(3), diff --git a/src/main/java/org/bukkit/Server.java b/src/main/java/org/bukkit/Server.java index a4f8ec29..ceec40cc 100644 --- a/src/main/java/org/bukkit/Server.java +++ b/src/main/java/org/bukkit/Server.java @@ -28,6 +28,8 @@ import org.bukkit.plugin.messaging.PluginMessageRecipient; import org.bukkit.scheduler.BukkitScheduler; import com.avaje.ebean.config.ServerConfig; +import org.bukkit.inventory.ItemFactory; +import org.bukkit.inventory.meta.ItemMeta; /** * Represents a server implementation @@ -679,4 +681,12 @@ public interface Server extends PluginMessageRecipient { * @return The configured WarningState */ public WarningState getWarningState(); + + /** + * Gets the instance of the item factory (for {@link ItemMeta}). + * + * @return the item factory + * @see ItemFactory + */ + ItemFactory getItemFactory(); } diff --git a/src/main/java/org/bukkit/Utility.java b/src/main/java/org/bukkit/Utility.java new file mode 100644 index 00000000..0e54481b --- /dev/null +++ b/src/main/java/org/bukkit/Utility.java @@ -0,0 +1,15 @@ +package org.bukkit; + +import java.lang.annotation.ElementType; +import java.lang.annotation.Retention; +import java.lang.annotation.RetentionPolicy; +import java.lang.annotation.Target; + +/** + * This annotation indicates a method (and sometimes constructor) will chain its internal operations. + * This is solely meant for identifying methods that don't need to be overridden / handled manually. + */ +@Target({ElementType.CONSTRUCTOR, ElementType.METHOD}) +@Retention(RetentionPolicy.SOURCE) +public @interface Utility { +} diff --git a/src/main/java/org/bukkit/configuration/ConfigurationSection.java b/src/main/java/org/bukkit/configuration/ConfigurationSection.java index c5a2a4c3..60b53524 100644 --- a/src/main/java/org/bukkit/configuration/ConfigurationSection.java +++ b/src/main/java/org/bukkit/configuration/ConfigurationSection.java @@ -3,6 +3,8 @@ package org.bukkit.configuration; import java.util.Map; import java.util.Set; import java.util.List; + +import org.bukkit.Color; import org.bukkit.OfflinePlayer; import org.bukkit.util.Vector; import org.bukkit.inventory.ItemStack; @@ -664,6 +666,43 @@ public interface ConfigurationSection { */ public boolean isItemStack(String path); + /** + * Gets the requested Color by path. + *

+ * If the Color does not exist but a default value has been specified, this + * will return the default value. If the Color does not exist and no default + * value was specified, this will return null. + * + * @param path Path of the Color to get. + * @return Requested Color. + */ + public Color getColor(String path); + + /** + * Gets the requested {@link Color} by path, returning a default value if not found. + *

+ * If the Color does not exist then the specified default value will returned + * regardless of if a default has been identified in the root {@link Configuration}. + * + * @param path Path of the Color to get. + * @param def The default value to return if the path is not found or is not an Color. + * @return Requested Color. + */ + public Color getColor(String path, Color def); + + /** + * Checks if the specified path is a Color. + *

+ * If the path exists but is not a Color, this will return false. If the path does not + * exist, this will return false. If the path does not exist but a default value + * has been specified, this will check if that default value is a Color and return + * appropriately. + * + * @param path Path of the Color to check. + * @return Whether or not the specified path is an Color. + */ + public boolean isColor(String path); + /** * Gets the requested ConfigurationSection by path. *

diff --git a/src/main/java/org/bukkit/configuration/MemorySection.java b/src/main/java/org/bukkit/configuration/MemorySection.java index f4a9592e..ddf7c515 100644 --- a/src/main/java/org/bukkit/configuration/MemorySection.java +++ b/src/main/java/org/bukkit/configuration/MemorySection.java @@ -1,16 +1,19 @@ package org.bukkit.configuration; +import static org.bukkit.util.NumberConversions.*; + import java.util.ArrayList; import java.util.LinkedHashMap; import java.util.LinkedHashSet; import java.util.List; import java.util.Map; import java.util.Set; + import org.apache.commons.lang.Validate; +import org.bukkit.Color; import org.bukkit.OfflinePlayer; import org.bukkit.inventory.ItemStack; import org.bukkit.util.Vector; -import static org.bukkit.util.NumberConversions.*; /** * A type of {@link ConfigurationSection} that is stored in memory. @@ -653,6 +656,21 @@ public class MemorySection implements ConfigurationSection { return val instanceof ItemStack; } + public Color getColor(String path) { + Object def = getDefault(path); + return getColor(path, (def instanceof Color) ? (Color) def : null); + } + + public Color getColor(String path, Color def) { + Object val = get(path, def); + return (val instanceof Color) ? (Color) val : def; + } + + public boolean isColor(String path) { + Object val = get(path); + return val instanceof Color; + } + public ConfigurationSection getConfigurationSection(String path) { Object val = get(path, null); if (val != null) { diff --git a/src/main/java/org/bukkit/configuration/serialization/ConfigurationSerialization.java b/src/main/java/org/bukkit/configuration/serialization/ConfigurationSerialization.java index abfc4b60..ca9f7b78 100644 --- a/src/main/java/org/bukkit/configuration/serialization/ConfigurationSerialization.java +++ b/src/main/java/org/bukkit/configuration/serialization/ConfigurationSerialization.java @@ -10,8 +10,10 @@ import java.util.logging.Level; import java.util.logging.Logger; import org.apache.commons.lang.Validate; +import org.bukkit.Color; import org.bukkit.configuration.Configuration; import org.bukkit.inventory.ItemStack; +import org.bukkit.potion.PotionEffect; import org.bukkit.util.BlockVector; import org.bukkit.util.Vector; @@ -27,6 +29,8 @@ public class ConfigurationSerialization { registerClass(Vector.class); registerClass(BlockVector.class); registerClass(ItemStack.class); + registerClass(Color.class); + registerClass(PotionEffect.class); } protected ConfigurationSerialization(Class clazz) { diff --git a/src/main/java/org/bukkit/inventory/ItemFactory.java b/src/main/java/org/bukkit/inventory/ItemFactory.java new file mode 100644 index 00000000..c6654c4a --- /dev/null +++ b/src/main/java/org/bukkit/inventory/ItemFactory.java @@ -0,0 +1,82 @@ +package org.bukkit.inventory; + +import org.bukkit.Material; +import org.bukkit.Server; +import org.bukkit.inventory.meta.BookMeta; +import org.bukkit.inventory.meta.ItemMeta; +import org.bukkit.inventory.meta.SkullMeta; + +/** + * An instance of the ItemFactory can be obtained with {@link Server#getItemFactory()}. + * The ItemFactory is solely responsible for creating item meta containers to apply on item stacks. + */ +public interface ItemFactory { + + /** + * This creates a new item meta for the material. + * @param material The material to consider as base for the meta + * @return a new ItemMeta that could be applied to an item stack of the specified material + */ + ItemMeta getItemMeta(final Material material); + + /** + * This method checks the item meta to confirm that it is applicable (no data lost if applied) to the specified ItemStack. + * A {@link SkullMeta} would not be valid for a sword, but a normal {@link ItemMeta} from an enchanted dirt block would. + * @param meta Meta to check + * @param stack Item that meta will be applied to + * @return true if the meta can be applied without losing data, false otherwise + * @throws IllegalArgumentException if the meta was not created by this factory + */ + boolean isApplicable(final ItemMeta meta, final ItemStack stack) throws IllegalArgumentException; + + /** + * This method checks the item meta to confirm that it is applicable (no data lost if applied) to the specified Material. + * A {@link SkullMeta} would not be valid for a sword, but a normal {@link ItemMeta} from an enchanted dirt block would. + * @param meta Meta to check + * @param material Material that meta will be applied to + * @return true if the meta can be applied without losing data, false otherwise + * @throws IllegalArgumentException if the meta was not created by this factory + */ + boolean isApplicable(final ItemMeta meta, final Material material) throws IllegalArgumentException; + + /** + * This method is used to compare two item meta data objects. + * @param meta1 First meta to compare, and may be null to indicate no data + * @param meta2 Second meta to compare, and may be null to indicate no data + * @return false if one of the meta has data the other does not, otherwise true + * @throws IllegalArgumentException if either meta was not created by this factory + */ + boolean equals(final ItemMeta meta1, final ItemMeta meta2) throws IllegalArgumentException; + + /** + * Returns an appropriate item meta for the specified stack. + * The item meta returned will always be a valid meta for a given item stack of the specified material. + * It may be a more or less specific meta, and could also be the same meta or meta type as the parameter. + * The item meta returned will also always be the most appropriate meta.
+ *
+ * Example, if a {@link SkullMeta} is being applied to a book, this method would return a {@link BookMeta} containing all + * information in the specified meta that is applicable to an {@link ItemMeta}, the highest common interface. + * + * @param meta the meta to convert + * @param stack the stack to convert the meta for + * @return An appropriate item meta for the specified item stack. No guarantees are made as to if a copy is returned. This will be null for a stack of air. + * @throws IllegalArgumentException if the specified meta was not created by this factory + */ + ItemMeta asMetaFor(final ItemMeta meta, final ItemStack stack) throws IllegalArgumentException; + + /** + * Returns an appropriate item meta for the specified material. + * The item meta returned will always be a valid meta for a given item stack of the specified material. + * It may be a more or less specific meta, and could also be the same meta or meta type as the parameter. + * The item meta returned will also always be the most appropriate meta.
+ *
+ * Example, if a {@link SkullMeta} is being applied to a book, this method would return a {@link BookMeta} containing all + * information in the specified meta that is applicable to an {@link ItemMeta}, the highest common interface. + * + * @param meta the meta to convert + * @param material the material to convert the meta for + * @return An appropriate item meta for the specified item material. No guarantees are made as to if a copy is returned. This will be null for air. + * @throws IllegalArgumentException if the specified meta was not created by this factory + */ + ItemMeta asMetaFor(final ItemMeta meta, final Material material) throws IllegalArgumentException; +} diff --git a/src/main/java/org/bukkit/inventory/ItemStack.java b/src/main/java/org/bukkit/inventory/ItemStack.java index db7c5005..58b7e992 100644 --- a/src/main/java/org/bukkit/inventory/ItemStack.java +++ b/src/main/java/org/bukkit/inventory/ItemStack.java @@ -1,48 +1,97 @@ package org.bukkit.inventory; import com.google.common.collect.ImmutableMap; -import java.util.HashMap; import java.util.LinkedHashMap; import java.util.Map; + +import org.apache.commons.lang.Validate; +import org.bukkit.Bukkit; import org.bukkit.Material; +import org.bukkit.Utility; import org.bukkit.configuration.serialization.ConfigurationSerializable; import org.bukkit.enchantments.Enchantment; +import org.bukkit.inventory.meta.ItemMeta; import org.bukkit.material.MaterialData; /** * Represents a stack of items */ public class ItemStack implements Cloneable, ConfigurationSerializable { - private int type; + private int type = 0; private int amount = 0; private MaterialData data = null; private short durability = 0; - private Map enchantments = new HashMap(); + private ItemMeta meta; + + @Utility + protected ItemStack() {} + /** + * Defaults stack size to 1, with no extra data + * + * @param type item material id + */ public ItemStack(final int type) { this(type, 1); } + /** + * Defaults stack size to 1, with no extra data + * + * @param type item material + */ public ItemStack(final Material type) { this(type, 1); } + /** + * An item stack with no extra data + * + * @param type item material id + * @param amount stack size + */ public ItemStack(final int type, final int amount) { this(type, amount, (short) 0); } + /** + * An item stack with no extra data + * + * @param type item material + * @param amount stack size + */ public ItemStack(final Material type, final int amount) { this(type.getId(), amount); } + /** + * An item stack with the specified damage / durability + * + * @param type item material id + * @param amount stack size + * @param damage durability / damage + */ public ItemStack(final int type, final int amount, final short damage) { - this(type, amount, damage, null); + this.type = type; + this.amount = amount; + this.durability = damage; } + /** + * An item stack with the specified damage / durabiltiy + * + * @param type item material + * @param amount stack size + * @param damage durability / damage + */ public ItemStack(final Material type, final int amount, final short damage) { this(type.getId(), amount, damage); } + /** + * @deprecated this method uses an ambiguous data byte object + */ + @Deprecated public ItemStack(final int type, final int amount, final short damage, final Byte data) { this.type = type; this.amount = amount; @@ -53,18 +102,29 @@ public class ItemStack implements Cloneable, ConfigurationSerializable { } } + /** + * @deprecated this method uses an ambiguous data byte object + */ + @Deprecated public ItemStack(final Material type, final int amount, final short damage, final Byte data) { this(type.getId(), amount, damage, data); } - public ItemStack(final ItemStack stack) { - this.type = stack.type; - this.amount = stack.amount; - this.durability = stack.durability; - if (stack.data != null) { - this.data = stack.data.clone(); + /** + * Creates a new item stack derived from the specified stack + * + * @param stack the stack to copy + * @throws IllegalArgumentException if the specified stack is null or returns an item meta not created by the item factory + */ + public ItemStack(final ItemStack stack) throws IllegalArgumentException { + Validate.notNull(stack, "Cannot copy null stack"); + this.type = stack.getTypeId(); + this.amount = stack.getAmount(); + this.durability = stack.getDurability(); + this.data = stack.getData(); + if (stack.hasItemMeta()) { + setItemMeta0(stack.getItemMeta(), getType0()); } - this.addUnsafeEnchantments(stack.getEnchantments()); } /** @@ -72,8 +132,18 @@ public class ItemStack implements Cloneable, ConfigurationSerializable { * * @return Type of the items in this stack */ + @Utility public Material getType() { - return Material.getMaterial(type); + return getType0(getTypeId()); + } + + private Material getType0() { + return getType0(this.type); + } + + private static Material getType0(int id) { + Material material = Material.getMaterial(id); + return material == null ? Material.AIR : material; } /** @@ -83,7 +153,9 @@ public class ItemStack implements Cloneable, ConfigurationSerializable { * * @param type New type to set the items in this stack to */ + @Utility public void setType(Material type) { + Validate.notNull(type, "Material cannot be null"); setTypeId(type.getId()); } @@ -105,6 +177,9 @@ public class ItemStack implements Cloneable, ConfigurationSerializable { */ public void setTypeId(int type) { this.type = type; + if (this.meta != null) { + this.meta = Bukkit.getItemFactory().asMetaFor(meta, getType0()); + } createData((byte) 0); } @@ -132,9 +207,9 @@ public class ItemStack implements Cloneable, ConfigurationSerializable { * @return MaterialData for this item */ public MaterialData getData() { - Material mat = Material.getMaterial(getTypeId()); - if (mat != null && mat.getData() != null) { - data = mat.getNewData((byte) this.durability); + Material mat = getType(); + if (data == null && mat != null && mat.getData() != null) { + data = mat.getNewData((byte) this.getDurability()); } return data; @@ -148,7 +223,7 @@ public class ItemStack implements Cloneable, ConfigurationSerializable { public void setData(MaterialData data) { Material mat = getType(); - if ((mat == null) || (mat.getData() == null)) { + if (data == null || mat == null || mat.getData() == null) { this.data = data; } else { if ((data.getClass() == mat.getData()) || (data.getClass() == MaterialData.class)) { @@ -183,12 +258,12 @@ public class ItemStack implements Cloneable, ConfigurationSerializable { * * @return The maximum you can stack this material to. */ + @Utility public int getMaxStackSize() { Material material = getType(); if (material != null) { return material.getMaxStackSize(); } - return -1; } @@ -203,19 +278,44 @@ public class ItemStack implements Cloneable, ConfigurationSerializable { } @Override + @Utility public String toString() { - return "ItemStack{" + getType().name() + " x " + getAmount() + "}"; + StringBuilder toString = new StringBuilder("ItemStack{").append(getType().name()).append(" x ").append(getAmount()); + if (hasItemMeta()) { + toString.append(", ").append(getItemMeta()); + } + return toString.append('}').toString(); } @Override + @Utility public boolean equals(Object obj) { + if (this == obj) { + return true; + } if (!(obj instanceof ItemStack)) { return false; } - ItemStack item = (ItemStack) obj; + ItemStack stack = (ItemStack) obj; + return getAmount() == stack.getAmount() && isSimilar(stack); + } - return item.getAmount() == getAmount() && item.getTypeId() == getTypeId() && getDurability() == item.getDurability() && getEnchantments().equals(item.getEnchantments()); + /** + * This method is the same as equals, but does not consider stack size (amount). + * + * @param stack the item stack to compare to + * @return true if the two stacks are equal, ignoring the amount + */ + @Utility + public boolean isSimilar(ItemStack stack) { + if (stack == null) { + return false; + } + if (stack == this) { + return true; + } + return getTypeId() == stack.getTypeId() && getDurability() == stack.getDurability() && hasItemMeta() == stack.hasItemMeta() && (hasItemMeta() ? Bukkit.getItemFactory().equals(getItemMeta(), stack.getItemMeta()) : true); } @Override @@ -223,7 +323,10 @@ public class ItemStack implements Cloneable, ConfigurationSerializable { try { ItemStack itemStack = (ItemStack) super.clone(); - itemStack.enchantments = new HashMap(this.enchantments); + if (this.meta != null) { + itemStack.meta = this.meta.clone(); + } + if (this.data != null) { itemStack.data = this.data.clone(); } @@ -235,11 +338,15 @@ public class ItemStack implements Cloneable, ConfigurationSerializable { } @Override - public int hashCode() { - int hash = 11; + @Utility + public final int hashCode() { + int hash = 1; + + hash = hash * 31 + getTypeId(); + hash = hash * 31 + getAmount(); + hash = hash * 31 + (getDurability() & 0xffff); + hash = hash * 31 + (hasItemMeta() ? (meta == null ? getItemMeta().hashCode() : meta.hashCode()) : 0); - hash = hash * 19 + 7 * getTypeId(); // Overriding hashCode since equals is overridden, it's just - hash = hash * 7 + 23 * getAmount(); // too bad these are mutable values... Q_Q return hash; } @@ -250,7 +357,7 @@ public class ItemStack implements Cloneable, ConfigurationSerializable { * @return True if this has the given enchantment */ public boolean containsEnchantment(Enchantment ench) { - return enchantments.containsKey(ench); + return meta == null ? false : meta.hasEnchant(ench); } /** @@ -260,7 +367,7 @@ public class ItemStack implements Cloneable, ConfigurationSerializable { * @return Level of the enchantment, or 0 */ public int getEnchantmentLevel(Enchantment ench) { - return enchantments.get(ench); + return meta == null ? 0 : meta.getEnchantLevel(ench); } /** @@ -269,7 +376,7 @@ public class ItemStack implements Cloneable, ConfigurationSerializable { * @return Map of enchantments. */ public Map getEnchantments() { - return ImmutableMap.copyOf(enchantments); + return meta == null ? ImmutableMap.of() : meta.getEnchants(); } /** @@ -279,8 +386,13 @@ public class ItemStack implements Cloneable, ConfigurationSerializable { * for each element of the map. * * @param enchantments Enchantments to add + * @throws IllegalArgumentException if the specified enchantments is null + * @throws IllegalArgumentException if any specific enchantment or level is null. + * Warning: Some enchantments may be added before this exception is thrown. */ + @Utility public void addEnchantments(Map enchantments) { + Validate.notNull(enchantments, "Enchantments cannot be null"); for (Map.Entry entry : enchantments.entrySet()) { addEnchantment(entry.getKey(), entry.getValue()); } @@ -293,8 +405,11 @@ public class ItemStack implements Cloneable, ConfigurationSerializable { * * @param ench Enchantment to add * @param level Level of the enchantment + * @throws IllegalArgumentException if enchantment null, or enchantment is not applicable */ + @Utility public void addEnchantment(Enchantment ench, int level) { + Validate.notNull(ench, "Enchantment cannot be null"); if ((level < ench.getStartLevel()) || (level > ench.getMaxLevel())) { throw new IllegalArgumentException("Enchantment level is either too low or too high (given " + level + ", bounds are " + ench.getStartLevel() + " to " + ench.getMaxLevel()); } else if (!ench.canEnchantItem(this)) { @@ -312,6 +427,7 @@ public class ItemStack implements Cloneable, ConfigurationSerializable { * * @param enchantments Enchantments to add */ + @Utility public void addUnsafeEnchantments(Map enchantments) { for (Map.Entry entry : enchantments.entrySet()) { addUnsafeEnchantment(entry.getKey(), entry.getValue()); @@ -330,7 +446,7 @@ public class ItemStack implements Cloneable, ConfigurationSerializable { * @param level Level of the enchantment */ public void addUnsafeEnchantment(Enchantment ench, int level) { - enchantments.put(ench, level); + (meta == null ? meta = Bukkit.getItemFactory().getItemMeta(getType0()) : meta).addEnchant(ench, level, true); } /** @@ -340,38 +456,43 @@ public class ItemStack implements Cloneable, ConfigurationSerializable { * @return Previous level, or 0 */ public int removeEnchantment(Enchantment ench) { - Integer previous = enchantments.remove(ench); - return (previous == null) ? 0 : previous; + int level = getEnchantmentLevel(ench); + if (level == 0 || meta == null) { + return level; + } + meta.removeEnchant(ench); + return level; } + @Utility public Map serialize() { Map result = new LinkedHashMap(); result.put("type", getType().name()); - if (durability != 0) { - result.put("damage", durability); + if (getDurability() != 0) { + result.put("damage", getDurability()); } - if (amount != 1) { - result.put("amount", amount); + if (getAmount() != 1) { + result.put("amount", getAmount()); } - Map enchants = getEnchantments(); - - if (enchants.size() > 0) { - Map safeEnchants = new HashMap(); - - for (Map.Entry entry : enchants.entrySet()) { - safeEnchants.put(entry.getKey().getName(), entry.getValue()); - } - - result.put("enchantments", safeEnchants); + ItemMeta meta = getItemMeta(); + if (!Bukkit.getItemFactory().equals(meta, null)) { + result.put("meta", meta); } return result; } + /** + * Required method for configuration serialization + * + * @param args map to deserialize + * @return deserialized item stack + * @see ConfigurationSerializable + */ public static ItemStack deserialize(Map args) { Material type = Material.getMaterial((String) args.get("type")); short damage = 0; @@ -387,7 +508,7 @@ public class ItemStack implements Cloneable, ConfigurationSerializable { ItemStack result = new ItemStack(type, amount, damage); - if (args.containsKey("enchantments")) { + if (args.containsKey("enchantments")) { // Backward compatiblity, @deprecated Object raw = args.get("enchantments"); if (raw instanceof Map) { @@ -401,8 +522,61 @@ public class ItemStack implements Cloneable, ConfigurationSerializable { } } } + } else if (args.containsKey("meta")) { // We cannot and will not have meta when enchantments (pre-ItemMeta) exist + Object raw = args.get("meta"); + if (raw instanceof ItemMeta) { + result.setItemMeta((ItemMeta) raw); + } } return result; } + + /** + * Get a copy of this ItemStack's {@link ItemMeta}. + * + * @return a copy of the current ItemStack's ItemData + */ + public ItemMeta getItemMeta() { + return this.meta == null ? Bukkit.getItemFactory().getItemMeta(getType0()) : this.meta.clone(); + } + + /** + * Checks to see if any meta data has been defined. + * + * @return Returns true if some meta data has been set for this item + */ + public boolean hasItemMeta() { + return !Bukkit.getItemFactory().equals(meta, null); + } + + /** + * Set the ItemMeta of this ItemStack. + * + * @param itemMeta new ItemMeta, or null to indicate meta data be cleared. + * @return True if successfully applied ItemMeta, see {@link ItemFactory#isApplicable(ItemMeta, ItemStack)} + * @throws IllegalArgumentException if the item meta was not created by the {@link ItemFactory} + */ + public boolean setItemMeta(ItemMeta itemMeta) { + return setItemMeta0(itemMeta, getType0()); + } + + /* + * Cannot be overridden, so it's safe for constructor call + */ + private boolean setItemMeta0(ItemMeta itemMeta, Material material) { + if (itemMeta == null) { + this.meta = null; + return true; + } + if (!Bukkit.getItemFactory().isApplicable(itemMeta, material)) { + return false; + } + this.meta = Bukkit.getItemFactory().asMetaFor(itemMeta, material); + if (this.meta == itemMeta) { + this.meta = itemMeta.clone(); + } + + return true; + } } diff --git a/src/main/java/org/bukkit/inventory/meta/BookMeta.java b/src/main/java/org/bukkit/inventory/meta/BookMeta.java new file mode 100644 index 00000000..987a2866 --- /dev/null +++ b/src/main/java/org/bukkit/inventory/meta/BookMeta.java @@ -0,0 +1,114 @@ +package org.bukkit.inventory.meta; + +import java.util.List; + +import org.bukkit.Material; + +/** + * Represents a book ({@link Material#BOOK_AND_QUILL} or {@link Material#WRITTEN_BOOK}) that can have a title, an author, and pages. + */ +public interface BookMeta extends ItemMeta { + + /** + * Checks for the existence of a title in the book. + * + * @return true if the book has a title + */ + boolean hasTitle(); + + /** + * Gets the title of the book. + * + * @return the title of the book + */ + String getTitle(); + + /** + * Sets the title of the book. Limited to 16 characters. + * + * @param title the title to set + * @return true if the title was successfully set + */ + boolean setTitle(String title); + + /** + * Checks for the existence of an author in the book. + * + * @return the author of the book + */ + boolean hasAuthor(); + + /** + * Gets the author of the book. + * + * @return the author of the book + */ + String getAuthor(); + + /** + * Sets the author of the book. + * + * @param author the author of the book + */ + void setAuthor(String author); + + /** + * Checks for the existence of pages in the book. + * + * @return true if the book has pages + */ + boolean hasPages(); + + /** + * Gets the specified page in the book. + * + * @param page the page number to get + * @return the page from the book + */ + String getPage(int page); + + /** + * Sets the specified page in the book. + * + * @param page the page number to set + * @param data the data to set for that page + */ + void setPage(int page, String data); + + /** + * Gets all the pages in the book. + * + * @return list of all the pages in the book + */ + List getPages(); + + /** + * Clears the existing book pages, and sets the book to use the provided pages. Maximum 50 pages with 256 characters per page. + * + * @param pages A list of pages to set the book to use + */ + void setPages(List pages); + + /** + * Clears the existing book pages, and sets the book to use the provided pages. Maximum 50 pages with 256 characters per page. + * + * @param pages A list of strings, each being a page + */ + void setPages(String... pages); + + /** + * Adds new pages to the end of the book. + * + * @param pages A list of strings, each being a page + */ + void addPage(String... pages); + + /** + * Gets the number of pages in the book. + * + * @return the number of pages in the book + */ + int getPageCount(); + + BookMeta clone(); +} diff --git a/src/main/java/org/bukkit/inventory/meta/ItemMeta.java b/src/main/java/org/bukkit/inventory/meta/ItemMeta.java new file mode 100644 index 00000000..fe744546 --- /dev/null +++ b/src/main/java/org/bukkit/inventory/meta/ItemMeta.java @@ -0,0 +1,108 @@ +package org.bukkit.inventory.meta; + +import java.util.List; +import java.util.Map; + +import org.bukkit.configuration.serialization.ConfigurationSerializable; +import org.bukkit.enchantments.Enchantment; + +/** + * This type represents the storage mechanism for auxiliary item data. + * An implementation will handle the creation and application for ItemMeta. + * This class should not be implemented by a plugin in a live environment. + */ +public interface ItemMeta extends Cloneable, ConfigurationSerializable { + + /** + * Checks for existence of a display name + * + * @return true if this has a display name + */ + boolean hasDisplayName(); + + /** + * Gets the display name that is set + * + * @return the display name that is set + */ + String getDisplayName(); + + /** + * Sets the display name + * + * @param name the name to set + */ + void setDisplayName(String name); + + /** + * Checks for existence of lore + * + * @return true if this has lore + */ + boolean hasLore(); + + /** + * Gets the lore that is set + * + * @return a list of lore that is set + */ + List getLore(); + + /** + * Sets the lore for this item + * + * @param lore the lore that will be set + */ + void setLore(List lore); + + /** + * Checks for the existence of any enchantments + * + * @return true if an enchantment exists on this meta + */ + boolean hasEnchants(); + + /** + * Checks for existence of the specified enchantment + * + * @param ench enchantment to check + * @return true if this enchantment exists for this meta + */ + boolean hasEnchant(Enchantment ench); + + /** + * Checks for the level of the specified enchantment + * + * @param ench enchantment to check + * @return The level that the specified enchantment has, or 0 if none + */ + int getEnchantLevel(Enchantment ench); + + /** + * This method gets a copy the enchantments in this ItemMeta + * + * @return An immutable copy of the enchantments + */ + Map getEnchants(); + + /** + * This method adds the specified enchantment to this item meta + * + * @param ench Enchantment to add + * @param level Level for the enchantment + * @param ignoreLevelRestriction this indicates the enchantment should be applied, ignoring the level limit + * @return true if the item meta changed as a result of this call, false otherwise + */ + boolean addEnchant(Enchantment ench, int level, boolean ignoreLevelRestriction); + + /** + * This method removes the specified enchantment from this item meta + * + * @param ench Enchantment to remove + * @return true if the item meta changed as a result of this call, false otherwise + */ + boolean removeEnchant(Enchantment ench); + + @SuppressWarnings("javadoc") + ItemMeta clone(); +} diff --git a/src/main/java/org/bukkit/inventory/meta/LeatherArmorMeta.java b/src/main/java/org/bukkit/inventory/meta/LeatherArmorMeta.java new file mode 100644 index 00000000..2ca6b6ec --- /dev/null +++ b/src/main/java/org/bukkit/inventory/meta/LeatherArmorMeta.java @@ -0,0 +1,26 @@ +package org.bukkit.inventory.meta; + +import org.bukkit.Color; +import org.bukkit.Material; + +/** + * Represents leather armor ({@link Material#LEATHER_BOOTS}, {@link Material#LEATHER_CHESTPLATE}, {@link Material#LEATHER_HELMET}, or {@link Material#LEATHER_LEGGINGS}) that can be colored. + */ +public interface LeatherArmorMeta extends ItemMeta { + + /** + * Gets the color of the armor + * + * @return the color of the armor, never null + */ + Color getColor(); + + /** + * Sets the color of the armor + * + * @param color the color to set, null makes it the default leather color + */ + void setColor(Color color); + + LeatherArmorMeta clone(); +} diff --git a/src/main/java/org/bukkit/inventory/meta/MapMeta.java b/src/main/java/org/bukkit/inventory/meta/MapMeta.java new file mode 100644 index 00000000..6ff13671 --- /dev/null +++ b/src/main/java/org/bukkit/inventory/meta/MapMeta.java @@ -0,0 +1,23 @@ +package org.bukkit.inventory.meta; + +/** + * Represents a map that can be scalable. + */ +public interface MapMeta extends ItemMeta { + + /** + * Checks to see if this map is scaling + * + * @return true if this map is scaling + */ + boolean isScaling(); + + /** + * Sets if this map is scaling or not + * + * @param value true to scale + */ + void setScaling(boolean value); + + MapMeta clone(); +} diff --git a/src/main/java/org/bukkit/inventory/meta/PotionMeta.java b/src/main/java/org/bukkit/inventory/meta/PotionMeta.java new file mode 100644 index 00000000..dad8fc30 --- /dev/null +++ b/src/main/java/org/bukkit/inventory/meta/PotionMeta.java @@ -0,0 +1,69 @@ +package org.bukkit.inventory.meta; + +import org.bukkit.Material; +import org.bukkit.potion.PotionEffect; +import org.bukkit.potion.PotionEffectType; + +import java.util.List; + +/** + * Represents a potion ({@link Material#POTION}) that can have custom effects. + */ +public interface PotionMeta extends ItemMeta { + + /** + * Checks for the presence of custom potion effects + * + * @return true if custom potion effects are applied + */ + boolean hasCustomEffects(); + + /** + * Gets an immutable list containing all custom potion effects applied to this potion + * + * @return the immutable list of custom potion effects + */ + List getCustomEffects(); + + /** + * Adds a custom potion effect to this potion + * + * @param effect the potion effect to add + * @param overwrite true if any existing effect of the same type should be overwritten + * @return true if the potion meta changed as a result of this call + */ + boolean addCustomEffect(PotionEffect effect, boolean overwrite); + + /** + * Removes a custom potion effect from this potion + * + * @param type the potion effect type to remove + * @return true if the potion meta changed as a result of this call + */ + boolean removeCustomEffect(PotionEffectType type); + + /** + * Checks for a specific custom potion effect type on this potion + * @param type the potion effect type to check for + * @return true if the potion has this effect + */ + boolean hasCustomEffect(PotionEffectType type); + + /** + * Moves a potion effect to the top of the potion effect list. + * This causes the client to display the potion effect in the potion's name. + * + * @param type the potion effect type to move + * @return true if the potion meta changed as a result of this call + */ + boolean setMainEffect(PotionEffectType type); + + /** + * Removes all custom potion effects from this potion + * + * @return true if the potion meta changed as a result of this call + */ + boolean clearCustomEffects(); + + PotionMeta clone(); +} diff --git a/src/main/java/org/bukkit/inventory/meta/Repairable.java b/src/main/java/org/bukkit/inventory/meta/Repairable.java new file mode 100644 index 00000000..c49844ed --- /dev/null +++ b/src/main/java/org/bukkit/inventory/meta/Repairable.java @@ -0,0 +1,31 @@ +package org.bukkit.inventory.meta; + +/** + * Represents an item that can be repaired at an anvil. + */ +public interface Repairable { + + /** + * Checks to see if this has a repair penalty + * + * @return true if this has a repair penalty + */ + boolean hasRepairCost(); + + /** + * Gets the repair penalty + * + * @return the repair penalty + */ + int getRepairCost(); + + /** + * Sets the repair penalty + * + * @param cost repair penalty + */ + void setRepairCost(int cost); + + @SuppressWarnings("javadoc") + Repairable clone(); +} diff --git a/src/main/java/org/bukkit/inventory/meta/SkullMeta.java b/src/main/java/org/bukkit/inventory/meta/SkullMeta.java new file mode 100644 index 00000000..42598576 --- /dev/null +++ b/src/main/java/org/bukkit/inventory/meta/SkullMeta.java @@ -0,0 +1,33 @@ +package org.bukkit.inventory.meta; + +import org.bukkit.Material; + +/** + * Represents a skull ({@link Material#SKULL_ITEM}) that can have an owner. + */ +public interface SkullMeta extends ItemMeta { + + /** + * Gets the owner of the skull + * + * @return the owner if the skull + */ + String getOwner(); + + /** + * Checks to see if the skull has an owner + * + * @return true if the skull has an owner + */ + boolean hasOwner(); + + /** + * Sets the owner of the skull + * + * @param owner the new owner of the skull + * @return true if the owner was successfully set + */ + boolean setOwner(String owner); + + SkullMeta clone(); +} diff --git a/src/main/java/org/bukkit/potion/PotionEffect.java b/src/main/java/org/bukkit/potion/PotionEffect.java index 6c738776..5182a074 100644 --- a/src/main/java/org/bukkit/potion/PotionEffect.java +++ b/src/main/java/org/bukkit/potion/PotionEffect.java @@ -1,24 +1,101 @@ package org.bukkit.potion; +import java.util.Map; +import java.util.NoSuchElementException; + import org.apache.commons.lang.Validate; +import org.bukkit.configuration.serialization.ConfigurationSerializable; +import org.bukkit.configuration.serialization.SerializableAs; import org.bukkit.entity.LivingEntity; +import com.google.common.collect.ImmutableMap; + /** * Represents a potion effect, that can be added to a {@link LivingEntity}. A * potion effect has a duration that it will last for, an amplifier that will * enhance its effects, and a {@link PotionEffectType}, that represents its * effect on an entity. */ -public class PotionEffect { +@SerializableAs("PotionEffect") +public class PotionEffect implements ConfigurationSerializable { + private static final String AMPLIFIER = "amplifier"; + private static final String DURATION = "duration"; + private static final String TYPE = "effect"; + private static final String AMBIENT = "ambient"; private final int amplifier; private final int duration; private final PotionEffectType type; + private final boolean ambient; - public PotionEffect(PotionEffectType type, int duration, int amplifier) { + /** + * Creates a potion effect. + * + * @param type effect type + * @param duration measured in ticks, see {@link PotionEffect#getDuration()} + * @param amplifier the amplifier, see {@link PotionEffect#getAmplifier()} + * @param ambient the ambient status, see {@link PotionEffect#isAmbient()} + */ + public PotionEffect(PotionEffectType type, int duration, int amplifier, boolean ambient) { Validate.notNull(type, "effect type cannot be null"); this.type = type; this.duration = duration; this.amplifier = amplifier; + this.ambient = ambient; + } + + /** + * Creates a potion affect. Assumes ambient is true. + * + * @param type Effect type + * @param duration measured in ticks + * @param amplifier the amplifier for the affect + * @see PotionEffect#PotionEffect(PotionEffectType, int, int, boolean) + */ + public PotionEffect(PotionEffectType type, int duration, int amplifier) { + this(type, duration, amplifier, true); + } + + /** + * Constructor for deserialization. + * + * @param map the map to deserialize from + */ + public PotionEffect(Map map) { + this(getEffectType(map), getInt(map, DURATION), getInt(map, AMPLIFIER), getBool(map, AMBIENT)); + } + + private static PotionEffectType getEffectType(Map map) { + int type = getInt(map, TYPE); + PotionEffectType effect = PotionEffectType.getById(type); + if (effect != null) { + return effect; + } + throw new NoSuchElementException(map + " does not contain " + TYPE); + } + + private static int getInt(Map map, Object key) { + Object num = map.get(key); + if (num instanceof Integer) { + return (Integer) num; + } + throw new NoSuchElementException(map + " does not contain " + key); + } + + private static boolean getBool(Map map, Object key) { + Object bool = map.get(key); + if (bool instanceof Boolean) { + return (Boolean) bool; + } + throw new NoSuchElementException(map + " does not contain " + key); + } + + public Map serialize() { + return ImmutableMap.of( + TYPE, type.getId(), + DURATION, duration, + AMPLIFIER, amplifier, + AMBIENT, ambient + ); } /** @@ -38,18 +115,11 @@ public class PotionEffect { if (this == obj) { return true; } - if (obj == null || getClass() != obj.getClass()) { - return false; - } - PotionEffect other = (PotionEffect) obj; - if (type == null) { - if (other.type != null) { - return false; - } - } else if (!type.equals(other.type)) { + if (!(obj instanceof PotionEffect)) { return false; } - return true; + PotionEffect that = (PotionEffect) obj; + return this.type.equals(that.type) && this.ambient == that.ambient && this.amplifier == that.amplifier && this.duration == that.duration; } /** @@ -80,8 +150,27 @@ public class PotionEffect { return type; } + /** + * Makes potion effect produce more, translucent, particles. + * + * @return if this effect is ambient + */ + public boolean isAmbient() { + return ambient; + } + @Override public int hashCode() { - return 31 + ((type == null) ? 0 : type.hashCode()); - }; + int hash = 1; + hash = hash * 31 + type.hashCode(); + hash = hash * 31 + amplifier; + hash = hash * 31 + duration; + hash ^= 0x22222222 >> (ambient ? 1 : -1); + return hash; + } + + @Override + public String toString() { + return type.getName() + (ambient ? ":(" : ":") + duration + "t-x" + amplifier + (ambient ? ")" : ""); + } } diff --git a/src/main/java/org/bukkit/potion/PotionEffectType.java b/src/main/java/org/bukkit/potion/PotionEffectType.java index 626987a9..b6e8a0de 100644 --- a/src/main/java/org/bukkit/potion/PotionEffectType.java +++ b/src/main/java/org/bukkit/potion/PotionEffectType.java @@ -115,6 +115,14 @@ public abstract class PotionEffectType { this.id = id; } + /** + * Creates a PotionEffect from this PotionEffectType, applying duration modifiers and checks. + * + * @see PotionBrewer#createEffect(PotionEffectType, int, int) + * @param duration time in ticks + * @param amplifier the effect's amplifier + * @return a resulting potion effect + */ public PotionEffect createEffect(int duration, int amplifier) { return Potion.getBrewer().createEffect(this, duration, amplifier); } diff --git a/src/test/java/org/bukkit/ColorTest.java b/src/test/java/org/bukkit/ColorTest.java new file mode 100644 index 00000000..8d9557af --- /dev/null +++ b/src/test/java/org/bukkit/ColorTest.java @@ -0,0 +1,365 @@ +package org.bukkit; + +import static org.hamcrest.Matchers.*; +import static org.junit.Assert.*; + +import org.bukkit.configuration.file.YamlConfiguration; +import org.junit.Test; + +@SuppressWarnings("javadoc") +public class ColorTest { + static class TestColor { + static int id = 0; + final String name; + final int rgb; + final int bgr; + final int r; + final int g; + final int b; + + TestColor(int rgb, int bgr, int r, int g, int b) { + this.rgb = rgb; + this.bgr = bgr; + this.r = r; + this.g = g; + this.b = b; + this.name = id + ":" + Integer.toHexString(rgb).toUpperCase() + "_" + Integer.toHexString(bgr).toUpperCase() + "-r" + Integer.toHexString(r).toUpperCase() + "-g" + Integer.toHexString(g).toUpperCase() + "-b" + Integer.toHexString(b).toUpperCase(); + } + } + + static TestColor[] examples = new TestColor[] { + /* 0xRRGGBB, 0xBBGGRR, 0xRR, 0xGG, 0xBB */ + new TestColor(0xFFFFFF, 0xFFFFFF, 0xFF, 0xFF, 0xFF), + new TestColor(0xFFFFAA, 0xAAFFFF, 0xFF, 0xFF, 0xAA), + new TestColor(0xFF00FF, 0xFF00FF, 0xFF, 0x00, 0xFF), + new TestColor(0x67FF22, 0x22FF67, 0x67, 0xFF, 0x22), + new TestColor(0x000000, 0x000000, 0x00, 0x00, 0x00) + }; + + @Test + public void testSerialization() throws Throwable { + for (TestColor testColor : examples) { + Color base = Color.fromRGB(testColor.rgb); + + YamlConfiguration toSerialize = new YamlConfiguration(); + toSerialize.set("color", base); + String serialized = toSerialize.saveToString(); + + YamlConfiguration deserialized = new YamlConfiguration(); + deserialized.loadFromString(serialized); + + assertThat(testColor.name + " on " + serialized, base, is(deserialized.getColor("color"))); + } + } + + // Equality tests + @Test + public void testEqualities() { + for (TestColor testColor : examples) { + Color fromRGB = Color.fromRGB(testColor.rgb); + Color fromBGR = Color.fromBGR(testColor.bgr); + Color fromRGBs = Color.fromRGB(testColor.r, testColor.g, testColor.b); + Color fromBGRs = Color.fromBGR(testColor.b, testColor.g, testColor.r); + + assertThat(testColor.name, fromRGB, is(fromRGBs)); + assertThat(testColor.name, fromRGB, is(fromBGR)); + assertThat(testColor.name, fromRGB, is(fromBGRs)); + assertThat(testColor.name, fromRGBs, is(fromBGR)); + assertThat(testColor.name, fromRGBs, is(fromBGRs)); + assertThat(testColor.name, fromBGR, is(fromBGRs)); + } + } + + @Test + public void testInequalities() { + for (int i = 1; i < examples.length; i++) { + TestColor testFrom = examples[i]; + Color from = Color.fromRGB(testFrom.rgb); + for (int j = i - 1; j >= 0; j--) { + TestColor testTo = examples[j]; + Color to = Color.fromRGB(testTo.rgb); + String name = testFrom.name + " to " + testTo.name; + assertThat(name, from, is(not(to))); + + Color transform = from.setRed(testTo.r).setBlue(testTo.b).setGreen(testTo.g); + assertThat(name, transform, is(not(sameInstance(from)))); + assertThat(name, transform, is(to)); + } + } + } + + // RGB tests + @Test + public void testRGB() { + for (TestColor testColor : examples) { + assertThat(testColor.name, Color.fromRGB(testColor.rgb).asRGB(), is(testColor.rgb)); + assertThat(testColor.name, Color.fromBGR(testColor.bgr).asRGB(), is(testColor.rgb)); + assertThat(testColor.name, Color.fromRGB(testColor.r, testColor.g, testColor.b).asRGB(), is(testColor.rgb)); + assertThat(testColor.name, Color.fromBGR(testColor.b, testColor.g, testColor.r).asRGB(), is(testColor.rgb)); + } + } + + @Test(expected=IllegalArgumentException.class) + public void testInvalidRGB1() { + Color.fromRGB(0x01000000); + } + + @Test(expected=IllegalArgumentException.class) + public void testInvalidRGB2() { + Color.fromRGB(Integer.MIN_VALUE); + } + + @Test(expected=IllegalArgumentException.class) + public void testInvalidRGB3() { + Color.fromRGB(Integer.MAX_VALUE); + } + + @Test(expected=IllegalArgumentException.class) + public void testInvalidRGB4() { + Color.fromRGB(-1); + } + + // BGR tests + @Test + public void testBGR() { + for (TestColor testColor : examples) { + assertThat(testColor.name, Color.fromRGB(testColor.rgb).asBGR(), is(testColor.bgr)); + assertThat(testColor.name, Color.fromBGR(testColor.bgr).asBGR(), is(testColor.bgr)); + assertThat(testColor.name, Color.fromRGB(testColor.r, testColor.g, testColor.b).asBGR(), is(testColor.bgr)); + assertThat(testColor.name, Color.fromBGR(testColor.b, testColor.g, testColor.r).asBGR(), is(testColor.bgr)); + } + } + + @Test(expected=IllegalArgumentException.class) + public void testInvalidBGR1() { + Color.fromBGR(0x01000000); + } + + @Test(expected=IllegalArgumentException.class) + public void testInvalidBGR2() { + Color.fromBGR(Integer.MIN_VALUE); + } + + @Test(expected=IllegalArgumentException.class) + public void testInvalidBGR3() { + Color.fromBGR(Integer.MAX_VALUE); + } + + @Test(expected=IllegalArgumentException.class) + public void testInvalidBGR4() { + Color.fromBGR(-1); + } + + // Red tests + @Test + public void testRed() { + for (TestColor testColor : examples) { + assertThat(testColor.name, Color.fromRGB(testColor.rgb).getRed(), is(testColor.r)); + assertThat(testColor.name, Color.fromBGR(testColor.bgr).getRed(), is(testColor.r)); + assertThat(testColor.name, Color.fromRGB(testColor.r, testColor.g, testColor.b).getRed(), is(testColor.r)); + assertThat(testColor.name, Color.fromBGR(testColor.b, testColor.g, testColor.r).getRed(), is(testColor.r)); + } + } + + @Test(expected=IllegalArgumentException.class) + public void testInvalidR01() { + Color.fromRGB(-1, 0x00, 0x00); + } + + @Test(expected=IllegalArgumentException.class) + public void testInvalidR02() { + Color.fromRGB(Integer.MAX_VALUE, 0x00, 0x00); + } + + @Test(expected=IllegalArgumentException.class) + public void testInvalidR03() { + Color.fromRGB(Integer.MIN_VALUE, 0x00, 0x00); + } + + @Test(expected=IllegalArgumentException.class) + public void testInvalidR04() { + Color.fromRGB(0x100, 0x00, 0x00); + } + + @Test(expected=IllegalArgumentException.class) + public void testInvalidR05() { + Color.fromBGR(0x00, 0x00, -1); + } + + @Test(expected=IllegalArgumentException.class) + public void testInvalidR06() { + Color.fromBGR(0x00, 0x00, Integer.MAX_VALUE); + } + + @Test(expected=IllegalArgumentException.class) + public void testInvalidR07() { + Color.fromBGR(0x00, 0x00, Integer.MIN_VALUE); + } + + @Test(expected=IllegalArgumentException.class) + public void testInvalidR08() { + Color.fromBGR(0x00, 0x00, 0x100); + } + + @Test(expected=IllegalArgumentException.class) + public void testInvalidR09() { + Color.WHITE.setRed(-1); + } + + @Test(expected=IllegalArgumentException.class) + public void testInvalidR10() { + Color.WHITE.setRed(Integer.MAX_VALUE); + } + + @Test(expected=IllegalArgumentException.class) + public void testInvalidR11() { + Color.WHITE.setRed(Integer.MIN_VALUE); + } + + @Test(expected=IllegalArgumentException.class) + public void testInvalidR12() { + Color.WHITE.setRed(0x100); + } + + // Blue tests + @Test + public void testBlue() { + for (TestColor testColor : examples) { + assertThat(testColor.name, Color.fromRGB(testColor.rgb).getBlue(), is(testColor.b)); + assertThat(testColor.name, Color.fromBGR(testColor.bgr).getBlue(), is(testColor.b)); + assertThat(testColor.name, Color.fromRGB(testColor.r, testColor.g, testColor.b).getBlue(), is(testColor.b)); + assertThat(testColor.name, Color.fromBGR(testColor.b, testColor.g, testColor.r).getBlue(), is(testColor.b)); + } + } + + @Test(expected=IllegalArgumentException.class) + public void testInvalidB01() { + Color.fromRGB(0x00, 0x00, -1); + } + + @Test(expected=IllegalArgumentException.class) + public void testInvalidB02() { + Color.fromRGB(0x00, 0x00, Integer.MAX_VALUE); + } + + @Test(expected=IllegalArgumentException.class) + public void testInvalidB03() { + Color.fromRGB(0x00, 0x00, Integer.MIN_VALUE); + } + + @Test(expected=IllegalArgumentException.class) + public void testInvalidB04() { + Color.fromRGB(0x00, 0x00, 0x100); + } + + @Test(expected=IllegalArgumentException.class) + public void testInvalidB05() { + Color.fromBGR(-1, 0x00, 0x00); + } + + @Test(expected=IllegalArgumentException.class) + public void testInvalidB06() { + Color.fromBGR(Integer.MAX_VALUE, 0x00, 0x00); + } + + @Test(expected=IllegalArgumentException.class) + public void testInvalidB07() { + Color.fromBGR(Integer.MIN_VALUE, 0x00, 0x00); + } + + @Test(expected=IllegalArgumentException.class) + public void testInvalidB08() { + Color.fromBGR(0x100, 0x00, 0x00); + } + + @Test(expected=IllegalArgumentException.class) + public void testInvalidB09() { + Color.WHITE.setBlue(-1); + } + + @Test(expected=IllegalArgumentException.class) + public void testInvalidB10() { + Color.WHITE.setBlue(Integer.MAX_VALUE); + } + + @Test(expected=IllegalArgumentException.class) + public void testInvalidB11() { + Color.WHITE.setBlue(Integer.MIN_VALUE); + } + + @Test(expected=IllegalArgumentException.class) + public void testInvalidB12() { + Color.WHITE.setBlue(0x100); + } + + // Green tests + @Test + public void testGreen() { + for (TestColor testColor : examples) { + assertThat(testColor.name, Color.fromRGB(testColor.rgb).getGreen(), is(testColor.g)); + assertThat(testColor.name, Color.fromBGR(testColor.bgr).getGreen(), is(testColor.g)); + assertThat(testColor.name, Color.fromRGB(testColor.r, testColor.g, testColor.b).getGreen(), is(testColor.g)); + assertThat(testColor.name, Color.fromBGR(testColor.b, testColor.g, testColor.r).getGreen(), is(testColor.g)); + } + } + + @Test(expected=IllegalArgumentException.class) + public void testInvalidG01() { + Color.fromRGB(0x00, -1, 0x00); + } + + @Test(expected=IllegalArgumentException.class) + public void testInvalidG02() { + Color.fromRGB(0x00, Integer.MAX_VALUE, 0x00); + } + + @Test(expected=IllegalArgumentException.class) + public void testInvalidG03() { + Color.fromRGB(0x00, Integer.MIN_VALUE, 0x00); + } + + @Test(expected=IllegalArgumentException.class) + public void testInvalidG04() { + Color.fromRGB(0x00, 0x100, 0x00); + } + + @Test(expected=IllegalArgumentException.class) + public void testInvalidG05() { + Color.fromBGR(0x00, -1, 0x00); + } + + @Test(expected=IllegalArgumentException.class) + public void testInvalidG06() { + Color.fromBGR(0x00, Integer.MAX_VALUE, 0x00); + } + + @Test(expected=IllegalArgumentException.class) + public void testInvalidG07() { + Color.fromBGR(0x00, Integer.MIN_VALUE, 0x00); + } + + @Test(expected=IllegalArgumentException.class) + public void testInvalidG08() { + Color.fromBGR(0x00, 0x100, 0x00); + } + + @Test(expected=IllegalArgumentException.class) + public void testInvalidG09() { + Color.WHITE.setGreen(-1); + } + + @Test(expected=IllegalArgumentException.class) + public void testInvalidG10() { + Color.WHITE.setGreen(Integer.MAX_VALUE); + } + + @Test(expected=IllegalArgumentException.class) + public void testInvalidG11() { + Color.WHITE.setGreen(Integer.MIN_VALUE); + } + + @Test(expected=IllegalArgumentException.class) + public void testInvalidG12() { + Color.WHITE.setGreen(0x100); + } +} diff --git a/src/test/java/org/bukkit/configuration/ConfigurationSectionTest.java b/src/test/java/org/bukkit/configuration/ConfigurationSectionTest.java index 4e3478fd..6dab4779 100644 --- a/src/test/java/org/bukkit/configuration/ConfigurationSectionTest.java +++ b/src/test/java/org/bukkit/configuration/ConfigurationSectionTest.java @@ -403,7 +403,7 @@ public abstract class ConfigurationSectionTest { map.put("two", "two"); map.put("three", 3.14); - List value = Arrays.asList((Object) "One", "Two", "Three", 4, "5", 6.0, true, "false", map); + List value = Arrays.asList("One", "Two", "Three", 4, "5", 6.0, true, "false", map); section.set(key, value); diff --git a/src/test/java/org/bukkit/configuration/file/TestEnchantment.java b/src/test/java/org/bukkit/configuration/file/TestEnchantment.java deleted file mode 100644 index 93569721..00000000 --- a/src/test/java/org/bukkit/configuration/file/TestEnchantment.java +++ /dev/null @@ -1,55 +0,0 @@ -package org.bukkit.configuration.file; - -import org.bukkit.enchantments.Enchantment; -import org.bukkit.enchantments.EnchantmentTarget; -import org.bukkit.inventory.ItemStack; - -public class TestEnchantment extends Enchantment { - - public static void registerEnchantments() { - Enchantment.registerEnchantment(new TestEnchantment(0, "DUMMY_0")); - Enchantment.registerEnchantment(new TestEnchantment(1, "DUMMY_1")); - Enchantment.registerEnchantment(new TestEnchantment(2, "DUMMY_2")); - Enchantment.registerEnchantment(new TestEnchantment(3, "DUMMY_3")); - Enchantment.registerEnchantment(new TestEnchantment(4, "DUMMY_4")); - Enchantment.registerEnchantment(new TestEnchantment(5, "DUMMY_5")); - } - - private final String name; - - private TestEnchantment(final int id, final String name) { - super(id); - this.name = name; - } - - @Override - public String getName() { - return name; - } - - @Override - public int getMaxLevel() { - return 5; - } - - @Override - public int getStartLevel() { - return 1; - } - - @Override - public EnchantmentTarget getItemTarget() { - throw new UnsupportedOperationException("Not supported yet."); - } - - @Override - public boolean canEnchantItem(ItemStack item) { - return true; - } - - @Override - public boolean conflictsWith(Enchantment other) { - return false; - } - -} diff --git a/src/test/java/org/bukkit/configuration/file/YamlConfigurationTest.java b/src/test/java/org/bukkit/configuration/file/YamlConfigurationTest.java index 7c4d7653..aa83af32 100644 --- a/src/test/java/org/bukkit/configuration/file/YamlConfigurationTest.java +++ b/src/test/java/org/bukkit/configuration/file/YamlConfigurationTest.java @@ -1,19 +1,9 @@ package org.bukkit.configuration.file; -import java.util.ArrayList; -import java.util.List; - -import org.bukkit.configuration.InvalidConfigurationException; -import org.bukkit.enchantments.Enchantment; -import org.bukkit.inventory.ItemStack; - import org.junit.Test; import static org.junit.Assert.*; public class YamlConfigurationTest extends FileConfigurationTest { - static { - TestEnchantment.registerEnchantments(); - } @Override public YamlConfiguration getConfig() { @@ -63,47 +53,4 @@ public class YamlConfigurationTest extends FileConfigurationTest { assertEquals(expected, result); } - - @Test - public void testSaveRestoreCompositeList() throws InvalidConfigurationException { - YamlConfiguration out = getConfig(); - - List stacks = new ArrayList(); - stacks.add(new ItemStack(1)); - stacks.add(new ItemStack(2)); - stacks.add(new ItemStack(3)); - stacks.add(new ItemStack(4, 17)); - stacks.add(new ItemStack(5, 63)); - stacks.add(new ItemStack(6, 1, (short) 1)); - stacks.add(new ItemStack(18, 32, (short) 2)); - - ItemStack item7 = new ItemStack(256); - item7.addEnchantment(Enchantment.getById(1), 1); - stacks.add(item7); - - ItemStack item8 = new ItemStack(257); - item8.addEnchantment(Enchantment.getById(2), 2); - item8.addEnchantment(Enchantment.getById(3), 1); - item8.addEnchantment(Enchantment.getById(4), 5); - item8.addEnchantment(Enchantment.getById(5), 4); - stacks.add(item8); - - out.set("composite-list.abc.def", stacks); - String yaml = out.saveToString(); - - YamlConfiguration in = new YamlConfiguration(); - in.loadFromString(yaml); - List raw = in.getList("composite-list.abc.def"); - - assertEquals(stacks.size(), raw.size()); - assertEquals(stacks.get(0), raw.get(0)); - assertEquals(stacks.get(1), raw.get(1)); - assertEquals(stacks.get(2), raw.get(2)); - assertEquals(stacks.get(3), raw.get(3)); - assertEquals(stacks.get(4), raw.get(4)); - assertEquals(stacks.get(5), raw.get(5)); - assertEquals(stacks.get(6), raw.get(6)); - assertEquals(stacks.get(7), raw.get(7)); - assertEquals(stacks.get(8), raw.get(8)); - } } -- cgit v1.2.3