跳到内容
新问题

对这个项目有疑问吗?注册一个免费的GitHub帐户以打开问题并联系其维护人员和社区。

单击“注册GitHub”,表示您同意我们的服务条款隐私声明。我们偶尔会向您发送与帐户相关的电子邮件。

已经在GitHub上了?登录到您的帐户

为缓存和瞬态添加pull/patch命令 #89

正常开放
想要合并26提交到
底座: 主要的
选择基础分支

对话

小矮人
复制链接
贡献者

@小矮人 小矮人 评论2023年7月25日

添加用于从对象缓存和瞬态中提取和修补值的命令。

修复 #26

@小矮人 小型php 用力推动的这个 功能/抓取-匹配-缓存-瞬态 分支 8787b05号 f31a3dd号 比较 2023年7月31日19:49
@小矮人 小矮人将此拉入请求标记为已准备好进行审阅 2023年8月23日22:18
@小矮人 小矮人请求审阅一个团队作为一个代码所有者 2023年8月23日22:18
@小型php
复制链接
贡献者 作者

小矮人 评论2023年8月28日

我无法在本地重现功能测试的失败:

$作曲家行为>运行behat测试...................................................................... 70...................................................................... 140...................................................................... 210...................................................................... 280...................................................................... 350................20张照片(20张照片)366é磁带(366个成功)4米20.27秒(10.47兆字节)

似乎用于设置这些场景的mu-plugins没有加载。

复制链接
成员

@丹尼尔巴赫伯 丹尼尔巴赫伯 留下了评论

选择隐藏此评论的原因

将显示原因,以便向其他人描述此评论。了解更多信息.

我无法在本地重现功能测试的失败:

是的,我在本地也通过了考试。

我不知道可能是什么问题。@wp-cli/committers有什么想法吗?

功能/缓存.feature 过时的 显示已解决 隐藏已解决
功能/缓存功能 过时的 显示已解决 隐藏已解决
特性/瞬态特性 过时的 显示已解决 隐藏已解决
特性/瞬态特性 过时的 显示已解决 隐藏已解决
功能/缓存.feature 过时的 显示已解决 隐藏已解决
@瑞士风格
复制链接
成员

不幸的是,也无法在本地复制。

看起来像是$当前值无效的,所以wp_cache_get()呼叫补丁()方法正在返回无效的出于某种原因。。?

添加一些WP_CLI::log(var_export(WP_cache_get('my_key'),true));在中调用$集合_foo可以帮助调试这个。

@小矮人
复制链接
贡献者 作者

小矮人 评论2023年9月7日

在分支中重新设置main的基址后,我设法在本地复制失败测试,我将对此进行调查。

刮一下,这是rebase的一个问题🥲

@小矮人 小矮人 用力推动的这个 功能/抓取-匹配-缓存-瞬态 分支 4f765d0 6e5秒df 比较 2023年9月7日08:06
@小矮人
复制链接
贡献者 作者

@丹尼尔巴赫伯我已将测试拆分为专用文件并更新它们以供使用当我跑步时如果他们被期望成功。

327天99a我添加了两条调试语句,试图理解为什么在Github CI中测试失败。

@小矮人 小矮人 用力推动的这个 功能/抓取-匹配-缓存-瞬态 分支10次,最近一次 6 cd56d8 e75电子7c1 比较 2023年9月18日14:11
@小矮人
复制链接
贡献者 作者

@丹尼尔巴赫伯我终于能够修复功能测试的失败。这与使用STDIN获取新值有关,因为STDIN没有正确处理空值的情况。

我的叉子上的所有测试都是绿色的。

复制链接
成员

@丹尼尔巴赫伯 丹尼尔巴赫伯 留下了评论

选择隐藏此评论的原因

将显示原因,以便向其他人描述此评论。了解更多信息.

做得好让测试正常进行,@小矮人!

我留下了几条评论要处理。

src/缓存命令.php 过时的 显示已解决 隐藏已解决
src/缓存命令.php 过时的 显示已解决 隐藏已解决
src/缓存命令.php 过时的 显示已解决 隐藏已解决
src/Cache/RecursiveDataStructureTraverser.php 过时的 显示已解决 隐藏已解决
src/缓存命令.php 过时的 显示已解决 隐藏已解决
set_transient('my_key_2',['foo'=>['bar'=>'baz']]);
};

WP_CLI::add_hook('before_invoke:transient patch',$set_foo);
复制链接
成员

选择隐藏此评论的原因

将显示原因,以便向其他人描述此评论。了解更多信息.

我们能用吗wp瞬态集设置这些初始瞬态值?临时缓存在普通WordPress中是持久的,所以我们不需要以这种方式挂接到命令调用中。

复制链接
贡献者 作者

@小型php 小矮人 2023年12月1日

选择隐藏此评论的原因

将显示原因,以便向其他人描述此评论。了解更多信息.

最初我计划使用wp瞬态集但不可能像使用时那样直接设置类似数组的数据wp选项添加/更新(也许是未来公关的一个想法)。

我通过使用wp评估命令执行设置_瞬态

给定WP安装然后我运行`wp-eval“set_transient('my_key',['foo'=>'bar']);”`然后我运行`wp-eval“set_transient('my_key_2',['foo'=>['bar'=>'baz']]);”`

特性/瞬态匹配特性 显示已解决 隐藏已解决
特性/瞬态匹配特性 显示已解决 隐藏已解决
特性/瞬态匹配特性 显示已解决 隐藏已解决
$traverser=新的RecursiveDataStructureTraverser($current_value);

尝试{
$traverser->$action($key_path,$patch_value);
复制链接
成员

选择隐藏此评论的原因

将显示原因,以便向其他人描述此评论。了解更多信息.

如何保存实际瞬态值?

复制链接
贡献者 作者

选择隐藏此评论的原因

将显示原因,以便向其他人描述此评论。了解更多信息.

瞬态值稍后在埃尔塞夫使用中的函数存储条件$write_func(写入函数)变量。

@约十亿美元
复制链接
贡献者

这个公关还有什么要做的?

@小矮人
复制链接
贡献者 作者

您好!@约十亿美元,抱歉耽搁了。

我还没有时间看丹尼尔的评论。我会在本周检查他们,并完成这张PR以进行新的审查。

@小矮人
复制链接
贡献者 作者

您好!@丹尼尔巴赫伯

我已经重构了特性测试来解决您的反馈,这应该很好。

所有使用MySQL的测试工作流都是绿色的,使用SQLite的测试工作流失败了,但似乎与此PR没有特别的关系(例如:功能性-WP最新版本8.2,带有SQLite).

@瑞士风格
复制链接
成员

已知SQLite错误,请参阅#92

@小矮人
复制链接
贡献者 作者

您好,此PR应准备好进行最终审查,失败测试与SQLite错误有关,而不是与此PR中的更改有关。

@小矮人 小型php 用力推动的这个 功能/抓取-匹配-缓存-瞬态 分支 b9d6f5e d8ef33a型 比较 2024年4月26日08:36
@小矮人 小矮人 用力推动的这个 功能/抓取-匹配-缓存-瞬态 分支 c0e3c62 a2018年a5 比较 2024年4月26日15:05
src/缓存命令.php 过时的 显示已解决 隐藏已解决
src/瞬态命令.php 过时的 显示已解决 隐藏已解决
小矮人 和其他补充提交 2024年4月29日14:36
合著人:Pascal Birchler<pascal.birchler@gmail.com>
合著人:Pascal Birchler<pascal.birchler@gmail.com>
@瑞士风格 瑞士风格修改了里程碑:2.1.32.2.0 2024年5月4日
免费注册 在GitHub上加入此对话.已经有帐户了吗?登录以发表评论
项目
还没有
开发

成功合并此请求可能会解决这些问题。

缓存和瞬变的补丁和抓取
5名参与者