Automattic / Automattic/VIP-Coding-Standards

Add warning for list_files function and related

Abierto
#704 6 comentarios 0 reacciones 0 asignados Ver en GitHub
Type: Enhancement
Lenguaje dominante
PHP
Estrellas
261
Forks
44
Merge medio
19 min
PR fusionados (30 d)
1

Descripción

## 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

Guía de contribución

Abrir la guía de contribución

Línea de trabajo

Localiza las reglas existentes de PHP_CodeSniffer para las comprobaciones de funciones específicas de WordPress y VIP, y revisa después cómo se definen y prueban las advertencias. Confirma el comportamiento esperado de las advertencias para list_files, scandir y opendir, incluido el ejemplo proporcionado de list_files; se considera terminado cuando esas llamadas se notifican de forma coherente.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
php
Área
tooling
Tipo de issue
Nueva funcionalidad
Dificultad
3/5
Tiempo estimado
1-2 días
Estado de actividad
Estancado
Claridad
Bastante claro
Aptitud para principiantes
35/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.