Automattic / Automattic/VIP-Coding-Standards
Sniff idea: Detect when constant is being used in constant()
- Lenguaje dominante
- PHP
- Estrellas
- 261
- Forks
- 44
- Merge medio
- 19 min
- PR fusionados (30 d)
- 1
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 );
```
Guía de contribución
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.
Evaluación
- Stack tecnológico
- php
- Área
- tooling
- Tipo de issue
- Nueva funcionalidad
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 42/100