Automattic / Automattic/VIP-Coding-Standards

Catch incorrect bloginfo() usage

Offen
#396 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Type: Enhancement
Vorherrschende Sprache
PHP
Sterne
261
Forks
44
Ø Merge
19 Min.
Gemergte PRs (30 T.)
1

Beschreibung

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

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Beginnen Sie mit den PHP_CodeSniffer-Regeln, die die Verwendung von Funktionen prüfen, und verwenden Sie bloginfo() und get_bloginfo() als Einstiegspunkte. Bestätigen Sie, dass echo bloginfo( 'name' ) mit dem Hinweis gemeldet wird, echo wegzulassen oder get_bloginfo() zu verwenden, während echo get_bloginfo( 'name' ) nicht gemeldet wird; führen Sie die relevanten Regeltests aus, falls verfügbar.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
php
Bereich
tooling
Issue-Typ
Feature
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Veraltet
Klarheit
Klar beschrieben
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.