3

次のMagentoコアパッチを受け取りました。

Index: app/code/core/Mage/Adminhtml/controllers/Sales/OrderController.php
===================================================================
--- app/code/core/Mage/Adminhtml/controllers/Sales/OrderController.php
    app/code/core/Mage/Adminhtml/controllers/Sales/OrderController.php  (working copy)
@@ -718,6 +718,7 @@
     $addressId = $this->getRequest()->getParam('address_id');
     $address = Mage::getModel('sales/order_address')
         ->getCollection()
+            ->addFilter('entity_id', $addressId)
         ->getItemById($addressId);
     if ($address) {
         Mage::register('order_address', $address);

私の意見では、これは同等です

$addressId = $this->getRequest()->getParam('address_id');
$address = Mage::getModel('sales/order_address')->load($addressId);
if ($address->getId()) {
    Mage::register('order_address', $address);

私のソリューションに対する元のパッチの利点は何ですか？

編集：

サポートからの説明は次のとおりです。

[...]この決定は設計によるものであり、このコードを作成するときに適切でした。

4

1 に答える 1

4

この種のパッチの唯一の理由は、最小限の行数を編集することであるように思われます。

Magento2では、これは2番目の方法で実装されます。

$address = Mage::getModel('Mage_Sales_Model_Order_Address')->load($addressId);
于 2012-10-01T08:36:59.977 に答える