Automattic / Automattic/VIP-Coding-Standards

switch_to_blog() - flag for missing subsequent restore_current_blog()

Offen
#651 3 Kommentare 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

## 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

Beitragsleitfaden öffnen

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

Neue Issues direkt in Ihr Postfach

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