Skip to content

docs(authorization): 移除README中已删除的注解和监听器引用 - #369

Merged
zhou-hao merged 1 commit into
hs-web:5.0.xfrom
Zir0-93:docs/authorization-readme-stale-refs
Oct 8, 2026
Merged

zhou-hao merged 1 commit into
hs-web:5.0.xfrom
Zir0-93:docs/authorization-readme-stale-refs

Conversation

@Zir0-93

@Zir0-93 Zir0-93 commented Sep 30, 2026 •

Copy link
Copy Markdown
Contributor

hsweb-authorization/hsweb-authorization-api/README.md 中有八个链接指向已不存在的源文件。

常用注解表中的 @RequiresExpression 和 @RequiresDataAccess 已在 1e2ec16 (重构权限, 2019-10) 中移除。现有的 @DataAccess 等数据权限注解都标记了 @Deprecated,所以这里直接删除这两行,没有改名。

Listener 一节链接的 AuthorizationListener 已在 238fe0a (3.0.1, 2018-10) 中删除,五个事件类在 2994556 中从 listener/event/ 移到了 events/ 包。现在事件由 AuthorizationController 通过 ApplicationEventPublisher 发布,框架自身用 @EventListener 监听(如 UserTokenWebFilter.handleUserSign)。本次修改把五个链接指向 events/,替换了提到 AuthorizationListener 的那一句,并把例子改为 @EventListener 方法。

未改动:表中 AuthorizationExitEvent 的说明是"用户注销时触发",但本仓库中没有代码发布该事件(UserOnSignOut 只是监听它),是否保留请维护者决定。

只修改了 README,未运行 mvn test。

我参与开发了 Striff,一个检查文档与代码是否一致的工具。它的本仓库报告标出了其中的 RequiresDataAccess。

@CLAassistant

CLAassistant commented Sep 30, 2026 •

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@zhou-hao
zhou-hao merged commit 72f42ef into hs-web:5.0.x Oct 8, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants