Is a good practice to instantiate a Magento help in the class constructor?
protected $helper
public function __construct()
{
$this->helper=Mage::helper('helper_name');
}
Add a comment
|
1 Answer
Magentos helpers are always singletons, so it's not necessary to instantiate them. Just using Mage::helper('helper_name')->someMethod() should be OK.
-
tnx, in terms of code readability I'd like to use $this->helper. Any contraindications with my code?– apedicCommented Aug 7, 2017 at 13:59
-
Imho, no, but i try to avoid such things :) How about an wrapper method
_getHelper() { return Mage::helper ...}and using$this->_getHelper()(it's not much longer)? Maybe wait for another answer?– sv3nCommented Aug 7, 2017 at 14:10 -