mineadmin / components

mine-admin library
Apache License 2.0
10 stars 21 forks source link

建议不要强制安装插件组件,在有使用这些代码时,加个判断 #116

Closed zoujingli closed 3 months ago

zoujingli commented 3 months ago

https://github.com/mineadmin/components/blob/94bb5e281ea399ae3211b87e841c1a8ba13fde55/src/mine-core/src/Command/InstallProjectCommand.php#L99

在未安装 AppStore 插件时,安装过程或部分操作有几个地方会报错,原因未加 class_exists 判断。

很多情况下系统是不让安装线上插件的,比如军队项目,政企内部项目,希望官方可以给予使用者选择的机会,不要强制必需安装在线 Store 类的插件中心组件,如果有使用这些代码的地方加个判断,不让它报错。

目前我是使用 class_alias 强行注册替换来避免报错,但这也不是长久之计。希望官方可以处理下....

boring-cyborg[bot] commented 3 months ago

Thanks for opening your first issue here! Be sure to follow the issue template!

zds-s commented 3 months ago

有道理,这里影响性不大。我改一下,感谢反馈