跳到内容
新问题

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

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

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

仅加载响应中出现的帖子缩略图。 #4210

正常开放
想要合并1承诺
基础: 大旅行箱
选择基础分支

对话

spacedmonkey(空格键)
复制链接
成员

Trac票:https://core.trac.wordpress.org/tickt/57900


此Pull请求仅用于代码审查。请将所有其他讨论保留在Trac记录单中。不要合并此Pull请求。请参见GitHub请求代码审查有关更多详细信息,请参阅《核心手册》。

复制链接
贡献者

@卡达姆怀特 卡达姆怀特 留下了评论

选择隐藏此评论的原因

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

这是否会给其他插件带来任何问题,例如当Yoast添加了注册的rest字段,其中包括日志图像(_I)信息?可能不会,我想他们会调用函数,如果还没有准备好的话,这些函数会填充这些数据。

if(post_type_supports($this->post_type,“缩略图”){
$fields=$this->get_fields_for_response($request);

if(rest_is_field_included('featured_media',$fields)||rest_is_field_icluded{
复制链接
贡献者

选择隐藏此评论的原因

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

我想我们还需要后类型支持我们也要检查一下,不是吗?如果帖子类型不支持缩略图缓存,那么更新缩略图缓存没有任何好处。

我还想知道_链接_嵌入。这些目标是否应该更具针对性_links.wp:功能媒体例如

它还提出了拥有rest_are_fields_included(字符串[]$field_names,$fields)功能

复制链接
成员 作者

选择隐藏此评论的原因

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

我们不需要后置类型支持。如果帖子类型不支持特色图像,则缺少此字段。

复制链接
贡献者

选择隐藏此评论的原因

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

@spacedmonkey(空格键)“失踪”怎么了?包含rest_is_field_included不知道post类型,因此它只考虑是否提供了参数。如果没有_字段则返回真的.这意味着我们最后打电话更新post_humbnail_cache对于不支持它们的帖子类型。

复制链接
成员 作者

选择隐藏此评论的原因

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

@卡达姆怀特只有架构中的特色媒体,如果帖子类型支持它,那么它就不存在。请参见

案例 '缩略图':
$模式['属性']['功能_媒体'] =阵列(
'描述'=>__('文章的特色媒体的ID。'),
'类型'=>'整数',
'上下文'=>阵列('看法','编辑','嵌入'),
);
打破;

复制链接
贡献者

选择隐藏此评论的原因

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

@spacedmonkey(空格键)啊,对了,包含rest_is_field_included可能没有任何架构上下文,但获取响应字段根据指定的架构生成包含的字段列表。说得好!担忧消失了。

复制链接
贡献者

选择隐藏此评论的原因

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

@spacedmonkey(空格键)我回过头来看,布尔逻辑看起来仍然是错误的。

rest_is_field_included('featured_media',$fields)||rest_is_field_icluded

这意味着如果_链接_嵌入请求,即使post类型不支持缩略图并且架构中没有featuredmedia。

我们可以恢复岗位类型支持检查吗?

复制链接
成员 作者

选择隐藏此评论的原因

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

所以

if(((rest_is_field_included('_embedded',$fields)||rest_is_field_icluded{

复制链接
贡献者

@卡达姆怀特 卡达姆怀特 留下了评论

选择隐藏此评论的原因

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

在这里更新了我的担忧。

你看到我在原始评论中的另一个问题了吗?

我还想知道_links和_embedded的全面检查。这些是否应该更多地针对_links.wp:featuredmedia,例如。

if(post_type_supports($this->post_type,“缩略图”){
$fields=$this->get_fields_for_response($request);

if(rest_is_field_included('featured_media',$fields)||rest_is_field_icluded{
复制链接
贡献者

选择隐藏此评论的原因

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

@spacedmonkey(空格键)我回过头来看,布尔逻辑看起来仍然是错误的。

rest_is_field_included('featured_media',$fields)||rest_is_field_icluded

这意味着如果_链接_嵌入请求,即使post类型不支持缩略图并且架构中没有featuredmedia。

我们可以恢复岗位类型支持检查吗?

免费注册 在GitHub上加入此对话.已经有帐户了吗?登录以发表评论
标签
还没有
项目
还没有
开发

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

2名参与者