Make suggestion when autoload resolves to true
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 48/100
Línea de trabajo
Localiza el sniff de PHPCS y las pruebas que gestionan add_option() y update_option(); inspecciona cómo se clasifican los argumentos omitidos, los valores literales yes/no y los booleanos. Se considera completado cuando los casos yes/true/implied indicados producen advertencias, los casos no/false no las producen y la advertencia puede hacer referencia a la documentación de autoloading proporcionada.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
What problem would the enhancement address for VIP?
Options get marked as autoload=yes when being added, unless there is an explicit setting to no.
For VIP clients, we should flag when it is yes, to remind and encourage developers to consider whether it should be autoload=no, which can help with performance.
Describe the solution you'd like
PHPCS Warning when 'yes' or true is supplied or implied when calling add_option() or update_option().
What code should be reported as a violation?
add_option( 'my_option', 'my_value', '', 'yes' );
add_option( 'my_option', 'my_value', '', true ); // Can be 'yes' string or boolean.
add_option( 'my_option', 'my_value', '' );
add_option( 'my_option', 'my_value' );
add_option( 'my_option' );
update_option( 'my_option', 'my_value', 'yes' );
update_option( 'my_option', 'my_value', true ); // Can be 'yes' string or boolean.
What code should not be reported as a violation?
add_option( 'my_option', 'my_value', '', 'no' );
add_option( 'my_option', 'my_value', '', false );
update_option( 'my_option', 'my_value', 'no' );
update_option( 'my_option', 'my_value', false );
Additional context
Some items may indeed be valid to be autoloaded, so it's not an Error in all cases.
This may seem like a bit of noise for some cases, but I think it counts as a good reminder. We can link the error message to our public docs about autoloading.
- 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 2 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 Un pull request vinculado a esta issue está abierto o ya se fusionó. 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 3 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 1-3 horas Aptitud para principiantes 84/100
PrestaShopCorp/ps_accounts#666 ·
-
Security SecurityBundle
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
symfony/symfony-docs#23173 ·
Los mantenedores suelen responder en 3 días
-
incomplete
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
opnsense/plugins#5763 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
fossology/fossology#3893 · 1 comentario ·
Los mantenedores suelen responder en 2 días
-
Documentation Feature: Self-Register / Verify UI
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día