Ttdfjt

I am trying to get up-sell product, but getting issue with invalid block type: MageplazaHelloWorldBlockHelloWorld in Magento 2.

<?php
namespace MageplazaHelloWorldBlock;
class Upsell extends MagentoFrameworkViewElementTemplate

 protected $_registry;

 public function __construct(
 MagentoBackendBlockTemplateContext $context, 
 MagentoFrameworkRegistry $registry,
 array $data = []
 )
 
 $this->_registry = $registry;
 parent::__construct($context, $data);
 

 public function _prepareLayout()
 
 return parent::_prepareLayout();
 

 public function getCurrentProduct()
 
 return $this->_registry->registry('current_product');
 


?>

Here is .phtml code

<?php
$myBlock = MagentoFrameworkAppObjectManager::getInstance()->get('MageplazaHelloWorldBlockUpsell');
$currentProduct = $myBlock->getCurrentProduct();

if ($currentProduct = $myBlock->getCurrentProduct()) 
 $upSellProducts = $currentProduct->getUpSellProducts();

 if (!empty($upSellProducts)) 
 echo '<h3>UpSell Products</h3>';
 foreach ($upSellProducts as $upSellProduct) 
 echo $upSellProduct->getSku() . '<br />'; 
 
 




?>

Class MagentoBackendBlockTemplateContext is for adminhtml block

You should use MagentoFrameworkViewElementTemplateContext class for frontend block

Here is your Final Block:

<?php

namespace MageplazaHelloWorldBlock;

class Upsell extends MagentoFrameworkViewElementTemplate

 protected $registry;

 protected $productFactory;

 public function __construct(
 MagentoFrameworkViewElementTemplateContext $context, 
 MagentoFrameworkRegistry $registry,
 MagentoCatalogModelProductFactory $productFactory,
 array $data = []
 )
 
 $this->registry = $registry;
 $this->productFactory = $productFactory;
 parent::__construct($context, $data);
 

 public function _prepareLayout()
 
 return parent::_prepareLayout();
 

 public function getCurrentProduct()
 
 return $this->registry->registry('current_product');
 

 public function getProduct($productId)
 
 $product = $this->productFactory->create()->load($productId);
 return $product;
 

Don't use ObjectManager directly in phtml file. Use $block for current template block.

Here is your Final Phtml:

<?php

$currentProduct = $block->getCurrentProduct();

if ($currentProduct) 

 $upSellProducts = $currentProduct->getUpSellProducts();

 if (!empty($upSellProducts)) 
 echo '<h3>UpSell Products</h3>';
 foreach ($upSellProducts as $upSellProduct) 
 $product = $block->getProduct($upSellProduct->getId());
 echo $product->getSku() . '<br />'; 
 
 

You shouldn't use ObjectManager to get block in template like that. It is bad practices you should avoid.
Use $block is corresponding to your current block . In your case is MageplazaHelloWorldBlockUpsell

Your Constructor in Block class need to modify

public function __construct(
 MagentoFrameworkViewElementTemplateContext $context, <== this class 
 MagentoFrameworkRegistry $registry,
 array $data = []
)

In your template

<?php /** @var MageplazaHelloWorldBlockUpsell $block **/

$currentProduct = $block->getCurrentProduct();

if ($currentProduct = $myBlock->getCurrentProduct()) 
 $upSellProducts = $currentProduct->getUpSellProducts();

 if (!empty($upSellProducts)) 
 echo '<h3>UpSell Products</h3>';
 foreach ($upSellProducts as $upSellProduct) 
 echo $upSellProduct->getSku() . '<br />'; 
 
 




?>

Finally, remove generation by command rm -r var/generation in order to update code