Recipe Refactor by Maxqia · Pull Request #1419 · SpongePowered/SpongeAPI

Maxqia · 2016-11-19T22:55:23Z

API | Common
Test Code : https://gist.github.com/Maxqia/5579f49f9cfd3f4b08d6912d899d259d

liach · 2016-11-19T23:06:07Z

src/main/java/org/spongepowered/api/item/recipe/SmeltingRegistry.java

+     * @param output the {@link ItemStack} the recipe gives
+     * @param exp the amount of exp the recipe gives
+     */
+    void addSmeltingRecipe(ItemStack input, ItemStack output, float exp);


Use ItemStackSnapshot.

kashike · 2016-11-19T23:06:00Z

src/main/java/org/spongepowered/api/item/recipe/SmeltingRegistry.java

@@ -0,0 +1,37 @@
+package org.spongepowered.api.item.recipe;


Missing licence header.

Cybermaxke · 2016-11-19T23:47:36Z

I would like to see a FurnaceRecipe and a Builder similar to the recipe system for crafting grids. I made the following draft to give a general idea, you can also checkout the package: https://github.com/SpongePowered/SpongeAPI/tree/bleeding/src/main/java/org/spongepowered/api/item/recipe

public interface SmeltingRecipeRegistry {

    void register(SmeltingRecipe recipe);

    void remove(SmeltingRecipe recipe);

    Set<SmeltingRecipe> getRecipes();

    Optional<SmeltingRecipe> match(ItemStack itemStack);

   // Move these to their own registry???

    void registerFuel(Fuel fuel);

    void removeFuel(Fuel fuel);

    Set<Fuel> getFuels();

    Optional<Fuel> matchFuel(ItemStack itemStack);
}

public interface SmeltingRecipe {

    ItemStackSnapshot getInput();

    ItemStackSnapshot getOutput();

    double getExperience();

    int getDuration(); // Duration may also be nice to have

    public interface Builder {

         // Builder stuff
    }
}

And maybe also custom fuel?

public interface Fuel {

     ItemStackSnapshot getInput();

    // The result item when the fuel is consumed, like a bucket from a lava bucket
     Optional<ItemStackSnapshot> getReturnItem();

     int getDuration();

     public interface Builder {

         // Builder stuff
    }
}

ST-DDT · 2016-11-19T23:49:19Z

src/main/java/org/spongepowered/api/item/recipe/SmeltingRegistry.java

+public interface SmeltingRegistry {
+    /**
+     * Adds a smelting recipe to this registry.
+     * @param input the input of the recipe


Needs an empty javadoc line before this line. Same goes for all methods in this class.
Also the param descriptions should start with a capital letter after the param name.

ST-DDT · 2016-11-19T23:53:20Z

src/main/java/org/spongepowered/api/item/recipe/SmeltingRegistry.java

+     * @return the experience of the {@link ItemStack}
+     */
+    float getExperience(ItemStack stack);
+}


Please add an empty line before this line.

ST-DDT · 2016-11-19T23:56:13Z

src/main/java/org/spongepowered/api/item/recipe/SmeltingRegistry.java

+     * @param the {@link ItemStack}
+     * @return the {@link ItemStack} it makes
+     */
+    ItemStack getMapping(ItemStack stack);


This should return Optionals for materials that are not registered/smeltable.

ST-DDT · 2016-11-19T23:56:52Z

src/main/java/org/spongepowered/api/item/recipe/SmeltingRegistry.java

+ * A RecipeRegistry holds all registered smelting recipes for a given game.
+ */
+public interface SmeltingRegistry {
+    /**


Missing newline before this line.

Maxqia · 2016-11-20T00:42:10Z

@Cybermaxke I was going to do that, until I realized that it would just be alot of = statements ...

liach · 2016-11-20T00:53:18Z

@Maxqia What are a lot of = statements? The suggested one is way more logical.

liach · 2016-11-20T00:57:13Z

@Cybermaxke Minecraft Forge manages fuel registry with a list of IFuelHandlers which returns the maximum burn time for a specific item stack, calling all registered handlers each time. Your Fuel interface probably cannot represent that.

Maxqia · 2016-11-20T00:59:55Z

eg.
SpongeSmeltingRecipe :

setInput(ItemStack input) {
this.input = input;
}
build() {
return new SpongeSmeltingStack(input,output,exp);
}

SpongeSmeltingStack :

ItemStack input,output;
float exp;
SpongeSmeltingStack(ItemStack input, ItemStack output, float exp) {
this.input = input; etc...
}

Maxqia · 2016-11-20T01:03:10Z

Also SmeltingRegistry.register(SmeltingRecipe.builder().setInput(input).setOutput(output).setExp(exp).build()) -> SmeltingRegistry.addSmeltingRecipe(input,output,exp)

kashike · 2016-11-20T04:01:54Z

src/main/java/org/spongepowered/api/item/recipe/SmeltingRegistry.java

+     * @param stack the {@link ItemStack}
+     * @return the {@link ItemStack} it makes
+     */
+    ItemStack getMapping(ItemStack stack);


This should return Optional<ItemStack>.

Oops, forgot about that ...

Cybermaxke · 2016-11-20T12:18:16Z

@Maxqia The sponge environment is built around Builders, this should't be an exception. You can always add helper methods to the SmeltingRecipe to simplify the creation.

@liach The implementation should be possible with the exception of the getFuels method, it would be a bit tricky though.

gabizou

Some discussion is necessary on the use of ItemStacks instead of ItemStackSnapshots. Since there's a big difference between the two (recipes in the underlying implementation aren't meant to change, but seeing as there's no immutable implementation aside from Sponge's, that's a different issue). To the API, we should be marking the objects as immutable as much as possible, when necessary.

gabizou · 2016-12-08T07:23:36Z

src/main/java/org/spongepowered/api/GameRegistry.java

-     * @return The recipe registry
+     * @return The smelting registry
     */
-    RecipeRegistry getRecipeRegistry();


This needs to be left in with @Deprecated to just redirect to the CraftingRegistry.

gabizou · 2016-12-08T07:24:11Z

src/main/java/org/spongepowered/api/item/recipe/Recipe.java

-     * @return The resultant list of item types
+     * @return the results of this {@link Recipe}
     */
-    List<ItemType> getResultTypes();


Likewise with the previous deprecated method, this should be left in with @Deprecated to use the getResults() list.

gabizou · 2016-12-08T07:24:40Z

src/main/java/org/spongepowered/api/item/recipe/Recipe.java

-     * @param grid The ItemGrid to check for validity
-     * @return True if the given input matches this recipe's requirements
-     */
-    boolean isValid(GridInventory grid);


Should be left in as deprecated, potentially default implemented to return true? I'm not sure about it, but try to have as minimal of a breakage as possible.

I mean there's nothing to break... it was always broken

@Maxqia There's a difference between binary compatibility and something being inherently broken. This is a binary breakage, anything relying on these methods (while they should be deprecated) would ultimately fail to work between API builds. I don't see the reason why this wouldn't be possible to retain for the sake of deprecation and uselessness (you can even print out a pretty printed log if you want)

gabizou · 2016-12-08T07:24:58Z

src/main/java/org/spongepowered/api/item/recipe/RecipeRegistry.java

     * @return All registered recipes
     */
    Set<Recipe> getRecipes();
-


Unnecessary whitespace removal.

gabizou · 2016-12-08T07:25:27Z

src/main/java/org/spongepowered/api/item/recipe/RecipeRegistry.java

+     *     type of recipe for this registry
     */
-    void register(Recipe recipe);
+    void register(Recipe recipe) throws IllegalArgumentException;


This is screaming about how it should use generics instead of just a raw Recipe type.

Was just about to say that to myself :P

gabizou · 2016-12-08T07:25:52Z

src/main/java/org/spongepowered/api/item/recipe/SmeltingRecipe.java

+     *
+     * @return the experience given when this recipe is made
+     */
+    float getExperience();


Never use float, you gain nothing from it. Use double.

gabizou · 2016-12-08T07:26:19Z

src/main/java/org/spongepowered/api/item/recipe/SmeltingRecipe.java

+     *
+     * @return item needed to make this recipe
+     */
+    ItemStack getIngredient();


I'm curious why you're using an ItemStack instead of an ItemStackSnapshot.

gabizou · 2016-12-08T07:26:28Z

src/main/java/org/spongepowered/api/item/recipe/SmeltingRecipe.java

+     *
+     * @return the result of this {@link Recipe}
+     */
+    ItemStack getResult();


Likewise here.

gabizou · 2016-12-08T07:27:32Z

src/main/java/org/spongepowered/api/item/recipe/SmeltingRegistry.java

+     * @param stack the {@link ItemStack}
+     * @return the experience of the {@link ItemStack}
+     */
+    Optional<Float> getExperience(ItemStack stack);


Should use Optional<Double> or if you want, java.util.OptionalDouble.

gabizou · 2016-12-08T07:27:52Z

src/main/java/org/spongepowered/api/item/recipe/SmeltingRegistry.java

+     * @return A ItemStack or {@link Optional#empty()} if the given
+     *          ItemStack does not match any recipes.
+     */
+    Optional<ItemStack> getResult(ItemStack stack);


Likewise, here, Optional<ItemStackSnapshot> instead.

Cybermaxke · 2016-12-11T14:56:41Z

src/main/java/org/spongepowered/api/item/recipe/SmeltingRecipe.java

+         * @param ingredient The ingredient
+         * @return fluent interface
+         */
+        Builder setIngredient(ItemStack ingredient);


You don't have to prefix the builder methods with set, just use for example ingredient.

You should not prefix builder methods with set.

liach · 2016-12-11T16:20:30Z

src/main/java/org/spongepowered/api/item/recipe/SmeltingRecipe.java

+     */
+    ItemStack getResult();
+
+    default List<ItemStack> getResults() {


@Override?

liach · 2016-12-11T16:21:38Z

src/main/java/org/spongepowered/api/item/recipe/SmeltingRecipe.java

+
+import java.util.List;
+
+public interface SmeltingRecipe extends Recipe {


ST-DDT · 2016-12-11T21:13:54Z

src/main/java/org/spongepowered/api/item/recipe/SmeltingRecipe.java

+         * Sets the ingredient for the requirements of this {@link SmeltingRecipe}.
+         *
+         * @param ingredient The ingredient
+         * @return fluent interface


Maybe change this to @return This instance for chaining (see other builders)

Oh, I just copied that part of the comment from the other recipe builders, I'll change them too while I'm at it

liach · 2016-12-12T02:59:31Z

src/main/java/org/spongepowered/api/item/recipe/smelting/SmeltingRegistry.java

+     * @param stack the {@link ItemStack}
+     * @return the experience of the {@link ItemStack}
+     */
+    Optional<Double> getExperience(ItemStack stack);


Use java.util.OptionalDouble

No, it doesn't have a bunch of functions that Optional has ...

liach · 2016-12-12T02:59:37Z

src/main/java/org/spongepowered/api/item/recipe/smelting/SmeltingRegistry.java

+     *          ItemStack does not match any recipes.
+     */
+    Optional<ItemStack> getResult(ItemStack stack);
+}


liach · 2016-12-12T02:59:52Z

src/main/java/org/spongepowered/api/item/recipe/smelting/SmeltingRecipe.java

+         * @return A new {@link SmeltingRecipe}
+         */
+        SmeltingRecipe build();
+    }


liach · 2016-12-12T02:59:59Z

src/main/java/org/spongepowered/api/item/recipe/smelting/SmeltingRecipe.java

+         * @param exp the experience
+         * @return This builder, for chaining
+         */
+        Builder experience(float exp);


accept double

missed that, will do

liach · 2016-12-12T03:00:27Z

src/main/java/org/spongepowered/api/item/recipe/smelting/SmeltingRecipe.java

+     *
+     * @return the result of this {@link Recipe}
+     */
+    ItemStack getResult();


ItemStackSnapshot

liach · 2016-12-12T03:01:10Z

src/main/java/org/spongepowered/api/item/recipe/crafting/CraftingRegistry.java

+     * @return A list of ItemStacks or {@link Optional#empty()} if the given
+     *          {@link GridInventory} does not match any recipes.
+     */
+    Optional<List<ItemStack>> getResults(GridInventory grid);


Can it return an empty list for nothing?

Yeah, have to agree here, why Optional instead of just an empty list?

IDK, just moved it from Recipe....

https://github.com/SpongePowered/SpongeAPI/pull/1098/files#diff-533288ff4558c56e3fb98c90df746cafR66

liach · 2016-12-12T03:01:16Z

src/main/java/org/spongepowered/api/item/recipe/crafting/CraftingRegistry.java

+     *          {@link GridInventory} does not match any recipes.
+     */
+    Optional<List<ItemStack>> getResults(GridInventory grid);
+}


gabizou · 2016-12-12T03:45:03Z

src/main/java/org/spongepowered/api/item/recipe/RecipeRegistry.java

     * Removes the given Recipe from registration in this registry.
     *
     * @param recipe The Recipe to unregister
+     * @return if it was successful or not


Capitalize the if

ST-DDT · 2017-01-03T11:50:45Z

src/main/java/org/spongepowered/api/item/recipe/Recipe.java

-     *          ItemGrid does not match this recipe's requirements.
-     */
-    Optional<List<ItemStack>> getResults(GridInventory grid);
+    RecipeRegistry<?> getRegistry();


If this recipe is restricted to a single registry you should throw an IllegalArgumentException if you try to register it in a different one.

Is there a builder method for this one?

ST-DDT · 2017-01-03T11:53:31Z

src/main/java/org/spongepowered/api/item/recipe/crafting/CraftingRegistry.java

+     * @param grid The ItemGrid to check for validity
+     * @return True if the given input matches this recipe's requirements
+     */
+    boolean isValid(GridInventory grid);


ImO this method would be better returning the matching recipe. The javadoc are strange currently.

Zidane · 2017-01-03T22:12:02Z

src/main/java/org/spongepowered/api/item/recipe/crafting/ShapelessCraftingRecipe.java

     * @return The ingredients
     */
-    Collection<ItemStack> getIngredients();
+    Collection<ItemStackSnapshot> getIngredients();


Getting ingredients as snapshots makes sense but I wouldn't require them when adding to the builder. Makes working with the API painful (when it could turn them into snapshots itself).

ST-DDT · 2017-01-07T13:48:02Z

src/main/java/org/spongepowered/api/item/recipe/Recipe.java

     * @return the results of this {@link Recipe}
     */
-    Collection<ItemStackSnapshot> getResults();
+    ImmutableCollection<ItemStackSnapshot> getResults();


I would prefer if this returns any kind of Collection instead of this specialized one.

jamierocks · 2017-01-07T17:45:27Z

src/main/java/org/spongepowered/api/item/recipe/RecipeRegistry.java


 /**
- * A RecipeRegistry holds all registered recipes for a given game.
+ * Used for {@link SmeltingRegistry} and {@link CraftingRegistry}


Missing full stop at end of sentence.

liach · 2017-01-07T17:56:58Z

src/main/java/org/spongepowered/api/item/recipe/Recipe.java

+     * @return the results of this {@link Recipe}
     */
-    boolean isValid(GridInventory grid);
+    ImmutableCollection<ItemStackSnapshot> getResults();


Collection

It's immutable because we don't want people changing it...

liach · 2017-01-07T17:57:17Z

src/main/java/org/spongepowered/api/item/recipe/RecipeRegistry.java

     * @return All registered recipes
     */
-    Set<Recipe> getRecipes();
+    ImmutableCollection<T> getRecipes();


Collection

Don't use ImmutableCollection, just Collection is fine (you can even say it's unmodifiable in the javadoc).

liach · 2017-01-07T17:58:11Z

src/main/java/org/spongepowered/api/item/recipe/crafting/CraftingRegistry.java

+     * @return A list of ItemStacks or {@link Optional#empty()} if the given
+     *          {@link GridInventory} does not match any recipes.
+     */
+    Optional<ImmutableCollection<ItemStackSnapshot>> getResults(GridInventory grid);


Collection<ItemStackSnapshot>

liach · 2017-01-07T17:58:30Z

src/main/java/org/spongepowered/api/item/recipe/crafting/ShapedCraftingRecipe.java

+     *
+     * @return The ingredients
+     */
+    ImmutableMap<Character, ItemStackSnapshot> getIngredients();


liach · 2017-01-07T17:58:46Z

src/main/java/org/spongepowered/api/item/recipe/crafting/ShapedCraftingRecipe.java

+     *
+     * @return The aisle
+     */
+    ImmutableCollection<String> getShape();


liach · 2017-01-07T17:59:51Z

src/main/java/org/spongepowered/api/item/recipe/crafting/ShapelessCraftingRecipe.java

+         * @return The builder
+         */
+        default Builder ingredients(ItemStack... ingredients) {
+            return ingredients(ingredients);


Meh, doesn't even compile

? It does...

Will cause StackOverflowException

liach · 2017-01-07T18:00:55Z

src/main/java/org/spongepowered/api/item/recipe/smelting/SmeltingRegistry.java

+     * @return A {@link ItemStackSnapshot} or {@link Optional#empty()} if the given
+     *          {@link ItemType} does not match any recipes.
+     */
+    Optional<ItemStackSnapshot> getResult(ItemType type);


Take ItemStackSnapshot as argument instead

Or ItemStack

I put ItemType there because it ignores stack size. I'll probably be changing getExperience to that too

No, meta is super important. Stack size can simply be ignored.

The data on the ItemStack is 100% very important, not to just the ItemType.

kashike · 2017-01-07T22:59:40Z

src/main/java/org/spongepowered/api/item/recipe/RecipeRegistry.java

+     * @return If it removed any recipes or not
     */
-    void remove(Recipe recipe);
+    boolean remove(Predicate<? super T> predicate);


Don't remove the previous version of this method, add a new one.

Seconded, the removal of the recipe can still be retained.

ST-DDT · 2017-01-08T15:54:51Z

src/main/java/org/spongepowered/api/item/recipe/crafting/ShapelessCraftingRecipe.java

+         * @param ingredients The ingredients
+         * @return The builder
+         */
+        Builder ingredients(Collection<ItemStack> ingredients);


No ItemStackSnapshot support? (Sorry for nagging)

Requested by Zidane to be an ItemStack :P https://github.com/SpongePowered/SpongeAPI/pull/1419/files/109bca54aacdfe4112c7a2b6ae536b5d86b62abb#diff-a6badc7df0748d1ae0caa7f85876c1c2

@Maxqia

You can do both lol.

ST-DDT · 2017-01-08T16:11:20Z

src/main/java/org/spongepowered/api/item/recipe/Recipe.java

+     * @return the {@link RecipeRegistry} that corresponds with this Recipe
     */
-    Optional<List<ItemStack>> getResults(GridInventory grid);
+    RecipeRegistry<?> getRegistry();


What is the reason for this method?

I'm not really against it being here, but IMO its simply not-required overhead.

If this is not yet/no longer registered, this would return null

(if ever supported) custom impls might return the wrong registry

I don't see a usecase

What are your thoughts adding this method?

I don't see the point in having this method. Just gut it.

gabizou · 2017-01-10T19:42:52Z

src/main/java/org/spongepowered/api/item/recipe/crafting/ShapedCraftingRecipe.java

+     *
+     * <p>This returns an unmodifiable copy of the original aisle.</p>
+     *
+     * @return The aisle


The aisle or shape?

gabizou · 2017-01-10T19:44:53Z

src/main/java/org/spongepowered/api/item/recipe/crafting/ShapelessCraftingRecipe.java

+         * @param ingredients The ingredients
+         * @return The builder
+         */
+        default Builder ingredients(ItemStack... ingredients) {


In general, avoid only adding vararg methods like this, if possible, you should have single argument methods and then two argument methods with the second argument being a vararg. This would simplify some logic.

gabizou · 2017-01-10T19:45:19Z

src/main/java/org/spongepowered/api/item/recipe/crafting/ShapelessCraftingRecipe.java

+         * @return The builder
+         */
+        default Builder ingredients(ItemType... ingredients) {
+            return ingredients(Lists.newArrayList(ingredients).stream().map(


Fix this formatting, it looks bad. Refer to other areas where streams are used.

gabizou · 2017-01-10T19:46:25Z

src/main/java/org/spongepowered/api/item/recipe/smelting/SmeltingRecipe.java

+     */
+    ItemStackSnapshot getResult();
+
+    default ImmutableCollection<ItemStackSnapshot> getResults() {


You javadoc everything else but not this method? I'm confused as to why there's a single result and then a collection of other results. What other results can there be?

I'm fairly certain there are a number of mods which give out more than one result in the furnace, can't think of any off the top of my head though.

Oh, It's a defaulted method from Recipe :P. not sure if @OverRide works here

@Maxqia Yes, @Override works here. You should always annotate the methods appropriately so it's plain and clear that the method doesn't originate from this interface/class and is an overridden method. Everything in Sponge follows this rule.

kashike · 2017-04-13T08:05:13Z

This is more than likely going to need changes with 1.12 arrives.

gabizou · 2017-04-21T05:09:04Z

@kashike While that is true, would it be worth having the changes being made so that recipes can be managed in API 6 and maybe 7?

kashike · 2017-06-17T01:10:13Z

I'm going to close this in favour of #1582.

Maxqia mentioned this pull request Nov 19, 2016

Implement Recipes SpongePowered/Sponge#1027

Closed

kashike changed the title ~~[WIP] Recipes~~ [WIP] Furnace Recipes Nov 19, 2016

kashike added the type: enhancement label Nov 19, 2016

liach reviewed Nov 19, 2016

View reviewed changes

kashike suggested changes Nov 19, 2016

View reviewed changes

kashike self-assigned this Nov 19, 2016

kashike added the status: wip label Nov 19, 2016

kashike changed the title ~~[WIP] Furnace Recipes~~ Furnace Recipes Nov 19, 2016

ST-DDT suggested changes Nov 19, 2016

View reviewed changes

kashike suggested changes Nov 20, 2016

View reviewed changes

Maxqia force-pushed the recipe branch from ebdbcb1 to 341b4dd Compare November 23, 2016 04:54

gabizou requested changes Dec 8, 2016

View reviewed changes

Cybermaxke reviewed Dec 11, 2016

View reviewed changes

liach suggested changes Dec 11, 2016

View reviewed changes

ST-DDT reviewed Dec 11, 2016

View reviewed changes

Maxqia force-pushed the recipe branch 2 times, most recently from 2c10b77 to 304af21 Compare December 12, 2016 02:38

liach suggested changes Dec 12, 2016

View reviewed changes

gabizou reviewed Dec 12, 2016

View reviewed changes

Maxqia mentioned this pull request Jan 3, 2017

Generalize recipes and implement them #1449

Closed

Add SmeltingRegistry and Recipe Refactor

5627e4b

Maxqia force-pushed the recipe branch from 304af21 to 5627e4b Compare January 3, 2017 08:03

Oh fiinnnnnnneeeeeeee

cb52f33

ST-DDT suggested changes Jan 3, 2017

View reviewed changes

Zidane reviewed Jan 3, 2017

View reviewed changes

It's getting along ...

1aca9b4

ST-DDT reviewed Jan 7, 2017

View reviewed changes

Predicates and Doubles

ecc5a22

Maxqia changed the title ~~Recipe Refactor~~ [Not WIP] Recipe Refactor Jan 7, 2017

stephan-gh removed the status: wip label Jan 7, 2017

stephan-gh changed the title ~~[Not WIP] Recipe Refactor~~ Recipe Refactor Jan 7, 2017

Javadoc and default methods

4d54ae9

jamierocks reviewed Jan 7, 2017

View reviewed changes

liach reviewed Jan 7, 2017

View reviewed changes

kashike suggested changes Jan 7, 2017

View reviewed changes

ST-DDT suggested changes Jan 8, 2017

View reviewed changes

gabizou reviewed Jan 10, 2017

View reviewed changes

gabizou mentioned this pull request Jan 10, 2017

Refactor recipes #1098

Closed

stephan-gh added the system: inventory label Apr 12, 2017

malte-laukoetter mentioned this pull request Apr 22, 2017

Add CraftItem to Possible actions malte-laukoetter/Laborus#8

Open

kashike closed this Jun 17, 2017


		import java.util.List;

		public interface SmeltingRecipe extends Recipe {

Uh oh!

Conversation

Maxqia commented Nov 19, 2016 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

liach Nov 19, 2016 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Cybermaxke commented Nov 19, 2016 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Maxqia commented Nov 20, 2016

Uh oh!

liach commented Nov 20, 2016

Uh oh!

liach commented Nov 20, 2016

Uh oh!

Maxqia commented Nov 20, 2016

Uh oh!

Maxqia commented Nov 20, 2016

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Cybermaxke commented Nov 20, 2016

Uh oh!

gabizou left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Maxqia commented Nov 19, 2016 •

edited

Loading

liach Nov 19, 2016 •

edited

Loading

Cybermaxke commented Nov 19, 2016 •

edited

Loading