Fix cs.

[successgo]

Fix cs.

[fancyecommerce]

1. https://github.com/fecshop/yii2_fecshop/pull/49/commits/312e63769b85bef1e8b955a5239acdd1aa8fae7d

use fecshop\services\page\Asset;
use fecshop\services\page\Breadcrumbs;
use fecshop\services\page\Currency;
use fecshop\services\page\Footer;
use fecshop\services\page\Menu;
use fecshop\services\page\Newsletter;
use fecshop\services\page\StaticBlock;
use fecshop\services\page\Theme;
use fecshop\services\page\Trace;
use fecshop\services\page\Translate;
use fecshop\services\page\Widget;

这个部分的代码需要去除，加这个是不对的，因为你不确定他到底用的是个子services，这个是通过配置来确定的，写这个存在误导。

为了IDE跳转， 试试这个

 * Page service.
 * 注意：下面是fecshop默认的子services路径文件，如果存在第三方插件，子services可能被重定向到其他地方的路径文件。
 * @property \fecshop\services\page\Asset $asset
 * @property \fecshop\services\page\Breadcrumbs $breadcrumbs
 * ....

然后去掉use的文件，上面加一个注意提示语，你看如何？

[fancyecommerce]

2.https://github.com/fecshop/yii2_fecshop/pull/49/commits/89a2db975c606c692de768252fe47ac11f357870

"ext-mongodb": "*",

这个部分required没有加的必要，因为存在依赖 "yiisoft/yii2-mongodb": "~2.1.7",

这个扩展： https://github.com/yiisoft/yii2-mongodb/blob/master/composer.json

里面有这个

[fancyecommerce]

3. https://github.com/fecshop/yii2_fecshop/blob/8c0c9b4dea4f4944fdecae2022de70daa8518c47/services/Category.php

可以将

    /**
     * @var CategoryInterface
     */
    protected $_category;

改成

    /**
    *  @notice：下面是fecshop默认的子services路径文件，如果存在第三方插件，子services可能被重定向到其他地方的路径文件。
     * @var \fecshop\services\category\CategoryInterface
     */
    protected $_category;

然后去掉use引入的部分

use fecshop\services\category\CategoryInterface;

[fancyecommerce]

4.https://github.com/fecshop/yii2_fecshop/pull/49/commits/6b287e23e8865000906c38f4d4b41a6ebb9acb02

\Exception $e

我已经改了，今天提交上来了

https://github.com/fecshop/yii2_fecshop/commit/7b64f6cfb8c1dc7ad89c7a3b02fb859d5776cdd6

[fancyecommerce]

其他的改的都很好， 第1和第3部分，你看看你的ide是否可以自动找文件。

[successgo]

2.89a2db9

"ext-mongodb": "*",

这个部分required没有加的必要，因为存在依赖 "yiisoft/yii2-mongodb": "~2.1.7",

这个扩展： https://github.com/yiisoft/yii2-mongodb/blob/master/composer.json

里面有这个

它有的那个 "ext-mongodb": ">=1.0.0" 跟本项目关系不是很大。 加上这样一句之后，有两个作用： 一是当在调用mongodb 相关函数方法时，不会有警告； 二是告诉别人本项目必须安装 mongodb 扩展。

可能重在于用处一吧。

[successgo]

1. 312e637

use fecshop\services\page\Asset;
use fecshop\services\page\Breadcrumbs;
use fecshop\services\page\Currency;
use fecshop\services\page\Footer;
use fecshop\services\page\Menu;
use fecshop\services\page\Newsletter;
use fecshop\services\page\StaticBlock;
use fecshop\services\page\Theme;
use fecshop\services\page\Trace;
use fecshop\services\page\Translate;
use fecshop\services\page\Widget;

这个部分的代码需要去除，加这个是不对的，因为你不确定他到底用的是个子services，这个是通过配置来确定的，写这个存在误导。

为了IDE跳转， 试试这个

 * Page service.
 * 注意：下面是fecshop默认的子services路径文件，如果存在第三方插件，子services可能被重定向到其他地方的路径文件。
 * @property \fecshop\services\page\Asset $asset
 * @property \fecshop\services\page\Breadcrumbs $breadcrumbs
 * ....

然后去掉use的文件，上面加一个注意提示语，你看如何？

的确，使用你的建议修改之后可以让 IDE 工作。但是两者之间仅有的一个区别是将命名空间放置到前头，use 进来不代表一定会 include，只是声明，只是一个别名。

在这里，两种方法的作用完全一致。

[successgo]

第 3 处，同 1.

[fancyecommerce]

2.89a2db9

"ext-mongodb": "*",

这个部分required没有加的必要，因为存在依赖 "yiisoft/yii2-mongodb": "~2.1.7", 这个扩展： https://github.com/yiisoft/yii2-mongodb/blob/master/composer.json 里面有这个

它有的那个 "ext-mongodb": ">=1.0.0" 跟本项目关系不是很大。 加上这样一句之后，有两个作用： 一是当在调用mongodb 相关函数方法时，不会有警告； 二是告诉别人本项目必须安装 mongodb 扩展。

可能重在于用处一吧。

当在调用mongodb 相关函数方法时，不会有警告, 可以列举一下在那些情况下吗？ 我没有遇到过你说的情况

[successgo]

警告来自来于 IDE 的提示。你如果没有用过 phpstorm 可以试用一下。这个是小事情。

[fancyecommerce]

我的环境在阿里云，都是用编辑器远程ftp编码

我先吧代码合并了，1和3部分，有时间按照讨论的方式修改下，重新发个PR吧