都道府県が不明だと東京都と想定して動作する関数

[Seasoft]

\data\class\helper\SC_Helper_DB.php

sfGetDelivFee()

            // 都道府県が指定されていない場合は、東京都の番号を指定しておく
            if($pref == "") {
                $pref = 13;
            }

どのような利用想定なのか分かりませんが、良いのでしょうか?

都道府県が不明ならば、送料を返さずエラー処理にまわすのが妥当では?

正常動作のふりをする動作は疑問を感じます。

[Seasoft]

これも、同様に疑問を持ちます。

        // 支払い方法が指定されていない場合は、先頭の配送業者を取得する
        } else {
            $where = "del_flg = 0";
            $objQuery->setOrder("rank DESC");
            $objQuery->setLimitOffset(1);
            $arrRet = $objQuery->select("deliv_id", "dtb_deliv", $where);
            $deliv_id = $arrRet[0]['deliv_id'];
        }

[kajiwara]

申し訳ございません。 こちら、マイルストーンを2.4.2に変更させていただきたいと思います。

[Yammy]

この処理は、支払方法、配送先が未確定の状態である、LC_Page_Cart クラスの処理で、SC_Helper_DB::sfTotalCart() を呼んでいるための回避方法ですかね？ 根本的に直す必要がありそうですね。

[nanasess]

根本的な仕様検討が必要ですので milestone:EC-CUBE2.5.0 へ変更します

[nanasess]

r20787 で解決