Magento non riesce quando si tenta di salvare il prodotto all'interno dell'osservatore di eventi sul frontend?


15

Ho una funzione legata a un osservatore di eventi

Uno dei requisiti è che quando scorre attraverso i dati dell'articolo dell'ordine, se un determinato attributo del prodotto è vuoto, emette una richiesta per ottenere dati specifici attraverso un modello separato (funzionando bene).

il codice problema si riduce sostanzialmente a questo

foreach ($order->getAllItems() as $key => $item) {
    /** @var Tantor_Catalog_Model_Product $productData */
    $productData = $item->getProduct();
    $_item = Mage::getModel('catalog/product')->load($productData->getId());
    $_item->setNetsuiteItemIid('foo');
    $_item->save();
}

tuttavia Magento sta gettando un'eccezione di

2014-03-05T21:14:14+00:00 ERR (3):
exception 'Exception' with message 'Warning: Invalid argument supplied for foreach()  in /var/www/html/app/code/core/Mage/Eav/Model/Entity/Abstract.php on line 1180' in /var/www/html/app/code/core/Mage/Core/functions.php:245
Stack trace:
#0 /var/www/html/app/code/core/Mage/Eav/Model/Entity/Abstract.php(1180): mageCoreErrorHandler(2, 'Invalid argumen...', '/var/www/html/a...', 1180, Array)
#1 /var/www/html/app/code/core/Mage/Eav/Model/Entity/Abstract.php(1123): Mage_Eav_Model_Entity_Abstract->_collectSaveData(Object(Tantor_Catalog_Model_Product))
#2 /var/www/html/app/code/core/Mage/Core/Model/Abstract.php(318): Mage_Eav_Model_Entity_Abstract->save(Object(Tantor_Catalog_Model_Product))
#3 /var/www/html/app/code/local/Tantor/Netsuite/Model/Observer.php(218): Mage_Core_Model_Abstract->save()
#4 /var/www/html/app/code/core/Mage/Core/Model/App.php(1338): Tantor_Netsuite_Model_Observer->saveOrder(Object(Varien_Event_Observer))
#5 /var/www/html/app/code/core/Mage/Core/Model/App.php(1317): Mage_Core_Model_App->_callObserverMethod(Object(Tantor_Netsuite_Model_Observer), 'saveOrder', Object(Varien_Event_Observer))
#6 /var/www/html/app/Mage.php(447): Mage_Core_Model_App->dispatchEvent('sales_order_pla...', Array)
#7 /var/www/html/app/code/core/Mage/Sales/Model/Order.php(1096): Mage::dispatchEvent('sales_order_pla...', Array)
#8 [internal function]: Mage_Sales_Model_Order->place()
#9 /var/www/html/app/code/core/Mage/Core/Model/Resource/Transaction.php(105): call_user_func(Array)
#10 /var/www/html/app/code/core/Mage/Core/Model/Resource/Transaction.php(159): Mage_Core_Model_Resource_Transaction->_runCallbacks()
#11 /var/www/html/app/code/core/Mage/Sales/Model/Service/Quote.php(189): Mage_Core_Model_Resource_Transaction->save()
#12 /var/www/html/app/code/core/Mage/Sales/Model/Service/Quote.php(249): Mage_Sales_Model_Service_Quote->submitOrder()
#13 /var/www/html/app/code/core/Mage/Checkout/Model/Type/Onepage.php(774): Mage_Sales_Model_Service_Quote->submitAll()
#14 /var/www/html/app/code/core/Mage/Checkout/controllers/OnepageController.php(511): Mage_Checkout_Model_Type_Onepage->saveOrder()
#15 /var/www/html/app/code/core/Mage/Core/Controller/Varien/Action.php(419): Mage_Checkout_OnepageController->saveOrderAction()
#16 /var/www/html/app/code/core/Mage/Core/Controller/Varien/Router/Standard.php(250): Mage_Core_Controller_Varien_Action->dispatch('saveOrder')
#17 /var/www/html/app/code/core/Mage/Core/Controller/Varien/Front.php(176): Mage_Core_Controller_Varien_Router_Standard->match(Object(Mage_Core_Controller_Request_Http))
#18 /var/www/html/app/code/core/Mage/Core/Model/App.php(354): Mage_Core_Controller_Varien_Front->dispatch()
#19 /var/www/html/app/Mage.php(683): Mage_Core_Model_App->run(Array)
#20 /var/www/html/index.php(86): Mage::run('', 'store')
#21 {main}

Perché dovrei essere in grado di utilizzare il codice identico in uno script di terze parti al di fuori dell'osservatore di Magento, ma quando provo a eseguirlo all'interno di quell'osservatore, non riesce con quel messaggio di errore casuale?


Questo errore è tipico di un non array che viene analizzato foreach. La traccia dello stack mostra che viene inviato un evento e l'osservatore è difettoso. Sei sicuro di $order->getAllItems()risintonizzare un array()? utilizzare: Zend_Debug::dump($order->getAllItems());. Tuttavia, potrebbe essere un altro osservatore con un codice errato?
ash

il problema è con il codice core di Magento, non con il mio frammento in alto.
Zxurian

Risposte:


33

Il problema è che non ti è permesso salvare prodotti dal frontend.

Ciò è dovuto al fatto che quando si carica un prodotto in frontend, la origDataproprietà non viene riempita:

public function setOrigData($key=null, $data=null)
{
    if (Mage::app()->getStore()->isAdmin()) {
        return parent::setOrigData($key, $data);
    }

    return $this;
}

Pertanto, quando si tenta di salvare il prodotto, viene generato l'errore descritto.

Puoi risolvere questo problema sia quando cambi l'archivio corrente in admin, ad esempio con il codice di @magboy:

Mage::app()->setCurrentStore(Mage_Core_Model_App::ADMIN_STORE_ID);

uso migliore Mage_Core_Model_App_Emulation

E la mia soluzione preferita è quella di estendere Mage_Catalog_Model_Producte sostituire il setOrigDatametodo

public function setOrigData($key = null, $data = null)
{
    if (is_null($key)) {
        $this->_origData = $this->_data;
    } else {
        $this->_origData[$key] = $data;
    }
    return $this;
}

NON sto parlando di una riscrittura qui! Usa il tuo modello solo su questo punto per abilitare il salvataggio. Quindi hai la funzione di sicurezza ancora attiva ovunque.


7

Prova ad aggiungere questa riga di codice:

Mage::app()->setCurrentStore(Mage_Core_Model_App::ADMIN_STORE_ID);

Non sono sicuro del motivo per cui devi farlo ora nella versione corrente di Magento. Forse qualcun altro può spiegare?


dopo aver impostato lo store corrente su Admin e aver salvato il prodotto, devo ripristinarlo al valore originale?
Giuseppe,

1
@Giuseppe Dipende da cosa farai in seguito con la richiesta; è più pulito e porterà a meno problemi se lo fai
simonthesorcerer
Utilizzando il nostro sito, riconosci di aver letto e compreso le nostre Informativa sui cookie e Informativa sulla privacy.
Licensed under cc by-sa 3.0 with attribution required.