Automattic / Automattic/VIP-Coding-Standards

Add warning for list_files function and related

未关闭
#704 6 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看
Type: Enhancement
主要语言
PHP
星标
261
派生
44
平均合并
19 分钟
30 天内合并 PR
1

描述

## What problem would the enhancement address for VIP?

I've recently been doing a code review and noticed the `list_files` function had no warning attached. On VIP Filesystem, the `list_files`, `scandir`, `opendir` will return empty result sets or false, so we should probably highlight this to customers to avoid unexpected results.

## Describe the solution you'd like
A warning should be added when the following functions are coded:
- list_files - WP
- scandir - PHP
- opendir - PHP

## What code should be reported as a violation?

```
$files = list_files( $folder, 2 );
```

## What code should *not* be reported as a violation?

## Additional context

贡献指南

打开贡献指南

调研方向

定位现有的 PHP_CodeSniffer 规则,这些规则用于检查 WordPress 和 VIP 特定的函数,然后检查警告是如何定义和测试的。确认对 list_files、scandir 和 opendir 的预期警告行为,包括提供的 list_files 示例;完成的标准是这些调用都能得到一致的报告。

由索引模型根据 Issue 内容生成。

评估

技术栈
php
领域
tooling
Issue 类型
功能
难度
3/5
预计耗时
1-2 天
活跃度
停滞
描述清晰度
基本清楚
新手友好度
35/100

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。