Automattic / Automattic/VIP-Coding-Standards
Catch incorrect bloginfo() usage
- 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?
When using `bloginfo()`, the caller should not echo the output, as `bloginfo()` does this internally (see [here](https://developer.wordpress.org/reference/functions/bloginfo/#source)). Callers should simply call `bloginfo()` without any `echo`, but if the callers want to control when the output is echoed, they should use `get_bloginfo()` ([#](https://developer.wordpress.org/reference/functions/get_bloginfo/)). Our standards should catch these kinds of issues in code.
## Describe the solution you'd like
The usage of `echo bloginfo()` should be caught and advice given on using `get_bloginfo()` instead or to skip the echo.
## What code should be reported as a violation?
echo bloginfo( 'name' );
## What code should *not* be reported as a violation?
echo get_bloginfo( 'name' );
Guía de contribución
Línea de trabajo
Comience con las reglas de PHP_CodeSniffer que inspeccionan el uso de funciones, utilizando bloginfo() y get_bloginfo() como puntos de entrada. Confirme que echo bloginfo( 'name' ) se informa con la recomendación de omitir echo o usar get_bloginfo(), mientras que echo get_bloginfo( 'name' ) no se informa; ejecute las pruebas de reglas pertinentes si están disponibles.
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
- Bien especificado
- Aptitud para principiantes
- 35/100