Sniff idea: Detect when constant is being used in constant()
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 42/100
Línea de trabajo
Comienza revisando las convenciones existentes de los sniffs de VIPCS-PHP_CodeSniffer y cómo se representan las llamadas a constant(). Confirma que se informa constant(FOO_BAR), mientras que las cadenas entre comillas y las variables no se informan, y verifica el comportamiento con el conjunto de pruebas del sniff correspondiente.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Describe the solution you'd like
Not sure if VIPCS is the best way forward, but I think this would be a worthy sniff.
When using constant(), we should throw a warning if a string is not being inputted. Of course it is valid PHP to do something like:
constant( FOO_BAR );
But I think most likely, what the user wants to do is:
constant( 'FOO_BAR' );
What code should be reported as a violation?
constant( FOO_BAR );
What code should not be reported as a violation?
constant( 'FOO_BAR' );
constant( "FOO_BAR" );
constant( $foo_bar );
- 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 3 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 4 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 72/100
thephpleague/commonmark#1159 ·
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
awslabs/aidlc-workflows#1879 ·
Los mantenedores suelen responder en 1 día
-
[Bug] The PHP file that lists DNS records truncates records to 12 characters?Posiblemente ocupada @sahsanu la tomó hoy. Abiertobug
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
hestiacp/hestiacp#5769 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
bug customer-reported
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
MagnaCapax/PMSS#1011 ·
Los mantenedores suelen responder en 5 días
-
Talk Review
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
socallinuxexpo/scale-drupal#351 ·