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

feat: thinkphp container 兼容 #669

Open
3 of 4 tasks
yansongda opened this issue Sep 4, 2022 · 13 comments · May be fixed by #675
Open
3 of 4 tasks

feat: thinkphp container 兼容 #669

yansongda opened this issue Sep 4, 2022 · 13 comments · May be fixed by #675

Comments

@yansongda
Copy link
Owner

yansongda commented Sep 4, 2022

  • 没有 set 方法,可以用 bind 等代替?
  • bind 后,执行 force 时,再次 bind 不生效,需要在 bind 前 delete 下
  • Pay 中如果未提前 bindget 时 必定失败,并不会自动注入新增
  • Pipeline 中如果未提前 bindget 时 必定失败,并不会自动注入新增
@yansongda yansongda linked a pull request Sep 8, 2022 that will close this issue
@axguowen
Copy link

可以试试使用instance方法来绑定一个类实例到容器

@yansongda
Copy link
Owner Author

yansongda commented Oct 23, 2023

可以试试使用instance方法来绑定一个类实例到容器

这周我再看看 tp 那边能否兼容了。

PS:如果感兴趣的化,也可以 PR 进来

@axguowen
Copy link

可以试试使用instance方法来绑定一个类实例到容器

这周我再看看 tp 那边能否兼容了。

PS:如果感兴趣的化,也可以 PR 进来

试了下好像还是不行,TP的Contanier有限制,必须是绑定了标识的类才能获取到
然而使用官方推荐的安装hyperf/pimple包会导致TP的助手函数env不能正常获取到环境变量,
所以卸载了hyperf/pimple之后安装illuminate/container包也能用,且不影响TP的env助手函数的使用

不知道在其它框架或者无框架的项目使用illuminate/container容器能不能用,如果可以用的话感觉都不需要hyperf/pimple包了,这个包稍微有点臃肿,可以直接改用illuminate/container容器替代

@yansongda
Copy link
Owner Author

然而使用官方推荐的安装hyperf/pimple包会导致TP的助手函数env不能正常获取到环境变量,

这个安装 hyperf/pimple 最新版的就行

不知道在其它框架或者无框架的项目使用illuminate/container容器能不能用,如果可以用的话感觉都不需要hyperf/pimple包了,这个包稍微有点臃肿,可以直接改用illuminate/container容器替代

illuminate/container 也可以用的。两个包其实都不臃肿,也都挺简单的,可以看看源码就知道了。不过看选择吧。我晚些时候可以改改文档,增加下,给多一个选择

@80boys

This comment was marked as abuse.

@ken678
Copy link
Contributor

ken678 commented May 28, 2024

目前thinkphp容器兼容有进度么 还是兼容不了

@yansongda
Copy link
Owner Author

目前兼容不了 tp 里的 container 还是得自行安装 hyperf/pimpleilluminate/container

@ken678
Copy link
Contributor

ken678 commented May 29, 2024

目前兼容不了tp里的container 还是得自行安装hyperf/pimpleilluminate/container

好的

@ken678
Copy link
Contributor

ken678 commented Dec 13, 2024

以前作者说不兼容tp容器 所以一直没关注
今天闲着没事 用了thinkphp8的容器 按照#675 按照这个逻辑修改了artful那边代码
然后Pipeline那边get改成make
image

发现能正常展示付款码
image

@ken678
Copy link
Contributor

ken678 commented Dec 13, 2024

@yansongda

@yansongda
Copy link
Owner Author

方便提个 PR 么?

以前作者说不兼容tp容器 所以一直没关注 今天闲着没事 用了thinkphp8的容器 按照#675 按照这个逻辑修改了artful那边代码 然后Pipeline那边get改成make image

发现能正常展示付款码 image

@ken678
Copy link
Contributor

ken678 commented Dec 16, 2024

好的 等我空了 我提交下

@ken678
Copy link
Contributor

ken678 commented Dec 16, 2024

已更新两个依赖yansongda/artful和yansongda/supports
thinkphp使用的最新8.1版本https://github.com/top-think/framework
thinkphp也有单独的容器版本 不过没测过https://github.com/top-think/think-container
@yansongda

其他支付还没有空测试过 就只测了支付宝付款码

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 a pull request may close this issue.

4 participants