New Sniff: wp_get_post_revisions memory issues
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 38/100
- Tipo di issue
- Funzionalità
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Ferma
- Stack tecnologico
- php
- Ambito
- performance, tooling
Direzione di ricerca
Inizia con gli esempi wp_get_post_revisions che violano la regola e quelli consentiti dell'issue, oltre che con il ticket WordPress Trac collegato. Il lavoro è completato quando vengono segnalate le chiamate senza un argomento fields, mentre non vengono segnalate le chiamate che richiedono ids o un altro valore di fields.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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' ) );
- Lingua principale
- PHP
- Stelle
- 261
- Fork
- 44
- Merge medio
- 3g 17h
- PR unite (30g)
- 9
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di Automattic/VIP-Coding-Standards
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 48/100
-
Breaking Change Type: Maintenance
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
Automattic/VIP-Coding-Standards#849 · 1 commento ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 45/100
-
WordPress.WP.AlternativeFunctions.file_system_operations error messages cannot be fixed anymoreAperta
Difficoltà 3/5 1-2 giorni Idoneità per principianti 48/100
-
Status: Awaiting feedback
Difficoltà 2/5 1-3 ore Idoneità per principianti 45/100
Automattic/VIP-Coding-Standards#754 · 3 commenti ·
Tutte le issue di Automattic/VIP-Coding-Standards
Issue simili
-
Translation TwigBridge
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
symfony/symfony-docs#23308 ·
I maintainer di solito rispondono entro 3 giorni
-
sync-en
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 84/100
I maintainer di solito rispondono entro 1 giorno
-
sync-en
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 84/100
I maintainer di solito rispondono entro 4 giorni
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 84/100
WordPress/hosting-handbook#506 ·
I maintainer di solito rispondono entro 2 giorni
-
在主页右上角的头像菜单栏中增加说说写作快捷入口Apertaenhancement
Difficoltà 2/5 1-3 ore Idoneità per principianti 79/100
mirai-mamori/Sakurairo#1435 · 1 commento ·