Suppress filters in get_posts false positive
Les mainteneurs répondent en général sous 1 jour
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 68/100
Piste de recherche
Reproduce the warning using the minimal get_posts() snippet against the VIP Coding Standards main branch, then trace how the phpcs:ignore comment is handled alongside suppress_filters. Done means the false-positive warning is no longer reported while the intended meta_key warning remains available.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Bug Description
This get_posts call generates a warning that you must use suppress_filters set to false for performance reasons, despite that already being the case:
$post_ids = get_posts( [
'post_type' => [ 'post' ],
'post_status' => 'publish',
'fields' => 'ids',
'meta_key' => 'foobar', // phpcs:ignore WordPress.DB.SlowDBQuery.slow_db_query_meta_key
'suppress_filters' => false,
] );
My suspicion is that phpcs ignore comment is breaking the parsing of the array somehow.
( Note this is legacy code, we're aware of the meta_key issue )
Minimal Code Snippet
$post_ids = get_posts( [
'post_type' => [ 'post' ],
'post_status' => 'publish',
'fields' => 'ids',
'meta_key' => 'foobar', // phpcs:ignore WordPress.DB.SlowDBQuery.slow_db_query_meta_key
'suppress_filters' => false,
] );
Error Code
⚠️ Warning( severity 3 ): get_posts() is uncached unless the "suppress_filters" parameter is set to false. If the suppress_filter parameter is set to false this can be safely ignored. More Info: https://docs.wpvip.com/technical-references/caching/uncached-functions/ (
WordPressVIPMinimum.Functions.RestrictedFunctions.get_posts_get_posts).
Environment
This occurred 1st October in the VIP Coding standards bot on a github pull request
Tested Against main branch?
- I have verified the issue still exists in the
mainbranch of VIPCS. - I have verified the issue still exists in the
developbranch of VIPCS.
- Langage dominant
- PHP
- Étoiles
- 261
- Forks
- 44
- Merge moyen
- 3 j 9 h
- PR mergées (30 j)
- 10
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
Automattic/VIP-Coding-Standards#876 ·
Les mainteneurs répondent en général sous 1 jour
-
Breaking Change Type: Maintenance
Difficulté 4/5 3-5 jours Accessibilité débutants 35/100
Automattic/VIP-Coding-Standards#849 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 3/5 1-2 jours Accessibilité débutants 45/100
Automattic/VIP-Coding-Standards#832 ·
Les mainteneurs répondent en général sous 1 jour
-
WordPress.WP.AlternativeFunctions.file_system_operations error messages cannot be fixed anymoreOuverte
Difficulté 3/5 1-2 jours Accessibilité débutants 48/100
Automattic/VIP-Coding-Standards#823 ·
Les mainteneurs répondent en général sous 1 jour
-
Status: Awaiting feedback
Difficulté 2/5 1-3 heures Accessibilité débutants 45/100
Automattic/VIP-Coding-Standards#754 · 3 commentaires ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de Automattic/VIP-Coding-Standards
Issues similaires
-
InvoiceSettled is answered 200 when the invoice cannot be re-fetched, so BTCPay never redelivers itOuverte
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
btcpayserver/woocommerce-greenfield-plugin#93 ·
Les mainteneurs répondent en général sous 1 jour
-
kind/bug status/triage
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
-
🐛 [Bug] Bulk CSV import stores hide_from_combined_view=FALSE as a string, which hides every account from Sent, Junk, Trash and DraftsPeut-être pris @exSnake l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
opnsense/core#11052 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
needs triage
Difficulté 2/5 1-3 heures Accessibilité débutants 66/100