Automattic / Automattic/VIP-Coding-Standards

Disallow deregistering/deqeueing of certain assets that break QM

Abierto
#695 0 comentarios 0 reacciones 0 asignados Ver en GitHub
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

Abrir la 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

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.