code review by fei.hong: do format new arrival codes

@@ -604,18 +604,14 @@ class Process
                 // single_image & floor & single_image
                 // single_image & floor & single_image
                 $list = array_merge($data[$key + 1]['data'], array_slice($data[$key + 2]['data'], 0, 6), $data[$key + 3]['data']);
                 $list = array_merge($data[$key + 1]['data'], array_slice($data[$key + 2]['data'], 0, 6), $data[$key + 3]['data']);
                 foreach ($list as $listKey => $val) {
                 foreach ($list as $listKey => $val) {
-                    $width = 185;
-                    $height = 248;
-                    $w = 377;
-                    $h = 504;
                     if ($listKey == 0 || $listKey == 7) {
                     if ($listKey == 0 || $listKey == 7) {
-                        $val['src'] = Images::getImageUrl($val['src'], $w, $h, 1);
+                        $val['src'] = Images::getImageUrl($val['src'], 377, 504, 1);
                         $result['newReport']['list'][] = array(
                         $result['newReport']['list'][] = array(
                             'href' => $val['url'],
                             'href' => $val['url'],
                             'img' => $val['src']
                             'img' => $val['src']
                         );
                         );
                     } else {
                     } else {
-                        $val['src'] = Images::getImageUrl($val['src'], $width, $height, 1);
+                        $val['src'] = Images::getImageUrl($val['src'], 185, 248, 1);
                         $result['newReport']['list'][] = array(
                         $result['newReport']['list'][] = array(
                             'href' => $val['url'],
                             'href' => $val['url'],
                             'img' => $val['src']
                             'img' => $val['src']
@@ -656,6 +652,8 @@ class Process
                     $result['category']['navs'] = self::mergeNavProcess($data[$key + 1], $type);
                     $result['category']['navs'] = self::mergeNavProcess($data[$key + 1], $type);
                 }
                 }
                 // floor模版
                 // floor模版
+                $width = 0; 
+                $height = 0;
                 foreach ($data[$key + 2]['data'] as $pos => $val) {
                 foreach ($data[$key + 2]['data'] as $pos => $val) {
                     $width = 185; $height = 510;
                     $width = 185; $height = 510;
                     if($pos == 1) {
                     if($pos == 1) {
@@ -761,8 +759,9 @@ class Process
     {
     {
         $result = array();
         $result = array();
         foreach ($data['data'] as $val) {
         foreach ($data['data'] as $val) {
-            if (empty($val))
+            if (empty($val)) {
                 continue;
                 continue;
+            }
             $result[] = array(
             $result[] = array(
                 'href' => $val['url'],
                 'href' => $val['url'],
                 'name' => $val['name']
                 'name' => $val['name']
@@ -790,8 +789,10 @@ class Process
             'client_type' => 'web'
             'client_type' => 'web'
         );
         );
         $goodsList = SearchData::searchElasticByCondition($params, false);
         $goodsList = SearchData::searchElasticByCondition($params, false);
-        $goodsList = $goodsList['data']['product_list'];
+        $goodsList = empty($goodsList) ? array() : $goodsList['data']['product_list'];
         $pos = 0;
         $pos = 0;
+        $url = '';
+        $oneGoods = array();
         foreach ($goodsList as $goods) 
         foreach ($goodsList as $goods) 
         {
         {
             $oneGoods = array();
             $oneGoods = array();
@@ -802,10 +803,10 @@ class Process
             if (empty($goods)) {
             if (empty($goods)) {
                 continue;
                 continue;
             }
             }
+            $url = Helpers::getUrlBySkc($goods['product_id'], $goods['goods_list'][0]['goods_id'], $goods['cn_alphabet']);
             $oneGoods['img'] = Images::getImageUrl($goods['goods_list'][0]['images_url'], 280, 373, 1);
             $oneGoods['img'] = Images::getImageUrl($goods['goods_list'][0]['images_url'], 280, 373, 1);
             $oneGoods['name'] = $goods['product_name'];
             $oneGoods['name'] = $goods['product_name'];
             $oneGoods['price'] = $goods['sales_price'];
             $oneGoods['price'] = $goods['sales_price'];
-            $url = Helpers::getUrlBySkc($goods['product_id'], $goods['goods_list'][0]['goods_id'], $goods['cn_alphabet']);
             $oneGoods['href'] = Helpers::transUrl($url, $type);
             $oneGoods['href'] = Helpers::transUrl($url, $type);
             $result[$goods['product_skn']] = $oneGoods;
             $result[$goods['product_skn']] = $oneGoods;
         }
         }
@@ -838,7 +839,6 @@ class Process
     public static function mergeHotBrandsData($key,array &$data,$type)
     public static function mergeHotBrandsData($key,array &$data,$type)
     {
     {
         $result = array();
         $result = array();
-        $temp = array();
         if ($data[$key]['template_name'] == 'custom_brands') 
         if ($data[$key]['template_name'] == 'custom_brands') 
         {
         {
             $result = array(
             $result = array(
@@ -858,9 +858,7 @@ class Process
             $brands = $data[$key]['data']['list'];
             $brands = $data[$key]['data']['list'];
             foreach ($brands as $val)
             foreach ($brands as $val)
                 {
                 {
-                    $width = 185;
-                    $height = 86;
-                    $val['src'] = Images::getImageUrl($val['src'], $width, $height, 2);
+                    $val['src'] = Images::getImageUrl($val['src'], 185, 86, 2);
                     $val['url'] = Helpers::transUrl($val['url'], $type);
                     $val['url'] = Helpers::transUrl($val['url'], $type);
                     $floor['logoBrand'][] = array(
                     $floor['logoBrand'][] = array(
                         'href' => $val['url'],
                         'href' => $val['url'],
@@ -868,10 +866,10 @@ class Process
                     );
                     );
                 }
                 }
             $floor['moreBrand'] = Helpers::url('/brands');
             $floor['moreBrand'] = Helpers::url('/brands');
-            $type_key = sprintf("%s_%s", $type, $key); 
-            $cacheKey = sprintf("%s_%s", WebCacheConfig::KEY_WEB_INDEX_BRANDS_LIST_DATA, $type_key);
+            $typeKey = sprintf("%s_%s", $type, $key); 
+            $cacheKey = sprintf("%s_%s", WebCacheConfig::KEY_WEB_INDEX_BRANDS_LIST_DATA, $typeKey);
             Cache::set($cacheKey, $floor, 86400);
             Cache::set($cacheKey, $floor, 86400);
-            $result['hotBrands']['brandUrl'] = '/common/getIndexResourceBrand?type=' . $type_key;
+            $result['hotBrands']['brandUrl'] = Helpers::url('/common/getIndexResourceBrand', array('type' => $typeKey));
         }
         }
         return $result;
         return $result;
     }
     }