Getting current customer with registry in Magento2

Question

below is my block file code

class View extends \Magento\Framework\View\Element\Template 
{ 
protected $_registry;
protected $_customerSession; 
protected $_customer; 
public function __construct(
    \Magento\Framework\App\Action\Context $context,
    \Magento\Framework\Registry $registry,
    \Magento\Customer\Model\Session $customerSession, 
    \Magento\Customer\Model\Customer $customer      
) {
    $this->_registry = $registry;
    $this->_customerSession = $customerSession; 
    $this->_customer = $customer; 
    parent::__construct($context);
}

public function getUser()
{
    if (!$this->_loadValidUser()) {
        return;
    }
    return $this->registry->registry('current_user');
}   

protected function _loadValidUser($userId = null)
{
     $userId = (int) $this->getRequest()->getParam('user_id');
    try {
        $user = $this->_customer->load($userId);
        $this->registry->register('current_user', $user);            
        return $user;
    }catch (Exception $e) {
       print_r($e->getMessage());
         return false;
    }

}

}

What is the mistake in above construction? I am getting the error like

PHP Fatal error: Uncaught TypeError: Argument 1 passed to Magento\Framework\View\Element\Template

Please accept the answer if it helped you so it will help others. — kunj, Aug 06 '18 at 12:21
This can help - https://magento.stackexchange.com/questions/236642/magento2-check-if-customer-is-logged-in-not-working — Gagan, Aug 06 '18 at 12:33
Use customerSession to get perfect output instead of registry. — Emipro Technologies Pvt. Ltd., Aug 06 '18 at 12:36

score 2 · Answer 1 · answered Aug 06 '18 at 12:18

Try this:

public function __construct(
    \Magento\Framework\View\Element\Template\Context $context,
    \Magento\Framework\Registry $registry,
    \Magento\Customer\Model\Session $customerSession,
    \Magento\Customer\Model\Customer $customer       
){
    $this->_registry = $registry;
    $this->_customerSession = $customerSession;
    $this->_customer = $customer;
    parent::__construct($context);
}

after that changes please run below commands

php bin/magento cache:flush and php bin/magento setup:di:compile

hi kunj, used above code no error but i am not getting current customer data. — Jafar Pinjar, Aug 06 '18 at 12:22

score 1 · Accepted Answer · answered Aug 06 '18 at 16:53

1

Use this code in your block file :

protected $_customerSession;

public function __construct(
    ......
    \Magento\Framework\View\Element\Template\Context $context,
    \Magento\Customer\Model\SessionFactory $customerSession,
    array $data = []
    ......
)
{
    ......
    $this->_customerSession = $customerSession->create();
    parent::__construct($context, $data);
    ......
}

public function getCustomerData() {
    if ($this->_customerSession->isLoggedIn()) {
        return $this->_customerSession->getCustomerData();
    }
    return false;
}

and display in your phtml file :

$customerData = $block->getCustomerData();
echo "<pre/>";
print_r($customerData);

answered Aug 06 '18 at 16:53

Rohan Hapani

17,388
9
54
96

I am getting userId in param and i need to register that user instead of current logged in customer. – Jafar Pinjar Aug 07 '18 at 05:11
Sorry ? I'm not getting exactly what you want to explain. – Rohan Hapani Aug 07 '18 at 05:41
can you explain in some more detail? – Rohan Hapani Aug 07 '18 at 05:41
In my code i used $userId = (int) $this->getRequest()->getParam('user_id'); to get parameter. from the parameter i am registering customer data. parameter is customer id, – Jafar Pinjar Aug 07 '18 at 05:44
Try to use this code $this->_registry->register('current_user', $user); in _loadValidUser function. – Rohan Hapani Aug 07 '18 at 05:47
And $this->_registry->registry('current_user'); use this code in getUser function. – Rohan Hapani Aug 07 '18 at 05:48
I am using same only now. please check my code once – Jafar Pinjar Aug 07 '18 at 05:49
In your code , you use like this => $this->registry->register('current_user', $user); . But, You need to use this line => $this->_registry->register('current_user', $user); – Rohan Hapani Aug 07 '18 at 05:50
Ok, I will use like that, this line is correct ? $user = $this->_customer->load($userId); I used customer , do we need to use customerSession? – Jafar Pinjar Aug 07 '18 at 05:53
Yes. That's correct. You just need to change that 2 line which I give you. – Rohan Hapani Aug 07 '18 at 05:53
Let us continue this discussion in chat. – Jafar Pinjar Aug 07 '18 at 05:57

Getting current customer with registry in Magento2

2 Answers2