Merge branch 'next-minor' into next-major

This commit is contained in:
Dylan K. Taylor
2022-10-31 19:11:39 +00:00
15 changed files with 130 additions and 64 deletions

View File

@ -52,6 +52,9 @@ use function json_decode;
final class CraftingManagerFromDataHelper{
private static function deserializeIngredient(RecipeIngredientData $data) : ?RecipeIngredient{
if(!isset($data->name)){
return null; //TODO: not yet implemented
}
if(isset($data->count) && $data->count !== 1){
//every case we've seen so far where this isn't the case, it's been a bug and the count was ignored anyway
//e.g. gold blocks crafted from 9 ingots, but each input item individually had a count of 9