Sniff for getimagesize()
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 35/100
Línea de trabajo
Comienza con los ejemplos de infracciones de getimagesize() y el enfoque propuesto con wp_get_attachment_metadata(), y después revisa cómo el conjunto de reglas identifica las llamadas a funciones de PHP. Aclara el comportamiento esperado para las imágenes de la Biblioteca multimedia frente a las URL externas antes de definir la finalización y añadir cobertura para ambos casos.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
What problem would the enhancement address for VIP?
getimagesize() holds up the PHP process downloading the entire image to just read its file size.
Describe the solution you'd like
Most images in WPs Media Library have width and height attributes already (like from wp_get_attachment_metadata()) & that existing data should be used instead.
If the image is external and not in WordPress' media library, perhapes:
- a VIP helper function could be created to download only first few bytes of image
- don't use this function, or do image sizing w/ JS or CSS instead
What code should be reported as a violation?
$size = getimagesize( $image_url );
list($width, $height) = getimagesize( $image_url );
//etc.
- Lenguaje dominante
- PHP
- Estrellas
- 261
- Forks
- 44
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de Automattic/VIP-Coding-Standards
-
AlwaysReturnInFilter: isInsideIfConditonal() guards the conditions array after reading itPosiblemente ocupada @tomjn la tomó hace 5 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
-
Bug: PreGetPosts warns when the early is_main_query() return is not the first statement in its ifPosiblemente ocupada @tomjn la tomó hace 5 días. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 76/100
-
Suppress filters in get_posts false positivePosiblemente ocupada @tomjn la tomó hace 6 días. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 68/100
-
Dificultad 3/5 1-2 días Aptitud para principiantes 48/100
-
Breaking Change Type: Maintenance
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
Automattic/VIP-Coding-Standards#849 · 1 comentario ·
Todos los issues de Automattic/VIP-Coding-Standards
Issues similares
-
Dificultad 2/5 Menos de una hora Aptitud para principiantes 78/100
opencart/opencart#15763 · 2 comentarios · 1 reacción ·
Los mantenedores suelen responder en 1 día
-
L: github:actions L: php:composer
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
dependabot/dependabot-core#16493 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
api-platform/core#8649 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 1/5 Menos de una hora Aptitud para principiantes 84/100
open-telemetry/opentelemetry-php#2071 ·
Los mantenedores suelen responder en 1 día
-
Always show measure tooltipsAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Los mantenedores suelen responder en 1 día