Authored by hf

code review by hf: fixes bug refs YW-1662 filter exception api return data

@@ -100,20 +100,23 @@ class UserModel @@ -100,20 +100,23 @@ class UserModel
100 /** 100 /**
101 * 处理个人中心页面优选新品数据 101 * 处理个人中心页面优选新品数据
102 * 102 *
103 - * @param int $yh_channel 频道,1代表男生,2代表女生,3代表潮童,4代表创意生活 103 + * @param int $channel 频道,1代表男生,2代表女生,3代表潮童,4代表创意生活
104 * @return array|mixed 处理之后的个人中心页面优选新品数据 104 * @return array|mixed 处理之后的个人中心页面优选新品数据
105 */ 105 */
106 - public static function getPreferenceData($yh_channel) 106 + public static function getPreferenceData($channel)
107 { 107 {
108 $result = array(); 108 $result = array();
109 109
110 // 调用接口获取个人中心页面优选新品数据 110 // 调用接口获取个人中心页面优选新品数据
111 - $preferenceData = UserData::preference($yh_channel); 111 + $preferenceData = UserData::preference($channel);
112 112
113 // 处理个人中心页面优选新品数据 113 // 处理个人中心页面优选新品数据
114 - if (isset($preferenceData['data']) && !empty($preferenceData['data'])) {  
115 - foreach ($preferenceData['data'] as $val) {  
116 - $result['recommendList'][] = Helpers::formatProduct($val, false, true, true, 299, 388, false, false); 114 + if (!empty($preferenceData['data'])) {
  115 + foreach ($preferenceData['data'] as $value) {
  116 + $value = Helpers::formatProduct($value, false, true, true, 299, 388, false, false);
  117 + if (false !== $value) {
  118 + $result['recommendList'][] = $value;
  119 + }
117 } 120 }
118 } 121 }
119 122
@@ -550,7 +550,10 @@ class DetailModel @@ -550,7 +550,10 @@ class DetailModel
550 $preference = DetailData::preference($productSkn, $channel, $brandId); 550 $preference = DetailData::preference($productSkn, $channel, $brandId);
551 if (!empty($preference)) { 551 if (!empty($preference)) {
552 foreach ($preference as $value) { 552 foreach ($preference as $value) {
553 - $result['recommendList'][] = Helpers::formatProduct($value, false, true, true, 299, 388, false, false); 553 + $value = Helpers::formatProduct($value, false, true, true, 299, 388, false, false);
  554 + if (false !== $value) {
  555 + $result['recommendList'][] = $value;
  556 + }
554 } 557 }
555 } 558 }
556 } 559 }