WordPress.org网站
获取WordPress

使WordPress成为核心

开的2年前

关闭2年前

#54746 关闭 任务（幸运的） (固定的)

wp-login.php中的注释错误

报告人： alkesh7的个人资料 碱性7 所有者： 谢尔盖比柳科夫的简介 谢尔盖·比尤科夫
里程碑： 6 优先： 正常的
严重程度： 少数的 版本：
组件： 登录和注册 关键词：
重点： 文档，编码标准 复写的副本：

描述

你好WordPress团队，

我看到WordPress在评论结构中有问题。

WordPress遵循注释结构。请参阅评论标准链接：​https://developer.wordpress.org/coding-standards/inline-documentation-standards.php/#5-内联注释

在WordPress 5.9候选版本中。根文件夹中的文件“wp-login.php”中存在注释结构问题。请参阅以下屏幕截图中的问题。
​网址：https://prnt.sc/263q9qs

请看一下我的请求并解决它。

谢谢您！

附件(3)

wp-login.php.补丁​(347字节)-由添加碱性7 2年前。
wp-org-ticket.png公司​(53.5 KB)-由添加碱性7 2年前。
54746.差异​(1.0 KB)-由添加彼得威尔逊公司 2年前。

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

更改历史记录(14)

@碱性7
2年以前

@碱性7
2年以前

#1 @audrasjb公司
2年以前

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

你好，欢迎回到Trac！
当我们进入5.9的候选发布周期时，让我们在6.0中解决这个问题。

#2 @普莱斯科普
2年以前

哦，我们在很多地方都有这样的产品！

这张票是在​松弛presskopp的#docs中。​查看日志。
2年以前

这张票是在​松弛presskopp的#核心编码标准。​查看日志。
2年以前

#5 @audrasjb公司
2年以前

  • 组件已从更改评论到登录和注册
  • 焦点 文档补充
  • 版本 大旅行箱删除

#6 @彼得威尔逊公司
2年以前

@Presskopp是的，它在核心的几个地方。我在49个文件中找到了它，但其中一些是第三方库。

出于本文的目的，让我们将重点放在wp-login.php上，任何进一步的修复都可以在通用编码标准修复单上进行#54728个。

@彼得威尔逊公司
2年以前

#7 @彼得威尔逊公司
2年以前

  • 所有者设置为彼得威尔逊公司
  • 分辨率设置为固定的
  • 状态已从更改新的到关闭

在52945:

注册/登录：的编码标准修复wp-login.php型。

道具alkesh7、Presskopp、audrasjb。
修复#54746。
请参见#54728。

#8 随访： @谢尔盖·比尤科夫
2年以前

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

我认为像“Main”这样的注释有一些好处，因为它们表示文件的各个部分，在本例中，将函数声明与主要部分分隔开来。

我认为我们可以根据文档标准将它们转换为多行格式，而不是删除它们：

/**主要。*/

#9个 答复：↑ 8 @彼得威尔逊公司
2年以前

  • 关键词 has-patch接口远离的
  • 所有者已从更改彼得威尔逊公司到谢尔盖·比尤科夫
  • 状态已从更改重新打开的到分配

回复谢尔盖·比尤科夫:

我认为像“Main”这样的注释有一些好处，因为它们表示文件的各个部分，在本例中，将函数声明与主要部分分隔开来。

有了额外的上下文，main这个词本身就没有意义了。

我认为我们可以根据文档标准将它们转换为多行格式，而不是删除它们：

/**主要。*/

关键是通过避免这种格式的单行注释来满足编码标准。鉴于评论需要额外的上下文才能有意义，我认为无论如何最好将其修改为多行评论。

#10 @成本开发
2年以前

  • 类型已从更改增强到任务（幸运的）

#11个 @谢尔盖·比尤科夫
2年以前

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

在53313:

文件：进一步澄清对主要部分的评论wp-login.php型。

后续行动[52945]。

修复#54746。

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