search

fecshop / yii2_fecshop

yii2 ( PHP ) fecmall（fecshop） core code used for ecommerce shop 多语言多货币多入口的开源电商 B2C 商城，支持移动端vue, app, html5，微信小程序微店，微信小程序商城等
http://www.fecmall.com
BSD 3-Clause "New" or "Revised" License
5.24k stars 1.42k forks source link

/customer/address/remove api 缺少return #32

Closed mistery-li closed 6 years ago

mistery-li commented 6 years ago

阅读源码时发现该api方法下面缺少return 响应

fancyecommerce commented 6 years ago

看不懂 http://www.fecshop.com/topic/624

mistery-li commented 6 years ago

很抱歉，因为是第一次弄这个，是我的错

我在github上clone的yii2_fecshop源码阅读的时候发现appserver\modules\controllers\AddressController.php文件发现actionRemove类方法缺少了return；

这个EditaccountController.php 我觉得应该加一个未登陆或者token过期的响应会不会更好 ？

2018-06-24 14:20 GMT+08:00 Terry notifications@github.com:

Closed #32 https://github.com/fecshop/yii2_fecshop/issues/32.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/fecshop/yii2_fecshop/issues/32#event-1697482784, or mute the thread https://github.com/notifications/unsubscribe-auth/ASz1dz9j6muLNk4iRQ_lg8DPCUooVKp4ks5t_y-rgaJpZM4UzZLZ .

mistery-li commented 6 years ago

好像图片崩了，不好意思，我还是去论坛注册一个账号