mirrors/PocketMine-MP
1
0
Fork 0
mirror of https://github.com/pmmp/PocketMine-MP.git synced 2025-06-26 05:14:05 +00:00
Code Issues Projects Releases Wiki Activity

Item: skip a step when decoding PC itemstacks

Browse Source
This commit is contained in:
Dylan K. Taylor 2020-07-10 21:32:39 +01:00
parent 279abb871d
commit 9b52af62b6
2 changed files with 9 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
src/item/Item.php
View File

@ -658,13 +658,12 @@ class Item implements \JsonSerializable{
if($idTag instanceof ShortTag){
$item = ItemFactory::getInstance()->get($idTag->getValue(), $meta, $count);
}elseif($idTag instanceof StringTag){ //PC item save format
try{
$item = LegacyStringToItemParser::getInstance()->parse($idTag->getValue() . ":$meta");
}catch(\InvalidArgumentException $e){
//TODO: improve error handling
//TODO: this isn't a very good mapping source, we need a dedicated mapping for PC
$id = LegacyStringToItemParser::getInstance()->parseId($idTag->getValue());
if($id === null){
return ItemFactory::air();
}
$item->setCount($count);
$item = ItemFactory::getInstance()->get($id, $meta, $count);
}else{
throw new \InvalidArgumentException("Item CompoundTag ID must be an instance of StringTag or ShortTag, " . get_class($idTag) . " given");
}

5
src/item/LegacyStringToItemParser.php
View File

@ -32,6 +32,7 @@ use function is_int;
use function is_numeric;
use function is_string;
use function json_decode;
use function mb_strtolower;
use function str_replace;
use function strtolower;
use function trim;
@ -78,6 +79,10 @@ final class LegacyStringToItemParser{
$this->map[$alias] = $id;
}
public function parseId(string $input) : ?int{
return $this->map[mb_strtolower($this->reprocess($input))] ?? null;
}
/**
* Tries to parse the specified string into Item types.
*