Disallow deregistering/deqeueing of certain assets that break QM
Maintainer antworten meist innerhalb von 1 Tag
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Anfängerfreundlichkeit
- 35/100
Rechercherichtung
Beginne mit der Überprüfung der angeforderten Fälle für wp_deregister_style und wp_dequeue_script sowie der verknüpften Query Monitor-Referenzen in dispatchers/Html.php für die erforderlichen Asset-Handles. Definiere das Sniff-Verhalten so, dass dashicons und jquery gemeldet werden, wenn sie deregistriert oder aus der Queue entfernt werden, während ähnlich benannte Handles wie not_dashicons und not_jquery nicht gemeldet werden; für den Abschluss ist eine Abdeckung sowohl von Verstößen als auch von Nicht-Verstößen erforderlich.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
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' );
- Vorherrschende Sprache
- PHP
- Sterne
- 261
- Forks
- 44
- Ø Merge
- 3 T. 9 Std.
- Gemergte PRs (30 T.)
- 10
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Keine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus Automattic/VIP-Coding-Standards
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 48/100
Automattic/VIP-Coding-Standards#876 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Breaking Change Type: Maintenance
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 35/100
Automattic/VIP-Coding-Standards#849 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 45/100
Automattic/VIP-Coding-Standards#832 ·
Maintainer antworten meist innerhalb von 1 Tag
-
WordPress.WP.AlternativeFunctions.file_system_operations error messages cannot be fixed anymoreOffen
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 48/100
Automattic/VIP-Coding-Standards#823 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Status: Awaiting feedback
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 45/100
Automattic/VIP-Coding-Standards#754 · 3 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in Automattic/VIP-Coding-Standards
Ähnliche Issues
-
Bug-Report
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
RSS-Bridge/rss-bridge#5158 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 82/100
octobercms/october#6150 ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
-
type/bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 62/100
flarum/framework#5301 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
Add TuneIn-DirMon in the bot listEvtl. vergeben @liviuconcioiu hat das heute übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
matomo-org/device-detector#8424 ·
Maintainer antworten meist innerhalb von 1 Tag