Automattic / Automattic/VIP-Coding-Standards
Disallow deregistering/deqeueing of certain assets that break QM
- Lingua principale
- PHP
- Stelle
- 261
- Fork
- 44
- Merge medio
- 19m
- PR unite (30g)
- 1
Descrizione
## What problem would the enhancement address for VIP?
Sometimes certain core assets are deregistered/dequeued which break Query Monitor, which adds an extra step during debugging to track that down.
Dependencies are:
- `dashicons` https://github.com/Automattic/vip-go-mu-plugins-built/blob/d38a2b0e4a1be875302942783cfe49f7f343e226/query-monitor/dispatchers/Html.php#L187-L192
- `jquery` https://github.com/Automattic/vip-go-mu-plugins-built/blob/d38a2b0e4a1be875302942783cfe49f7f343e226/query-monitor/dispatchers/Html.php#L170-L199
## Describe the solution you'd like
It'd be good to have a sniff that warns for specific assets that we do not recommend removing.
## What code should be reported as a violation?
```
wp_deregister_style( 'dashicons' );
```
```
wp_dequeue_script( 'jquery' );
```
## What code should *not* be reported as a violation?
```
wp_dequeue_style( 'not_dashicons' );
```
```
wp_dequeue_script( 'not_jquery' );
```
Guida per i contributori
Apri la guida per i contributori
Direzione di ricerca
Inizia esaminando i casi richiesti di wp_deregister_style e wp_dequeue_script e i riferimenti collegati di Query Monitor in dispatchers/Html.php per gli handle degli asset richiesti. Definisci il comportamento dello sniff in modo che dashicons e jquery vengano segnalati quando vengono deregistrati o rimossi dalla coda, mentre gli handle con nomi simili, come not_dashicons e not_jquery, non vengano segnalati; il completamento richiede la copertura di esempi sia di violazione sia di non violazione.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- php
- Ambito
- tooling
- Tipo di issue
- Funzionalità
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 35/100