Skip to content
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

fix: 修复多语言在使用全局引入的 LoadLangPack 时无法自动检测的问题 #46

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ZoftTy
Copy link

@ZoftTy ZoftTy commented May 13, 2024

fixed #32

fixed top-think/framework#2953

多应用中无需再次初始化默认语言 App.php 中已经初始化过。

https://github.com/top-think/framework/blob/05942a3c64639d8d2c4c2b44489ca27e65f690f6/src/think/App.php#L450-L455

再次初始化会导致全局引入的 LoadLangPack 中间件解析的语言被覆盖。

@liu21st

@ZoftTy
Copy link
Author

ZoftTy commented May 17, 2024

@big-dream

@hulang
Copy link

hulang commented Jun 6, 2024

你艾特他个锤子,他就会点击一下关闭issues

@big-dream
Copy link
Contributor

你艾特他个锤子,他就会点击一下关闭issues

确实,At我没用,因为我只是参与者,并不是主导者,我贡献代码也是提交PR,并不是自己想怎么改就怎么改。

我不清楚你为什么对我这么大敌意,如果你对我关闭 Issues 不满意,你可以重新打开。

@ZoftTy
Copy link
Author

ZoftTy commented Jun 6, 2024

你艾特他个锤子,他就会点击一下关闭issues

确实,At我没用,因为我只是参与者,并不是主导者,我贡献代码也是提交PR,并不是自己想怎么改就怎么改。

我不清楚你为什么对我这么大敌意,如果你对我关闭 Issues 不满意,你可以重新打开。

我只想找个人来处理这个 PR,如果你能帮我找到可以处理的人就更好了,这个项目的维护者 @liu21st 好像没有时间处理这个问题。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants