Magento - действие редактирования нового модуля создает новый элемент

о сохранении вашего баннера, адаптируйте следующий код к вашей ситуации: Здесь важно использовать реестр при сохранении. Убедитесь также, что идентификатор баннера указан в $data при сохранении. Ваш form.php должен предоставить его, добавьте приведенный ниже код перед строкой $form->setValues(...):

$model = Mage::registry('BannerManagement_data');
if ($model->getEntityId()) {
   $fieldset->addField('entity_id', 'hidden', array('name' => 'entity_id'));// or banner_id depends on what id title you gave in your database table
}

Конечно, вы должны проверить ввод пользователей перед сохранением. Используйте защищенный метод _beforeSave() в вашей модели баннера, чтобы реализовать эту проверку ввода, или в контроллере непосредственно в действии сохранения.

/**
 * Common init to almost all actions
 */
protected function _initAction(){
    $this->_title ($this->__("Banner"));

    $this->loadLayout();
    $this->_setActiveMenu('mymenu/banner');
    $this->_addBreadcrumb(Mage::helper('banner')->__('Banners'), Mage::helper('banner')->__('Items'));
    }

    if(! Mage::registry('current_banner')){
        Mage::register('current_banner', Mage::getModel('banner/item'));
    }

    $id = $this->getRequest()->getParam('id');
    if (!is_null($id)) {
        $model = Mage::registry('current_banner')->load($id);

        if (! $model->getId()) {
            $this->_getSession()->addError(Mage::helper('banner')->__('This banner item no longer exists'));
            $this->_redirect('*/*/');
            return;
        }
    }

    return $this;
}


/**
 * Banner edit page
 */ 
public function editAction(){
    $this->_initAction();
    $this->_title('Banner Edit');

    // 1. Get ID and create model
    $banner = Mage::registry('current_banner');

    // 2. set entered data if there had errors when we do save
    $data = $this->_getSession()->getBannerData(true);

    // 3. restore data from SESSION and provide a correct date format
    if (!empty($data)) {
        $banner->addData($data);
    }

    // 4. Build Edit form
    $this->_addBreadcrumb(Mage::helper('banner')->__('Edit banner Item'), Mage::helper('banner')->__('Edit Banner Item'));
    $this->_addContent($this->getLayout()->createBlock('banner/adminhtml_banner'));
    $this->renderLayout();
}

/**
 * Subscritpion save process
 */ 
public function saveAction(){

    $this->_initAction();
    $banner = Mage::registry('current_banner');
    $data = $this->getRequest()->getParams();

    if ($data) {

        try {
            $banner->addData($data);
            $banner->save();

            $this->_getSession()->addSuccess(Mage::helper('banner')->__('The banner item has been saved.'));

            if ($this->getRequest()->getParam('back', false)) {
                $this->_redirect('*/*/edit', array('id'    => $banner->getId(), '_current'=>true));
                return;
            }
        } catch (Exception $e) {
            $this->_getSession()->addError($e->getMessage());
            $this->_getSession()->setBannerData($banner->getData());
            $this->_redirectUrl($this->getUrl('*/*/edit', array('id' => $banner->getId())));
            return;
        }
    }
    $this->_redirectUrl($this->getUrl('*/adminhtml_overview'));
}