Finalize factory generated Keys to use lazy loading tostring values.

Signed-off-by: Gabriel Harris-Rouquette <gabizou@me.com>
SpongePowered · Sep 16, 2016 · b8c1838 · b8c1838
1 parent 5f31153
commit b8c1838
Show file tree

Hide file tree

Showing 4 changed files with 27 additions and 18 deletions.
diff --git a/src/main/java/org/spongepowered/api/data/key/KeyFactory.java b/src/main/java/org/spongepowered/api/data/key/KeyFactory.java
@@ -133,6 +133,7 @@ public String toString() {
     public static <E> Key<ListValue<E>> makeListKey(final TypeToken<? extends List<E>> elementToken, final TypeToken<ListValue<E>> valueToken,
             final DataQuery query, final String id, final String name) {
         return new Key<ListValue<E>>() {
+            @Nullable private String string;
 
             @Override
             public String getId() {
@@ -169,7 +170,10 @@ public int hashCode() {
 
             @Override
             public String toString() {
-                return "Key{Value:" + "ListValue<" + elementToken.getSimpleName() + ">, Query: " + query.toString() + "}";
+                if (this.string == null) {
+                    this.string = "Key{Value:ListValue<" + elementToken.toString() + ">, Query: " + query.toString() + "}";
+                }
+                return this.string;
             }
         };
     }
@@ -187,6 +191,8 @@ public static <E> Key<SetValue<E>> makeSetKey(final TypeToken<? extends Set<E>>
             final DataQuery query, final String id, final String name) {
         return new Key<SetValue<E>>() {
 
+            @Nullable private String string;
+
             @Override
             public String getId() {
                 return id;
@@ -222,7 +228,10 @@ public int hashCode() {
 
             @Override
             public String toString() {
-                return "Key{Value:" + "SetValue<" + elementToken.getSimpleName() + ">, Query: " + query.toString() + "}";
+                if (this.string == null) {
+                    this.string = "Key{Value:" + "SetValue<" + elementToken.toString() + ">, Query: " + query.toString() + "}";
+                }
+                return this.string;
             }
         };
     }
@@ -242,6 +251,8 @@ public String toString() {
     public static <K, V> Key<MapValue<K, V>> makeMapKey(final TypeToken<Map<K, V>> elementToken, final TypeToken<MapValue<K, V>> valueToken, final DataQuery query, final String id, final String name) {
         return new Key<MapValue<K, V>>() {
 
+            @Nullable private String string;
+
             @Override
             public String getId() {
                 return id;
@@ -278,8 +289,11 @@ public int hashCode() {
 
             @Override
             public String toString() {
-                return "Key{Value:" + "MapValue<" + keyToken.getSimpleName() + "," + valueToken.getSimpleName() + ">, Query: " + query.toString()
-                       + "}";
+                if (this.string == null) {
+                    this.string = "Key{Value:" + "MapValue<" + elementToken.toString() + ","
+                                  + valueToken.toString() + ">, Query: " + query.toString() + "}";
+                }
+                return this.string;
             }
         };
     }
@@ -299,6 +313,7 @@ public static <E> Key<OptionalValue<E>> makeOptionalKey(final TypeToken<Optional
             final DataQuery query, final String id,
             final String name) {
         return new Key<OptionalValue<E>>() {
+            @Nullable private String string;
 
             @Override
             public String getId() {
@@ -334,7 +349,10 @@ public int hashCode() {
 
             @Override
             public String toString() {
-                return "Key{Value:" + "OptionalValue<" + elementToken.getSimpleName() + ">, Query: " + query.toString() + "}";
+                if (this.string == null) {
+                    this.string = "Key{Value:" + "OptionalValue<" + elementToken.toString() + ">, Query: " + query.toString() + "}";
+                }
+                return this.string;
             }
         };
     }

diff --git a/...org/spongepowered/api/data/manipulator/immutable/common/AbstractImmutableBooleanData.java b/...org/spongepowered/api/data/manipulator/immutable/common/AbstractImmutableBooleanData.java
@@ -57,11 +57,4 @@ protected final ImmutableValue<Boolean> getValueGetter() {
         return this.immutableValue;
     }
 
-    @Override
-    public int compareTo(I o) {
-        return ComparisonChain.start()
-            .compare(o.get(this.usedKey).get(), this.getValue())
-            .result();
-    }
-
 }
diff --git a/...ongepowered/api/data/manipulator/immutable/common/AbstractImmutableSingleCatalogData.java b/...ongepowered/api/data/manipulator/immutable/common/AbstractImmutableSingleCatalogData.java
@@ -60,11 +60,6 @@ protected AbstractImmutableSingleCatalogData(E value, E defaultValue, Key<? exte
 
     }
 
-    @Override
-    public int compareTo(I o) {
-        return o.get(this.usedKey).get().getId().compareToIgnoreCase(this.getValue().getId());
-    }
-
     @Override
     protected ImmutableValue<E> getValueGetter() {
         return this.immutableValue;

diff --git a/src/main/java/org/spongepowered/api/item/inventory/ItemStack.java b/src/main/java/org/spongepowered/api/item/inventory/ItemStack.java
@@ -141,7 +141,10 @@ static ItemStack of(ItemType itemType, int quantity) {
     ItemStack copy();
 
     interface Builder extends DataBuilder<ItemStack> {
-
+
+        @Override
+        Builder from(ItemStack value);
+
         /**
          * Sets the {@link ItemType} of the item stack.
          *