使WordPress成为核心

开的3年前

关闭3年前

上次修改时间3年前

#53022 关闭 缺陷(bug) (固定的)

列出活动图像编辑器支持的文件类型

报告人: 德斯罗什的个人资料 德斯罗什 所有者: 德斯罗什的个人资料 德斯罗什
里程碑: 5.8 优先: 正常的
严重程度: 正常的 版本:
组件: 站点运行状况 关键词: has-patch接口 犯罪 需求-法典
重点: 复写的副本:

描述(上次修改者德斯罗什)

我正在做一些与#35725查看各种服务器配置是否支持WebP,并意识到服务器的Imagick配置支持的图像格式列表不会显示在任何地方。

在某些情况下,容易获得此列表可能会很有用。

例如,这可能是一种很好的方法,可以帮助用户在WebP功能可用之前验证其服务器是否支持WebP(以及其他更现代的图像格式),以便他们能够联系到主机。

当Imagick不是站点的活动编辑器时,可以显示GD支持的格式,但GD处理这一点有所不同(a而是返回位字段).

附件(6)

53022.差异(1001字节)-由添加德斯罗什 3年前.
53022.2.diff(如果有)(2.1 KB)-由添加德斯罗什 3年前.
53022.3-差异(1.2 KB)-由添加亚当西尔弗斯坦 3年前.
53022.4差异(1.3 KB)-由添加亚当西尔弗斯坦 3年前.
站点健康信息测试-WordPress 2021-06-11 20-35-00.jpg(33.4 KB)-由添加亚当西尔弗斯坦 3年前.
53022.5.差异(1.2 KB)-由添加亚当西尔弗斯坦 3年前.

将所有附件下载为:.zip文件

更改历史记录(23)

@德斯罗什
3年以前

#1 @德斯罗什
3年以前

  • 描述修改(差异)
  • 关键词 has-patch接口补充;需要-补丁远离的

#2 @Clorith公司
3年以前

这看起来是一件非常有用的事情,即使在添加了最终的WebP核心支持之后也可以添加它,当您觉得应该有图像时,可以看到它不受支持,这对用户来说是有益的。

路径看起来也很简单👍

我不确定我们现在是否需要GD支持,看看另一个问题,GD似乎只支持有损转换,所以我怀疑目前不会实现。

#3 @德斯罗什
3年以前

  • 里程碑已从更改等待审查5.8

#4 @德斯罗什
3年以前

53022.2.diff(如果有)稍微清理原始补丁,使标签更准确,并从中重命名变量$imagick_version($imagick版本)$imagemagick_版本,这更准确地反映了所存储的值。

我还添加了一个Imagick版本到调试数据。目前,还没有正式支持PHP8的Imagick扩展版本(目前需要从源代码构建)。随着越来越多的人寻求升级到PHP8,了解Imagick扩展的安装版本将有所帮助。

@Clorith如果你认为最后一项应该再讨论一点,或者分开讨论,我很乐意把它分出来。

#5 @Clorith公司
3年以前

啊,聪明,我没有考虑过你可能也想要PHP扩展版本的关系,这似乎是合理的,然后,我认为没有理由不把它添加到这个票据旁边,因为它们是相关的,因为它们都是媒体处理调试字段。

#6 @德斯罗什
3年以前

  • 关键词 犯罪补充
  • 所有者设置为德斯罗什
  • 状态已从更改新的分配

#7 @德斯罗什
3年以前

  • 分辨率设置为固定的
  • 状态已从更改分配关闭

50817:

站点运行状况:在“媒体处理”部分包含更多ImageMagick/Imagick信息。

这会将其他信息添加到“站点运行状况信息”页面的“媒体处理”部分。当ImageMagick用作站点的图像编辑器时,将显示支持的文件格式的完整列表。这将有助于站点所有者调试他们遇到的任何问题,因为添加了对更新、更现代的图像格式的支持(如中的WebP[50810]).

此外,还安装了Imagick的版本。这将帮助站点所有者调试在PHP端生成图像的问题。

一些变量也被重命名,以更准确地表示存储的内容。

道具Clorith,desrosj。
修复#53022.

#8 @米兰_卡普
3年以前

  • 关键词 需要编码补充

一旦我们真正为Site Health创建了最终用户文档,就可以将人们引导到可以查看其安装中支持哪些文件类型的位置。我想支持团队也会发现这很有用。

#9 @亚当西尔弗斯坦
3年以前

太棒了,谢谢@desrosj!当LibGD是活动图像编辑器时,您对列出图像格式支持有何看法?我们已经这样做了吗?

#11 @亚当西尔弗斯坦
3年以前

  • 分辨率 固定的删除
  • 状态已从更改关闭重新打开的

@desrosj进入53022.4差异我添加了对GD图像检测的支持。由于gd_info数据如下所示:

{“GD版本”:“捆绑(2.1.0兼容)”,“FreeType支持”:true,“FreeType链接”:“带FreeType”,“GIF读取支持”:true,“GIF创建支持”:true,“JPEG支持”:true,“PNG支持”:true,“WBMP支持”:true,“XPM支持”:false,...}

代码迭代已知格式,寻找设置为true的匹配数组变量。我还稍微清理了一下GIF图像支持字符串,跳过了我们不需要/使用的格式,比如“FreeType”。

随着新格式的添加,我们需要保持更新。

重新开放考虑5.8,罚款5.9。

#12 @约宾
3年以前

53022.4.diff在我看来很不错。我认为5.8版本最好能实现这个功能,因为这是一个新功能的bug,所以在beta周期的早期应该不会出现问题。

#13 @德斯罗什
3年以前

谢谢@adamsilverstein!

让我们继续添加这个。我认为这很有意义,这样用户就可以获得他们需要的信息。

对于53022.4差异,我认为调试可以从中删除gd_格式数组。什么时候?调试不存在的值价值而是使用。由于看起来两者之间永远不会有区别,调试这里可以跳过。

是否值得允许$gd_supported_formats要筛选的数组?这将允许插件添加对其他格式的支持,以将这些格式添加到列表中。

#14 @亚当西尔弗斯坦
3年以前

对于53022.4.diff,我认为可以从gd_formats数组中删除调试。如果不存在调试,则使用value的值。因为看起来两者之间永远不会有区别,所以这里可以跳过调试。

谢谢你的澄清,我会删除它。我不确定这些字段是如何使用的。

允许过滤$gd_supported_formats数组是否值得?这将允许插件添加对其他格式的支持,以将这些格式添加到列表中。

我不这么认为。这是一个服务器级别的功能检查,插件甚至可以在这个级别添加图像格式支持吗?

#15 @亚当西尔弗斯坦
3年以前

53022.5.差异我删除了调试数据行,并验证了我仍然可以在站点健康中看到正确的信息。

这应该可以提交了。

#16 @德斯罗什
3年以前

  • 分辨率设置为固定的
  • 状态已从更改重新打开的关闭

51143:

站点运行状况:显示GD库支持的文件格式列表。

这会将GD库支持的文件格式列表添加到Site Health的Media Handling部分。

这将有助于站点所有者调试他们遇到的任何问题,因为添加了对更新、更现代的图像格式的支持(如中的WebP[50810]).

后续行动[50817].
道具亚当·西尔弗斯坦(jorbin adamsilverstein)。
修复#53022.

#17 @德斯罗什
3年以前

我觉得包括在内很好航空工业联合会在这个列表中,但只是想注意一下GD仅在PHP 8.1及以上版本中支持此功能.

注:请参见TracTickets公司有关使用的帮助门票。