New Sniff: wp_get_post_revisions memory issues
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 38/100
- Type d'issue
- Fonctionnalité
- Clarté
- Clairement spécifiée
- Activité
- À l'abandon
- Stack technique
- php
- Domaine
- performance, tooling
Piste de recherche
Commencez par les exemples wp_get_post_revisions en infraction et autorisés de l'issue, ainsi que par le ticket WordPress Trac lié. C'est terminé lorsque les appels sans argument fields sont signalés, tandis que les appels demandant ids ou une autre valeur de fields ne le sont pas.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
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' ) );
- Langage dominant
- PHP
- Étoiles
- 261
- Forks
- 44
- Merge moyen
- 3 j 17 h
- PR mergées (30 j)
- 9
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de Automattic/VIP-Coding-Standards
-
Difficulté 3/5 1-2 jours Accessibilité débutants 48/100
-
Breaking Change Type: Maintenance
Difficulté 4/5 3-5 jours Accessibilité débutants 35/100
Automattic/VIP-Coding-Standards#849 · 1 commentaire ·
-
Difficulté 3/5 1-2 jours Accessibilité débutants 45/100
-
WordPress.WP.AlternativeFunctions.file_system_operations error messages cannot be fixed anymoreOuverte
Difficulté 3/5 1-2 jours Accessibilité débutants 48/100
-
Status: Awaiting feedback
Difficulté 2/5 1-3 heures Accessibilité débutants 45/100
Automattic/VIP-Coding-Standards#754 · 3 commentaires ·
Toutes les issues de Automattic/VIP-Coding-Standards
Issues similaires
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 85/100
-
cloning objets from planningOuverte
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
glpi-project/glpi#25922 ·
Les mainteneurs répondent en général sous 1 jour
-
Bug
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
Automattic/safe-publish#726 ·
Les mainteneurs répondent en général sous 1 jour
-
Translation TwigBridge
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
symfony/symfony-docs#23308 ·
Les mainteneurs répondent en général sous 3 jours
-
sync-en
Difficulté 1/5 Moins d'une heure Accessibilité débutants 84/100
Les mainteneurs répondent en général sous 1 jour