8000 [建议] 废弃并删除Pimple & 重构 Biz Framework,更多使用Symfony本身的DI · Issue #212 · edusoho/edusoho · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
[建议] 废弃并删除Pimple & 重构 Biz Framework,更多使用Symfony本身的DI #212
Open
@tourze

Description

@tourze

https://developer.edusoho.com/biz-framework/overview.html 这里可以了解到 Biz Framework 的一些由来和实现。
但是目前 php 社区的 psr 容器方案已经足够完善,biz framework这种基于约定和命名(如 createService('XXX:YYY'))已经不是一个主流方案,对新接触的开发者来说额外带来了理解成本。

建议:

  1. 删除Pimple;
  2. Biz改为一个更加轻量的容器,同时注入Symfony自有的container,offsetGet/offsetExists等方法,优先从 symfony container中取值;
  3. 逐步把 Biz 相关的服务声明迁移到 yaml,使用 symfony 来管理;
  4. 逐步废弃 createService / createDao 之类的用法,改用更加直接的构造函数注入/控制器方案注入;
  5. 升级Symfony版本;

按照我最近阅读代码的理解,使用 Pimple 绕过 Symfony Dependency Compile,唯一的收益就是Plugin/Theme在管理上可以更加灵活,但是带来的问题真的太多了。

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions

      0