Skip to content

递归中间件(Ergodic)执行顺序错误 #365

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
crazywhalecc opened this issue Jun 11, 2023 · 0 comments · Fixed by #366 or #367
Closed

递归中间件(Ergodic)执行顺序错误 #365

crazywhalecc opened this issue Jun 11, 2023 · 0 comments · Fixed by #366 or #367
Labels
bug 意外行为或意外问题

Comments

@crazywhalecc
Copy link
Member

crazywhalecc commented Jun 11, 2023

受影响版本

2.x 全版本,3.x 全版本

描述

如题。比如 class 有 A 注解、B 注解,方法 foo() 分别有 C、D 注解,理论上注解的执行顺序应该是 ABCD,但实际上是 CDAB。在 3.2.2 修复了一个倒序执行的问题,但是没有从根本上解决执行顺序的问题。(3.2.1 及以前的 3.x 版本执行顺序是 BACD,也是不对的)

复现步骤

如题。

解决方案

重写注解加载顺序部分的代码。

附加信息

No response

@crazywhalecc crazywhalecc added the bug 意外行为或意外问题 label Jun 11, 2023
@github-project-automation github-project-automation bot moved this to 📋 Backlog in 3.0 计划内容 Jun 11, 2023
@crazywhalecc crazywhalecc changed the title 3.2.2 版本的递归中间件(Ergodic)被放在了方法中间件的后面,逻辑上错误 3.2.2 版本的递归中间件(Ergodic)执行顺序错误 Jun 11, 2023
@crazywhalecc crazywhalecc changed the title 3.2.2 版本的递归中间件(Ergodic)执行顺序错误 递归中间件(Ergodic)执行顺序错误 Jun 11, 2023
@github-project-automation github-project-automation bot moved this from 📋 Backlog to ✅ Done in 3.0 计划内容 Jun 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 意外行为或意外问题
Projects
Status: Done
1 participant