Automattic / Automattic/VIP-Coding-Standards
Disallow deregistering/deqeueing of certain assets that break QM
- Lenguaje dominante
- PHP
- Estrellas
- 261
- Forks
- 44
- Merge medio
- 19 min
- PR fusionados (30 d)
- 1
Descripción
## 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' );
```
Guía de contribución
Línea de trabajo
Comienza revisando los casos solicitados de wp_deregister_style y wp_dequeue_script y las referencias vinculadas de Query Monitor en dispatchers/Html.php para los handles de assets requeridos. Define el comportamiento del sniff de modo que dashicons y jquery se notifiquen cuando se desregistren o se retiren de la cola, mientras que los handles con nombres similares, como not_dashicons y not_jquery, no se notifiquen; la finalización requiere cobertura tanto para ejemplos de infracción como de no infracción.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- php
- Área
- tooling
- Tipo de issue
- Nueva funcionalidad
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100