Admin Form Field Value Not Saving to Database – Magento Solution

adminformadminhtmlimage-previewmodel

This is what I'm trying to achieve as the end result by the way:-

Magento Image Upload Preview

I think I may have just realised why $this->getImageHtml($model->getImgMain()) returns nothing. Is this meant to be getting the value from the database? If so, there is no value stored in the database. So the image uploader is uploading the image but not saving the name in the database. I think I'm getting somewhere?

Edit

I can confirm, that the value as per above is returned empty because the image filename is not being saved to the database:-

        $fldUpload = $form->addFieldset('upload', array('legend'=> $hlp->__('Image Upload')));
        $fldUpload->addField('img_main', 'file', array(
        'label'     => $hlp->__('Image'),
        'name'      => 'img_main',
        'required'  => false,
        'after_element_html' => $this->getImageHtml($model->getImgMain()),
        )); 
        $fldUpload->addField('remove_img_main', 'checkbox', array(
            'label'     => $hlp->__('Remove Image'),
            'name'      => 'remove_img_main',
            'value'     => 1,
        ));

And I created the table in the database img_main and whilst the image uploads, the filename is not saved to the database…

Really stumped on this one, I know the image filename is being generated correctly, I know that the field name is the same as the table name in the database. I can't figure out why the image filename is not being saved in the database table.

Best Answer

You are missing the necessary bit to actually write the data to the model/table.

On line $data[$field] = $newName; it alters the $data array that is used to store values pair and later you should call $model->setData($data) and then you need to actually save with $model->save();

You could pass a single value to the $models table column with $model->setImgMain("text here");. Note the camelcazing. After saving you could call $model->getImageMain() to get the stored value.

Related Solutions

Magento – pull database data into a text field

To answer your second question first :)

The first and last name information is added via a widget:

app/code/core/Mage/Customer/Block/Widget/Name.php
app/design/frontend/base/default/template/customer/widget/name.phtml

In the registration process these are added via:

echo $this->getLayout()->createBlock('customer/widget_name')
         ->setObject($this->getFormData())
         ->setForceUseCustomerAttributes(true)->toHtml()

I guess this was done as the name can be used in many locations, registration, address and checkout. By doing this via a widget you only need once place that adds the form fields and will cut down on duplication of code.

Now onto your first question. To pass data from the database into a textbox I would suggest that you add a new function into your block for this example getPrefcentreData.

What this function would do would take your snippet of code and load the information that you need from the database.

Then in your template you can simply call something like:

<input type="text" name="your_field" id="your_field" value="<?php $this->getPrefcentreData() ?>" class="text" />

Magento – How to Create a Custom Grid Filter

try adding a custom filter callback:

in your grid:

$this->addColumn('url', array(
    'header'   => Mage::helper('companymodule')->__('Has Website'),
    'align'    => 'left',
    'index'    => 'url',
    'renderer' => new Company_Module_Block_Adminhtml_Module_Grid_Renderer_HasUrl(),
    'type'      => 'options',
    'options'   => Mage::getSingleton('companymodule/hasurl')->getOptionArray(),
    'filter_condition_callback' => array($this, '_filterHasUrlConditionCallback')
));

and add a method like this:

protected function _filterHasUrlConditionCallback($collection, $column)
{
    if (!$value = $column->getFilter()->getValue()) {
        return $this;
    }
    if (empty($value)) {
        $this->getCollection()->getSelect()->where(
             "main_table.url IS NULL");
    }
    else {
        $this->getCollection()->getSelect()->where(
             "main_table.url IS NOT NULL");
    }

    return $this;
}

note it's not tested; you may have to check for empty rather than null on the database; also make sure you have the correct table alias (if URL is not on the main_table).

Edit

Best Answer

Related Solutions

Magento – pull database data into a text field

Magento – How to Create a Custom Grid Filter

Related Topic