wechatpay-apiv3 / wechatpay-php

微信支付 APIv3 的官方 PHP Library,同时也支持 APIv2
Apache License 2.0
496 stars 100 forks source link

fixed -o option problem #89

Closed maxsky closed 2 years ago

TheNorthMemory commented 2 years ago

这个PR的意图应该是,通过指定-o参数,指定文件名形式的存证书,可API接口返回的是list,不能通过写死-o filename形式去存文件,不严谨,有可能达不到获取新证书的目的。

xy-peng commented 2 years ago

@maxsky 请先描述下pr解决的问题,你的原因分析,以及处理思路?以便于我们理解和review代码。

maxsky commented 2 years ago

@maxsky 请先描述下pr解决的问题,你的原因分析,以及处理思路?以便于我们理解和review代码。

楼上已经描述很清晰了,修正了输出路径问题,加入了输出路径判断

xy-peng commented 2 years ago

目前实现方式,有两个问题:

  1. 修改了原有的行为,升级后会破坏已有的很多程序。这一点在不知道使用者是如何使用的情况下,非常可怕。
  2. 程序的行为不确定了,没有明确的预期,调用程序可能会变复杂

如果指定文件名的需求是合理,希望能使用一个新的独立的参数。

xy-peng commented 2 years ago

暂时先关闭了。如果有新的想法可以再打开。