Consistently declare BlockBreakInfo at the constructor call site

instead of inside the class
This commit is contained in:
Dylan K. Taylor
2021-05-20 22:49:51 +01:00
parent 8b4aa36e16
commit 376d2c4cd4
121 changed files with 267 additions and 657 deletions

View File

@ -27,7 +27,6 @@ use pocketmine\block\tile\Furnace as TileFurnace;
use pocketmine\block\utils\FacesOppositePlacingPlayerTrait;
use pocketmine\block\utils\NormalHorizontalFacingInMetadataTrait;
use pocketmine\item\Item;
use pocketmine\item\ToolTier;
use pocketmine\math\Vector3;
use pocketmine\player\Player;
@ -43,8 +42,8 @@ class Furnace extends Opaque{
/** @var bool */
protected $lit = false; //this is set based on the blockID
public function __construct(BlockIdentifier $idInfo, string $name, ?BlockBreakInfo $breakInfo = null){
parent::__construct($idInfo, $name, $breakInfo ?? new BlockBreakInfo(3.5, BlockToolType::PICKAXE, ToolTier::WOOD()->getHarvestLevel()));
public function __construct(BlockIdentifierFlattened $idInfo, string $name, BlockBreakInfo $breakInfo){
parent::__construct($idInfo, $name, $breakInfo);
}
public function getId() : int{