使WordPress成为核心

开的13个月前

关闭13个月前

#58400 关闭 缺陷(bug) (馄饨)

当前版本只能来自变量或常量。

报告人: 奥斯姆卡卡迪的简介 奥斯姆卡卡迪 所有者:
里程碑: 优先: 正常的
严重程度: 正常的 版本:
组件: 帮助/关于 关键词: has-屏幕截图 关闭
重点: 编码标准 复写的副本:

说明

从WordPress下载的存储库中使用了静态版本号。在wp-admin/about.php中,有两个地方使用了静态版本号。我们应该使用变量来定义同一文件中$display_version已经用于定义当前版本。请参阅下面使用静态版本号的两行。我还附上了屏幕截图,请看。

注意:这是https://build.trac.wordpress.org。如果我需要在其他地方创建票证,请告诉我。

https://build.trac.wordpress.org/browser/brances/6.2/wp-admin/about.php#L68
https://build.trac.wordpress.org/browser/brances/6.2/wp-admin/about.php#L68

附件(1)

58400磅/加仑(84.1 KB)-由添加奥斯姆卡卡迪 13个月前.

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

更改历史记录(3)

#1 @谢尔盖·比尤科夫
13个月以前

  • 组件已从更改管理帮助/关于
  • 关键词 关闭补充;需要-补丁远离的

大家好,欢迎回到WordPress Trac!谢谢你的票。

当查看整个文件时,您可能会发现$显示版本用于以下字符串中:

  • 欢迎使用WordPress%s
  • 了解有关WordPress%s的更多信息

但是,维护和安全发布条目(请参见6.2.1例如)使用硬编码版本号。使用$显示版本它似乎没有提供任何好处,只会为发布团队增加更多工作,因为一旦新版本可用,它就不再适用于以前的版本,因此每次新版本都必须更换它,而且很容易丢失。

#2 @audrasjb公司
13个月以前

  • 里程碑 等待审查删除
  • 分辨率设置为馄饨
  • 状态已从更改新的关闭
  • 版本 6.2.2删除

感谢您的门票和补丁建议,

然而,让我们将其解析为馄饨按照上面的注释:将此字符串设置为非静态字符串没有任何价值。如果您有任何异议,请重新打开票据:)

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