SC_Helper_Customer::sfGetCustomerData() 一致無しを考慮していない

[seasoftjapan]

中途半端な配列が返る。
false を返すとか、例外投げるとかすべき。

@@ -239,15 +239,17 @@ class SC_Helper_Customer
      * @param mixed $customer_id
      * @param boolean $mask_flg
      * @access public
-     * @return array 会員情報の配列を返す
+     * @return array|boolean 会員情報の配列を返す。該当する会員が存在しない場合、false を返する。
      */
     public function sfGetCustomerData($customer_id, $mask_flg = true)
     {
         $objQuery       =& SC_Query_Ex::getSingletonInstance();

         // 会員情報DB取得
-        $ret        = $objQuery->select('*', 'dtb_customer', 'customer_id=?', array($customer_id));
-        $arrForm    = $ret[0];
+        $arrForm = $objQuery->getRow('*', 'dtb_customer', 'customer_id=?', array($customer_id));
+        if (empty($arrForm)) {
+            return false;
+        }

         // 確認項目に複製
         $arrForm['email02'] = $arrForm['email'];

sfGetCustomerDataFromId() も似た状況だと思うが、empty な何か (null ?) が返りそう。(動作未確認) まだ、マシか。

[so-amuamu]

実害は現状でも無いと見られるが、 #118 の対応と共に修正したい