Disallow deregistering/deqeueing of certain assets that break QM
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 35/100
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.
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:
dashiconshttps://github.com/Automattic/vip-go-mu-plugins-built/blob/d38a2b0e4a1be875302942783cfe49f7f343e226/query-monitor/dispatchers/Html.php#L187-L192jqueryhttps://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' );
- Lingua principale
- PHP
- Stelle
- 261
- Fork
- 44
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di Automattic/VIP-Coding-Standards
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
-
Bug: PreGetPosts warns when the early is_main_query() return is not the first statement in its ifAperta
Difficoltà 3/5 1-2 giorni Idoneità per principianti 76/100
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 68/100
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 48/100
-
Breaking Change Type: Maintenance
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
Automattic/VIP-Coding-Standards#849 · 1 commento ·
Tutte le issue di Automattic/VIP-Coding-Standards
Issue simili
-
sync-en
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 92/100
I maintainer di solito rispondono entro 1 giorno
-
sync-en
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
I maintainer di solito rispondono entro 3 giorni
-
Form
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
symfony/symfony-docs#23159 ·
I maintainer di solito rispondono entro 3 giorni
-
bug component: bulk editor support
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
Yoast/wordpress-seo#23669 ·
I maintainer di solito rispondono entro 3 giorni
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
php/frankenphp#2688 ·
I maintainer di solito rispondono entro 1 giorno