I followed this link to make image upload work within my Magento module, but the greatest thing all that code does is passing selected file name via POST: $_FILES
array is empty whenever i add image
or file
fields.
Here's my code:
<?php
class Moo_News_Block_Adminhtml_News_Edit_Tab_Form extends Mage_Adminhtml_Block_Widget_Form
{
protected function _prepareForm()
{
$form = new Varien_Data_Form();
$this->setForm($form);
$fieldset = $form->addFieldset('news_form', array('legend' => Mage::helper('news')->__('Item information')));
$fieldset->addField('title', 'text', array(
'label' => Mage::helper('news')->__('Title'),
'class' => 'required-entry',
'required' => true,
'name' => 'title',
));
$fieldset->addField('published', 'select', array(
'label' => Mage::helper('news')->__('Published'),
'name' => 'published',
'required' => true,
'values' => array(
array(
'value' => 1,
'label' => Mage::helper('news')->__('Yes'),
),
array(
'value' => 0,
'label' => Mage::helper('news')->__('No'),
),
),
));
$fieldset->addField('short_description', 'editor', array(
'name' => 'short_description',
'label' => Mage::helper('news')->__('Short Description'),
'title' => Mage::helper('news')->__('Short Description'),
'style' => 'width:98%; height:200px;',
'wysiwyg' => false,
'required' => true,
));
$fieldset->addField('body', 'editor', array(
'name' => 'body',
'label' => Mage::helper('news')->__('Body'),
'title' => Mage::helper('news')->__('Body'),
'style' => 'width:98%; height:400px;',
'wysiwyg' => false,
'required' => true,
));
$fieldset->addField('image', 'file', array(
'label' => Mage::helper('news')->__('Image'),
'required' => false,
'name' => 'image',
));
if ( Mage::getSingleton('adminhtml/session')->getNewsData() )
{
$form->setValues(Mage::getSingleton('adminhtml/session')->getNewsData());
Mage::getSingleton('adminhtml/session')->setNewsData(null);
} elseif ( Mage::registry('news_data') ) {
$form->setValues(Mage::registry('news_data')->getData());
}
return parent::_prepareForm();
}
}
and the controller code:
<?php
/**
*
*
*/
class Moo_News_Adminhtml_NewsController extends Mage_Adminhtml_Controller_Action
{
protected function _initAction()
{
$this->loadLayout()
->_setActiveMenu('news/items')
->_addBreadcrumb(Mage::helper('adminhtml')->__('Items Manager'), Mage::helper('adminhtml')->__('Item Manager'));
return $this;
}
public function indexAction() {
$this->_initAction();
$this->_addContent($this->getLayout()->createBlock('news/adminhtml_news'));
$this->renderLayout();
}
public function editAction()
{
$newsId = $this->getRequest()->getParam('id');
$newsModel = Mage::getModel('news/post')->load($newsId);
if ($newsModel->getId() || $newsId == 0) {
Mage::register('news_data', $newsModel);
$this->loadLayout();
$this->_setActiveMenu('news/it开发者_如何学Pythonems');
$this->_addBreadcrumb(Mage::helper('adminhtml')->__('Item Manager'), Mage::helper('adminhtml')->__('Item Manager'));
$this->_addBreadcrumb(Mage::helper('adminhtml')->__('Item News'), Mage::helper('adminhtml')->__('Item News'));
$this->getLayout()->getBlock('head')->setCanLoadExtJs(true);
$this->_addContent($this->getLayout()->createBlock('news/adminhtml_news_edit'))
->_addLeft($this->getLayout()->createBlock('news/adminhtml_news_edit_tabs'));
$this->renderLayout();
} else {
Mage::getSingleton('adminhtml/session')->addError(Mage::helper('news')->__('Item does not exist'));
$this->_redirect('*/*/');
}
}
public function newAction()
{
$this->_forward('edit');
}
public function saveAction()
{
if ( $this->getRequest()->getPost() ) {
try {
$postData = $this->getRequest()->getPost();
$newsModel = Mage::getModel('news/post');
$imgFilename = NULL;
//if (isset($postData['image']) && $postData['image'] != '') {
//if (isset($_FILES['image']['name']) && $_FILES['image']['name'] != '') {
if (false) {
try {
$uploader = new Varien_File_Uploader('image');
$uploader->setAllowedExtensions(array('jpg','jpeg','gif','png'));
$uploader->setAllowRenameFiles(false);
$uploader->setFilesDispersion(false);
// Set media as the upload dir
$media_path = Mage::getBaseDir('media');
$imgFilename = $media_path . $postData['image'];
while (file_exists($filename)) {
$pieces = array();
$res = preg_match('/^(.+)_(\d+)$/', $filename, $pieces);
if (!$res) {
$filename .= '_1';
} else {
$filename .= '_' . strval(intval($pieces[2]) + 1);
}
}
// Upload the image
$uploader->save($media_path, $postData['image']);
}
catch (Exception $e) {
Mage::log($e);
$this->redirectError(502);
}
} else {
if(isset($postData['image']['delete']) && $postData['image']['delete'] == 1) {
$imgFilename = NULL;
}
}
if(isset($imgFilename))
$newsModel->setImage($imgFilename);
$newsModel->setId($this->getRequest()->getParam('id'))
->setTitle($postData['title'])
->setBody($postData['body'])
->setShortDescription($postData['short_description'])
->save();
Mage::getSingleton('adminhtml/session')->addSuccess(Mage::helper('adminhtml')->__('Item was successfully saved'));
Mage::getSingleton('adminhtml/session')->setNewsData(false);
$this->_redirect('*/*/');
return;
} catch (Exception $e) {
Mage::getSingleton('adminhtml/session')->addError($e->getMessage());
Mage::getSingleton('adminhtml/session')->setNewsData($this->getRequest()->getPost());
$this->_redirect('*/*/edit', array('id' => $this->getRequest()->getParam('id')));
return;
}
}
$this->_redirect('*/*/');
}
public function deleteAction()
{
if( $this->getRequest()->getParam('id') > 0 ) {
try {
$newsModel = Mage::getModel('news/post');
$newsModel->setId($this->getRequest()->getParam('id'))
->delete();
Mage::getSingleton('adminhtml/session')->addSuccess(Mage::helper('adminhtml')->__('Item was successfully deleted'));
$this->_redirect('*/*/');
} catch (Exception $e) {
Mage::getSingleton('adminhtml/session')->addError($e->getMessage());
$this->_redirect('*/*/edit', array('id' => $this->getRequest()->getParam('id')));
}
}
$this->_redirect('*/*/');
}
/**
* Product grid for AJAX request.
* Sort and filter result for example.
*/
public function gridAction()
{
$this->loadLayout();
$this->getResponse()->setBody(
$this->getLayout()->createBlock('importedit/adminhtml_news_grid')->toHtml()
);
}
}
So, the question is: how should i do this in a right way?
The trick was to change the default form creation code
$form = new Varien_Data_Form(array(
'id' => 'edit_form',
'action' => $this->getUrl('*/*/save', array('id' => $this->getRequest()->getParam('id'))),
'method' => 'post',
));
to
$form = new Varien_Data_Form(array(
'id' => 'edit_form',
'action' => $this->getUrl('*/*/save', array('id' => $this->getRequest()->getParam('id'))),
'method' => 'post',
'enctype' => 'multipart/form-data'
));
Not within the code i've provided, but within the caller for that partial (..._Adminhtml_News_Edit_Form
instead of ..._Adminhtml_News_Edit_Tab_Form
).
精彩评论