Automattic / Automattic/VIP-Coding-Standards

using curl should suggest using vip_safe_wp_remote_get instead of wp_remote_get

Offen
#754 3 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Status: Awaiting feedback
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 `curl` the suggested fix is to use `wp_remote_get`, however, `wp_remote_get` is discouraged and `vip_safe_wp_remote_get` is suggested instead.

## Describe the solution you'd like
Suggest the use of `vip_safe_wp_remote_get` when using `curl` to avoid a secondary violation for using `wp_remote_get`

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Es ist keine Datei oder kein Test benannt. Finde die PHP_CodeSniffer-Regel, die die Verwendung von curl meldet und derzeit wp_remote_get empfiehlt, und überprüfe anschließend deren bestehende Testabdeckung. Fertig ist die Änderung, wenn die Diagnose stattdessen vip_safe_wp_remote_get empfiehlt und keine nachgelagerte wp_remote_get-Verletzung mehr verursacht.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
php, wordpress
Bereich
tooling
Issue-Typ
Feature
Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
45/100

Neue Issues direkt in Ihr Postfach

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