Fixies v2 (merged into Core r635+)
#8
Still, xoft had to work out how to handle that weird bug with no clearing crafting grid on crafting plugin recipe.

Because the most flexible solution would be to allow plugins somehow clear crafting grid manually, but making hook-hook interactions within a plugin could be painful (imagine that we added OnGetCraftedItem hook - we now got to check if we're dealing with same player that had placed items in crafting grid when OnCraftingNoRecipe was called. We should store player and crafting details, and check them all).

The other way would be flushing whole crafting grid when item is crafted through such "plugin" recipe. But then we'll loose additional "multiple items in slop per recipe" functionality (which is nice thing to have, actually!). And we'll run into troubles with situation when player tries to use stacked items in crafting (meaning he want to craft only one item).

Just a thought: could we somehow requie plugin to provide crafting grid output AFTER crafting happened? That would be flexible and (in my mind) easy to implement enough. If plugin returns true - use it's output in recipe and in crafting grid after crafting.

Although maybe I'm just missing something, and there is already some way to handle that... Xoft?
Reply
Thanks given by:


Messages In This Thread
Fixies v2 (merged into Core r635+) - by Taugeshtu - 06-15-2012, 01:56 AM
RE: Fixies - by NiLSPACE - 06-15-2012, 05:26 AM
RE: Fixies - by Taugeshtu - 06-15-2012, 05:44 AM
RE: Fixies - by xoft - 06-15-2012, 06:37 AM
RE: Fixies - by cedeel - 06-15-2012, 06:40 AM
RE: Fixies - by xoft - 06-15-2012, 06:46 AM
RE: Fixies - by FakeTruth - 06-15-2012, 07:09 AM
RE: Fixies - by Taugeshtu - 06-15-2012, 08:53 AM
RE: Fixies - by xoft - 06-16-2012, 06:12 AM
RE: Fixies - by Taugeshtu - 06-16-2012, 06:44 AM
RE: Fixies - by xoft - 06-16-2012, 07:01 AM
RE: Fixies v2 (fixed) - by Taugeshtu - 06-16-2012, 07:01 AM
RE: Fixies v2 (fixed) - by xoft - 06-16-2012, 07:14 AM
RE: Fixies v2 (fixed) - by xoft - 06-20-2012, 06:41 AM
RE: Fixies v2 (fixed) - by Taugeshtu - 06-20-2012, 07:04 AM
RE: Fixies v2 (fixed) - by xoft - 06-20-2012, 07:59 AM
RE: Fixies v2 (merged into Core r635+) - by xoft - 06-20-2012, 05:49 PM



Users browsing this thread: 1 Guest(s)