无法清除的Magento系统日志错误

Question

我的系统有两个问题。记录我正在解决一些问题。该网站似乎运行良好，但我非常希望从日志中删除这些正在进行的错误。

注意：未定义变量：order in/var/www/。。。在线17

// line 17
$merchantnumber = $standard->getConfigData('merchantnumber', $order ? $order->getStoreId() : null);

警告：在/var/www/…中被零除。。。在线267

// line 267
"vat" => (float)round((string)((round($order->getBaseShippingInclTax(),2)-round($order->getBaseShippingAmount(),2))/round((string)$order->getBaseShippingAmount(),2))*100, 2)

更新

// line 258-281 
                $items = $order->getAllItems();
    foreach ($items as $itemId => $item)
    {
        $invoice["lines"][] = array
        (
            "id" => $item->getSku(),
            "description" => $item->getName(),
            "quantity" => round($item->getQtyOrdered(), 0),
            "price" => $item->getBasePrice()*100,
            "vat" => (float)round((string)((round($item->getBasePriceInclTax(),2)-round($item->getBasePrice(),2))/round((string)$item->getBasePrice(),2))*100, 2)
        );
    }

    $invoice["lines"][] = array
    (
        "id" => $order->getShippingMethod(),
        "description" => $order->getShippingDescription(),
        "quantity" => 1,
        "price" => $order->getBaseShippingAmount()*100,
        "vat" => (float)round((string)((round($order->getBaseShippingInclTax(),2)-round($order->getBaseShippingAmount(),2))/round((string)$order->getBaseShippingAmount(),2))*100, 2)
    );

    return json_encode($invoice);
}

我之前发布了错误的代码对不起，我在查看错误日志时感到困惑，因为相同的（Devider）错误出现在商品和订单部分

Answer 1

通知不是错误。您可以选择忽略它，也可以使用isset检查变量是否存在：

 ...er', isset($order) ? $...

应始终避免被零除，因为答案通常是未定义的（至少在数学上是未定义的）。在尝试除法之前，请检查分母是否为零。

Answer 2

在第一种情况下，您可以尝试将该行更改为

$merchantnumber = $standard->getConfigData('merchantnumber', (isset($order) && $order) ? $order->getStoreId() : null);

在第二种情况下，如果订单金额为美元，我无法准确地说出“vat”的值-

if (!$order->getBaseShippingAmount()) {
   return;
}

添加到第267行上方的某个位置，但如果没有看到代码，就很难确定。

Answer 3

不幸的是，Magento代码似乎经常依赖于PHP对html" target="_blank">逻辑错误的容忍度，这被视为注意事项。

 $merchantnumber = $standard->getConfigData('merchantnumber', $order ?
   $order->getStoreId() : null);

如果未设置$订单，我们希望使用NULL而不是订单的存储Id：

 $merchantnumber = $standard->getConfigData('merchantnumber',
   isset($order) ? $order->getStoreId() : null);

在第二个文件中：

更新检查您是否指的是$order或$Item。如果它在循环中，它可能是后者。

"vat" => (float)round((string)((round($item->getBaseShippingInclTax(),2)-round($item->getBaseShippingAmount(),2))/round((string)$item->getBaseShippingAmount(),2))*100, 2)

增值税%从ShippingInclTax（即ShippingAmount增值税）中反算得出。仅当getBaseShippingAmount（）为零时，计算才会崩溃。

为了应对这种情况，我们应该：

 'vat' => (0 == round($item->getBaseShippingAmount(),2)) ? 0 :
          (float)round((string)(
 (round($item->getBaseShippingInclTax(),2)
 -round($item->getBaseShippingAmount (),2)
 )/round((string)$item->getBaseShippingAmount(),2)
 )*100,2),

...我对所有这些四舍五入不太满意，但它们很可能存在，这样打印出来的账户就可以“检查”到最后一位小数，避免出现奇怪的结果，例如10.33 10.33 9.33=30.00而不是29.99。

我会写

 'vat' => (0 == $item->getBaseShippingAmount()) ? 0
        : round(
             100.0*(
               $item->getBaseShippingInclTax()/$item->getBaseShippingAmount()
               -1.0
             )
          ,2),

但即使在数学上更合理，我担心结果可能与Magento在其他地方打印的结果不匹配。

如果您在循环中使用$订单（意味着无论项目是什么，您都有相同的值），最好在循环之前计算增值税，然后在循环中使用它。