Changeset 23477

Timestamp:

2014/05/30 18:14:20 (10 years ago)

Author:

pineray

Message:

#2448 typo修正・ソース整形・ソースコメントの改善 for 2.13.3

Location:

branches/version-2_13-dev/data/class

Files:

• helper/SC_Helper_DB.php (modified) (1 diff)
• pages/admin/order/LC_Page_Admin_Order_Edit.php (modified) (7 diffs)
• util/GC_Utils.php (modified) (1 diff)
• util/SC_Utils.php (modified) (8 diffs)

branches/version-2_13-dev/data/class/helper/SC_Helper_DB.php

@@ -1461 +1461 @@
      * SC_Utils_Ex::sfCalcIncTax とどちらか統一したほうが良い
      *
-     * @param  integer $price 計算対象の金額
-     * @return integer 税金付与した金額
-     */
-    public function sfCalcIncTax($price, $tax = null, $tax_rule = null)
+     * @param  int $price 計算対象の金額
+     * @param  int $tax
+     * @param  int $tax_rule
+     * @return int 税金付与した金額
+     */
+    public static function sfCalcIncTax($price, $tax = null, $tax_rule = null)
     {
         // 店舗基本情報を取得

branches/version-2_13-dev/data/class/pages/admin/order/LC_Page_Admin_Order_Edit.php

@@ -977 +977 @@
      *
      * @param  SC_FormParam                   $objFormParam SC_FormParam インスタンス
-     * @return アンカーキーの文字列
+     * @return string アンカーキーの文字列
      */
     public function getAnchorKey(&$objFormParam)
@@ -1034 +1034 @@
      *
      * @param  SC_FormParam $objFormParam         SC_FormParam インスタンス
-     * @param  integer      $add_product_class_id 変更商品規格ID
+     * @param  integer      $edit_product_class_id 変更商品規格ID
      * @param  integer      $change_no            変更対象
      * @return void
@@ -1115 +1115 @@
      * 変更対象のデータを上書きする
      *
-     * @param  array   $arrShipmentProducts 変更対象配列
-     * @param  array   $arrProductInfo      上書きデータ
-     * @param  integer $shipping_id         配送先ID
-     * @param  array   $no                  変更対象
+     * @param  array    $arrShipmentProducts 変更対象配列
+     * @param  array    $arrProductInfo      上書きデータ
+     * @param  int      $shipping_id         配送先ID
+     * @param  int      $no                  変更対象
      * @return void
      */
@@ -1144 +1144 @@
         // 配送先が存在する時のみ、商品個数の再設定を行います
         if(!SC_Utils_Ex::isBlank($arrShipmentsItems)) {
+            $arrUpdateQuantity = array();
             foreach ($arrShipmentsItems as $arritems) {
                 foreach ($arritems['shipment_product_class_id'] as $relation_index => $shipment_product_class_id) {
@@ -1151 +1152 @@
 
             $arrProductsClass = $objFormParam->getValue('product_class_id');
-            $arrProductsQuantity = $objFormParam->getValue('quantity');
+            $arrQuantity = array();
             foreach ($arrProductsClass as $relation_key => $product_class_id) {
-                $arrQuantity['quantity'][$relation_key] = $arrUpdateQuantity[$product_class_id];
-            }
-            $objFormParam->setParam($arrQuantity);
+                $arrQuantity[$relation_key] = isset($arrUpdateQuantity[$product_class_id]) ? $arrUpdateQuantity[$product_class_id] : 0;
+            }
+            $objFormParam->setParam(array('quantity' => $arrQuantity));
         }
     }
@@ -1175 +1176 @@
                 foreach ($arrDeleteKeys as $delete_key) {
                     $arrProducts = $objFormParam->getValue($delete_key);
+                    $arrUpdateParams = array();
                     foreach ($arrProducts as $index => $product_info) {
                         if ($index != $relation_index) {
@@ -1192 +1194 @@
      * @param  SC_FormParam $objFormParam          SC_FormParam インスタンス
      * @param  array        $arrShipmentDeleteKeys 削除項目
-     * @param  integer      $delete_shipping_id　 削除配送ID
-     * @param  array        $delete_no             削除対象
-     * @return void
-     */
-    public function deleteShipment(&$objFormParam, $arrShipmentDeletKeys, $delete_shipping_id, $delete_no)
-    {
-            foreach ($arrShipmentDeletKeys as $delete_key) {
-                $arrShipments = $objFormParam->getValue($delete_key);
-                foreach ($arrShipments as $shipp_id => $arrKeyData) {
-                    if (empty($arrKeyData)) continue;
-                    foreach ($arrKeyData as $relation_index => $shipment_info) {
-                        if ($relation_index != $delete_no || $shipp_id != $delete_shipping_id) {
-                            $arrUpdateParams[$delete_key][$shipp_id][] = $shipment_info;
-                        }
+     * @param  int          $delete_shipping_id　  削除配送ID
+     * @param  int          $delete_no             削除対象
+     * @return array
+     */
+    public function deleteShipment(&$objFormParam, $arrShipmentDeleteKeys, $delete_shipping_id, $delete_no)
+    {
+        $arrUpdateParams = array();
+        foreach ($arrShipmentDeleteKeys as $delete_key) {
+            $arrShipments = $objFormParam->getValue($delete_key);
+            foreach ($arrShipments as $shipp_id => $arrKeyData) {
+                if (empty($arrKeyData)) continue;
+                foreach ($arrKeyData as $relation_index => $shipment_info) {
+                    if ($relation_index != $delete_no || $shipp_id != $delete_shipping_id) {
+                        $arrUpdateParams[$delete_key][$shipp_id][] = $shipment_info;
                     }
                 }
             }
-            //$objFormParam->setParam($arrUpdateParams);
-            return $arrUpdateParams;
-        }
+        }
+        //$objFormParam->setParam($arrUpdateParams);
+        return $arrUpdateParams;
+    }
 
     /**
      * 受注商品一覧側に商品を追加
      *
-     * @param  SC_FormParam $objFormParam                    SC_FormParam インスタンス
-     * @param  array        $arrProductClassIds　           対象配列の商品規格ID
-     * @param  integer      $indert_product_class_id　追?? 商品規格ID
-     * @param  array        $arrAddProductInfo               追加データ
-     * @return array        $arrAddProducts           更新データ
+     * @param SC_FormParam  $objFormParam               SC_FormParam インスタンス
+     * @param array         $arrProductClassIds 　      対象配列の商品規格ID
+     * @param int           $insert_product_class_id    追加商品規格ID
+     * @param array         $arrAddProductInfo          追加データ
+     * @return array|null   $arrAddProducts             更新データ
      */
     public function checkInsertOrderProducts(&$objFormParam, $arrProductClassIds, $insert_product_class_id, $arrAddProductInfo)

branches/version-2_13-dev/data/class/util/GC_Utils.php

@@ -254 +254 @@
      * [注釈] -
      *----------------------------------------------------------------------*/
-    public function gfMakePassword($pwLength)
+    public static function gfMakePassword($pwLength)
     {
         // 乱数表のシードを決定

branches/version-2_13-dev/data/class/util/SC_Utils.php

@@ -693 +693 @@
      * @return integer 税金額
      */
-    public function sfTax($price, $tax, $tax_rule)
+    public static function sfTax($price, $tax, $tax_rule)
     {
         $real_tax = $tax / 100;
@@ -713 +713 @@
      * @return integer 税金付与した金額
      */
-    public function sfCalcIncTax($price, $tax, $tax_rule)
+    public static function sfCalcIncTax($price, $tax, $tax_rule)
     {
         return $price + SC_Utils_Ex::sfTax($price, $tax, $tax_rule);
@@ -723 +723 @@
         $adjust = pow(10 ,$pow-1);
 
-        // 整数且つ0出なければ桁数指定を行う
+        // 整数且つ0でなければ桁数指定を行う
         if (SC_Utils_Ex::sfIsInt($adjust) and $pow > 1) {
             $ret = (round($value * $adjust)/$adjust);
@@ -741 +741 @@
      * @return int
      */
-    public function sfPrePoint($price, $point_rate, $rule = POINT_RULE)
+    public static function sfPrePoint($price, $point_rate, $rule = POINT_RULE)
     {
         $real_point = $point_rate / 100;
@@ -813 +813 @@
     }
 
-    /* 配列をキー名ごとの配列に変更する */
-    public function sfSwapArray($array, $isColumnName = true)
+    /**
+     * 配列をキー名ごとの配列に変更する
+     *
+     * @param array $array
+     * @param bool $isColumnName
+     * @return array
+     */
+    public static function sfSwapArray($array, $isColumnName = true)
     {
         $arrRet = array();
@@ -837 +843 @@
      * $requires が指定された場合, $requires に含まれるキーの値のみを返す.
      *
-     * @param array 連想配列
-     * @param array 必須キーの配列
+     * @param array $hash 連想配列
+     * @param array $requires 必須キーの配列
      * @return array 連想配列の値のみの配列
      */
@@ -899 +905 @@
     }
 
-    // 二回以上繰り返されているスラッシュ[/]を一つに変換する。
+    /**
+     * 二回以上繰り返されているスラッシュ[/]を一つに変換する。
+     *
+     * @param string $istr
+     * @return string
+     */
     public function sfRmDupSlash($istr)
     {
@@ -910 +921 @@
         } else {
             $str = $istr;
+            $head = '';
         }
         $str = preg_replace('|[/]+|', '/', $str);