Ticket #410 (closed バグ指摘: 修正済)

Opened 14 years ago

Last modified 11 years ago

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

Reported by: Seasoft Owned by: somebody
Priority: Milestone: EC-CUBE2.11.1
Component: フロント Version: 2.4.0
Keywords: Cc:
修正済み: yes

Description (last modified by Seasoft) (diff)

\data\class\helper\SC_Helper_DB.php

sfGetDelivFee()

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

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

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

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

Change History

comment:1 Changed 14 years ago by 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'];
        }

comment:2 Changed 14 years ago by Seasoft

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

comment:3 Changed 13 years ago by kajiwara

  • Milestone changed from EC-CUBE2.3.4 to EC-CUBE2.x.x

comment:4 Changed 13 years ago by Seasoft

  • Version changed from 2.3.0 to 2.4.0

comment:5 Changed 13 years ago by kajiwara

  • Milestone changed from EC-CUBE2.4.1 to EC-CUBE2.4.2

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

comment:6 Changed 13 years ago by Yammy

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

comment:7 Changed 13 years ago by kajiwara

  • Milestone changed from EC-CUBE2.4.3 to EC-CUBE2.4.4

comment:8 Changed 12 years ago by nanasess

  • Milestone changed from EC-CUBE2.4.4 to EC-CUBE2.5.0

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

comment:9 Changed 12 years ago by Seasoft

  • Description modified (diff)

comment:10 Changed 12 years ago by nanasess

  • Milestone changed from EC-CUBE2.5.0beta to EC-CUBE2.6.0(仮)

comment:11 Changed 11 years ago by nanasess

  • Status changed from new to closed
  • 修正済み set
  • Resolution set to 修正済
  • Milestone changed from EC-CUBE2.12.0(仮) to EC-CUBE2.11.1

r20787 で解決

Note: See TracTickets for help on using tickets.