Automattic / Automattic/VIP-Coding-Standards
switch_to_blog() - flag for missing subsequent restore_current_blog()
- Vorherrschende Sprache
- PHP
- Sterne
- 261
- Forks
- 44
- Ø Merge
- 19 Min.
- Gemergte PRs (30 T.)
- 1
Beschreibung
## Describe the solution you'd like
From https://github.com/Automattic/VIP-Coding-Standards/pull/649#discussion_r606174833, it would be worth flagging that `restore_current_blog()` should be called subsequently (if not already).
## What code should be reported as a violation?
```
function switch_blog() {
$id = get_current_blog_id();
if ( $id === 2 ) {
return;
}
switch_to_blog( 2 );
}
```
## What code should *not* be reported as a violation?
```
function switch_blog() {
$id = get_current_blog_id();
if ( $id === 2 ) {
return;
}
switch_to_blog( 2 );
restore_current_blog();
}
```
Beitragsleitfaden
Rechercherichtung
Finde die bestehende Regel oder den bestehenden Sniff, der switch_to_blog() behandelt, und prüfe anschließend, wie er bestimmt, ob ein Aufruf ausgeglichen ist. Füge Abdeckung für die zwei Beispiele im Issue hinzu: Melde einen switch ohne nachfolgenden restore_current_blog(), und melde den wiederhergestellten Fall nicht.
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
- Größtenteils klar
- Anfängerfreundlichkeit
- 45/100