3.2.4版本的路由失效

[hepeichun]

我正尝试将程序由3.2.2升级到3.2.4，中间发现我们的路由规则失效

[bigiot]

我的也有失效现象：
'URL_ROUTE_RULES'=>array(
'talk/:id\d' => 'Talk/talk',
'talkcate/:id\d' => 'Talk/category',
'mobile' => 'Index/mobile',
'donate' => 'Index/donate'
)
前两个的配置方法继续有效，后两个找不到控制器。

[hepeichun]

我的也有失效现象：
'URL_ROUTE_RULES'=> array（
'talk /：id \ d'=>'Talk / talk'，
'talkcate /：id \ d'=>'Talk / category'，
'mobile'=> 'index / mobile'，
'donate'=>'索引/捐赠'
）
前两个的配置方法继续有效，后两个找不到控制器。

坐等官方修复吧，手动无奈

[liu21st]

静态路由规则应该统一在URL_MAP_RULES配置里面定义的

[blackpuppy]

我也遇到同样的动态路由失效的问题，同样的无法加载控制器的错误。看3.2.4的9个commits和3.2.5的3个commits，完全没有改到路由，但是框架路由的行为发生了变化。原来应用程序的代码可以正常工作，升级到3.2.4之后却不行了。查看了一下2015年10月8日3.2.3发布之后的commits，其实有很多改动，远不止上述的12个commits，路由的部分，比如Library/Think/Dispatcher.class.php和ThinkPHP/Library/Think/Route.class.php都有比较大的变动。是否有相应的文档，说明这些框架的变化对应用程序有哪些影响，需要相应作出什么样的改动呢？

谢谢！

[blackpuppy]

@jdkysq @bigiot

经过一番调试，发现对框架做如下改动，可以修复我的问题：
ThinkPHP/Library/Think/Route.class.php第61行由：

if ($matches = self::checkUrlMatch($rule, $args, $regx)) {

改为：

$matches = self::checkUrlMatch($rule, $args, $regx);
if ($matches !== false) {

道理也很简单，我猜测私有方法checkUrlMatch()返回false表示动态路由未匹配成功，而在返回值的判断上，误把空数组也当成未匹配成功。

我还要测试一下，看这样是否适用于我所有的近300个动态路由(有web，也有API)。如果的确如此，我会提交一个PR。可能的话，希望大家也可以帮我测试一下，看是否能解决问题，另外是否还有其他问题。

既然3.2官方已经不维护了，所以要靠社区中还在使用该版本的我们自己来承担这个责任了。

谢谢！

[blackpuppy]

之前对路由的修正仍不完整。又经过几天折磨人的调试，终于使我近300个动态路由(有web，也有API)又能够在3.2.5下正常工作了，通过单元测试。下面是我对框架的完整修正(包括之前提到的)，只涉及一个文件ThinkPHP/Library/Think/Route.class.php：

ThinkPHP/Library/Think/Route.class.php第61行由：

                if ($matches = self::checkUrlMatch($rule, $args, $regx)) {

改为：

                $matches = self::checkUrlMatch($rule, $args, $regx);
                if ($matches !== false) {  // 严格检查false才表示动态路由未匹配成功

ThinkPHP/Library/Think/Route.class.php第440行由：

                            // 如果值在排除的名单里
                            if (in_array($var, $val[2])) {

改为：

                            // 如果值在排除的名单里
                            if (is_array($val[2]) && in_array($var, $val[2])) {

在ruleCache(...)中读取路由配置应当避免改变路由定义顺序，因为顺序也会影响路由的解析。ThinkPHP/Library/Think/Route.class.php第273行由：

            // 动态路由
            $result[1] = C('URL_ROUTE_RULES');
            if (!empty($result[1])) {
                foreach ($result[1] as $rule => $route) {

改为：

            // 动态路由
            $result[1] = [];
            $dynamicRoutes = C('URL_ROUTE_RULES');
            if (!empty($dynamicRoutes)) {
                foreach ($dynamicRoutes as $rule => $route) {

并删除ThinkPHP/Library/Think/Route.class.php第350-351行。

我会提交一个PR。希望大家帮助测试，以覆盖更多不同的使用用例。

谢谢！

[blackpuppy]

@liu21st 我已经提交了PR #535，请检查，如需改进请告知。
另外，建议对框架的修改，必须分外谨慎，因为框架是很多用户开发应用的基础，一旦引入了哪怕很小的错误，也有可能影响很多人。尤其是3.2已经过了维护生命周期，更多的可能是对安全性等不得不做的更新，核心团队对其关注度肯定不如正在开发和使用的5.0等版本，相比而言3.2也缺乏单元测试等比较好的质量验证手段，使得任何改动都更难验证。
谢谢！

[blackpuppy]

又有一些新的对路由的修正，详见PR #535。
关于路由的单元测试，可见示例项目blackpuppy/thinkblog。

[cnjackven]

这个路由的调整应用公告一下，同是3.2.*的版本更新的时候应该考虑下兼容问题，更新后直接大部分路由干不起也不科学啊

[hepeichun]

又有一些新的对路由的修正，详见PR #535。
关于路由的单元测试，可见示例项目blackpuppy/thinkblog。

谢谢

[hepeichun]

还发现一个问题，我将路由定义在子模块配置文件，这样就失效了，需要将路由定义到主配置文件

[blackpuppy]

@jdkysq

还发现一个问题，我将路由定义在子模块配置文件，这样就失效了，需要将路由定义到主配置文件

有测试用例吗？有空的时候，我也试一下。我用过不同的模块，但没用过子模块。

[hepeichun]

@jdkysq

还发现一个问题，我将路由定义在子模块配置文件，这样就失效了，需要将路由定义到主配置文件

有测试用例吗？有空的时候，我也试一下。我用过不同的模块，但没用过子模块。

我在Application/Plugin/Conf/config.php里面添加路由

'URL_ROUTER_ON' => true,
'URL_ROUTE_RULES'=>array(
'Plugin'=>'Plugin/Load/index'
),
发现无法调用

将上面的代码放入到Application/Common/Common/function.php里面问题解决。