[現象]
複数配送設定を有効にしたところ、
異なる配送方法の商品をカート内に入れることはできたのですが、
「レジに進む」ボタンをクリックすると500エラーが出てしまいます。




エラー：
FatalErrorException in DeliveryRepository.php line 106:
Error: Call to a member function getId() on null
in DeliveryRepository.php line 106

EC-CUBE自体の経験もとても浅く、
エラーの内容はIDを取れて来ていないということだと思っているのですが、
どのようにすればエラー回避ができるのか、ご教授いただけますようお願い申し上げます。


該当箇所
/**
* 選択可能な配送業者を取得
*
* @param $payments
* @return array
*/
public function findAllowedDeliveries($productTypes, $payments)
{

$d = $this->getDeliveries($productTypes);
$arr = array();

foreach ($d as $Delivery) {
$paymentOptions = $Delivery->getPaymentOptions();

foreach ($paymentOptions as $PaymentOption) {
foreach ($payments as $Payment) {
if ($PaymentOption->getPayment()->getId() == $Payment['id']) {
$arr[$Delivery->getId()] = $Delivery;
break;
}
}
}

}

return array_values($arr);

}



---------------------------------

[EC-CUBE] 　　　　EC-CUBE 3.0.10
[レンタルサーバ] Xserver
[データベース] 　　　　MySQL 5.5.45
[ブラウザ] 　　　　Chrome
[導入プラグインの有無] GmoEpsilon,おすすめ商品管理プラグイン,サイトマップ,定休日管理プラグイン,関連商品プラグイン

最新の3.0.17で動作確認をしてみましたが500のエラーは発生しませんでした。

バグが修正された直接的なプルリクは見つけられなかったのですが、3.0.14で商品追加のバリデーション周りが修正されているようです。
https://github.com/EC-CUBE/ec-cube/pull/1665

お使いのバージョンが3.0.10と少し古いバージョンをお使いになられているので、一度バージョンアップを検討されてみてはいかがでしょうか？

管理画面の設定＞基本情報設定＞配送方法設定の配送先編集画面にて、
支払方法設定のチェックボックスが1件も選択されていない配送先が存在していませんか？
配送方法に紐づく支払方法のIDを取得しようとしてエラーになっていると思います。

$PaymentOption->getPayment()->getId()
の箇所で、
$PaymentOption->getPayment()の返り値がnullになっており、
nullにはgetId()というメソッドはありませんという旨のエラーが発生しております。

通常、画面操作ではこのようなデータは登録できないと思うので、
データベースでデータを確認されたほうがよいかもしれません。

----------------
株式会社シロハチ
■ECCUBE2系、3系構築カスタマイズご相談ください。
■EC-CUBE3マニュアル
■blog

okazy様

ご返信ありがとうございます。


仰る通りですね...


以前プラグインとの兼ね合いでエラーを起こしたこともあるので、
早速、ローカルを作成して一度バージョンアップしてみようと思います。


結果について追ってご報告させていただきます。

株式会社シロハチ様

ご返信ありがとうございます。



管理画面の設定＞基本情報設定＞配送方法設定の配送先編集画面にて、
支払方法設定のチェックボックスが1件も選択されていない配送先が存在していませんか？

＞確認致しましたが、すべて何かしらの支払い方法設定のチェックボックスが選択されておりました。。。




配送方法に紐づく支払方法のIDを取得しようとしてエラーになっていると思います。

$PaymentOption->getPayment()->getId()
の箇所で、
$PaymentOption->getPayment()の返り値がnullになっており、
nullにはgetId()というメソッドはありませんという旨のエラーが発生しております。

通常、画面操作ではこのようなデータは登録できないと思うので、
データベースでデータを確認されたほうがよいかもしれません。

＞エラーの意味はそのような内容だったのですね。 ご丁寧にご説明いただき、ありがとうございます。
　データベースで確認した方が良いとのことですが、具体的な確認事項をお教えいただくことは可能でしょうか？
　何から何まで申し訳ございませんが、ご教授いただけますと幸いです。

データベースのテーブル構造を確認して説明をまとめますので、少しお待ちください。

----------------
株式会社シロハチ
■ECCUBE2系、3系構築カスタマイズご相談ください。
■EC-CUBE3マニュアル
■blog

関連のありそうなテーブルは以下のテーブルかと思います。

dtb_delivery = 配送業者マスタ
dtb_payment = 支払方法マスタ
dtb_payment_option = 配送業者毎の支払方法(有効)のデータ
(delivery_id, payment_idで繋がりを管理)

例えば、dtb_deliveryテーブルにレコードが存在するがdtb_payment_optionテーブルに1件もレコードが存在しないような
配送業者はありませんか？
もしくはdtb_payment_optionテーブルにdtb_paymentテーブルに存在しないレコードが登録されていたりしないでしょうか？

----------------
株式会社シロハチ
■ECCUBE2系、3系構築カスタマイズご相談ください。
■EC-CUBE3マニュアル
■blog

株式会社シロハチ様

ご返信ありがとうございます。
また、原因についてお調いただき、誠にありがとうございます。


確認致しましたが、いずれもそのような状態は確認できませんでした。。。

ですが、DeliveryRepository.phpを最新バージョンのものに差し替えたところ、
上手く動作するようになりました。




ご教授頂きありがとうございました。