Fix Wrong Tab Configuration Error in Magento

adminhtmlblocksoverridestabs

I am trying to extend Mage_Adminhtml_Block_Customer_Edit_Tab_View class in order to pass in some functions and display them using a layout update in

adminhtml/default/default/template/namespace/mymodule/edit/tab/view.phtml

I've used a rewrite in my config.xml

<blocks>
    <mymodule>
        <class>Mymodule_Customer_Block</class>
    </mymodule>
    <adminhtml>
        <rewrite>
            <customer_edit_tab_view>Mymodule_Customer_Block_Adminhtml_Customer_Edit_Tab_View</customer_edit_tab_view>
        </rewrite>
    </adminhtml>
</blocks>

And here is the View.php file

class Mymodule_Customer_Block_Adminhtml_Customer_Edit_Tab_View extends   Mage_Adminhtml_Block_Customer_Edit_Tab_View
{
    public function getCredit(){

        $customerId = $this->getCustomer()->getId();
        $model = Mage::registry('current_customer');
        $customerId = $model->getId();
        $data = $model->getData();

        $helper = Mage::helper('customercredit');
        $creditValue = (float)$helper->getCreditValue($customerId, $model->getWebsiteId());

        return $creditValue;
    }    
}

Whenever I go to view the customer edit view tab in magento admin I get an error:

Wrong tab configuration.

I must have missed something out of my setup but I am not sure what?

Any one have any idea please let me know…

Best Answer

You'll need to do a little detective work. It's very possible a different change is responsible for the error you're seeing, or that something about your replacement triggers this error.

To start, I'd remove any custom functions you have in Mymodule_Customer_Block_Adminhtml_Customer_Edit_Tab_View and concentrate on getting the class rewrite working without a Wrong tab Configuration exception — that way if the error is tied to a method you've added, you'll know which one when you add your methods back one by one.

The exception you're seeing comes from this method

#File: app/code/core/Mage/Adminhtml/Block/Widget/Tabs.php
public function addTab($tabId, $tab)
{
    if (is_array($tab)) {
        $this->_tabs[$tabId] = new Varien_Object($tab);
    }
    elseif ($tab instanceof Varien_Object) {
        $this->_tabs[$tabId] = $tab;
        if (!$this->_tabs[$tabId]->hasTabId()) {
            $this->_tabs[$tabId]->setTabId($tabId);
        }
    }
    elseif (is_string($tab)) {
        if (strpos($tab, '/')) {
            $this->_tabs[$tabId] = $this->getLayout()->createBlock($tab);
        }
        elseif ($this->getChild($tab)) {
            $this->_tabs[$tabId] = $this->getChild($tab);
        }
        else {
            $this->_tabs[$tabId] = null;
        }

        if (!($this->_tabs[$tabId] instanceof Mage_Adminhtml_Block_Widget_Tab_Interface)) {
            throw new Exception(Mage::helper('adminhtml')->__('Wrong tab configuration.'));
        }
    }
    else {
        throw new Exception(Mage::helper('adminhtml')->__('Wrong tab configuration.'));
    }

    if (is_null($this->_tabs[$tabId]->getUrl())) {
        $this->_tabs[$tabId]->setUrl('#');
    }

    if (!$this->_tabs[$tabId]->getTitle()) {
        $this->_tabs[$tabId]->setTitle($this->_tabs[$tabId]->getLabel());
    }

    $this->_tabs[$tabId]->setId($tabId);
    $this->_tabs[$tabId]->setTabId($tabId);

    if (is_null($this->_activeTab)) $this->_activeTab = $tabId;
    if (true === $this->_tabs[$tabId]->getActive()) $this->setActiveTab($tabId);

    return $this;
}

And occurs when Magento can't instantiate a tab from information passed into addTab. My guess is your error comes from this bit of layout XML in catalog.xml

<action method="addTab"><name>customer_edit_tab_view</name><block>customer_edit_tab_view</block></action>

Although adding some temporary debugging to the addTab method above will help with that

var_dump($tabId, $tab);

If that's the case, it sounds like Magento may be having trouble instantiating your new block class. I'd try running the following code

$block = new Mymodule_Customer_Block_Adminhtml_Customer_Edit_Tab_View;
var_dump($block);

$block = Mage::getSingleton('adminhtml/customer_edit_tab_view');
var_dump($block);

and looking for instantiation errors (each call should return a block object).

My best guess as to why this is happening is you have your View.php file for Mymodule_Customer_Block_Adminhtml_Customer_Edit_Tab_View created in the wrong folder.

Good luck!

Related Solutions

Magento Admin – Add System Configuration to Existing Tab

Assuming you mean the 404 page where there's admin chrome and 404 text in the content area, it sounds like your ACL is setup incorrectly or your current user account lacks the proper ACL credentials to view your new System Configuration section.

First, if your user has the all ACL permissions (i.e. is a system root user and/or the default user created during setup), you'll need to clear your admin session before being able to see the page. Magento caches the individual permissions an admin user has in the session, not the cache. This happens when you login — so your new ACL rule hasn't been added to the list of rules allowed by your user.

Second, it's always best to go right to the source of an error. Take a look at the following section of code.

#File: app/code/core/Mage/Adminhtml/controllers/System/ConfigController.php
protected function _isSectionAllowed($section)
{
    try {
        $session = Mage::getSingleton('admin/session');
        $resourceLookup = "admin/system/config/{$section}";
        if ($session->getData('acl') instanceof Mage_Admin_Model_Acl) {
            $resourceId = $session->getData('acl')->get($resourceLookup)->getResourceId();
            if (!$session->isAllowed($resourceId)) {
                throw new Exception('');
            }
            return true;
        }
    }
    catch (Zend_Acl_Exception $e) {
        $this->norouteAction();
        $this->setFlag('', self::FLAG_NO_DISPATCH, true);
        return false;
    }
    catch (Exception $e) {
        $this->deniedAction();
        $this->setFlag('', self::FLAG_NO_DISPATCH, true);
        return false;
    }
}

This is what triggers a 404 page if you're missing an ACL role. A

var_dump($e->getMessage());
var_dump($resourceLookup);

in each of the catch blocks will tell you why Magento thinks your user doesn't have the right access permissions.

Magento Grid Inside Catalog Product Edit Tab

The final solution ended up being modifying Tab.php to look like:

    <?php

class Company_WordpressIntegration_Block_Adminhtml_Catalog_Product_Tab
    extends Mage_Adminhtml_Block_Widget_Grid
    implements Mage_Adminhtml_Block_Widget_Tab_Interface
{

    public function __construct()
    {
        parent::__construct();
        $this->setId('company_wordpressintegration_grid');
        $this->setDefaultSort('ID');
        $this->setDefaultDir('DESC');
        $this->setSaveParametersInSession(true);
        $this->setUseAjax(true);
    }

    protected function _prepareCollection()
    {
        $collection = Mage::getModel('wordpressintegration/post')->getCollection();

        $this->setCollection($collection);
        parent::_prepareCollection();
        return $this;
    }

    protected function _prepareColumns()
    {
        $helper = Mage::helper('company_wordpressintegration');
        $currency = (string) Mage::getStoreConfig(Mage_Directory_Model_Currency::XML_PATH_CURRENCY_BASE);

        $this->addColumn('itemCheckbox', array(
            'index'     => 'ID',
            'type'      => 'checkbox',
            'width'     => 20,
            'sortable'  => false,
            'field_name' => 'map[]'
        ));

        $this->addColumn('ID', array(
            'header' => $helper->__('ID'),
            'index'  => 'ID'
        ));

        $this->addColumn('post_author', array(
            'header' => $helper->__('Author'),
            'index'  => 'post_author'
        ));

        $this->addColumn('post_date', array(
            'header' => $helper->__('Date'),
            'type'   => 'datetime',
            'index'  => 'post_date'
        ));

        $this->addColumn('post_title', array(
            'header' => $helper->__('Title'),
            'index'  => 'post_title'
        ));

        $this->addColumn('post_content', array(
            'header' => $helper->__('Content'),
            'index'  => 'post_content'
        ));

        $this->addColumn('post_status', array(
            'header' => $helper->__('Status'),
            'index'  => 'post_status'
        ));



        $this->addExportType('*/*/exportRelatedCsv', $helper->__('CSV'));
        $this->addExportType('*/*/exportRelatedExcel', $helper->__('Excel XML'));

        return parent::_prepareColumns();
    }

    public function getGridUrl()
    {
        return $this->getUrl('*/*/grid', array('_current'=>true));
    }


    /**
     * Retrieve the label used for the tab relating to this block
     *
     * @return string
     */
    public function getTabLabel()
    {
        return $this->__('Related Projects');
    }

    /**
     * Retrieve the title used by this tab
     *
     * @return string
     */
    public function getTabTitle()
    {
        return $this->__('Click here to view your custom tab content');
    }

    /**
     * Determines whether to display the tab
     * Add logic here to decide whether you want the tab to display
     *
     * @return bool
     */
    public function canShowTab()
    {
        return true;
    }

    /**
     * Stops the tab being hidden
     *
     * @return bool
     */
    public function isHidden()
    {
        return false;
    }

}

This negates the need for the additional grid and grid containers. Notice aside from the grid functions being moved into the tab file, the file now extends the grid, but continues to implement the tab interface.

Best Answer

Related Solutions

Magento Admin – Add System Configuration to Existing Tab

Magento Grid Inside Catalog Product Edit Tab

Related Topic