Item: use static-analysis-friendly method for retrieving custom block data

fixes 2 phpstan level 8 errors
This commit is contained in:
Dylan K. Taylor 2020-06-27 13:46:04 +01:00
parent 0000783926
commit c040248dbd
2 changed files with 2 additions and 12 deletions

View File

@ -351,8 +351,8 @@ class Item implements \JsonSerializable{
$tag->removeTag(self::TAG_ENCH);
}
$this->hasCustomBlockData() ?
$tag->setTag(self::TAG_BLOCK_ENTITY_TAG, clone $this->getCustomBlockData()) :
($blockData = $this->getCustomBlockData()) !== null ?
$tag->setTag(self::TAG_BLOCK_ENTITY_TAG, clone $blockData) :
$tag->removeTag(self::TAG_BLOCK_ENTITY_TAG);
if(!$this->canPlaceOn->isEmpty()){

View File

@ -145,16 +145,6 @@ parameters:
count: 1
path: ../../../src/inventory/transaction/CraftingTransaction.php
-
message: "#^Cannot clone pocketmine\\\\nbt\\\\tag\\\\CompoundTag\\|null\\.$#"
count: 1
path: ../../../src/item/Item.php
-
message: "#^Parameter \\#2 \\$tag of method pocketmine\\\\nbt\\\\tag\\\\CompoundTag\\:\\:setTag\\(\\) expects pocketmine\\\\nbt\\\\tag\\\\Tag, pocketmine\\\\nbt\\\\tag\\\\CompoundTag\\|null given\\.$#"
count: 1
path: ../../../src/item/Item.php
-
message: "#^Parameter \\#1 \\$object of function get_class expects object, pocketmine\\\\nbt\\\\tag\\\\Tag\\|null given\\.$#"
count: 1