フロントエンドのイベントオブザーバー内で製品を保存しようとすると、Magentoが失敗しますか?


15

イベントオブザーバーに関連付けられている関数があります

要件の1つは、注文項目データをループするときに、特定の製品属性が空の場合、別のモデルを介して特定のデータを取得する要求を発行することです(正常に動作します)。

問題コードは基本的にこれに要約されます

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();
}

ただし、Magentoは次の例外をスローしています。

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}

magentoオブザーバーの外部のサードパーティスクリプトで同一のコードを使用できるのはなぜですか?しかし、そのオブザーバー内で実行しようとすると、そのランダムエラーメッセージで失敗しますか?


このエラーは、非配列がに解析される典型的なものですforeach。スタックトレースは、イベントがディスパッチされ、オブザーバーに障害があることを示しています。本当に$order->getAllItems()調整し直していarray()ますか?使用:Zend_Debug::dump($order->getAllItems());。しかし、それはいくつかの悪いコードを持つ別のオブザーバーかもしれませんか?
アッシュ14年

問題は、上部のスニペットではなく、Magentoコアコードにあります。
ズクスリアン14年

回答:


33

問題は、フロントエンドから製品を保存できないことです。

これは、フロントエンドで製品をロードするときにorigDataプロパティが入力されないという事実によって行われます。

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

    return $this;
}

そのため、製品を保存しようとすると、説明したエラーが発生します。

この問題は、現在のストアをadminに変更するときに、たとえば@magboyのコードを使用して解決できます。

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

より良い使用 Mage_Core_Model_App_Emulation

そして、私の好みの解決策はMage_Catalog_Model_ProductsetOrigDataメソッドを拡張して置き換えることです

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

私はここで書き換えについて話していません!保存を有効にするには、この1か所でのみモデルを使用してください。その後、他のすべての場所でセキュリティ機能をアクティブにします。


7

次のコード行を追加してみてください。

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

Magentoの現在のバージョンでこれを行う必要がある理由はわかりません。たぶん他の誰かが説明できますか?


currentstoreをAdminに設定して製品を保存した後、元の値に戻す必要がありますか?
ジュゼッペ

1
@Giuseppeそれはあなたが後でリクエストで何をしているのかに依存します。よりクリーンで、あなたがそれをすればより少ない問題につながります
-simonthesorcerer
弊社のサイトを使用することにより、あなたは弊社のクッキーポリシーおよびプライバシーポリシーを読み、理解したものとみなされます。
Licensed under cc by-sa 3.0 with attribution required.