Authored by Rock Zhang

排除浏览记录中下架的商品

Code Review By Rock Zhang
@@ -301,31 +301,49 @@ class UserModel @@ -301,31 +301,49 @@ class UserModel
301 301
302 $records = UserData::browseRecord($uid, $udid, $page, $limit); 302 $records = UserData::browseRecord($uid, $udid, $page, $limit);
303 303
304 - if (!$records) {  
305 - $result['walkwayUrl'] = Helpers::url('/product/new');  
306 - $result['noRecord'] = true;  
307 -  
308 - return $result;  
309 - }  
310 -  
311 // 处理数据 304 // 处理数据
312 - if (isset($records['data']['product_list'])) { 305 + do {
  306 + if (!$records) {
  307 + $result['walkwayUrl'] = Helpers::url('/product/new');
  308 + $result['noRecord'] = true;
  309 + break;
  310 + }
  311 +
  312 + if (!isset($records['data']['product_list'])) {
  313 + break;
  314 + }
313 315
314 // 不能再查到结果了 316 // 不能再查到结果了
315 if ($page == 1 && $records['data']['total'] === 0) { 317 if ($page == 1 && $records['data']['total'] === 0) {
316 $result['walkwayUrl'] = Helpers::url('/product/new'); 318 $result['walkwayUrl'] = Helpers::url('/product/new');
317 $result['noRecord'] = true; 319 $result['noRecord'] = true;
318 - } else {  
319 - $data = $records['data']['product_list'];  
320 - foreach ($data as &$val) {  
321 - $val['link'] = Helpers::url('/product/show_' . $val['product_skn'] . '.html');  
322 - $val['image'] = !empty($val['image']) ? Helpers::getImageUrl($val['image'], 447, 596) : '';  
323 - $val['sales_price'] = Helpers::transPrice($val['sales_price']);  
324 - $val['market_price'] = ($val['market_price'] - $val['sales_price'] > 0) ? Helpers::transPrice($val['market_price']) : false; 320 + break;
  321 + }
  322 +
  323 + $data = $records['data']['product_list'];
  324 + $allRecords = array();
  325 + $record = array();
  326 + foreach ($data as &$val) {
  327 + // 排除下架的商品
  328 + if ($val['status'] == 0) {
  329 + continue;
325 } 330 }
326 - !empty($data) && $result['browseRecord'] = $data; 331 +
  332 + $record = array();
  333 + $record['product_name'] = $val['product_name'];
  334 + $record['product_id'] = $val['product_id'];
  335 + $record['link'] = Helpers::url('/product/show_' . $val['product_skn'] . '.html');
  336 + $record['image'] = !empty($val['image']) ? Helpers::getImageUrl($val['image'], 447, 596) : '';
  337 + $record['sales_price'] = Helpers::transPrice($val['sales_price']);
  338 + $record['market_price'] = ($val['market_price'] - $val['sales_price'] > 0) ? Helpers::transPrice($val['market_price']) : false;
  339 + $record['storage'] = $val['storage'];
  340 + $allRecords[] = $record;
327 } 341 }
328 - } 342 +
  343 + if (!empty($allRecords)) {
  344 + $result['browseRecord'] = $allRecords;
  345 + }
  346 + } while(false);
329 347
330 return $result; 348 return $result;
331 } 349 }