使WordPress成为核心

开的8年前

关闭5个月前

#39653 关闭 增强 (馄饨)

WP_Http_Cookie更改

报告人: 塞巴斯蒂安皮苏拉的侧面 塞巴斯蒂安·皮苏拉 所有者:
里程碑: 优先: 正常的
严重程度: 正常的 版本:
组件: HTTP API 关键词: 报告人反馈
重点: 复写的副本:

描述

  1. WordPress约定是使用下划线代替camelCase来更改函数和方法
  2. 在getattributes方法中向返回数组添加新参数
  3. 将parseurl更改为wp_parseurl

附件(1)

39653.补丁(3.4 KB)-由添加塞巴斯蒂安·皮苏拉 8年前.

将所有附件下载为:.zip

更改历史记录(3)

#1 @海洋90
8年以前

  • 关键词 报告人反馈补充

有关1),请参阅https://make.wordpress.org/core/handbook/contribute/code-refacting/.

你能解释一下你为什么想要名称价值也可通过以下方式退回获取属性()以及为什么使用wp_parse_url()这里有必要吗?

#2 @你好,Tonya
5个月以前

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

你好,@sebastianpisula,

谢谢你给我这张票和补丁。

对于上下文WP_Http_Cookie课程介绍于[10512]通过#9049回到2009年。虽然是的,Core的编码标准确实需要snake_case命名,但为了符合这一要求,已经谨慎地重命名了以前发布的方法。

为什么?

影响与利益。
弃用旧方法而改名为snake_case方法将影响:

  • 用户和扩展器。插件、主题等中的现有代码将抛出弃用通知(当WP_DEBUG公司启用),这可能会导致噪音、服务器日志报告增加以及分类、支持、文档、代码更改等额外工作。
  • 贡献者和维护者。额外的代码需要长期维护。需要更新文档。额外工作和文件大小。

这些影响是否超过了使代码完全符合编码标准的好处?

这是一个有时很难回答的问题。这个代码重构准则手册中提供了帮助。同时评估其他因素,如意外错误或副作用。

在这个特定的类和实例中,我认为影响大于好处。这可能就是决定在中添加忽略的原因[47632]:

//phpcs:忽略WordPress。命名约定。有效功能名称。方法名称无效

还需要注意的是,还有其他使用旧方法的类也在使用ignore,例如。WP_网址WP_HTTP_响应.

这并不理想,因为所有代码都完全兼容将是非常棒的。但当存在影响时,需要考虑利益与影响。

关闭这张罚单,认为它是馄饨。如果其他委员不同意或希望继续讨论,欢迎重新考虑。

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