mirrors/PocketMine-MP
1
0
Fork 0
mirror of https://github.com/pmmp/PocketMine-MP.git synced 2025-04-21 08:17:34 +00:00
Code Issues Projects Releases Wiki Activity

LegacySkinAdapter: an extra check for resourcePatch (fixes 3 phpstan explicitMixed errors)

Browse Source
This commit is contained in:
Dylan K. Taylor 2020-06-18 20:08:38 +01:00
parent a9d98bdf73
commit e105578be0
2 changed files with 2 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/pocketmine/network/mcpe/protocol/types/LegacySkinAdapter.php
View File

@ -26,6 +26,7 @@ namespace pocketmine\network\mcpe\protocol\types;
use pocketmine\entity\InvalidSkinException;
use pocketmine\entity\Skin;
use function is_array;
use function is_string;
use function json_decode;
use function json_encode;
@ -58,7 +59,7 @@ class LegacySkinAdapter implements SkinAdapter{
$capeData = $data->isPersonaCapeOnClassic() ? "" : $data->getCapeImage()->getData();
$resourcePatch = json_decode($data->getResourcePatch(), true);
if(isset($resourcePatch["geometry"]["default"]) && is_string($resourcePatch["geometry"]["default"])){
if(is_array($resourcePatch) && isset($resourcePatch["geometry"]["default"]) && is_string($resourcePatch["geometry"]["default"])){
$geometryName = $resourcePatch["geometry"]["default"];
}else{
throw new InvalidSkinException("Missing geometry name field");

5
tests/phpstan/configs/check-explicit-mixed-baseline.neon
View File

@ -455,11 +455,6 @@ parameters:
count: 1
path: ../../../src/pocketmine/network/mcpe/protocol/StartGamePacket.php
-
message: "#^Cannot access offset 'geometry' on mixed\\.$#"
count: 3
path: ../../../src/pocketmine/network/mcpe/protocol/types/LegacySkinAdapter.php
-
message: "#^Cannot access offset string on mixed\\.$#"
count: 1