New Sniff: wp_get_post_revisions memory issues
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 38/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Bien especificado
- Estado de actividad
- Estancado
- Stack tecnológico
- php
- Área
- performance, tooling
Línea de trabajo
Comienza con los ejemplos de wp_get_post_revisions que infringen la regla y los permitidos del issue, y con el ticket enlazado de WordPress Trac. Se considera terminado cuando se informan las llamadas sin un argumento fields, mientras que las llamadas que solicitan ids u otro valor de fields no se informan.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
What problem would the enhancement address for VIP?
wp_get_post_revisions can be problematic because it loads the entire post object for each revision. I've seen this in CLI commands and other places, where you might be looping through a set of posts, and then suddenly have to load the entire post objects for every revision a post has. If those revisions are large and numerous - it quickly fails.
Related trac: https://core.trac.wordpress.org/ticket/34560
Describe the solution you'd like
We should suggest they only get the IDs for the revisions instead of the whole object:
wp_get_post_revisions( $post->ID, array( 'fields' => 'ids' ) );
What code should be reported as a violation?
Anything that doesn't have the fields argument, like:
wp_get_post_revisions( $post->ID );
What code should not be reported as a violation?
Getting the ids field or other fields instead of the whole object.
wp_get_post_revisions( $post->ID, array( 'fields' => 'ids' ) );
- Lenguaje dominante
- PHP
- Estrellas
- 261
- Forks
- 44
- Merge medio
- 3 d 17 h
- PR fusionados (30 d)
- 9
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
-
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 ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 45/100
-
WordPress.WP.AlternativeFunctions.file_system_operations error messages cannot be fixed anymoreAbierto
Dificultad 3/5 1-2 días Aptitud para principiantes 48/100
-
Status: Awaiting feedback
Dificultad 2/5 1-3 horas Aptitud para principiantes 45/100
Automattic/VIP-Coding-Standards#754 · 3 comentarios ·
Todos los issues de Automattic/VIP-Coding-Standards
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Los mantenedores suelen responder en 2 días
-
b:1.9 t:enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
Los mantenedores suelen responder en 3 días
-
backend bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
-
bug
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100