Skip to content

🐛 修复控制器在遭遇事件操作响应器时出现的错误#4

Merged
A-kirami merged 1 commit into
mainfrom
fix/tmp-matcher
Aug 19, 2023
Merged

🐛 修复控制器在遭遇事件操作响应器时出现的错误#4
A-kirami merged 1 commit into
mainfrom
fix/tmp-matcher

Conversation

@A-kirami

Copy link
Copy Markdown
Owner

这个 PR 带来了什么样的更改?

  • 错误修复
  • 新功能
  • 文档/注释
  • 代码格式
  • 代码重构
  • 测试用例
  • 性能优化
  • 外观样式
  • 项目构建
  • 依赖环境
  • 持续集成/部署
  • 其他,请描述:

这个 PR 是否存在破坏性变更?

  • 是的,并已在 issue #___ 号中获得批准
  • 没有

描述

当事件响应器的优先级为0,且为临时响应器时,不接受控制器检查

动机和背景

事件响应器操作如 gotreject 等,会创造一个新的临时响应器,这个临时响应器没有经过服务化,在尝试通过 Ability.got 获取时会出现 KeyError 错误

其他信息

检查工作

  • 我对我的代码进行了注释,特别是在难以理解的部分
  • 我的更改需要更新文档,并且已对文档进行了相应的更改
  • 我添加了测试并且已经在本地通过,以证明我的修复补丁或新功能有效
  • 我已检查并确保更改没有与其他打开的 Pull Requests 重复

@A-kirami A-kirami added the bug Something isn't working label Aug 19, 2023
@A-kirami
A-kirami merged commit d044f62 into main Aug 19, 2023
@A-kirami
A-kirami deleted the fix/tmp-matcher branch August 19, 2023 15:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant