Automattic / Automattic/VIP-Coding-Standards

Catch incorrect bloginfo() usage

Ouverte
#396 1 commentaire 0 réactions 0 personnes assignées Voir sur GitHub
Type: Enhancement
Langage dominant
PHP
Étoiles
261
Forks
44
Merge moyen
19 min
PR mergées (30 j)
1

Description

## 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' );

Guide de contribution

Ouvrir le guide de contribution

Piste de recherche

Commencez par les règles de PHP_CodeSniffer qui inspectent l’utilisation des fonctions, en utilisant bloginfo() et get_bloginfo() comme points d’entrée. Vérifiez que echo bloginfo( 'name' ) est signalé avec la recommandation d’omettre echo ou d’utiliser get_bloginfo(), tandis que echo get_bloginfo( 'name' ) ne l’est pas ; exécutez les tests de règles pertinents s’ils sont disponibles.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
php
Domaine
tooling
Type d'issue
Fonctionnalité
Difficulté
3/5
Temps estimé
1-2 jours
Activité
À l'abandon
Clarté
Clairement spécifiée
Accessibilité débutants
35/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.