Magento – Remove and recreate custom option programatically on product before save event

custom-optionsevent-observermagento-1.7option

I m creating a custom option programmatically using observer with the catalog_product_save_before event.The custom option was created successfully on product save. but the problem is whenever the product saves a second time it will create the same option on every save action.So I want to remove all the options first before save and recreate all the options on save. My observer code for creating custom option is as below

public function createMyCustomOption(Varien_Event_Observer $observer)
{
    //Mage::getSingleton('catalog/product_option')->unsetOptions();
    $this->saveProductTabData($observer);
    $product = $observer->getProduct();
    $globlaOptions = unserialize(Mage::getStoreConfig("toursection/tourgroup/product_option"));
    $globlaOptionsArray = array();
    foreach ($globlaOptions as $key=>$value){
        $item['optiontitle'] = $value['optiontitle'];
        $item['optionprice'] = $value['optionprice'];
        $globlaOptionsArray[] = $item;
    }
    //print_r($globlaOptionsArray);
   
    //print_r($options);exit;
    
        $options = $product->getProductOptions();
    echo '<pre>';
    
      print_r($options);exit;
        /*if ($options) {
            foreach ($options as $option) {
                if ($option['title'] == 'Adult' || $option['title'] == 'Child') {
                    return;
                }
            }
        }*/
        
        $noOfAdult         = Mage::getStoreConfig("toursection/tourgroup/noofadult");
        $noOfchild         = Mage::getStoreConfig("toursection/tourgroup/nofochild");
        $adultPrice    = $product->getAdultOurprice();
        $childPrice    = $product->getChildOurprice();
        if(!$product_id = Mage::app()->getRequest()->getParam('id')){
                $productModel = Mage::getModel('catalog/product');
                $product_entity_table = $productModel->getResource()->getEntityTable();    
                $resource = Mage::getSingleton('core/resource');
                $connection = $resource->getConnection('core_read');
                $result = $connection->showTableStatus($product_entity_table);
                $product_id = $result['Auto_increment'];
            }
        $optionModel = Mage::getModel('touroptions/touroptions')->load($product_id,'product_id');
        $tourOptionArray = unserialize($optionModel->getTouroptions());
        print_r($tourOptionArray);
        $optionsTitle = array('Adult','Child');
        $optionsPrice = array($adultPrice,$childPrice);
        $noOfOption = array(
            $noOfAdult,
            $noOfchild
        );
        $optionsSku = array(NULL,NULL);
        $isRequired    = array(
            1
        );
        $typeArray = array(
        'drop_down',
        'drop_down'
        );
        $inc = 0;
        foreach($globlaOptionsArray as $title){
            if(!in_array($title['optiontitle'],$tourOptionArray)){
                continue;
            }
            $optionsTitle[]=$title['optiontitle'];
            $optionsPrice[]=$title['optionprice'];
            $noOfOption[] = '1';
            $optionsSku[] = $tourOptionArray['optionsku'][$inc];
            $isRequired[] = 0;
            $typeArray[] = 'checkbox';
            $inc++;
            
        }
        
      
        $optionsArray  = $this->getOptions($optionsTitle, $typeArray, $noOfOption, $optionsPrice, $isRequired , $optionsSku);
        //echo '<pre>';
        //print_r($optionsArray);exit;
        try {
            $product->setCanSaveCustomOptions(true);
            foreach ($optionsArray as $option) {
                $product->getOptionInstance()->addOption($option);
            }
            $product->setHasOptions(true);
        }
        catch (Exception $e) {
            Mage::Log($e->getMessage());
        }
       
    
    
}
protected function getOptions($optionsTitle = '', $typeArray = '', $noOfOption, $optionsPrice, $isRequired, $optionsSku)
{
    
    if (is_array($optionsTitle)) {
        $j = 0;
        foreach ($optionsTitle as $ttl) {
            $option[$j]               = array();
            $optionvalue              = array();
            $option[$j]['title']      = $ttl;
            $option[$j]['type']       = $typeArray[$j];
            $option[$j]['is_require'] = $isRequired[$j];
            $option[$j]['sort_order'] = $j;
            for ($i = 1; $i <= $noOfOption[$j]; $i++) {
                $optionvalue[$i]['title']      = $i;
                $optionvalue[$i]['price']      = $i * $optionsPrice[$j];
                $optionvalue[$i]['price_type'] = 'fixed';
                $optionvalue[$i]['sku']        = $optionsSku[$j];
                $optionvalue[$i]['sort_order'] = $i;
                
            }
            $option[$j]['values'] = $optionvalue;
            $j++;
        }
    } 
    return $option;
}

any help would be much appreciated.
Thanks.

Best Answer

Try this before adding your new options:

$oldOptions = $product->getOptionInstance()->getOptions();
foreach ($oldOptions as $option){
    $option->setIsDelete(true);
}

Related Solutions

Magento – Product Custom Option Duplication on Save Within Loop

OK so I finally came back to this issue and the following post helped me figure this one out: https://stackoverflow.com/questions/4006260/magento-accumulating-custom-options-in-script

As the user in the post above says, the Magento product option model was 'designed with a singleton pattern', which means for each loop I am adding the product options again, then setting these duplicated options to the following product.

So the fix is simply to call Mage::getSingleton('catalog/product_option')->unsetOptions(); before we call setProductOptions. There is no need to call load() either, the model returned in the collection is adequate.

So yeah, simple when you know how eh! I was really tearing my hair out on this one, thanks Magento!

Creating Custom Option on Product Save in Magento 1.7

Until I find the exact problem here is the solution. Instead catalog_product_save_before use the event catalog_product_prepare_save. The downside of this is that the prepare_save event is dispatched only when saving a product from the admin interface or the API. So if you are saving from a custom code this won't get triggered unless you trigger it manually.

I have a hunch that the problem has something to do with the Mage_Catalog_Model_Product::_beforeSave() method. In there there is some processing of the custom options.
But catalog_product_save_before is dispatched after this processing occurs, so while the custom options are processed in Mage_Catalog_Model_Product::_beforeSave() they are actually empty because the event didn't fire yet so they are not added.
If you move the line parent::_beforeSave(); in the method I mentioned at the top of the method, the options are added (still twice, but they are added). I will post more when/if I find the problem.

[EDIT]
Found it. I was somehow right in the lines above.
Like I said the problem is that catalog_product_save_before is dispatched after the custom options are processed. but this is why it doesn't work.
The custom options are saved in Mage_Catalog_Model_Product::_afterSave() by this code:

$this->getOptionInstance()->setProduct($this)
            ->saveOptions();

But $this->getOptionInstance() is populated with options in _beforeSave when the options array is empty in your case. Hence...nothing to save.
If you still want to use the catalog_product_save_before here is the code that should work.

//check that we haven't made the option already
$options = $product->getOptions();
if ($options){
    foreach ($options as $option) {
        if ($option['title'] == 'Auto Date & Time' && $option['type'] == 'date_time' && !$option['is_delete']) {
            //we've already added the option
            return;
        }
    }
}
$option = array(
    'title' => 'Auto Date & Time',
    'type' => 'date_time',
    'is_require' => 1,
    'sort_order' => 0,
    'is_delete' => '',
    'previous_type' => '',
    'previous_group' => '',
    'price' => '0.00',
    'price_type' => 'fixed',
    'sku' => ''
);
//don't use it like this because it has no effect
//$product->setProductOptions($options);
$product->setCanSaveCustomOptions(true);
//use it this way. Populate `$product->getOptionInstance()` directly
$product->getOptionInstance()->addOption($option);
//don't forget to state that the product has custom options
$product->setHasOptions(true);

Best Answer

Related Solutions

Magento – Product Custom Option Duplication on Save Within Loop

Creating Custom Option on Product Save in Magento 1.7

Related Topic